On Wed, Jun 22, 2011 at 7:24 PM, Segher Boessenkool <seg...@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 > > _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev