Hi Mark, On Thu, 25 Oct 2007 16:39:48 -0700 "Mark A. Greer" <[EMAIL PROTECTED]> wrote: > > +static int __init prpmc2800_register_mtd(void) > +{ > + struct device_node *np = NULL; ^^^^^^^ Not needed if you use for_each_compatible_node().
> + > + while ((np = of_find_compatible_node(np, NULL, "cfi-flash")) != NULL) for_each_compatible_node(np, NULL, "cfi-flash") > + of_platform_device_create(np, NULL, NULL); > + > + of_node_put(np); Not needed as np must be NULL when you get here. -- Cheers, Stephen Rothwell [EMAIL PROTECTED] http://www.canb.auug.org.au/~sfr/
pgpSga2dWmVUl.pgp
Description: PGP signature
_______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailman/listinfo/linuxppc-dev