[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: devices: pass gn_config to open and following devices: *: use instan
Re: devices: pass gn_config to open and following devices: *: use instance data commits
Sun, 09 Jan 2022 21:47:03 +0100
(and welcome back :-))
I just pushed my last changes here:
Sorry for not doing that earlier...
As for changes you are asking about:
- gn_statemachine was removed as parameter of device drivers as it was
pulling pretty high level stuff into low level. Driver is just a simple
driver, it does not need to be aware at all about gnokii itself.
- as for usbfbus_find_interfaces I believe you misread patch. Logic is
the same, I just modified interface to be the same for all drivers, so
I could use plugin architecture.
From: Pawel Kot <email@example.com>
To: Ladislav Michl <firstname.lastname@example.org>, Discussion forum for gnokii
Subject: devices: pass gn_config to open and following devices: *: use
instance data commits
Date: Sun, 9 Jan 2022 19:55:40 +0100
I'm in the process of pulling patches from you to the mainline. I think
I got most of them already read, understood, and committed (will push
them to my GitHub repo any time soon - will need to figure out how to
push them to the official repo easily). However, a couple of questions
popped up (raising them in separate emails).
I can see that you are getting rid of gn_statemachine as a parameter to
device drivers. What is the reason for this? I think we have this
rather convenient and universal method of passing gn_statemachine as a
context (agree that device drivers interface could be simplified
Also, why do you move logic between usbfbus_find_interfaces() and