[PATCH] powerpc: Fix cell's mmio nvram to properly parse dt

Benjamin Herrenschmidt benh at kernel.crashing.org
Thu Jan 11 16:08:41 EST 2007


The mmio nvram driver (used by cell only atm) isn't properly parsing the
device-tree. It works ok for old blades where the nvram is at the root
of the DT but fails on Malta and CAB when it's hanging off axon. This
fixes it by using the proper OF parsing functions.

Signed-off-by: Benjamin Herrenschmidt <benh at kernel.crashing.org>
---

It should be good for 2.6.20 but if not, I won't complain.

Index: linux-work/arch/powerpc/sysdev/mmio_nvram.c
===================================================================
--- linux-work.orig/arch/powerpc/sysdev/mmio_nvram.c	2007-01-02 11:12:52.000000000 +1100
+++ linux-work/arch/powerpc/sysdev/mmio_nvram.c	2007-01-10 15:34:11.000000000 +1100
@@ -80,33 +80,39 @@ static ssize_t mmio_nvram_get_size(void)
 int __init mmio_nvram_init(void)
 {
 	struct device_node *nvram_node;
-	const unsigned long *buffer;
-	int proplen;
 	unsigned long nvram_addr;
+	struct resource r;
 	int ret;
 
-	ret = -ENODEV;
 	nvram_node = of_find_node_by_type(NULL, "nvram");
-	if (!nvram_node)
+	if (!nvram_node) {
+		printk(KERN_WARNING "nvram: no node found in device-tree\n");
+		return -ENODEV;
+	}
+
+	ret = of_address_to_resource(nvram_node, 0, &r);
+	if (ret) {
+		printk(KERN_WARNING "nvram: failed to get address (err %d)\n",
+		       ret);
 		goto out;
-
-	ret = -EIO;
-	buffer = get_property(nvram_node, "reg", &proplen);
-	if (proplen != 2*sizeof(unsigned long))
-		goto out;
-
-	ret = -ENODEV;
-	nvram_addr = buffer[0];
-	mmio_nvram_len = buffer[1];
-	if ( (!mmio_nvram_len) || (!nvram_addr) )
+	}
+	nvram_addr = r.start;
+	mmio_nvram_len = r.end - r.start + 1;
+	if ( (!mmio_nvram_len) || (!nvram_addr) ) {
+		printk(KERN_WARNING "nvram: address or lenght is 0\n");
+		ret = -EIO;
 		goto out;
+	}
 
 	mmio_nvram_start = ioremap(nvram_addr, mmio_nvram_len);
-	if (!mmio_nvram_start)
+	if (!mmio_nvram_start) {
+		printk(KERN_WARNING "nvram: failed to ioremap\n");
+		ret = -ENOMEM;
 		goto out;
+	}
 
-	printk(KERN_INFO "mmio NVRAM, %luk mapped to %p\n",
-	       mmio_nvram_len >> 10, mmio_nvram_start);
+	printk(KERN_INFO "mmio NVRAM, %luk at 0x%lx mapped to %p\n",
+	       mmio_nvram_len >> 10, nvram_addr, mmio_nvram_start);
 
 	ppc_md.nvram_read	= mmio_nvram_read;
 	ppc_md.nvram_write	= mmio_nvram_write;





More information about the Linuxppc-dev mailing list