[PATCH V2 2/2] ARM: SAMSUNG: Cleanup resources by using macro

Arnd Bergmann wrote:

On Monday 03 October 2011 12:41:22 Kukjin Kim wrote:

quoted

This patch adds macro SAMSUNG_RES_MEM, SAMSUNG_RES_IRQ
and so on to cleanup regarding 'struct resource' by using
defined helpers at <linux/ioport.h>.

Cc: Ben Dooks <ben-linux@fluff.org>
Signed-off-by: Kukjin Kim <redacted>

When we introduced those macros, it was specifically so that platforms
would be migrated to use them instead of providing their own.

quoted

index 8f19241..a506831 100644

--- a/arch/arm/plat-samsung/include/plat/devs.h
+++ b/arch/arm/plat-samsung/include/plat/devs.h

@@ -18,6 +18,17 @@

 #include <linux/platform_device.h>

+#define SAMSUNG_RES_MEM(soc, ip, sz)

DEFINE_RES_MEM(soc##_PA_##ip, sz)

quoted

+#define SAMSUNG_RES_IRQ(ip)            DEFINE_RES_IRQ(IRQ_##ip)
+
+#define SAMSUNG_RES_MEM_NAMED(soc, ip, sz, name)       \
+

DEFINE_RES_MEM_NAMED(soc##_PA_##ip, sz, name)

quoted

+#define SAMSUNG_RES_IRQ_NAMED(ip, name)                        \
+                               DEFINE_RES_IRQ_NAMED(IRQ_##ip,

name)

quoted

+#define SAMSUNG_RES_DMA_NAMED(ch, name)

\

quoted

+

DEFINE_RES_DMA_NAMED(DMACH_##ch, name)

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.

Thanks.

Best regards,
Kgene.
--
Kukjin Kim [off-list ref], Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.