[PATCH V2 2/2] ARM: SAMSUNG: Cleanup resources by using macro
Kukjin Kim
kgene.kim at samsung.com
Tue Oct 4 21:17:21 EDT 2011
Arnd Bergmann wrote:
>
> On Tuesday 04 October 2011, Kukjin Kim wrote:
> > > I think the string concatenation really just obfuscates the code, and
> > > it does not actually save much at all. When you replace
> > >
> > > + [0] = SAMSUNG_RES_MEM(S3C, WDT, SZ_1K),
> > > + [1] = SAMSUNG_RES_IRQ(WDT),
> > >
> > > with
> > >
> > > + [0] = DEFINE_RES_MEM(S3C_PA_WDT, SZ_1K),
> > > + [1] = DEFINE_RES_IRQ(IRQ_WDT),
> > >
> > > you need practically no extra space, but you gain the advantages that
> > >
> > > * Someone using grep for DEFINE_RES_MEM finds all memory resources
> without
> > > having to look up what your macros do an where they are used.
> > > * Someone using grep to look for S3C_PA_WDT finds the place where it
is
> > used.
> > > * Someone reading the resource definition immediately knows what the
> > > macro does if familiar with other platforms using that macro.
> >
> > Yes, right. But I'm preparing to reduce the 'soc' part to consolidate
some
> > duplicated resources and platform data after this and the new
SAMSUNG_RES
> > macro will be used.
>
> Hmm, can't you instead change the names of these constants to be
> always the same? That would let you use the regular DEFINE_RES_*
> definitions without having to introduce your own.
>
OK, I agree with you. And if introducing Samsung own is required, will do it
later.
Thanks.
Best regards,
Kgene.
--
Kukjin Kim <kgene.kim at samsung.com>, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.
More information about the linux-arm-kernel
mailing list