* [PATCH 4/5] OMAP: Zoom2: Fix serial break
@ 2009-08-21 18:11 Vikram Pandita
2009-08-22 14:46 ` Kevin Hilman
0 siblings, 1 reply; 3+ messages in thread
From: Vikram Pandita @ 2009-08-21 18:11 UTC (permalink / raw)
To: linux-omap, linux-arm-kernel; +Cc: Vikram Pandita, Kevin Hilman
Fix the break in Zoom2 Quad uart debug port support.
Patch by Kevin Hilman fd455ea899b5a14 caused a break as it introuduced
per port PLATFORM id and Zoom2 was not tested with this change.
For Zoom2 serial port, add platform id as 4
Signed-off-by: Vikram Pandita <vikram.pandita@ti.com>
cc: Kevin Hilman <khilman@deeprootsystems.com>
---
arch/arm/mach-omap2/board-zoom-debugboard.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/arch/arm/mach-omap2/board-zoom-debugboard.c b/arch/arm/mach-omap2/board-zoom-debugboard.c
index f8b0726..4dd714d 100644
--- a/arch/arm/mach-omap2/board-zoom-debugboard.c
+++ b/arch/arm/mach-omap2/board-zoom-debugboard.c
@@ -97,7 +97,7 @@ static struct plat_serial8250_port serial_platform_data[] = {
static struct platform_device zoom2_debugboard_serial_device = {
.name = "serial8250",
- .id = PLAT8250_DEV_PLATFORM1,
+ .id = PLAT8250_DEV_FOURPORT,
.dev = {
.platform_data = serial_platform_data,
},
--
1.6.3.3.334.g916e1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH 4/5] OMAP: Zoom2: Fix serial break
2009-08-21 18:11 [PATCH 4/5] OMAP: Zoom2: Fix serial break Vikram Pandita
@ 2009-08-22 14:46 ` Kevin Hilman
2009-08-24 11:50 ` Kevin Hilman
0 siblings, 1 reply; 3+ messages in thread
From: Kevin Hilman @ 2009-08-22 14:46 UTC (permalink / raw)
To: Vikram Pandita; +Cc: linux-omap, linux-arm-kernel
Vikram Pandita <vikram.pandita@ti.com> writes:
> Fix the break in Zoom2 Quad uart debug port support.
>
> Patch by Kevin Hilman fd455ea899b5a14 caused a break as it introuduced
> per port PLATFORM id and Zoom2 was not tested with this change.
...because working zoom2 UART support was not yet upstream.
> For Zoom2 serial port, add platform id as 4
>
> Signed-off-by: Vikram Pandita <vikram.pandita@ti.com>
> cc: Kevin Hilman <khilman@deeprootsystems.com>
> ---
> arch/arm/mach-omap2/board-zoom-debugboard.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/arch/arm/mach-omap2/board-zoom-debugboard.c b/arch/arm/mach-omap2/board-zoom-debugboard.c
> index f8b0726..4dd714d 100644
> --- a/arch/arm/mach-omap2/board-zoom-debugboard.c
> +++ b/arch/arm/mach-omap2/board-zoom-debugboard.c
> @@ -97,7 +97,7 @@ static struct plat_serial8250_port serial_platform_data[] = {
>
> static struct platform_device zoom2_debugboard_serial_device = {
> .name = "serial8250",
> - .id = PLAT8250_DEV_PLATFORM1,
> + .id = PLAT8250_DEV_FOURPORT,
Using FOURPORT isn't right either. Just use a number.
If you're OK with the version below, I'll pull it into my fixes queue.
I've also updated the changelog to explain the problem instead of just
"fix the break..." This fix messages are not useful when looking back
through the changelog history.
I don't currently have access to a Zoom2, so this has not been tested.
Kevin
commit c52bc1989540d1305aa5372a73bc6096636c8432
Author: Vikram Pandita <vikram.pandita@ti.com>
Date: Fri Aug 21 13:11:06 2009 -0500
OMAP: Zoom2: update serial platform_data id for external UART
Now that on-chip UARTs each have separate platform_data, the external
UART needs an non-conflicting ID. Since there are 3 on-chip UARTs,
the Zoom2 external UART will be registered after as the fourth.
Signed-off-by: Vikram Pandita <vikram.pandita@ti.com>
Signed-off-by: Kevin Hilman <khilman@deeprootsystems.com>
diff --git a/arch/arm/mach-omap2/board-zoom-debugboard.c b/arch/arm/mach-omap2/board-zoom-debugboard.c
index bac5c43..66fe940 100644
--- a/arch/arm/mach-omap2/board-zoom-debugboard.c
+++ b/arch/arm/mach-omap2/board-zoom-debugboard.c
@@ -94,7 +94,7 @@ static struct plat_serial8250_port serial_platform_data[] = {
static struct platform_device zoom2_debugboard_serial_device = {
.name = "serial8250",
- .id = PLAT8250_DEV_PLATFORM1,
+ .id = 3,
.dev = {
.platform_data = serial_platform_data,
},
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH 4/5] OMAP: Zoom2: Fix serial break
2009-08-22 14:46 ` Kevin Hilman
@ 2009-08-24 11:50 ` Kevin Hilman
0 siblings, 0 replies; 3+ messages in thread
From: Kevin Hilman @ 2009-08-24 11:50 UTC (permalink / raw)
To: Vikram Pandita; +Cc: linux-omap, linux-arm-kernel
Kevin Hilman <khilman@deeprootsystems.com> writes:
> Vikram Pandita <vikram.pandita@ti.com> writes:
>
>> Fix the break in Zoom2 Quad uart debug port support.
>>
>> Patch by Kevin Hilman fd455ea899b5a14 caused a break as it introuduced
>> per port PLATFORM id and Zoom2 was not tested with this change.
>
> ...because working zoom2 UART support was not yet upstream.
>
>> For Zoom2 serial port, add platform id as 4
>>
>> Signed-off-by: Vikram Pandita <vikram.pandita@ti.com>
>> cc: Kevin Hilman <khilman@deeprootsystems.com>
>> ---
>> arch/arm/mach-omap2/board-zoom-debugboard.c | 2 +-
>> 1 files changed, 1 insertions(+), 1 deletions(-)
>>
>> diff --git a/arch/arm/mach-omap2/board-zoom-debugboard.c b/arch/arm/mach-omap2/board-zoom-debugboard.c
>> index f8b0726..4dd714d 100644
>> --- a/arch/arm/mach-omap2/board-zoom-debugboard.c
>> +++ b/arch/arm/mach-omap2/board-zoom-debugboard.c
>> @@ -97,7 +97,7 @@ static struct plat_serial8250_port serial_platform_data[] = {
>>
>> static struct platform_device zoom2_debugboard_serial_device = {
>> .name = "serial8250",
>> - .id = PLAT8250_DEV_PLATFORM1,
>> + .id = PLAT8250_DEV_FOURPORT,
>
> Using FOURPORT isn't right either. Just use a number.
>
> If you're OK with the version below, I'll pull it into my fixes queue.
> I've also updated the changelog to explain the problem instead of just
> "fix the break..." This fix messages are not useful when looking back
> through the changelog history.
>
> I don't currently have access to a Zoom2, so this has not been tested.
>
> Kevin
>
>
> commit c52bc1989540d1305aa5372a73bc6096636c8432
> Author: Vikram Pandita <vikram.pandita@ti.com>
> Date: Fri Aug 21 13:11:06 2009 -0500
>
> OMAP: Zoom2: update serial platform_data id for external UART
>
> Now that on-chip UARTs each have separate platform_data, the external
> UART needs an non-conflicting ID. Since there are 3 on-chip UARTs,
> the Zoom2 external UART will be registered after as the fourth.
>
> Signed-off-by: Vikram Pandita <vikram.pandita@ti.com>
> Signed-off-by: Kevin Hilman <khilman@deeprootsystems.com>
>
> diff --git a/arch/arm/mach-omap2/board-zoom-debugboard.c b/arch/arm/mach-omap2/board-zoom-debugboard.c
> index bac5c43..66fe940 100644
> --- a/arch/arm/mach-omap2/board-zoom-debugboard.c
> +++ b/arch/arm/mach-omap2/board-zoom-debugboard.c
> @@ -94,7 +94,7 @@ static struct plat_serial8250_port serial_platform_data[] = {
>
> static struct platform_device zoom2_debugboard_serial_device = {
> .name = "serial8250",
> - .id = PLAT8250_DEV_PLATFORM1,
> + .id = 3,
> .dev = {
> .platform_data = serial_platform_data,
> },
Queuing this in my fixes queue for the next merge window.
Kevin
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2009-08-24 11:50 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-08-21 18:11 [PATCH 4/5] OMAP: Zoom2: Fix serial break Vikram Pandita
2009-08-22 14:46 ` Kevin Hilman
2009-08-24 11:50 ` Kevin Hilman
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.