From: shawn.guo@linaro.org (Shawn Guo)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] ARM: dts: imx: Add dr_mode host setting to all host-only usb instances
Date: Tue, 3 Mar 2015 11:41:35 +0800 [thread overview]
Message-ID: <20150303034133.GJ20455@dragon> (raw)
In-Reply-To: <1425045960-14191-1-git-send-email-mporter@konsulko.com>
Peter,
Can you take a look at Matt's patch below?
Shawn
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.
>
> Signed-off-by: Matt Porter <mporter@konsulko.com>
> ---
> 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
>
WARNING: multiple messages have this Message-ID (diff)
From: Shawn Guo <shawn.guo@linaro.org>
To: Matt Porter <mporter@konsulko.com>,
Peter Chen <peter.chen@freescale.com>
Cc: Sascha Hauer <kernel@pengutronix.de>,
Devicetree List <devicetree@vger.kernel.org>,
Linux ARM Kernel List <linux-arm-kernel@lists.infradead.org>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] ARM: dts: imx: Add dr_mode host setting to all host-only usb instances
Date: Tue, 3 Mar 2015 11:41:35 +0800 [thread overview]
Message-ID: <20150303034133.GJ20455@dragon> (raw)
In-Reply-To: <1425045960-14191-1-git-send-email-mporter@konsulko.com>
Peter,
Can you take a look at Matt's patch below?
Shawn
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.
>
> Signed-off-by: Matt Porter <mporter@konsulko.com>
> ---
> 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
>
next prev parent reply other threads:[~2015-03-03 3:41 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-02-27 14:06 [PATCH] ARM: dts: imx: Add dr_mode host setting to all host-only usb instances Matt Porter
2015-02-27 14:06 ` Matt Porter
2015-03-03 3:41 ` Shawn Guo [this message]
2015-03-03 3:41 ` Shawn Guo
2015-03-03 4:43 ` Peter Chen
2015-03-03 4:43 ` Peter Chen
2015-03-03 4:43 ` Peter Chen
2015-03-03 15:08 ` Matt Porter
2015-03-03 15:08 ` Matt Porter
2015-03-04 0:51 ` Peter Chen
2015-03-04 0:51 ` Peter Chen
2015-03-06 4:40 ` Shawn Guo
2015-03-06 4:40 ` Shawn Guo
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20150303034133.GJ20455@dragon \
--to=shawn.guo@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.