[PATCH V2 2/5] include/video: Add Exynos5 specific FIMD register offsets
Sylwester Nawrocki
s.nawrocki at samsung.com
Tue Jul 31 09:03:47 EDT 2012
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
More information about the linux-arm-kernel
mailing list