* [U-Boot] [PATCH V5] ARM: Various: Future-proof serial platdata
@ 2016-03-02 11:38 Adam Ford
2016-03-02 15:15 ` Alexander Graf
` (3 more replies)
0 siblings, 4 replies; 5+ messages in thread
From: Adam Ford @ 2016-03-02 11:38 UTC (permalink / raw)
To: u-boot
A few boards still use ns16550_platdata structures, but assume the structure
is going to be in a specific order. By explicitly naming each entry,
this should also help 'future-proof' in the event the structure changes.
Tested on the Logic PD Torpedo + Wireless.
I only changed a handful of devices that used the same syntax as the Logic
board. Appologies if I missed one or stepped on toes. Thanks to Derald Woods
and Alexander Graf.
Signed-off-by: Adam Ford <aford173@gmail.com>
V5: Add fix to arch/arm/cpu/arm926ejs/lpc32xx/devices.c
V4: Fix subject heading
V3: Remove reg_offset out in all the structs. It was reverted out, and and if
it did exist, it would get initialized to 0 by default.
V2: I hastily copy-pasted the boards without looking at the UART number.
This addresses 3 boards that use UART3 and not UART1.
---
arch/arm/cpu/arm926ejs/lpc32xx/devices.c | 8 ++++----
board/isee/igep00x0/igep00x0.c | 6 +++---
board/logicpd/omap3som/omap3logic.c | 6 +++---
board/logicpd/zoom1/zoom1.c | 6 +++---
board/overo/overo.c | 6 +++---
board/quipos/cairo/cairo.c | 6 +++---
board/ti/beagle/beagle.c | 6 +++---
board/timll/devkit8000/devkit8000.c | 6 +++---
8 files changed, 25 insertions(+), 25 deletions(-)
diff --git a/arch/arm/cpu/arm926ejs/lpc32xx/devices.c b/arch/arm/cpu/arm926ejs/lpc32xx/devices.c
index b6db23e..399b07c 100644
--- a/arch/arm/cpu/arm926ejs/lpc32xx/devices.c
+++ b/arch/arm/cpu/arm926ejs/lpc32xx/devices.c
@@ -45,10 +45,10 @@ void lpc32xx_uart_init(unsigned int uart_id)
#if !CONFIG_IS_ENABLED(OF_CONTROL)
static const struct ns16550_platdata lpc32xx_uart[] = {
- { UART3_BASE, 2, CONFIG_SYS_NS16550_CLK },
- { UART4_BASE, 2, CONFIG_SYS_NS16550_CLK },
- { UART5_BASE, 2, CONFIG_SYS_NS16550_CLK },
- { UART6_BASE, 2, CONFIG_SYS_NS16550_CLK },
+ { .base = UART3_BASE, .reg_shift = 2, .clock = CONFIG_SYS_NS16550_CLK },
+ { .base = UART4_BASE, .reg_shift = 2, .clock = CONFIG_SYS_NS16550_CLK },
+ { .base = UART5_BASE, .reg_shift = 2, .clock = CONFIG_SYS_NS16550_CLK },
+ { .base = UART6_BASE, .reg_shift = 2, .clock = CONFIG_SYS_NS16550_CLK },
};
#if defined(CONFIG_LPC32XX_HSUART)
diff --git a/board/isee/igep00x0/igep00x0.c b/board/isee/igep00x0/igep00x0.c
index e2fce50..d1a6a6f 100644
--- a/board/isee/igep00x0/igep00x0.c
+++ b/board/isee/igep00x0/igep00x0.c
@@ -34,9 +34,9 @@ static const u32 gpmc_lan_config[] = {
#endif
static const struct ns16550_platdata igep_serial = {
- OMAP34XX_UART3,
- 2,
- V_NS16550_CLK
+ .base = OMAP34XX_UART3,
+ .reg_shift = 2,
+ .clock = V_NS16550_CLK
};
U_BOOT_DEVICE(igep_uart) = {
diff --git a/board/logicpd/omap3som/omap3logic.c b/board/logicpd/omap3som/omap3logic.c
index 668f684..51d2987 100644
--- a/board/logicpd/omap3som/omap3logic.c
+++ b/board/logicpd/omap3som/omap3logic.c
@@ -47,9 +47,9 @@ DECLARE_GLOBAL_DATA_PTR;
*/
static const struct ns16550_platdata omap3logic_serial = {
- OMAP34XX_UART1,
- 2,
- V_NS16550_CLK
+ .base = OMAP34XX_UART1,
+ .reg_shift = 2,
+ .clock = V_NS16550_CLK
};
U_BOOT_DEVICE(omap3logic_uart) = {
diff --git a/board/logicpd/zoom1/zoom1.c b/board/logicpd/zoom1/zoom1.c
index 4040114..982619c 100644
--- a/board/logicpd/zoom1/zoom1.c
+++ b/board/logicpd/zoom1/zoom1.c
@@ -44,9 +44,9 @@ static const u32 gpmc_lab_enet[] = {
};
static const struct ns16550_platdata zoom1_serial = {
- OMAP34XX_UART3,
- 2,
- V_NS16550_CLK
+ .base = OMAP34XX_UART3,
+ .reg_shift = 2,
+ .clock = V_NS16550_CLK
};
U_BOOT_DEVICE(zoom1_uart) = {
diff --git a/board/overo/overo.c b/board/overo/overo.c
index a38b959..40f13e5 100644
--- a/board/overo/overo.c
+++ b/board/overo/overo.c
@@ -68,9 +68,9 @@ static struct {
} expansion_config = {0x0};
static const struct ns16550_platdata overo_serial = {
- OMAP34XX_UART3,
- 2,
- V_NS16550_CLK
+ .base = OMAP34XX_UART3,
+ .reg_shift = 2,
+ .clock = V_NS16550_CLK
};
U_BOOT_DEVICE(overo_uart) = {
diff --git a/board/quipos/cairo/cairo.c b/board/quipos/cairo/cairo.c
index 21793e8..77e4482 100644
--- a/board/quipos/cairo/cairo.c
+++ b/board/quipos/cairo/cairo.c
@@ -91,9 +91,9 @@ void get_board_mem_timings(struct board_sdrc_timings *timings)
#endif
static const struct ns16550_platdata cairo_serial = {
- OMAP34XX_UART2,
- 2,
- V_NS16550_CLK
+ .base = OMAP34XX_UART2,
+ .reg_shift = 2,
+ .clock = V_NS16550_CLK
};
U_BOOT_DEVICE(cairo_uart) = {
diff --git a/board/ti/beagle/beagle.c b/board/ti/beagle/beagle.c
index ff317ef..0ed4f52 100644
--- a/board/ti/beagle/beagle.c
+++ b/board/ti/beagle/beagle.c
@@ -73,9 +73,9 @@ static struct {
} expansion_config;
static const struct ns16550_platdata beagle_serial = {
- OMAP34XX_UART3,
- 2,
- V_NS16550_CLK
+ .base = OMAP34XX_UART3,
+ .reg_shift = 2,
+ .clock = V_NS16550_CLK
};
U_BOOT_DEVICE(beagle_uart) = {
diff --git a/board/timll/devkit8000/devkit8000.c b/board/timll/devkit8000/devkit8000.c
index 1a447c7..965252f 100644
--- a/board/timll/devkit8000/devkit8000.c
+++ b/board/timll/devkit8000/devkit8000.c
@@ -46,9 +46,9 @@ static u32 gpmc_net_config[GPMC_MAX_REG] = {
};
static const struct ns16550_platdata devkit8000_serial = {
- OMAP34XX_UART3,
- 2,
- V_NS16550_CLK
+ .base = OMAP34XX_UART3,
+ .reg_shift = 2,
+ .clock = V_NS16550_CLK
};
U_BOOT_DEVICE(devkit8000_uart) = {
--
1.9.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [U-Boot] [PATCH V5] ARM: Various: Future-proof serial platdata
2016-03-02 11:38 [U-Boot] [PATCH V5] ARM: Various: Future-proof serial platdata Adam Ford
@ 2016-03-02 15:15 ` Alexander Graf
2016-03-02 22:11 ` Vladimir Zapolskiy
` (2 subsequent siblings)
3 siblings, 0 replies; 5+ messages in thread
From: Alexander Graf @ 2016-03-02 15:15 UTC (permalink / raw)
To: u-boot
> Am 02.03.2016 um 12:38 schrieb Adam Ford <aford173@gmail.com>:
>
> A few boards still use ns16550_platdata structures, but assume the structure
> is going to be in a specific order. By explicitly naming each entry,
> this should also help 'future-proof' in the event the structure changes.
>
> Tested on the Logic PD Torpedo + Wireless.
>
> I only changed a handful of devices that used the same syntax as the Logic
> board. Appologies if I missed one or stepped on toes. Thanks to Derald Woods
> and Alexander Graf.
>
> Signed-off-by: Adam Ford <aford173@gmail.com>
Reviewed-by: Alexander Graf <agraf@suse.de>
Alex
^ permalink raw reply [flat|nested] 5+ messages in thread
* [U-Boot] [PATCH V5] ARM: Various: Future-proof serial platdata
2016-03-02 11:38 [U-Boot] [PATCH V5] ARM: Various: Future-proof serial platdata Adam Ford
2016-03-02 15:15 ` Alexander Graf
@ 2016-03-02 22:11 ` Vladimir Zapolskiy
2016-03-05 17:44 ` Tom Rini
2016-03-07 2:38 ` Simon Glass
3 siblings, 0 replies; 5+ messages in thread
From: Vladimir Zapolskiy @ 2016-03-02 22:11 UTC (permalink / raw)
To: u-boot
Hi Adam,
On 02.03.2016 13:38, Adam Ford wrote:
> A few boards still use ns16550_platdata structures, but assume the structure
> is going to be in a specific order. By explicitly naming each entry,
> this should also help 'future-proof' in the event the structure changes.
>
> Tested on the Logic PD Torpedo + Wireless.
>
> I only changed a handful of devices that used the same syntax as the Logic
> board. Appologies if I missed one or stepped on toes. Thanks to Derald Woods
> and Alexander Graf.
>
> Signed-off-by: Adam Ford <aford173@gmail.com>
>
> V5: Add fix to arch/arm/cpu/arm926ejs/lpc32xx/devices.c
>
> V4: Fix subject heading
>
> V3: Remove reg_offset out in all the structs. It was reverted out, and and if
> it did exist, it would get initialized to 0 by default.
>
> V2: I hastily copy-pasted the boards without looking at the UART number.
> This addresses 3 boards that use UART3 and not UART1.
> ---
> arch/arm/cpu/arm926ejs/lpc32xx/devices.c | 8 ++++----
> board/isee/igep00x0/igep00x0.c | 6 +++---
> board/logicpd/omap3som/omap3logic.c | 6 +++---
> board/logicpd/zoom1/zoom1.c | 6 +++---
> board/overo/overo.c | 6 +++---
> board/quipos/cairo/cairo.c | 6 +++---
> board/ti/beagle/beagle.c | 6 +++---
> board/timll/devkit8000/devkit8000.c | 6 +++---
> 8 files changed, 25 insertions(+), 25 deletions(-)
>
> diff --git a/arch/arm/cpu/arm926ejs/lpc32xx/devices.c b/arch/arm/cpu/arm926ejs/lpc32xx/devices.c
> index b6db23e..399b07c 100644
> --- a/arch/arm/cpu/arm926ejs/lpc32xx/devices.c
> +++ b/arch/arm/cpu/arm926ejs/lpc32xx/devices.c
> @@ -45,10 +45,10 @@ void lpc32xx_uart_init(unsigned int uart_id)
>
> #if !CONFIG_IS_ENABLED(OF_CONTROL)
> static const struct ns16550_platdata lpc32xx_uart[] = {
> - { UART3_BASE, 2, CONFIG_SYS_NS16550_CLK },
> - { UART4_BASE, 2, CONFIG_SYS_NS16550_CLK },
> - { UART5_BASE, 2, CONFIG_SYS_NS16550_CLK },
> - { UART6_BASE, 2, CONFIG_SYS_NS16550_CLK },
> + { .base = UART3_BASE, .reg_shift = 2, .clock = CONFIG_SYS_NS16550_CLK },
> + { .base = UART4_BASE, .reg_shift = 2, .clock = CONFIG_SYS_NS16550_CLK },
> + { .base = UART5_BASE, .reg_shift = 2, .clock = CONFIG_SYS_NS16550_CLK },
> + { .base = UART6_BASE, .reg_shift = 2, .clock = CONFIG_SYS_NS16550_CLK },
> };
For LPC32xx part
Acked-by: Vladimir Zapolskiy <vz@mleia.com>
--
With best wishes,
Vladimir
^ permalink raw reply [flat|nested] 5+ messages in thread
* [U-Boot] [PATCH V5] ARM: Various: Future-proof serial platdata
2016-03-02 11:38 [U-Boot] [PATCH V5] ARM: Various: Future-proof serial platdata Adam Ford
2016-03-02 15:15 ` Alexander Graf
2016-03-02 22:11 ` Vladimir Zapolskiy
@ 2016-03-05 17:44 ` Tom Rini
2016-03-07 2:38 ` Simon Glass
3 siblings, 0 replies; 5+ messages in thread
From: Tom Rini @ 2016-03-05 17:44 UTC (permalink / raw)
To: u-boot
On Wed, Mar 02, 2016 at 05:38:18AM -0600, Adam Ford wrote:
> A few boards still use ns16550_platdata structures, but assume the structure
> is going to be in a specific order. By explicitly naming each entry,
> this should also help 'future-proof' in the event the structure changes.
>
> Tested on the Logic PD Torpedo + Wireless.
>
> I only changed a handful of devices that used the same syntax as the Logic
> board. Appologies if I missed one or stepped on toes. Thanks to Derald Woods
> and Alexander Graf.
>
> Signed-off-by: Adam Ford <aford173@gmail.com>
>
> V5: Add fix to arch/arm/cpu/arm926ejs/lpc32xx/devices.c
>
> V4: Fix subject heading
>
> V3: Remove reg_offset out in all the structs. It was reverted out, and and if
> it did exist, it would get initialized to 0 by default.
>
> V2: I hastily copy-pasted the boards without looking at the UART number.
> This addresses 3 boards that use UART3 and not UART1.
Reviewed-by: Tom Rini <trini@konsulko.com>
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160305/c0ec6dc5/attachment.sig>
^ permalink raw reply [flat|nested] 5+ messages in thread
* [U-Boot] [PATCH V5] ARM: Various: Future-proof serial platdata
2016-03-02 11:38 [U-Boot] [PATCH V5] ARM: Various: Future-proof serial platdata Adam Ford
` (2 preceding siblings ...)
2016-03-05 17:44 ` Tom Rini
@ 2016-03-07 2:38 ` Simon Glass
3 siblings, 0 replies; 5+ messages in thread
From: Simon Glass @ 2016-03-07 2:38 UTC (permalink / raw)
To: u-boot
On 2 March 2016 at 04:38, Adam Ford <aford173@gmail.com> wrote:
> A few boards still use ns16550_platdata structures, but assume the structure
> is going to be in a specific order. By explicitly naming each entry,
> this should also help 'future-proof' in the event the structure changes.
>
> Tested on the Logic PD Torpedo + Wireless.
>
> I only changed a handful of devices that used the same syntax as the Logic
> board. Appologies if I missed one or stepped on toes. Thanks to Derald Woods
> and Alexander Graf.
>
> Signed-off-by: Adam Ford <aford173@gmail.com>
>
> V5: Add fix to arch/arm/cpu/arm926ejs/lpc32xx/devices.c
>
> V4: Fix subject heading
>
> V3: Remove reg_offset out in all the structs. It was reverted out, and and if
> it did exist, it would get initialized to 0 by default.
>
> V2: I hastily copy-pasted the boards without looking at the UART number.
> This addresses 3 boards that use UART3 and not UART1.
> ---
> arch/arm/cpu/arm926ejs/lpc32xx/devices.c | 8 ++++----
> board/isee/igep00x0/igep00x0.c | 6 +++---
> board/logicpd/omap3som/omap3logic.c | 6 +++---
> board/logicpd/zoom1/zoom1.c | 6 +++---
> board/overo/overo.c | 6 +++---
> board/quipos/cairo/cairo.c | 6 +++---
> board/ti/beagle/beagle.c | 6 +++---
> board/timll/devkit8000/devkit8000.c | 6 +++---
> 8 files changed, 25 insertions(+), 25 deletions(-)
Reviewed-by: Simon Glass <sjg@chromium.org>
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2016-03-07 2:38 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-03-02 11:38 [U-Boot] [PATCH V5] ARM: Various: Future-proof serial platdata Adam Ford
2016-03-02 15:15 ` Alexander Graf
2016-03-02 22:11 ` Vladimir Zapolskiy
2016-03-05 17:44 ` Tom Rini
2016-03-07 2:38 ` Simon Glass
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.