Re: [PATCH] ARM: dts: imx: Add dr_mode host setting to all host-only usb instances

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On Tue, Mar 03, 2015 at 11:41:35AM +0800, Shawn Guo wrote:
> 
> On Fri, Feb 27, 2015 at 09:06:00AM -0500, Matt Porter wrote:
> > The chipidea driver adds an extra line of spam to the log when a
> > host-only chipidea instance is left set to the default of a dual role
> > controller.
> > 
> > [    2.010873] ci_hdrc ci_hdrc.1: doesn't support gadget
> > 
> > Set the dr_mode property to host on all the host-only nodes
> > to avoid this warning.

It is not an warning, it is dev_info.

In fact, imx28, imx6sl and imx6sx's second controller is dual-role
controller, we only set dr_mode at board's dts according to design
unless the controller's capability register is incorrect.

So, sorry, I don't think this change is necessary.

> > 
> > Signed-off-by: Matt Porter <mporter@xxxxxxxxxxxx>
> > ---
> >  arch/arm/boot/dts/imx27.dtsi   | 2 ++
> >  arch/arm/boot/dts/imx28.dtsi   | 1 +
> >  arch/arm/boot/dts/imx35.dtsi   | 1 +
> >  arch/arm/boot/dts/imx50.dtsi   | 3 +++
> >  arch/arm/boot/dts/imx51.dtsi   | 3 +++
> >  arch/arm/boot/dts/imx53.dtsi   | 3 +++
> >  arch/arm/boot/dts/imx6qdl.dtsi | 3 +++
> >  arch/arm/boot/dts/imx6sl.dtsi  | 1 +
> >  arch/arm/boot/dts/imx6sx.dtsi  | 1 +
> >  9 files changed, 18 insertions(+)
> > 
> > diff --git a/arch/arm/boot/dts/imx27.dtsi b/arch/arm/boot/dts/imx27.dtsi
> > index 4b063b6..6951b66 100644
> > --- a/arch/arm/boot/dts/imx27.dtsi
> > +++ b/arch/arm/boot/dts/imx27.dtsi
> > @@ -488,6 +488,7 @@
> >  				interrupts = <54>;
> >  				clocks = <&clks IMX27_CLK_USB_IPG_GATE>;
> >  				fsl,usbmisc = <&usbmisc 1>;
> > +				dr_mode = "host";
> >  				status = "disabled";
> >  			};
> >  
> > @@ -497,6 +498,7 @@
> >  				interrupts = <55>;
> >  				clocks = <&clks IMX27_CLK_USB_IPG_GATE>;
> >  				fsl,usbmisc = <&usbmisc 2>;
> > +				dr_mode = "host";
> >  				status = "disabled";
> >  			};
> >  
> > diff --git a/arch/arm/boot/dts/imx28.dtsi b/arch/arm/boot/dts/imx28.dtsi
> > index 47f68ac..02330f4 100644
> > --- a/arch/arm/boot/dts/imx28.dtsi
> > +++ b/arch/arm/boot/dts/imx28.dtsi
> > @@ -1197,6 +1197,7 @@
> >  			interrupts = <92>;
> >  			clocks = <&clks 61>;
> >  			fsl,usbphy = <&usbphy1>;
> > +			dr_mode = "host";
> >  			status = "disabled";
> >  		};
> >  
> > diff --git a/arch/arm/boot/dts/imx35.dtsi b/arch/arm/boot/dts/imx35.dtsi
> > index 6932928..b6478e9 100644
> > --- a/arch/arm/boot/dts/imx35.dtsi
> > +++ b/arch/arm/boot/dts/imx35.dtsi
> > @@ -318,6 +318,7 @@
> >  				clocks = <&clks 73>;
> >  				fsl,usbmisc = <&usbmisc 1>;
> >  				fsl,usbphy = <&usbphy1>;
> > +				dr_mode = "host";
> >  				status = "disabled";
> >  			};
> >  
> > diff --git a/arch/arm/boot/dts/imx50.dtsi b/arch/arm/boot/dts/imx50.dtsi
> > index 620b0f0..e245713 100644
> > --- a/arch/arm/boot/dts/imx50.dtsi
> > +++ b/arch/arm/boot/dts/imx50.dtsi
> > @@ -197,6 +197,7 @@
> >  				reg = <0x53f80200 0x0200>;
> >  				interrupts = <14>;
> >  				clocks = <&clks IMX5_CLK_USB_PHY2_GATE>;
> > +				dr_mode = "host";
> >  				status = "disabled";
> >  			};
> >  
> > @@ -205,6 +206,7 @@
> >  				reg = <0x53f80400 0x0200>;
> >  				interrupts = <16>;
> >  				clocks = <&clks IMX5_CLK_USBOH3_GATE>;
> > +				dr_mode = "host";
> >  				status = "disabled";
> >  			};
> >  
> > @@ -213,6 +215,7 @@
> >  				reg = <0x53f80600 0x0200>;
> >  				interrupts = <17>;
> >  				clocks = <&clks IMX5_CLK_USBOH3_GATE>;
> > +				dr_mode = "host";
> >  				status = "disabled";
> >  			};
> >  
> > diff --git a/arch/arm/boot/dts/imx51.dtsi b/arch/arm/boot/dts/imx51.dtsi
> > index c0116cf..f46fe9b 100644
> > --- a/arch/arm/boot/dts/imx51.dtsi
> > +++ b/arch/arm/boot/dts/imx51.dtsi
> > @@ -265,6 +265,7 @@
> >  				interrupts = <14>;
> >  				clocks = <&clks IMX5_CLK_USBOH3_GATE>;
> >  				fsl,usbmisc = <&usbmisc 1>;
> > +				dr_mode = "host";
> >  				status = "disabled";
> >  			};
> >  
> > @@ -274,6 +275,7 @@
> >  				interrupts = <16>;
> >  				clocks = <&clks IMX5_CLK_USBOH3_GATE>;
> >  				fsl,usbmisc = <&usbmisc 2>;
> > +				dr_mode = "host";
> >  				status = "disabled";
> >  			};
> >  
> > @@ -283,6 +285,7 @@
> >  				interrupts = <17>;
> >  				clocks = <&clks IMX5_CLK_USBOH3_GATE>;
> >  				fsl,usbmisc = <&usbmisc 3>;
> > +				dr_mode = "host";
> >  				status = "disabled";
> >  			};
> >  
> > diff --git a/arch/arm/boot/dts/imx53.dtsi b/arch/arm/boot/dts/imx53.dtsi
> > index ff4fa7e..c3e3ca9 100644
> > --- a/arch/arm/boot/dts/imx53.dtsi
> > +++ b/arch/arm/boot/dts/imx53.dtsi
> > @@ -309,6 +309,7 @@
> >  				clocks = <&clks IMX5_CLK_USBOH3_GATE>;
> >  				fsl,usbmisc = <&usbmisc 1>;
> >  				fsl,usbphy = <&usbphy1>;
> > +				dr_mode = "host";
> >  				status = "disabled";
> >  			};
> >  
> > @@ -318,6 +319,7 @@
> >  				interrupts = <16>;
> >  				clocks = <&clks IMX5_CLK_USBOH3_GATE>;
> >  				fsl,usbmisc = <&usbmisc 2>;
> > +				dr_mode = "host";
> >  				status = "disabled";
> >  			};
> >  
> > @@ -327,6 +329,7 @@
> >  				interrupts = <17>;
> >  				clocks = <&clks IMX5_CLK_USBOH3_GATE>;
> >  				fsl,usbmisc = <&usbmisc 3>;
> > +				dr_mode = "host";
> >  				status = "disabled";
> >  			};
> >  
> > diff --git a/arch/arm/boot/dts/imx6qdl.dtsi b/arch/arm/boot/dts/imx6qdl.dtsi
> > index d6c69ec..9347c93 100644
> > --- a/arch/arm/boot/dts/imx6qdl.dtsi
> > +++ b/arch/arm/boot/dts/imx6qdl.dtsi
> > @@ -845,6 +845,7 @@
> >  				clocks = <&clks IMX6QDL_CLK_USBOH3>;
> >  				fsl,usbphy = <&usbphy2>;
> >  				fsl,usbmisc = <&usbmisc 1>;
> > +				dr_mode = "host";
> >  				status = "disabled";
> >  			};
> >  
> > @@ -854,6 +855,7 @@
> >  				interrupts = <0 41 IRQ_TYPE_LEVEL_HIGH>;
> >  				clocks = <&clks IMX6QDL_CLK_USBOH3>;
> >  				fsl,usbmisc = <&usbmisc 2>;
> > +				dr_mode = "host";
> >  				status = "disabled";
> >  			};
> >  
> > @@ -863,6 +865,7 @@
> >  				interrupts = <0 42 IRQ_TYPE_LEVEL_HIGH>;
> >  				clocks = <&clks IMX6QDL_CLK_USBOH3>;
> >  				fsl,usbmisc = <&usbmisc 3>;
> > +				dr_mode = "host";
> >  				status = "disabled";
> >  			};
> >  
> > diff --git a/arch/arm/boot/dts/imx6sl.dtsi b/arch/arm/boot/dts/imx6sl.dtsi
> > index 36ab8e0..9bf1db3 100644
> > --- a/arch/arm/boot/dts/imx6sl.dtsi
> > +++ b/arch/arm/boot/dts/imx6sl.dtsi
> > @@ -699,6 +699,7 @@
> >  				interrupts = <0 40 IRQ_TYPE_LEVEL_HIGH>;
> >  				clocks = <&clks IMX6SL_CLK_USBOH3>;
> >  				fsl,usbmisc = <&usbmisc 2>;
> > +				dr_mode = "host";
> >  				status = "disabled";
> >  			};
> >  
> > diff --git a/arch/arm/boot/dts/imx6sx.dtsi b/arch/arm/boot/dts/imx6sx.dtsi
> > index 7a24fee..fdadd060 100644
> > --- a/arch/arm/boot/dts/imx6sx.dtsi
> > +++ b/arch/arm/boot/dts/imx6sx.dtsi
> > @@ -763,6 +763,7 @@
> >  				fsl,usbmisc = <&usbmisc 2>;
> >  				phy_type = "hsic";
> >  				fsl,anatop = <&anatop>;
> > +				dr_mode = "host";
> >  				status = "disabled";
> >  			};
> >  
> > -- 
> > 1.8.4
> > 

-- 

Best Regards,
Peter Chen
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux