Re: [PATCH v1 8/9] usb: phy: dts: Adding usbphy DT bindings for am33xx

From: Sergei Shtylyov
Date: Thu May 23 2013 - 14:18:01 EST


Hello.

On 05/23/2013 09:13 PM, B, Ravi wrote:


+ phy1: usbphy-gs70@44e10620 {
+ compatible = "ti,dsps-usbphy";
+ reg = <0x44e10620 0x8
+ 0x44e10648 0x4>;
+ reg-names = "phy_ctrl","phy_wkup";
+ id = <0>;
+ };
+
+ phy2: usbphy-gs70@44e10628 {
+ compatible = "ti,dsps-usbphy";
+ reg = <0x44e10628 0x8
+ 0x44e10648 0x4>;
The second register conflicts with phy1.
The two instances of phy uses common phy wakeup register.

That's why there is a resource conflict. Have you actually tried to instantiate the devices out of such tree?
This register should be declared somewhere above the PHYs I think...

--
Ravi B

WBR, Sergei

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/