Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1458283 > unrolled thread
| Started by | John Stultz <john.stultz@linaro.org> |
|---|---|
| First post | 2016-08-09 00:40 +0200 |
| Last post | 2016-08-10 06:10 +0200 |
| Articles | 2 — 2 participants |
Back to article view | Back to linux.kernel
[PATCH 1/2] device-tree: aqp8064.dtsi: Remove usb phy dr_mode = "host" John Stultz <john.stultz@linaro.org> - 2016-08-09 00:40 +0200
Re: [PATCH 1/2] device-tree: aqp8064.dtsi: Remove usb phy dr_mode = "host" Bjorn Andersson <bjorn.andersson@linaro.org> - 2016-08-10 06:10 +0200
| From | John Stultz <john.stultz@linaro.org> |
|---|---|
| Date | 2016-08-09 00:40 +0200 |
| Subject | [PATCH 1/2] device-tree: aqp8064.dtsi: Remove usb phy dr_mode = "host" |
| Message-ID | <s41nb-7sd-3@gated-at.bofh.it> |
Most 8064 devices have micro-usb ports for phy1, so setting the dr_mode to host here seems incorrect. Leaving it unspecified should default to otg, and then any boards that wish to specify something else, can override it in their dts file. Cc: Rob Herring <robh+dt@kernel.org> Cc: Andy Gross <agross@codeaurora.org> Cc: Bjorn Andersson <bjorn.andersson@linaro.org> Cc: Stephen Boyd <stephen.boyd@linaro.org> Cc: linux-arm-msm@vger.kernel.org Cc: devicetree@vger.kernel.org Signed-off-by: John Stultz <john.stultz@linaro.org> --- arch/arm/boot/dts/qcom-apq8064.dtsi | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/arm/boot/dts/qcom-apq8064.dtsi b/arch/arm/boot/dts/qcom-apq8064.dtsi index 9dc83b0..7e43416 100644 --- a/arch/arm/boot/dts/qcom-apq8064.dtsi +++ b/arch/arm/boot/dts/qcom-apq8064.dtsi @@ -750,7 +750,6 @@ reg = <0x12500000 0x400>; interrupts = <GIC_SPI 100 IRQ_TYPE_NONE>; status = "disabled"; - dr_mode = "host"; clocks = <&gcc USB_HS1_XCVR_CLK>, <&gcc USB_HS1_H_CLK>; -- 1.9.1
[toc] | [next] | [standalone]
| From | Bjorn Andersson <bjorn.andersson@linaro.org> |
|---|---|
| Date | 2016-08-10 06:10 +0200 |
| Subject | Re: [PATCH 1/2] device-tree: aqp8064.dtsi: Remove usb phy dr_mode = "host" |
| Message-ID | <s4t05-bD-1@gated-at.bofh.it> |
| In reply to | #1458283 |
On Mon 08 Aug 15:34 PDT 2016, John Stultz wrote: Changes in this file is commonly prefixed "ARM: dts: apq8064:", please follow that. Perhaps: ARM: dts: apq8064: Drop dr_mode property from usb phy > Most 8064 devices have micro-usb ports for phy1, so setting > the dr_mode to host here seems incorrect. > > Leaving it unspecified should default to otg, and then > any boards that wish to specify something else, can > override it in their dts file. > > Cc: Rob Herring <robh+dt@kernel.org> > Cc: Andy Gross <agross@codeaurora.org> > Cc: Bjorn Andersson <bjorn.andersson@linaro.org> > Cc: Stephen Boyd <stephen.boyd@linaro.org> > Cc: linux-arm-msm@vger.kernel.org > Cc: devicetree@vger.kernel.org > Signed-off-by: John Stultz <john.stultz@linaro.org> Reviewed-by: Bjorn Andersson <bjorn.andersson@linaro.org> Regards, Bjorn > --- > arch/arm/boot/dts/qcom-apq8064.dtsi | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/arch/arm/boot/dts/qcom-apq8064.dtsi b/arch/arm/boot/dts/qcom-apq8064.dtsi > index 9dc83b0..7e43416 100644 > --- a/arch/arm/boot/dts/qcom-apq8064.dtsi > +++ b/arch/arm/boot/dts/qcom-apq8064.dtsi > @@ -750,7 +750,6 @@ > reg = <0x12500000 0x400>; > interrupts = <GIC_SPI 100 IRQ_TYPE_NONE>; > status = "disabled"; > - dr_mode = "host"; > > clocks = <&gcc USB_HS1_XCVR_CLK>, > <&gcc USB_HS1_H_CLK>; > -- > 1.9.1 >
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web