Hi Mark,

On Thu, 25 Oct 2007 16:39:48 -0700 "Mark A. Greer" [off-list ref] 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                    sfr@canb.auug.org.au
http://www.canb.auug.org.au/~sfr/