On 11.06.2012 20:29, Wolfram Sang wrote: > On Wed, May 30, 2012 at 11:43:05AM +0200, Karol Lewandowski wrote: >> This commit fixes warning introduced in 27452498a ("i2c-s3c2410: Rework >> device type handling"): >> >> drivers/i2c/busses/i2c-s3c2410.c: In function 's3c24xx_get_device_quirks': >> drivers/i2c/busses/i2c-s3c2410.c:125: warning: passing argument 1 of 'of_match_node' from incompatible pointer type >> include/linux/of.h:245: note: expected 'const struct of_device_id *' but argument is of type 'const struct of_device_id (*)[4]' >> >> Signed-off-by: Karol Lewandowski <k.lewandowsk@xxxxxxxxxxx> >> Signed-off-by: Kyungmin Park <kyungmin.park@xxxxxxxxxxx> > > Ah, now I understand why my build-test didn't spot the warning. > What config can I use to test CONFIG_OF with s3c? Currently no config for samsung hardware has OF enabled by default. However, you could use exynos4_defconfig plus following (that's what I use): --- a/arch/arm/configs/exynos4_defconfig +++ b/arch/arm/configs/exynos4_defconfig @@ -14,6 +14,7 @@ CONFIG_MACH_ORIGEN=y CONFIG_MACH_SMDK4412=y CONFIG_NO_HZ=y CONFIG_HIGH_RES_TIMERS=y +CONFIG_MACH_EXYNOS4_DT=y CONFIG_SMP=y CONFIG_NR_CPUS=2 CONFIG_PREEMPT=y @@ -37,6 +38,7 @@ CONFIG_SERIAL_SAMSUNG=y CONFIG_SERIAL_SAMSUNG_CONSOLE=y CONFIG_HW_RANDOM=y CONFIG_I2C=y +CONFIG_I2C_S3C2410=y # CONFIG_HWMON is not set # CONFIG_MFD_SUPPORT is not set # CONFIG_HID_SUPPORT is not set Regards, -- Karol Lewandowski | Samsung Poland R&D Center | Linux/Platform -- To unsubscribe from this list: send the line "unsubscribe linux-i2c" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html