On 12.08.2020 11:14, Arnd Bergmann wrote: > On Wed, Aug 12, 2020 at 9:59 AM Stephen Boyd <sboyd@xxxxxxxxxx> wrote: >> Quoting Krzysztof Kozlowski (2020-08-04 12:26:54) >>> The s3c_camif_gpio_put() declaration in >>> include/media/drv-intf/s3c_camif.h header was different than definition. >>> Fixing this allows to include that header to also fix the W=1 compile >>> warnings: >>> >>> arch/arm/mach-s3c24xx/setup-camif.c:28:5: warning: no previous prototype for 's3c_camif_gpio_get' [-Wmissing-prototypes] >>> 28 | int s3c_camif_gpio_get(void) >>> arch/arm/mach-s3c24xx/setup-camif.c:56:6: warning: no previous prototype for 's3c_camif_gpio_put' [-Wmissing-prototypes] >>> 56 | void s3c_camif_gpio_put(void) >> >> Maybe it should have been void all along? > > It seems there have never been any callers and the entire file > can just be removed, with the rest of that platform_data header > file moved to drivers/media/platform/s3c-camif/. Yes, it seems that patch never made it to mainline: https://git.linuxtv.org/snawrocki/media.git/commit/?h=testing/s3c-camif&id=355cbf835aff2aabf78390931cbbaa608af77967 I doubt there are still users of camera on the s3c2440 boards with current mainline kernels, if any at all, there are much better HW alternatives right now.
