On Fri, Sep 18, 2009 at 06:13,  <[email protected]> wrote:
> @@ -1331,8 +1331,8 @@  bfin_earlyserial_init
>       bfin_serial_console.flags &= ~CON_PRINTBUFFER;
>  #endif
>
> -     if (port == -1 || port >= nr_active_ports)
> -             port = 0;
> +     if (port == -1 || port >= nr_ports)
> +             return -ENODEV;

this change is incorrect.  the early init func returns a pointer, not
an errno.  so this should return NULL.
-mike
_______________________________________________
Linux-kernel-commits mailing list
[email protected]
https://blackfin.uclinux.org/mailman/listinfo/linux-kernel-commits

Reply via email to