Hi Kukjin,
your commit a1bd8d8bb890 ("ARM: SAMSUNG: make local setup-camif in
mach-s3c24xx") is in today's linux-next tree (i.e., next-20150731) and
it adds the following lines of code with
arch/arm/mach-s3c24xx/setup-camif.c:
+#ifdef CONFIG_ARCH_S3C24XX
+ *gpio_start = S3C2410_GPJ(0);
+ *gpio_reset = S3C2410_GPJ(12);
+#else
+ /* s3c64xx */
+ *gpio_start = S3C64XX_GPF(0);
+ *gpio_reset = S3C64XX_GPF(3);
+#endif
The problem is that the #else block cannot be compiled at the current
state, since the #ifdef block will _always_ be selected by the C
preprocessor due to some build constraints from Makefiles, see
arch/arm/Makefile line 196:
machine-$(CONFIG_ARCH_S3C24XX)
Hence CONFIG_ARCH_S3C24XX will always be set when compiling
arch/arm/mach-s3c24xx/setup-camif.c.
I detected this issue with undertaker-checkpatch from [1].
Kind regards,
Valentin
[1] https://undertaker.cs.fau.de