Re: [PATCH] staging: media: remove unused VIDEO_ATOMISP_OV8858 kconfig

From: Andy Shevchenko
Date: Tue Jan 23 2018 - 12:31:34 EST


On Tue, Jan 23, 2018 at 4:37 PM, Corentin Labbe <clabbe@xxxxxxxxxxxx> 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