lkml.org 
[lkml]   [2005]   [Aug]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [patch] fix ucb1x00 support on collie
Hi!

> >
> > > 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.
>
> Ok, new set of patches on the ftp site with the above two items resolved.
> I've also moved it into drivers/mfd and linked that directory into
> kbuild.

Thanks, it seems to work okay here.

You still do:

Ser4MCCR0 = data->mccr0 | 0x7f7f;

...I'm not sure, but it seems to me I want MMCR0 to be set to
0x60000. Would it be better to move 7f7f constant (what is it, anyway)
to machine-specific code?

Pavel
--
if you have sharp zaurus hardware you don't need... you know my address
-
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/

\
 
 \ /
  Last update: 2005-08-12 13:14    [W:0.076 / U:0.116 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site