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/

Attachment: pgpSga2dWmVUl.pgp
Description: PGP signature

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

Reply via email to