* Stephen Rothwell ([EMAIL PROTECTED]) wrote:
> 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.

I've changed the patch to ifdef out CMO for legacy iSeries.  This should
keep things cleaner.

> 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) {

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

Reply via email to