* Tony Lindgren <t...@atomide.com> [091020 15:15]:
> * ladislav.mi...@seznam.cz <ladislav.mi...@seznam.cz> [091020 14:53]:
> > On Fri, Oct 09, 2009 at 06:46:23PM -0400, Tony Lindgren wrote:
> > > This patch has been applied to the linux-omap
> > > by youw fwiendly patch wobot.
> > >
> > > Branch in linux-omap: omap2-upstream
> > >
> > > Initial commit ID (Likely to change):
> > > 439d2c69335a28ffdb5a9795ff384b6755ca0f7f
> > >
> > > PatchWorks
> > > http://patchwork.kernel.org/patch/52477/
> > >
> > > Git (Likely to change, and takes a while to get mirrored)
> > > http://git.kernel.org/?p=linux/kernel/git/tmlind/linux-omap-2.6.git;a=commit;h=439d2c69335a28ffdb5a9795ff384b6755ca0f7f
> >
> > This patch broke all OMAP1 boards. NULL terminator entry for omap1
> > serial_platform_data cannot be removed just because serial driver uses
> > is as last entry marker. Without it we end this way:
> > Unable to handle kernel NULL pointer dereference at virtual address 00000028
> > pgd = c0004000
> > [00000028] *pgd=00000000
> > Internal error: Oops: 80000005 [#1]
> > last sysfs file:
> > Modules linked in:
> > CPU: 0 Not tainted (2.6.32-rc5-06314-g4155da6-dirty #12)
> > PC is at 0x28
> > LR is at serial8250_config_port+0x184/0xc34
> > (...etc...)
> >
> > Please consider following fix (and while there fix OMAP2 too as patch broke
> > it as well (untested))
>
> Thanks, I already refreshed the original patch with the same fix few
> days ago :) It should be there in for-next branch and master branch.
Correction, sorry looks like I did not really read your patch. It seems
to be the right solution for mach-omap1, but not needed for mach-omap2
because the array is not plat_serial8250_port on mach-omap2.
> Tony
>
> > Signed-off-by: Ladislav Michl <la...@linux-mips.org>
> >
> > diff --git a/arch/arm/mach-omap1/serial.c b/arch/arm/mach-omap1/serial.c
> > index 9c59332..0392ff5 100644
> > --- a/arch/arm/mach-omap1/serial.c
> > +++ b/arch/arm/mach-omap1/serial.c
> > @@ -86,7 +86,9 @@ static struct plat_serial8250_port serial_platform_data[]
> > = {
> > .iotype = UPIO_MEM,
> > .regshift = 2,
> > .uartclk = OMAP16XX_BASE_BAUD * 16,
> > - },
> > + }, {
> > + }
> > +
> > };
> >
> > static struct platform_device serial_device = {
> > @@ -119,7 +121,7 @@ void __init omap_serial_init(void)
> > serial_platform_data[2].uartclk = OMAP1510_BASE_BAUD * 16;
> > }
> >
> > - for (i = 0; i < ARRAY_SIZE(serial_platform_data); i++) {
> > + for (i = 0; i < ARRAY_SIZE(serial_platform_data) - 1; i++) {
> > unsigned char reg;
> >
> > /* Static mapping, never released */
> > diff --git a/arch/arm/mach-omap2/serial.c b/arch/arm/mach-omap2/serial.c
> > index dabc089..ca69ffa 100644
> > --- a/arch/arm/mach-omap2/serial.c
> > +++ b/arch/arm/mach-omap2/serial.c
> > @@ -585,7 +585,7 @@ void __init omap_serial_early_init(void)
> > * if not needed.
> > */
> >
> > - for (i = 0; i < ARRAY_SIZE(omap_uart); i++) {
> > + for (i = 0; i < ARRAY_SIZE(omap_uart) - 1; i++) {
> > struct omap_uart_state *uart = &omap_uart[i];
> > struct platform_device *pdev = &uart->pdev;
> > struct device *dev = &pdev->dev;
> > @@ -637,7 +637,7 @@ void __init omap_serial_init(void)
> > {
> > int i;
> >
> > - for (i = 0; i < ARRAY_SIZE(omap_uart); i++) {
> > + for (i = 0; i < ARRAY_SIZE(omap_uart) - 1; i++) {
> > struct omap_uart_state *uart = &omap_uart[i];
> > struct platform_device *pdev = &uart->pdev;
> > struct device *dev = &pdev->dev;
> --
> 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
--
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
