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 _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel
