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 [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
