RE: [PATCH v2 6/6] TI816X: Add low level debug support

2010-10-19 Thread Pedanekar, Hemant
Tony Lindgren wrote on Friday, September 17, 2010 3:58 AM:

 * Hemant Pedanekar hema...@ti.com [100811 10:04]:
 This patch adds support for low level debugging on TI816X boards.
 Currently the support for UART3 console on TI816X EVM is added.
 
 --- a/arch/arm/mach-omap2/include/mach/debug-macro.S
 +++ b/arch/arm/mach-omap2/include/mach/debug-macro.S
 @@ -66,6 +66,12 @@ omap_uart_lsr:.word   0
  beq 34f @ configure OMAP3UART4
  cmp \rx, #OMAP4UART4@ only on 44xx
  beq 44f @ configure OMAP4UART4
 +cmp \rx, #TI816XUART1   @ ti816x UART offsets different
 +beq 81f @ configure UART1
 +cmp \rx, #TI816XUART2   @ ti816x UART offsets different
 +beq 82f @ configure UART2
 +cmp \rx, #TI816XUART3   @ ti816x UART offsets different
 +beq 83f @ configure UART3
  cmp \rx, #ZOOM_UART @ only on zoom2/3
  beq 95f @ configure ZOOM_UART
 
 
 Would be nice to have the TI816XUART defines in this patch too
 instead of defining them in some other patch earlier.
 
 Tony

Can you please refer to the patch 
@https://patchwork.kernel.org/patch/118855/

The file arch/arm/plat-omap/include/plat/serial.h which has these defined, 
is part of this patch.

Thanks
-
Hemant

--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v2 6/6] TI816X: Add low level debug support

2010-09-16 Thread Tony Lindgren
* Hemant Pedanekar hema...@ti.com [100811 10:04]:
 This patch adds support for low level debugging on TI816X boards. Currently 
 the
 support for UART3 console on TI816X EVM is added.

 --- a/arch/arm/mach-omap2/include/mach/debug-macro.S
 +++ b/arch/arm/mach-omap2/include/mach/debug-macro.S
 @@ -66,6 +66,12 @@ omap_uart_lsr: .word   0
   beq 34f @ configure OMAP3UART4
   cmp \rx, #OMAP4UART4@ only on 44xx
   beq 44f @ configure OMAP4UART4
 + cmp \rx, #TI816XUART1   @ ti816x UART offsets different
 + beq 81f @ configure UART1
 + cmp \rx, #TI816XUART2   @ ti816x UART offsets different
 + beq 82f @ configure UART2
 + cmp \rx, #TI816XUART3   @ ti816x UART offsets different
 + beq 83f @ configure UART3
   cmp \rx, #ZOOM_UART @ only on zoom2/3
   beq 95f @ configure ZOOM_UART
  

Would be nice to have the TI816XUART defines in this patch too
instead of defining them in some other patch earlier.

Tony
--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html