Dear Grant Likely,

In message <fa686aa41001270843u3b4e9687k699ad579de446...@mail.gmail.com> you 
wrote:
>
> > +       if (rev != 2) {
> > +               dev_err(dev, "SoC revision %u is not supported!\n", rev);
> > +               return -ENXIO;
> > +       }
>
> *Only* revision 2?  Are future revisions of silicon assumed to be broken then?

I vote for keeping it this way - if you look at the diffferences
between revision 1 and 2, or at differences between MPC5121/3 and
MPC5125, it is more than likely that revision 3, should it ever come
out, will be incompatible and require driver changes.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
Landru! Guide us!
        -- A Beta 3-oid, "The Return of the Archons", stardate 3157.4
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev

Reply via email to