Hi, On 07/31/2012 02:51 PM, Sergei Shtylyov wrote: >> +/*EXYNOS5 FIMD REG OFFSET */ How about changing it to, e.g. /* EXYNOS5 specific register offset definitions */ (but just in case you happen to resend this patch series) >> +#define EXYNOS5_VIDTCON0 (0x20010) >> +#define EXYNOS5_VIDTCON1 (0x20014) >> +#define EXYNOS5_VIDTCON2 (0x20018) >> +#define EXYNOS5_VIDTCON3 (0x2001C) >> +#define EXYNOS5_VIDCON1 (0x20004) > > Parens not needed around simple literals. Yes, I agree it looks insane. But, I think we could allow this for consistency with the remaining definitions. -- Regards, Sylwester -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html