Re: [patch] fix ucb1x00 support on collie

From: Russell King (rmk+lkml_at_arm.linux.org.uk)
Date: 07/31/05

  • Next message: Jar: "Re: [git patches] new wireless stuffs"
    Date:	Sun, 31 Jul 2005 16:42:45 +0100
    To: Pavel Machek <pavel@ucw.cz>
    
    

    On Sun, Jul 31, 2005 at 03:46:17PM +0200, Pavel Machek wrote:
    > Collie is slightly strange; it does not seem to have proper ucb1x00
    > ID. With this patch, basic ucb support seems to work and I can get
    > interrupts from battery.

    Out of interest, what ID does it appear to have?

    > diff --git a/drivers/misc/mcp-sa1100.c b/drivers/misc/mcp-sa1100.c
    > --- a/drivers/misc/mcp-sa1100.c
    > +++ b/drivers/misc/mcp-sa1100.c
    > @@ -149,7 +149,7 @@ static int mcp_sa1100_probe(struct devic
    > !machine_is_graphicsmaster() && !machine_is_lart() &&
    > !machine_is_omnimeter() && !machine_is_pfs168() &&
    > !machine_is_shannon() && !machine_is_simpad() &&
    > - !machine_is_yopy())
    > + !machine_is_yopy() && !machine_is_collie())

    I think it's about time we did something better with this, like only
    registering the platform device on those which use it.

    > @@ -181,7 +187,10 @@ static int mcp_sa1100_probe(struct devic
    >
    > Ser4MCSR = -1;
    > Ser4MCCR1 = 0;
    > - Ser4MCCR0 = 0x00007f7f | MCCR0_ADM;
    > + if (machine_is_collie())
    > + Ser4MCCR0 = MCCR0_ADM | MCCR0_ExtClk;
    > + else
    > + Ser4MCCR0 = 0x00007f7f | MCCR0_ADM;

    And this setup should probably be passed as part of the platform device
    data.

    -- 
    Russell King
     Linux kernel    2.6 ARM Linux   - http://www.arm.linux.org.uk/
     maintainer of:  2.6 Serial core
    -
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at  http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at  http://www.tux.org/lkml/
    

  • Next message: Jar: "Re: [git patches] new wireless stuffs"

    Relevant Pages

    • Re: [PATCH 0/3] Couple of sysfs patches
      ... >> Now that I can see the platform device interfaces multipling like rabbits, ... > it's almost just as much work to manage it as a normal platform device. ... here's a patch so you can see what I'm suggesting above. ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)
    • Re: [patch] fix ucb1x00 support on collie
      ... basic ucb support seems to work and I can get ... > registering the platform device on those which use it. ... the audio driver isn't expected to build without Nico's sa11x0 ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)
    • Re: [CFT] Move platform device to separate header
      ... Russell> Here's a patch which moves the platform device out of ... doesn't appear in your diffstat and as far as I can tell, ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)
    • Re: serial8250_init and platform_device
      ... Someone send me a patch:) ... Separate platform device name from platform device number such that ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)
    • [CFT] Move platform device to separate header
      ... Here's a patch which moves the platform device out of device.h and ... This patch is far too large for lkml ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)