Hi Robert,

Firstly, can all this new stuff be ifdef'ed out if not needed as the
vio infrastructure is also used on legacy iSeries and this adds quite a
bit of stuff that won't ever be used there.

On Thu, 12 Jun 2008 17:19:59 -0500 Robert Jennings <[EMAIL PROTECTED]> wrote:
>
> +static int vio_cmo_num_OF_devs(void)
> +{
> +     struct device_node *node_vroot;
> +     int count = 0;
> +
> +     /*
> +      * Count the number of vdevice entries with an
> +      * ibm,my-dma-window OF property
> +      */
> +     node_vroot = of_find_node_by_name(NULL, "vdevice");
> +     if (node_vroot) {
> +             struct device_node *of_node;
> +             struct property *prop;
> +
> +             for (of_node = node_vroot->child; of_node != NULL;
> +                             of_node = of_node->sibling) {

Use:
                for_each_child_of_node(node_vroot, of_node) {


-- 
Cheers,
Stephen Rothwell                    [EMAIL PROTECTED]
http://www.canb.auug.org.au/~sfr/

Attachment: pgpU9rH40jVeP.pgp
Description: PGP signature

_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev

Reply via email to