[RFC PATCH 1/3] Add StorCenter DTS first draft.
Jon Loeliger
jdl at jdl.com
Tue Jan 8 07:09:09 EST 2008
So, like, the other day "Grant Likely" mumbled:
> > +
> > + ranges = <80000000 80000000 70000000 /* pci mem space */
> > + fdf00000 fdf00000 00100000 /* EUMB */
> > + fe000000 fe000000 00c00000 /* pci i/o space */
> > + fec00000 fec00000 00300000 /* pci cfg regs */
> > + fef00000 fef00000 00100000>; /* pci iack */
>
> This doesn't look nice. On the other ppc boards, Kumar moved the pci
> bus node out of the soc node so the the internal register range is
> separate from the soc.
Oh crap. I forgot about that. Yes, you are right.
> I think it should look like this:
>
> {
> soc at fdf00000 {
> compatible = "fsl,mpc8241-immr";
> ranges = <0 fe000000 00100000>;
> serial at 4500 {
> blah....
> };
> blah....
> };
> pci at 800000000 {
> ranges = blah....
> };
> };
So, I'll definitely rework that part as in indicated
> > + clock-frequency = <d# 97553800>; /* Hz */
>
> That's kind of an odd number for clock frequency. Usually clock
> frequencies are *big* and *round*. :-)
And this one, according to Andy, is nice-n-measured right
off the crystal! I had 10M in there originally even.
But I'll double check that with him to be sure.
Thanks,
jdl
More information about the Linuxppc-dev
mailing list