Thread (13 messages) flat view 13 messages, 3 authors, 2013-08-21
DORMANTno replies

Revision v5 of 11 in this series.

Revisions (11)
  1. v1 [diff vs current]
  2. v2 [diff vs current]
  3. v2 [diff vs current]
  4. v2 [diff vs current]
  5. v3 [diff vs current]
  6. v4 [diff vs current]
  7. v5 [diff vs current]
  8. v5 [diff vs current]
  9. v5 current
  10. v6 [diff vs current]
  11. v8 [diff vs current]

[PATCH v5 1/8] usb: phy: omap-control: Get rid of platform data

From: Roger Quadros <hidden>
Date: 2013-08-21 06:46:29
Also in: linux-omap

Hi Thomas,

On 08/20/2013 06:04 PM, thomas.langer at lantiq.com wrote:
Hello Roger,

this will not work!

Roger Quadros wrote on 2013-08-20:
quoted
-	struct omap_control_usb_platform_data *pdata =
-			dev_get_platdata(&pdev->dev);
+
+	if (np) {
+		of_property_read_u32(np, "ti,type", &control_usb->type);
here you are dereferencing control_usb
quoted
+	} else {
+		/* We only support DT boot */
+		return -EINVAL;
+	}

 	control_usb = devm_kzalloc(&pdev->dev, sizeof(*control_usb),
 		GFP_KERNEL);
and here it is allocated!
Good catch :).
The problem is eliminated by other patches in this series, so tests ran fine.
But I'll fix this one.

cheers,
-roger
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help