[lkml]   [2005]   [Jul]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    Patch in this message
    SubjectRe: [patch 2.6.13-git] 8250 tweaks
    On Wed, Jul 13, 2005 at 03:51:26AM -0700, Sam Song wrote:
    > Russell King <> wrote:
    > > However, if you merely lifted the later 8250.c and
    > > put it into a previous kernel (which looks like the
    > > case), there's other changes in addition which are
    > > required.
    > Good catch. I tried 2.6.13-rc1 and the newest version
    > 2.6.13-rc3 on the same target[MPC8241]. The whining
    > remained the same.

    v. whining

    1. To utter a plaintive, high-pitched, protracted sound, as in pain,
    fear, supplication, or complaint.
    2. To complain or protest in a childish fashion.
    3. To produce a sustained noise of relatively high pitch: jet engines

    The kernel isn't doing any of those.

    Anyway, you're going to have to help me out a lot - I don't know a
    thing about PPC, so I don't know what a MPC8241 is. I don't know
    your kernel configuration (could you send it please?) so I don't
    know which files are trying to be built.

    Also, having the contents of /sys/devices/platform or
    /sys/bus/platform/* would be useful.

    For some reason, it appears that the serial driver is being asked to
    register two serial ports at MMIO address 0, from one platform device,
    which it apparantly detects as being present. I suspect these are
    coming from some table included via asm-ppc/serial.h, but where that
    is I've no idea.

    It's then asked to add two more ports from the serial8250.0 device,
    which doesn't exist. These come from a platform device in arch/ppc.
    Again, where these come from I don't know.

    So. The serial driver is being asked to create _four_ ports. It's
    created two, but can't create the other two, failing with error -22.
    22 is EINVAL, which means there was something wrong with what was
    requested. That generally points to uartclk being zero, which would
    be a bug in the PPC architecture code. You can confirm this by
    applying this patch:

    diff --git a/drivers/serial/8250.c b/drivers/serial/8250.c
    --- a/drivers/serial/8250.c
    +++ b/drivers/serial/8250.c
    @@ -2333,6 +2333,7 @@ static int __devinit serial8250_probe(st
    dev_err(dev, "unable to register port at index %d "
    "(IO%lx MEM%lx IRQ%d): %d\n", i,
    p->iobase, p->mapbase, p->irq, ret);
    + printk(KERN_ERR "uartclk was %d\n", port.uartclk);
    return 0;
    (For those cogito fanatics, the above simple patch took 1min 22sec for
    cg-diff to spit out!)

    Russell King
    Linux kernel 2.6 ARM Linux -
    maintainer of: 2.6 Serial core
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to
    More majordomo info at
    Please read the FAQ at

     \ /
      Last update: 2005-07-13 14:54    [W:0.027 / U:16.424 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site