* [PATCH] staging: media: remove unused VIDEO_ATOMISP_OV8858 kconfig @ 2018-01-23 14:37 Corentin Labbe 2018-01-23 17:31 ` Andy Shevchenko 0 siblings, 1 reply; 6+ messages in thread From: Corentin Labbe @ 2018-01-23 14:37 UTC (permalink / raw) To: gregkh, mchehab; +Cc: devel, linux-kernel, linux-media, Corentin Labbe Nothing in kernel use VIDEO_ATOMISP_OV8858 since commit 3a81c7660f80 ("media: staging: atomisp: Remove IMX sensor support") Lets remove this kconfig option. Signed-off-by: Corentin Labbe <clabbe@baylibre.com> --- drivers/staging/media/atomisp/i2c/Kconfig | 12 ------------ 1 file changed, 12 deletions(-) diff --git a/drivers/staging/media/atomisp/i2c/Kconfig b/drivers/staging/media/atomisp/i2c/Kconfig index db054d3c7ed6..f7f7177b9b37 100644 --- a/drivers/staging/media/atomisp/i2c/Kconfig +++ b/drivers/staging/media/atomisp/i2c/Kconfig @@ -28,18 +28,6 @@ config VIDEO_ATOMISP_GC2235 It currently only works with the atomisp driver. -config VIDEO_ATOMISP_OV8858 - tristate "Omnivision ov8858 sensor support" - depends on ACPI - depends on I2C && VIDEO_V4L2 && VIDEO_ATOMISP - ---help--- - This is a Video4Linux2 sensor-level driver for the Omnivision - ov8858 RAW sensor. - - OV8858 is a 8M raw sensor. - - It currently only works with the atomisp driver. - config VIDEO_ATOMISP_MSRLIST_HELPER tristate "Helper library to load, parse and apply large register lists." depends on I2C -- 2.13.6 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] staging: media: remove unused VIDEO_ATOMISP_OV8858 kconfig 2018-01-23 14:37 [PATCH] staging: media: remove unused VIDEO_ATOMISP_OV8858 kconfig Corentin Labbe @ 2018-01-23 17:31 ` Andy Shevchenko 2018-01-23 18:20 ` Greg Kroah-Hartman 2018-01-24 8:35 ` Dan Carpenter 0 siblings, 2 replies; 6+ messages in thread From: Andy Shevchenko @ 2018-01-23 17:31 UTC (permalink / raw) To: Corentin Labbe Cc: devel, Greg Kroah-Hartman, Mauro Carvalho Chehab, Linux Kernel Mailing List, Linux Media Mailing List On Tue, Jan 23, 2018 at 4:37 PM, Corentin Labbe <clabbe@baylibre.com> wrote: > Nothing in kernel use VIDEO_ATOMISP_OV8858 since commit 3a81c7660f80 ("media: staging: atomisp: Remove IMX sensor support") > Lets remove this kconfig option. First of all, I hardly understand how that change is related. Second, did you check Makefile? Third, the files are in the folder (for OV8858). Taking all above into account, it seems NACK, though regression might be made by renaming patch from Sakari (no, it's not). So, it looks like it was never enabled in the first place. Anyway, do you have hardware to test? This is *most* important reason why to accept or decline a change to the driver. -- With Best Regards, Andy Shevchenko _______________________________________________ devel mailing list devel@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] staging: media: remove unused VIDEO_ATOMISP_OV8858 kconfig 2018-01-23 17:31 ` Andy Shevchenko @ 2018-01-23 18:20 ` Greg Kroah-Hartman 2018-01-26 12:49 ` LABBE Corentin 2018-01-24 8:35 ` Dan Carpenter 1 sibling, 1 reply; 6+ messages in thread From: Greg Kroah-Hartman @ 2018-01-23 18:20 UTC (permalink / raw) To: Andy Shevchenko Cc: devel, Mauro Carvalho Chehab, Linux Media Mailing List, Corentin Labbe, Linux Kernel Mailing List On Tue, Jan 23, 2018 at 07:31:27PM +0200, Andy Shevchenko wrote: > On Tue, Jan 23, 2018 at 4:37 PM, Corentin Labbe <clabbe@baylibre.com> wrote: > > Nothing in kernel use VIDEO_ATOMISP_OV8858 since commit 3a81c7660f80 ("media: staging: atomisp: Remove IMX sensor support") > > Lets remove this kconfig option. > > First of all, I hardly understand how that change is related. > Second, did you check Makefile? I don't see it being used in any Makefile, what file do you see it: $ git grep VIDEO_ATOMISP_OV8858 drivers/staging/media/atomisp/i2c/Kconfig:config VIDEO_ATOMISP_OV8858 So it should be removed. thanks, greg k-h _______________________________________________ devel mailing list devel@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] staging: media: remove unused VIDEO_ATOMISP_OV8858 kconfig 2018-01-23 18:20 ` Greg Kroah-Hartman @ 2018-01-26 12:49 ` LABBE Corentin 2018-01-26 16:25 ` Andy Shevchenko 0 siblings, 1 reply; 6+ messages in thread From: LABBE Corentin @ 2018-01-26 12:49 UTC (permalink / raw) To: Greg Kroah-Hartman Cc: devel, Andy Shevchenko, Mauro Carvalho Chehab, Linux Kernel Mailing List, Linux Media Mailing List On Tue, Jan 23, 2018 at 07:20:12PM +0100, Greg Kroah-Hartman wrote: > On Tue, Jan 23, 2018 at 07:31:27PM +0200, Andy Shevchenko wrote: > > On Tue, Jan 23, 2018 at 4:37 PM, Corentin Labbe <clabbe@baylibre.com> wrote: > > > Nothing in kernel use VIDEO_ATOMISP_OV8858 since commit 3a81c7660f80 ("media: staging: atomisp: Remove IMX sensor support") > > > Lets remove this kconfig option. > > > > First of all, I hardly understand how that change is related. > > Second, did you check Makefile? > > I don't see it being used in any Makefile, what file do you see it: > $ git grep VIDEO_ATOMISP_OV8858 > drivers/staging/media/atomisp/i2c/Kconfig:config VIDEO_ATOMISP_OV8858 > > So it should be removed. > > thanks, > Hello I just see that 3a81c7660f80 left ov8858.c so do you agree that I send a v2 which remove also this file ? av8858.c is useless without dw9718.c and vcm.c which 3a81c7660f80 removed. Regards _______________________________________________ devel mailing list devel@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] staging: media: remove unused VIDEO_ATOMISP_OV8858 kconfig 2018-01-26 12:49 ` LABBE Corentin @ 2018-01-26 16:25 ` Andy Shevchenko 0 siblings, 0 replies; 6+ messages in thread From: Andy Shevchenko @ 2018-01-26 16:25 UTC (permalink / raw) To: LABBE Corentin Cc: Greg Kroah-Hartman, devel, Mauro Carvalho Chehab, Linux Kernel Mailing List, Linux Media Mailing List On Fri, Jan 26, 2018 at 2:49 PM, LABBE Corentin <clabbe@baylibre.com> wrote: > On Tue, Jan 23, 2018 at 07:20:12PM +0100, Greg Kroah-Hartman wrote: >> On Tue, Jan 23, 2018 at 07:31:27PM +0200, Andy Shevchenko wrote: >> > On Tue, Jan 23, 2018 at 4:37 PM, Corentin Labbe <clabbe@baylibre.com> wrote: >> > > Nothing in kernel use VIDEO_ATOMISP_OV8858 since commit 3a81c7660f80 ("media: staging: atomisp: Remove IMX sensor support") >> > > Lets remove this kconfig option. >> > >> > First of all, I hardly understand how that change is related. >> > Second, did you check Makefile? >> >> I don't see it being used in any Makefile, what file do you see it: >> $ git grep VIDEO_ATOMISP_OV8858 >> drivers/staging/media/atomisp/i2c/Kconfig:config VIDEO_ATOMISP_OV8858 >> >> So it should be removed. >> >> thanks, > I just see that 3a81c7660f80 left ov8858.c so do you agree that I send a v2 which remove also this file ? > av8858.c is useless without dw9718.c and vcm.c which 3a81c7660f80 removed. Removal of dead code is pretty fine to me, though the decision is to Alan and Sakari. -- With Best Regards, Andy Shevchenko ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] staging: media: remove unused VIDEO_ATOMISP_OV8858 kconfig 2018-01-23 17:31 ` Andy Shevchenko 2018-01-23 18:20 ` Greg Kroah-Hartman @ 2018-01-24 8:35 ` Dan Carpenter 1 sibling, 0 replies; 6+ messages in thread From: Dan Carpenter @ 2018-01-24 8:35 UTC (permalink / raw) To: Andy Shevchenko Cc: Corentin Labbe, devel, Greg Kroah-Hartman, Mauro Carvalho Chehab, Linux Kernel Mailing List, Linux Media Mailing List On Tue, Jan 23, 2018 at 07:31:27PM +0200, Andy Shevchenko wrote: > On Tue, Jan 23, 2018 at 4:37 PM, Corentin Labbe <clabbe@baylibre.com> wrote: > > Nothing in kernel use VIDEO_ATOMISP_OV8858 since commit 3a81c7660f80 ("media: staging: atomisp: Remove IMX sensor support") > > Lets remove this kconfig option. > > First of all, I hardly understand how that change is related. It's pretty obvious if you look at the commit. -obj-$(CONFIG_VIDEO_ATOMISP_OV8858) += atomisp-ov8858.o It sounds like you deleted that line by mistake and re-added it to your local Makefile? regards, dan carpenter ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2018-01-26 16:25 UTC | newest] Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2018-01-23 14:37 [PATCH] staging: media: remove unused VIDEO_ATOMISP_OV8858 kconfig Corentin Labbe 2018-01-23 17:31 ` Andy Shevchenko 2018-01-23 18:20 ` Greg Kroah-Hartman 2018-01-26 12:49 ` LABBE Corentin 2018-01-26 16:25 ` Andy Shevchenko 2018-01-24 8:35 ` Dan Carpenter
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).