[RFC] X Input Driver specification

Magnus Vigerlöf Magnus.Vigerlof at home.se
Thu Apr 5 14:30:50 PDT 2007


On Thursday 05 April 2007 22:39, Simon Thum wrote:
> Hi Magnus,
>
> I'm in charge of bug#8583, which would benefit from some single point
> that would be called on device init (or just before). Currently my
> integration is a little hacky and I think I've seen other code which
> might be better placed in such a function.

Currently (even without my patch-set) the xserver will call PreInit and 
DEVICE_INIT at startup and at an input device hotplug add request, it will 
also call DEVICE_ON in that case. DEVICE_ON/OFF will be called at every 
VT-switch as well, maybe one/several of these points is good enough?

> What I have in mind is a single point (in DIX as well as backends) to
> perform device-class specific stuff just before the device comes to action.
> Maybe I just missed it, but I haven't seen something like that in the
> code and your proposal. So I'd just like to suggest it :)

If there's a need to make changes in DIX, maybe the functions that make the 
actual calls for DEVICE_* would be a suitable place? In DEVICE_INIT you will 
have access to all structures that will be allocated for the input device by 
the core part of the xserver, so that's the last place you may have to make 
device-specific initialization of the allocated structures.

ActivateDevice [+187 dix/devices.c] (DEVICE_INIT)
EnableDevice [+142 dix/devices.c] (DEVICE_ON)
DisableDevice [+168 dix/devices.c] (DEVICE_OFF)
CloseDevice [+408 dix/devices.c] (DEVICE_CLOSE)

I hope the current callbacks will prove sufficient for your needs.


The proposal is more a try to define what the xserver does in the current 
implementation so that the different input devices knows what they should be 
doing at the different callbacks, rather than to try to introduce changes to 
the interface. The wacom driver (at least) seems to have suffered from this 
lack of a spec, and my gut tells me that we're not the only one that might be 
affected by this.

Cheers
 Magnus V



More information about the xorg mailing list