> -----Original Message-----
> From: Hiremath, Vaibhav
> Sent: Saturday, November 05, 2011 3:11 PM
> To: Hiremath, Vaibhav; [email protected]
> Cc: Hilman, Kevin; [email protected]; [email protected]; linux-arm-
> [email protected]; Mohammed, Afzal
> Subject: RE: [PATCH-V3 2/4] arm:omap:am33xx: Update common OMAP machine
> specific sources
>
> > -----Original Message-----
> > From: Hiremath, Vaibhav
> > Sent: Tuesday, September 20, 2011 8:02 PM
> > To: [email protected]
> > Cc: Hilman, Kevin; [email protected]; [email protected]; linux-arm-
> > [email protected]; Mohammed, Afzal; Hiremath, Vaibhav
> > Subject: [PATCH-V3 2/4] arm:omap:am33xx: Update common OMAP machine
> > specific sources
> >
> > From: Afzal Mohammed <[email protected]>
> >
> > This patch updates the common machine specific source files for
> > support for AM33XX/AM335x with cpu type, macros for identification of
> > AM33XX/AM335X device.
> >
> > Signed-off-by: Afzal Mohammed <[email protected]>
> > Signed-off-by: Vaibhav Hiremath <[email protected]>
> > ---
> > arch/arm/mach-omap2/clock3xxx_data.c | 6 +++++-
> > arch/arm/mach-omap2/common.c | 16 ++++++++++++++++
> > arch/arm/mach-omap2/id.c | 10 ++++++++--
> > arch/arm/mach-omap2/io.c | 25
> > +++++++++++++++++++++++++
> > arch/arm/mach-omap2/serial.c | 6 +++---
> > arch/arm/plat-omap/include/plat/am33xx.h | 25
> > +++++++++++++++++++++++++
> > arch/arm/plat-omap/include/plat/common.h | 1 +
> > arch/arm/plat-omap/include/plat/hardware.h | 1 +
> > arch/arm/plat-omap/include/plat/io.h | 20 ++++++++++++++++++++
> > arch/arm/plat-omap/include/plat/omap34xx.h | 2 ++
> > arch/arm/plat-omap/io.c | 5 +++++
> > 11 files changed, 111 insertions(+), 6 deletions(-)
> > create mode 100644 arch/arm/plat-omap/include/plat/am33xx.h
> >
<snip>
> > diff --git a/arch/arm/mach-omap2/serial.c b/arch/arm/mach-omap2/serial.c
> > index 466fc72..b7782ee 100644
> > --- a/arch/arm/mach-omap2/serial.c
> > +++ b/arch/arm/mach-omap2/serial.c
> > @@ -486,7 +486,7 @@ static void omap_uart_idle_init(struct
> omap_uart_state
> > *uart)
> > mod_timer(&uart->timer, jiffies + uart->timeout);
> > omap_uart_smart_idle_enable(uart, 0);
> >
> > - if (cpu_is_omap34xx() && !cpu_is_ti816x()) {
> > + if (cpu_is_omap34xx() && !(cpu_is_ti816x() || cpu_is_am33xx())) {
> > u32 mod = (uart->num > 1) ? OMAP3430_PER_MOD : CORE_MOD;
> > u32 wk_mask = 0;
> > u32 padconf = 0;
> > @@ -768,7 +768,7 @@ void __init omap_serial_init_port(struct
> > omap_board_data *bdata)
> > */
> > uart->regshift = p->regshift;
> > uart->membase = p->membase;
> > - if (cpu_is_omap44xx() || cpu_is_ti816x())
> > + if (cpu_is_omap44xx() || cpu_is_ti816x() || cpu_is_am33xx())
> > uart->errata |= UART_ERRATA_FIFO_FULL_ABORT;
> > else if ((serial_read_reg(uart, UART_OMAP_MVER) & 0xFF)
> > >= UART_OMAP_NO_EMPTY_FIFO_READ_IP_REV)
> > @@ -851,7 +851,7 @@ void __init omap_serial_init_port(struct
> > omap_board_data *bdata)
> > }
>
> Can I get rid of this if condition as well here with below change -
>
> - * omap44xx, ti816x: Never read empty UART fifo
> + * all >omap3 family of devices: Never read empty UART fifo
> * omap3xxx: Never read empty UART fifo on UARTs
> * with IP rev >=0x52
> */
> uart->regshift = p->regshift;
> uart->membase = p->membase;
> - if (cpu_is_omap44xx() || cpu_is_ti816x() || cpu_is_am33xx())
> - uart->errata |= UART_ERRATA_FIFO_FULL_ABORT;
> - else if ((serial_read_reg(uart, UART_OMAP_MVER) & 0xFF)
> - >= UART_OMAP_NO_EMPTY_FIFO_READ_IP_REV)
> +
> + if ((serial_read_reg(uart, UART_OMAP_MVER) & 0xFF)
> + < UART_OMAP_NO_EMPTY_FIFO_READ_IP_REV)
> + uart->errata &= ~UART_ERRATA_FIFO_FULL_ABORT;
> + else
> uart->errata |= UART_ERRATA_FIFO_FULL_ABORT;
>
> if (uart->errata & UART_ERRATA_FIFO_FULL_ABORT) {
>
Hit send button bit early, realized that, on all other devices (like AM33xx)
the rev id is not incremental order, it has been reseted again, sp right
condition here would be -
- if (cpu_is_omap44xx() || cpu_is_ti816x() || cpu_is_am33xx())
- uart->errata |= UART_ERRATA_FIFO_FULL_ABORT;
- else if ((serial_read_reg(uart, UART_OMAP_MVER) & 0xFF)
- >= UART_OMAP_NO_EMPTY_FIFO_READ_IP_REV)
+
+ if (cpu_is_omap34xx() && (serial_read_reg(uart, UART_OMAP_MVER) & 0xFF)
+ < UART_OMAP_NO_EMPTY_FIFO_READ_IP_REV)
+ uart->errata &= ~UART_ERRATA_FIFO_FULL_ABORT;
+ else
uart->errata |= UART_ERRATA_FIFO_FULL_ABORT;
Thanks,
Vaibhav
>
> Thanks,
> Vaibhav
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
