Skip to contentSkip to conversation

ARM: fix mach-bcmring UART declarations

PATCH New Paul Gortmaker 3 participants 4 messages Apr 16, 2012 – Apr 20, 2012 1 message not in the archive

Subsystems (2)
lore
Share & tools
lore
#0 lore raw

PATCH 1/1 ARM: fix mach-bcmring UART declarations

This error appeared in the bcmring_defconfig build:

CC arch/arm/mach-bcmring/core.o
arch/arm/mach-bcmring/core.c:55: error: macro "AMBA_APB_DEVICE" requires 6 arguments, but only 5 given arch/arm/mach-bcmring/core.c:55: warning: type defaults to 'int' in declaration of 'AMBA_APB_DEVICE' arch/arm/mach-bcmring/core.c:56: error: macro "AMBA_APB_DEVICE" requires 6 arguments, but only 5 given arch/arm/mach-bcmring/core.c:56: warning: type defaults to 'int' in declaration of 'AMBA_APB_DEVICE' arch/arm/mach-bcmring/core.c:134: error: 'uartA_device' undeclared here (not in a function) arch/arm/mach-bcmring/core.c:135: error: 'uartB_device' undeclared here (not in a function) make[2]: *** [arch/arm/mach-bcmring/core.o] Error 1

It appeared as of commit 8ede1ae65e61282cddba39bde4142be3885a6f5a

"ARM: amba: bcmring: use common amba device initializers"

Note that in include/linux/amba/bus.h we have:
#define AMBA_APB_DEVICE(name, busid, id, base, irqs, data) ...

There is an a --> A case error in the busid and a missing zero placeholder for the id field.

arch/arm/mach-bcmring/core.c +2 -2
index 22e4e0a..adbfb19 100644
@@ -52,8 +52,8 @@
 #include <mach/csp/chipcHw_inline.h>
 #include <mach/csp/tmrHw_reg.h>
 
-static AMBA_APB_DEVICE(uartA, "uarta", MM_ADDR_IO_UARTA, { IRQ_UARTA }, NULL);
-static AMBA_APB_DEVICE(uartB, "uartb", MM_ADDR_IO_UARTB, { IRQ_UARTB }, NULL);
+static AMBA_APB_DEVICE(uartA, "uartA", 0, MM_ADDR_IO_UARTA, {IRQ_UARTA}, NULL);
+static AMBA_APB_DEVICE(uartB, "uartB", 0, MM_ADDR_IO_UARTB, {IRQ_UARTB}, NULL);
 
 static struct clk pll1_clk = {
 	.name = "PLL1",
Signature · 1 line
1.7.9.1

A message that is not in the archive (sent off-list or not yet received): [email protected]

#2 lore raw

PATCH 1/1 ARM: fix mach-bcmring UART declarations

Quoted 2 lines · Jiandong Zheng
Quoted 1 line · Paul Gortmaker

This error appeared in the bcmring_defconfig build:

[...]

Quoted 1 line

This is already fixed by Olof Johansson.

Thanks, I guess it just hasn't made it into linux-next yet.

Paul.

Quoted 2 lines
#3 lore raw

PATCH 1/1 ARM: fix mach-bcmring UART declarations

Quoted 9 lines · Paul Gortmaker
Quoted 2 lines · Jiandong Zheng
Quoted 1 line · Paul Gortmaker

This error appeared in the bcmring_defconfig build:

[...]

Quoted 1 line

This is already fixed by Olof Johansson.

Thanks, I guess it just hasn't made it into linux-next yet.

Actually, I didn't fix the bus id case error, so I'll apply your version. I could just fix up mine the same way, but git am -s is quicker. :)

-Olof

#4 lore raw

PATCH RESEND 1/1 ARM: BCMRING: Add debug macro

Add debug macro to bcmring so that it supports LL debug and early printk.

---
arch/arm/mach-bcmring/include/mach/debug-macro.S | 19 +++++++++++++++++++

  1 files changed, 19 insertions(+), 0 deletions(-)

create mode 100755 arch/arm/mach-bcmring/include/mach/debug-macro.S

(quoted diff) +0 -0

b/arch/arm/mach-bcmring/include/mach/debug-macro.S new file mode 100755
index 0000000..fae01fd

arch/arm/mach-bcmring/include/mach/debug-macro.S new file +19 -0
@@ -0,0 +1,19 @@
+/* arch/arm/mach-bcmring/include/mach/debug-macro.S
+ *
+ * Debugging macro include header
+ *
+ *  Based on arch/arm/mach-versatile/include/mach/debug-macro.S
+ *  by Jiandong Zheng
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2 as
+ * published by the Free Software Foundation.
+ *
+*/
+        .macro    addruart, rp, rv, tmp
+        mov    \rp,      #0x00012000
+        orr    \rv, \rp, #0xf8000000    @ virtual base
+        orr    \rp, \rp, #0x80000000    @ physical base
+        .endm
+
+#include <asm/hardware/debug-pl01x.S>
Signature · 1 line
1.7.9.1