Hi Jon, On Fri, Nov 11, 2011 at 5:14 AM, Jon Hunter <jon-hunter@xxxxxx> wrote: > Hi Govindraj, > > On 10/18/2011 10:26, Govindraj.R wrote: >> >> Move the errata handling mechanism from serial.c to omap-serial file >> and utilise the same func in driver file. >> >> Errata i202, i291 are moved to be handled with omap-serial >> Moving the errata macro from serial.c file to driver header file >> as from on errata will be handled in driver file itself. >> Corrected errata id from chapter reference 2.15 to errata id i291. >> >> Removed errata and dma_enabled feilds from omap_uart_state struct >> as they are no more needed with errata handling done within omap-serial. >> >> Acked-by: Alan Cox<alan@xxxxxxxxxxxxxxx> >> Signed-off-by: Govindraj.R<govindraj.raja@xxxxxx> >> --- >> arch/arm/mach-omap2/serial.c | 97 >> +++++++------------------ >> arch/arm/plat-omap/include/plat/omap-serial.h | 6 ++ >> drivers/tty/serial/omap-serial.c | 66 ++++++++++++++++- >> 3 files changed, 95 insertions(+), 74 deletions(-) > > [snip] > >> + /* Enable the MDR1 errata for OMAP3 */ >> + if (cpu_is_omap34xx()&& !cpu_is_ti816x()) >> + omap_up.errata |= UART_ERRATA_i202_MDR1_ACCESS; > > This errata (i202) is applicable to all devices from OMAP2430 to OMAP4460 > (apparently this one is not getting fixed). So could you make sure this is > enabled for 2430, 3430, 3630, 4430 and 4460? > > [snip] Thanks for the info, Have posted a updated the patch [1] > >> +/* >> + * Work Around for Errata i202 (3430 - 1.12, 3630 - 1.6) > > We should update the comment to be "Work Around for Errata i202 (2430, 3430, > 3630, 4430 and 4460)". The "1.12" and "1.6" are just the section references > in the errata docs, but I think that you can drop these as you can just > search for the errata ID which is i202. > yes corrected [1] -- Thanks, Govindraj.R [1]: http://www.spinics.net/lists/linux-serial/msg04841.html -- 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
