On 11/10/2010 12:51 PM, Carmine IASCONE wrote:
> Hi Greg, Hi JS,
> Matteo and I have started to develop linux device drivers for our 
> STMicroelectronics sensors about one year ago. Our main target is the Android 
> platform (mobile phone or tablet pc), and for this reason the drivers are 
> thought to be used on I2C bus. The drivers are enough stable, we have several 
> customers that are using them, and also with the advice of these customers, 
> we would like to make available them for all the linux community, merging 
> them in the kernel upstream to be used in all linux supported platforms.
> The drivers have had a initial review by Alan Cox, that gives us some very 
> precious advices to improve the style and robustness of the drivers. 
> We are newbies in patch generation and submission: we have followed the 
> instructions in the Greg's video on you tube to create this first patch, so 
> sorry if there is something that we missed. We thought to put the driver in 
> staging directory, because before merging them in the main tree we would like 
> to have a general revision, and also because in this first release we haven't 
> managed the device interrupts yet. At the end the right position for these 
> drivers could be drivers/input/misc.
> How do we proceed now? Do we need to generate a new patch adding the TODO 
> file? Please advice.

Hi, well, I don't think this should go into staging at all as I think
it's clean enough to go upstream directly (but I repeat I'm no IIC
expert). So please resend to IIC people:
I2C SUBSYSTEM
M:      "Jean Delvare (PC drivers, core)" <[email protected]>
M:      "Ben Dooks (embedded platforms)" <[email protected]>
L:      [email protected]

but before that, move that out of staging to drivers/i2c/ (or anywhere
where it make sense).

It's perfectly OK to add functionality later (IRQs).

regards,
-- 
js
_______________________________________________
devel mailing list
[email protected]
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel

Reply via email to