On 10/3/07, Peter Korsgaard <[EMAIL PROTECTED]> wrote: > >>>>> "Grant" == Grant Likely <[EMAIL PROTECTED]> writes: > > Hi, > > Grant> Why not just: > > Grant> np = of_find_node_by_type(NULL, "qe"); > Grant> + if (!np) > Grant> + np = of_find_node_by_type(NULL, "soc"); > Grant> if (!np) > Grant> return -ENODEV; > > My first iteration did it like that, but then you don't get a -ENODEV > if the node is missing (and you'll end up using the wrong clock) and > it doesn't support processors with SPI on and off QE (if that > exists/will ever exist).
Okay, but you should at least be able confine your determination of which sysclk value to use to one part of the function. Otherwise, it looks good. Cheers, g. -- Grant Likely, B.Sc., P.Eng. Secret Lab Technologies Ltd. [EMAIL PROTECTED] (403) 399-0195 _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailman/listinfo/linuxppc-dev
