On Thu, Oct 19, 2017 at 02:40:31PM +0300, Andy Shevchenko wrote: > On Wed, 2017-10-18 at 23:53 +0300, Sakari Ailus wrote: > > On Wed, Sep 27, 2017 at 09:24:55PM +0300, Andy Shevchenko wrote: > > > The driver has been submitted with a limitation to few platforms and > > > sensors which it does support. Even though two sensor drivers have > > > no > > > users neither on ACPI-enabled platforms, nor in current Linux kernel > > > code. Patches 1 and 2 removes those drivers for now. > > > > > > It seems new contributors follow cargo cult programming done by the > > > original driver developers. It's neither good for code, nor for > > > reviewing process. To avoid such issues in the future here are few > > > clean > > > up patches, i.e. patches 3, 4, 6. 13. > > > > > > On top of this here are clean ups with regard to GPIO use. One may > > > consider this as an intermediate clean up. This part toughly related > > > to > > > removal of unused sensor drivers in patches 1 and 2. > > > > > > Patch series has been partially compile tested. It would be nice to > > > see > > > someone with hardware to confirm it doesn't break anything. > > > > > > Andy Shevchenko (13): > > > staging: atomisp: Remove IMX sensor support > > > staging: atomisp: Remove AP1302 sensor support > > > staging: atomisp: Use module_i2c_driver() macro > > > staging: atomisp: Switch i2c drivers to use ->probe_new() > > > staging: atomisp: Do not set GPIO twice > > > staging: atomisp: Remove unneeded gpio.h inclusion > > > staging: atomisp: Remove ->gpio_ctrl() callback > > > staging: atomisp: Remove ->power_ctrl() callback > > > staging: atomisp: Remove unused members of > > > camera_sensor_platform_data > > > staging: atomisp: Remove Gmin dead code #1 > > > staging: atomisp: Remove Gmin dead code #2 > > > staging: atomisp: Remove duplicate declaration in header > > > staging: atomisp: Remove FSF snail address > > > > After chatting with Andy we figured out the first patch was actually > > missing from the set, both on the mailing list and Patchwork. I've > > uploaded > > it to the same branch, and the patch itself is here: > > > > <URL:https://git.linuxtv.org/sailus/media_tree.git/commit/?h=atomisp-a > > ndy&id=5ef68fbdbb80e72f3239363289fbf12f673988a1>; > > > > Looks good, thanks! Thanks, Andy! I'll add these to the next atomisp pull request. -- Sakari Ailus e-mail: sakari.ailus@xxxxxx
