Fwd: [PATCH] powerpc, 460gt: Add 460gt as compatible in the check for 460ex-compatible crypto

Mike Williams mikebwilliams at gmail.com
Fri Jun 24 00:21:44 EST 2011


On Wed, Jun 22, 2011 at 7:24 PM, Segher Boessenkool
<segher at kernel.crashing.org> wrote:
>> -       if (of_find_compatible_node(NULL, NULL, "amcc,ppc460ex-crypto")) {
>> +       if (of_find_compatible_node(NULL, NULL, "amcc,ppc460ex-crypto") ||
>> +           of_find_compatible_node(NULL, NULL, "amcc,ppc460gt-crypto")) {
>
> If the device is actually compatible, the device tree node should claim
> it is, and you do not need this code change.

That was actually my first instinct, however I tried to follow the
current convention in the glacier and canyonlands DTS files, which is
to set every device compatible to 460gt or 460ex, depending on the
processor. Many of the devices are identical between the two, since
they are variations of the same SoC, so which is the preferred method?
Follow the device tree convention and add the compatibility check in
the driver, or alter the device trees? I'll send another patch if it's
the latter.

Thanks,
Mike

>
>
> Segher
>
>


More information about the Linuxppc-dev mailing list