| Message ID | 1398693512-5355-1-git-send-email-guido@vanguardiasur.com.ar |
|---|---|
| State | New |
| Headers | show |
On 4/28/2014 7:28 PM, Guido Martínez wrote: > Use phandles instead of unit adresses to reference usb and dma nodes. > This makes the DT more robust and readable. The series will give dtb build errors Error: arch/arm/boot/dts/am335x-bone-common.dtsi:186.2-15 syntax error FATAL ERROR: Unable to parse input tree Error: arch/arm/boot/dts/am335x-evmsk.dts:367.2-15 syntax error FATAL ERROR: Unable to parse input tree make[1]: *** [arch/arm/boot/dts/am335x-bone.dtb] Error 1 make[1]: *** Waiting for unfinished jobs.... Error: arch/arm/boot/dts/am335x-evm.dts:334.2-15 syntax error FATAL ERROR: Unable to parse input tree make[1]: *** [arch/arm/boot/dts/am335x-evmsk.dtb] Error 1 make[1]: *** [arch/arm/boot/dts/am335x-evm.dtb] Error 1 Error: arch/arm/boot/dts/am335x-igep0033.dtsi:204.2-15 syntax error FATAL ERROR: Unable to parse input tree make[1]: *** [arch/arm/boot/dts/am335x-base0033.dtb] Error 1 Error: arch/arm/boot/dts/am335x-bone-common.dtsi:186.2-15 syntax error FATAL ERROR: Unable to parse input tree make[1]: *** [arch/arm/boot/dts/am335x-boneblack.dtb] Error 1 make: *** [dtbs] Error 2 > > Signed-off-by: Guido Martínez <guido@vanguardiasur.com.ar> > --- > arch/arm/boot/dts/am335x-bone-common.dtsi | 12 ++++++------ > 1 file changed, 6 insertions(+), 6 deletions(-) > > diff --git a/arch/arm/boot/dts/am335x-bone-common.dtsi b/arch/arm/boot/dts/am335x-bone-common.dtsi > index 2e7d932..b95bc9a 100644 > --- a/arch/arm/boot/dts/am335x-bone-common.dtsi > +++ b/arch/arm/boot/dts/am335x-bone-common.dtsi > @@ -183,28 +183,28 @@ > &usb { > status = "okay"; > > - control@44e10620 { > + &usb_ctrl_mod { > status = "okay"; > }; > > - usb-phy@47401300 { > + &usb0_phy { > status = "okay"; > }; > > - usb-phy@47401b00 { > + &usb1_phy { > status = "okay"; > }; > > - usb@47401000 { > + &usb0 { > status = "okay"; > }; > > - usb@47401800 { > + &usb1 { > status = "okay"; > dr_mode = "host"; > }; > > - dma-controller@47402000 { > + &cppi41dma { > status = "okay"; > }; > }; Instead the patch should be diff --git a/arch/arm/boot/dts/am335x-bone-common.dtsi b/arch/arm/boot/dts/am335x-bone-common.dtsi index 2e7d932..06be20a 100644 --- a/arch/arm/boot/dts/am335x-bone-common.dtsi +++ b/arch/arm/boot/dts/am335x-bone-common.dtsi @@ -182,31 +182,30 @@ &usb { status = "okay"; +}; +&usb_ctrl_mod { + status = "okay"; +}; - control@44e10620 { - status = "okay"; - }; - - usb-phy@47401300 { - status = "okay"; - }; +&usb0_phy { + status = "okay"; +}; - usb-phy@47401b00 { - status = "okay"; - }; +&usb1_phy { + status = "okay"; +}; - usb@47401000 { - status = "okay"; - }; +&usb { + status = "okay"; +}; - usb@47401800 { - status = "okay"; - dr_mode = "host"; - }; +&usb1 { + status = "okay"; + dr_mode = "host"; +}; - dma-controller@47402000 { - status = "okay"; - }; +&cppi41dma { + status = "okay"; };
On Mon, Apr 28, 2014 at 08:11:33PM +0530, George Cherian wrote: > On 4/28/2014 7:28 PM, Guido Martínez wrote: > >Use phandles instead of unit adresses to reference usb and dma nodes. > >This makes the DT more robust and readable. > The series will give dtb build errors > Error: arch/arm/boot/dts/am335x-bone-common.dtsi:186.2-15 syntax error > FATAL ERROR: Unable to parse input tree > Error: arch/arm/boot/dts/am335x-evmsk.dts:367.2-15 syntax error > FATAL ERROR: Unable to parse input tree > make[1]: *** [arch/arm/boot/dts/am335x-bone.dtb] Error 1 > make[1]: *** Waiting for unfinished jobs.... > Error: arch/arm/boot/dts/am335x-evm.dts:334.2-15 syntax error > FATAL ERROR: Unable to parse input tree > make[1]: *** [arch/arm/boot/dts/am335x-evmsk.dtb] Error 1 > make[1]: *** [arch/arm/boot/dts/am335x-evm.dtb] Error 1 > Error: arch/arm/boot/dts/am335x-igep0033.dtsi:204.2-15 syntax error > FATAL ERROR: Unable to parse input tree > make[1]: *** [arch/arm/boot/dts/am335x-base0033.dtb] Error 1 > Error: arch/arm/boot/dts/am335x-bone-common.dtsi:186.2-15 syntax error > FATAL ERROR: Unable to parse input tree > make[1]: *** [arch/arm/boot/dts/am335x-boneblack.dtb] Error 1 > make: *** [dtbs] Error 2 > Terribly sorry about this. I'll send a v2. > > > >Signed-off-by: Guido Martínez <guido@vanguardiasur.com.ar> > >--- > > arch/arm/boot/dts/am335x-bone-common.dtsi | 12 ++++++------ > > 1 file changed, 6 insertions(+), 6 deletions(-) > > > >diff --git a/arch/arm/boot/dts/am335x-bone-common.dtsi b/arch/arm/boot/dts/am335x-bone-common.dtsi > >index 2e7d932..b95bc9a 100644 > >--- a/arch/arm/boot/dts/am335x-bone-common.dtsi > >+++ b/arch/arm/boot/dts/am335x-bone-common.dtsi > >@@ -183,28 +183,28 @@ > > &usb { > > status = "okay"; > >- control@44e10620 { > >+ &usb_ctrl_mod { > > status = "okay"; > > }; > >- usb-phy@47401300 { > >+ &usb0_phy { > > status = "okay"; > > }; > >- usb-phy@47401b00 { > >+ &usb1_phy { > > status = "okay"; > > }; > >- usb@47401000 { > >+ &usb0 { > > status = "okay"; > > }; > >- usb@47401800 { > >+ &usb1 { > > status = "okay"; > > dr_mode = "host"; > > }; > >- dma-controller@47402000 { > >+ &cppi41dma { > > status = "okay"; > > }; > > }; > Instead the patch should be > > diff --git a/arch/arm/boot/dts/am335x-bone-common.dtsi > b/arch/arm/boot/dts/am335x-bone-common.dtsi > index 2e7d932..06be20a 100644 > --- a/arch/arm/boot/dts/am335x-bone-common.dtsi > +++ b/arch/arm/boot/dts/am335x-bone-common.dtsi > @@ -182,31 +182,30 @@ > > &usb { > status = "okay"; > +}; > +&usb_ctrl_mod { > + status = "okay"; > +}; > > - control@44e10620 { > - status = "okay"; > - }; > - > - usb-phy@47401300 { > - status = "okay"; > - }; > +&usb0_phy { > + status = "okay"; > +}; > > - usb-phy@47401b00 { > - status = "okay"; > - }; > +&usb1_phy { > + status = "okay"; > +}; > > - usb@47401000 { > - status = "okay"; > - }; > +&usb { > + status = "okay"; > +}; > > - usb@47401800 { > - status = "okay"; > - dr_mode = "host"; > - }; > +&usb1 { > + status = "okay"; > + dr_mode = "host"; > +}; > > - dma-controller@47402000 { > - status = "okay"; > - }; > +&cppi41dma { > + status = "okay"; > }; > > > > -- > -George >
On Mon, Apr 28, 2014 at 10:58:29AM -0300, Guido Martínez wrote: > Use phandles instead of unit adresses to reference usb and dma nodes. > This makes the DT more robust and readable. > > Signed-off-by: Guido Martínez <guido@vanguardiasur.com.ar> Acked-by: Felipe Balbi <balbi@ti.com> > --- > arch/arm/boot/dts/am335x-bone-common.dtsi | 12 ++++++------ > 1 file changed, 6 insertions(+), 6 deletions(-) > > diff --git a/arch/arm/boot/dts/am335x-bone-common.dtsi b/arch/arm/boot/dts/am335x-bone-common.dtsi > index 2e7d932..b95bc9a 100644 > --- a/arch/arm/boot/dts/am335x-bone-common.dtsi > +++ b/arch/arm/boot/dts/am335x-bone-common.dtsi > @@ -183,28 +183,28 @@ > &usb { > status = "okay"; > > - control@44e10620 { > + &usb_ctrl_mod { > status = "okay"; > }; > > - usb-phy@47401300 { > + &usb0_phy { > status = "okay"; > }; > > - usb-phy@47401b00 { > + &usb1_phy { > status = "okay"; > }; > > - usb@47401000 { > + &usb0 { > status = "okay"; > }; > > - usb@47401800 { > + &usb1 { > status = "okay"; > dr_mode = "host"; > }; > > - dma-controller@47402000 { > + &cppi41dma { > status = "okay"; > }; > }; > -- > 1.9.2 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-omap" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html
On Mon, Apr 28, 2014 at 10:58:31AM -0300, Guido Martínez wrote: > Use phandles instead of unit adresses to reference usb and dma nodes. > This makes the DT more robust and readable. > > Signed-off-by: Guido Martínez <guido@vanguardiasur.com.ar> Acked-by: Felipe Balbi <balbi@ti.com> > --- > arch/arm/boot/dts/am335x-evmsk.dts | 12 ++++++------ > 1 file changed, 6 insertions(+), 6 deletions(-) > > diff --git a/arch/arm/boot/dts/am335x-evmsk.dts b/arch/arm/boot/dts/am335x-evmsk.dts > index ab23885..44b8d75 100644 > --- a/arch/arm/boot/dts/am335x-evmsk.dts > +++ b/arch/arm/boot/dts/am335x-evmsk.dts > @@ -364,28 +364,28 @@ > &usb { > status = "okay"; > > - control@44e10620 { > + &usb_ctrl_mod { > status = "okay"; > }; > > - usb-phy@47401300 { > + &usb0_phy { > status = "okay"; > }; > > - usb-phy@47401b00 { > + &usb1_phy { > status = "okay"; > }; > > - usb@47401000 { > + &usb0 { > status = "okay"; > }; > > - usb@47401800 { > + &usb1 { > status = "okay"; > dr_mode = "host"; > }; > > - dma-controller@47402000 { > + &cppi41dma { > status = "okay"; > }; > }; > -- > 1.9.2 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > Please read the FAQ at http://www.tux.org/lkml/
diff --git a/arch/arm/boot/dts/am335x-bone-common.dtsi b/arch/arm/boot/dts/am335x-bone-common.dtsi index 2e7d932..b95bc9a 100644 --- a/arch/arm/boot/dts/am335x-bone-common.dtsi +++ b/arch/arm/boot/dts/am335x-bone-common.dtsi @@ -183,28 +183,28 @@ &usb { status = "okay"; - control@44e10620 { + &usb_ctrl_mod { status = "okay"; }; - usb-phy@47401300 { + &usb0_phy { status = "okay"; }; - usb-phy@47401b00 { + &usb1_phy { status = "okay"; }; - usb@47401000 { + &usb0 { status = "okay"; }; - usb@47401800 { + &usb1 { status = "okay"; dr_mode = "host"; }; - dma-controller@47402000 { + &cppi41dma { status = "okay"; }; };
Use phandles instead of unit adresses to reference usb and dma nodes. This makes the DT more robust and readable. Signed-off-by: Guido Martínez <guido@vanguardiasur.com.ar> --- arch/arm/boot/dts/am335x-bone-common.dtsi | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-)