Kevin Hilman <khilman@xxxxxxxxxxxxxxxxxxx> writes:
> Vikram Pandita <vikram.pandita@xxxxxx> 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@xxxxxx>
>> cc: Kevin Hilman <khilman@xxxxxxxxxxxxxxxxxxx>
>> ---
>> 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@xxxxxx>
> 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@xxxxxx>
> Signed-off-by: Kevin Hilman <khilman@xxxxxxxxxxxxxxxxxxx>
>
> 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
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
