* Re: [PATCH mvebu-dt64 1/1] arm64: dts: armada-3720-turris-mox: convert usb-phy to phy-supply
[not found] <20191008145944.21659-1-marek.behun@nic.cz>
@ 2019-10-08 16:37 ` Gregory CLEMENT
2019-10-08 16:49 ` Marek Behún
0 siblings, 1 reply; 4+ messages in thread
From: Gregory CLEMENT @ 2019-10-08 16:37 UTC (permalink / raw)
To: Marek Behún, Miquel Raynal; +Cc: Andrew Lunn, linux-arm-kernel
Hi Marek,
> Update Turris Mox device tree to use the phy-supply property of the
> generic PHY framework instead of the legacy usb-phy property.
>
> This is needed since it caused a regression on Turris Mox since commit
> eb6c2eb6c7fb ("usb: host: xhci-plat: Prevent an abnormally...").
>
I am not against this patch and if it the right fix for the regression I
will applied it on my mvebu/fixes branch to be part of v5.4. However I'd
like to have the feedback from Miquel on this one, as for me it is not
obvious that it is te correct fix.
Gregory
> Signed-off-by: Marek Behún <marek.behun@nic.cz>
> Fixes: eb6c2eb6c7fb ("usb: host: xhci-plat: Prevent an abnormally...")
> Cc: Miquel Raynal <miquel.raynal@bootlin.com>
> Cc: Gregory CLEMENT <gregory.clement@bootlin.com>
> Cc: Andrew Lunn <andrew@lunn.ch>
> ---
> .../boot/dts/marvell/armada-3720-turris-mox.dts | 13 +++++++------
> 1 file changed, 7 insertions(+), 6 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/marvell/armada-3720-turris-mox.dts b/arch/arm64/boot/dts/marvell/armada-3720-turris-mox.dts
> index d105986c6be1..5f350cc71a2f 100644
> --- a/arch/arm64/boot/dts/marvell/armada-3720-turris-mox.dts
> +++ b/arch/arm64/boot/dts/marvell/armada-3720-turris-mox.dts
> @@ -60,11 +60,6 @@
> gpio = <&gpiosb 0 GPIO_ACTIVE_HIGH>;
> };
>
> - usb3_phy: usb3-phy {
> - compatible = "usb-nop-xceiv";
> - vcc-supply = <&exp_usb3_vbus>;
> - };
> -
> vsdc_reg: vsdc-reg {
> compatible = "regulator-gpio";
> regulator-name = "vsdc";
> @@ -255,10 +250,16 @@
> status = "okay";
> };
>
> +&comphy2 {
> + connector {
> + compatible = "usb-a-connector";
> + phy-supply = <&exp_usb3_vbus>;
> + };
> +};
> +
> &usb3 {
> status = "okay";
> phys = <&comphy2 0>;
> - usb-phy = <&usb3_phy>;
> };
>
> &mdio {
> --
> 2.21.0
>
--
Gregory Clement, Bootlin
Embedded Linux and Kernel engineering
http://bootlin.com
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH mvebu-dt64 1/1] arm64: dts: armada-3720-turris-mox: convert usb-phy to phy-supply
2019-10-08 16:37 ` [PATCH mvebu-dt64 1/1] arm64: dts: armada-3720-turris-mox: convert usb-phy to phy-supply Gregory CLEMENT
@ 2019-10-08 16:49 ` Marek Behún
2019-10-08 16:54 ` Miquel Raynal
2019-10-08 16:58 ` Gregory CLEMENT
0 siblings, 2 replies; 4+ messages in thread
From: Marek Behún @ 2019-10-08 16:49 UTC (permalink / raw)
To: Gregory CLEMENT; +Cc: Andrew Lunn, linux-arm-kernel, Miquel Raynal
Gregory,
I communicated with Miquel when I first discovered this regression, see
https://marc.info/?l=linux-usb&m=156966328310472&w=2
In this mail Miquel is mentioning a patch
(https://patchwork.kernel.org/patch/11067687/) which does the same for
Armada 7k/8k. He says his patch should enter next release, but since on
Mox this caused a regression, I would like it to be merged now.
Anyway if you want to still wait for Miquels review to be sure, I am ok
with that.
What do you think about the "arm64: dts: armada-3720-turris-mox: add
firmware node" which I sent last month?
Marek
On Tue, 08 Oct 2019 18:37:03 +0200
Gregory CLEMENT <gregory.clement@bootlin.com> wrote:
> Hi Marek,
>
> > Update Turris Mox device tree to use the phy-supply property of the
> > generic PHY framework instead of the legacy usb-phy property.
> >
> > This is needed since it caused a regression on Turris Mox since
> > commit eb6c2eb6c7fb ("usb: host: xhci-plat: Prevent an
> > abnormally...").
>
> I am not against this patch and if it the right fix for the
> regression I will applied it on my mvebu/fixes branch to be part of
> v5.4. However I'd like to have the feedback from Miquel on this one,
> as for me it is not obvious that it is te correct fix.
>
> Gregory
>
> > Signed-off-by: Marek Behún <marek.behun@nic.cz>
> > Fixes: eb6c2eb6c7fb ("usb: host: xhci-plat: Prevent an
> > abnormally...") Cc: Miquel Raynal <miquel.raynal@bootlin.com>
> > Cc: Gregory CLEMENT <gregory.clement@bootlin.com>
> > Cc: Andrew Lunn <andrew@lunn.ch>
> > ---
> > .../boot/dts/marvell/armada-3720-turris-mox.dts | 13
> > +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-)
> >
> > diff --git a/arch/arm64/boot/dts/marvell/armada-3720-turris-mox.dts
> > b/arch/arm64/boot/dts/marvell/armada-3720-turris-mox.dts index
> > d105986c6be1..5f350cc71a2f 100644 ---
> > a/arch/arm64/boot/dts/marvell/armada-3720-turris-mox.dts +++
> > b/arch/arm64/boot/dts/marvell/armada-3720-turris-mox.dts @@ -60,11
> > +60,6 @@ gpio = <&gpiosb 0 GPIO_ACTIVE_HIGH>;
> > };
> >
> > - usb3_phy: usb3-phy {
> > - compatible = "usb-nop-xceiv";
> > - vcc-supply = <&exp_usb3_vbus>;
> > - };
> > -
> > vsdc_reg: vsdc-reg {
> > compatible = "regulator-gpio";
> > regulator-name = "vsdc";
> > @@ -255,10 +250,16 @@
> > status = "okay";
> > };
> >
> > +&comphy2 {
> > + connector {
> > + compatible = "usb-a-connector";
> > + phy-supply = <&exp_usb3_vbus>;
> > + };
> > +};
> > +
> > &usb3 {
> > status = "okay";
> > phys = <&comphy2 0>;
> > - usb-phy = <&usb3_phy>;
> > };
> >
> > &mdio {
> > --
> > 2.21.0
> >
>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH mvebu-dt64 1/1] arm64: dts: armada-3720-turris-mox: convert usb-phy to phy-supply
2019-10-08 16:49 ` Marek Behún
@ 2019-10-08 16:54 ` Miquel Raynal
2019-10-08 16:58 ` Gregory CLEMENT
1 sibling, 0 replies; 4+ messages in thread
From: Miquel Raynal @ 2019-10-08 16:54 UTC (permalink / raw)
To: Marek Behún; +Cc: Andrew Lunn, Gregory CLEMENT, linux-arm-kernel
Hi Gregory,
Marek Behún <marek.behun@nic.cz> wrote on Tue, 8 Oct 2019 18:49:21
+0200:
> Gregory,
>
> I communicated with Miquel when I first discovered this regression, see
> https://marc.info/?l=linux-usb&m=156966328310472&w=2
>
> In this mail Miquel is mentioning a patch
> (https://patchwork.kernel.org/patch/11067687/) which does the same for
> Armada 7k/8k. He says his patch should enter next release, but since on
> Mox this caused a regression, I would like it to be merged now.
>
> Anyway if you want to still wait for Miquels review to be sure, I am ok
> with that.
Reviewed-by: Miquel Raynal <miquel.raynal@bootlin.com>
I think it is a good candidate for a fixes PR.
Thanks,
Miquèl
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH mvebu-dt64 1/1] arm64: dts: armada-3720-turris-mox: convert usb-phy to phy-supply
2019-10-08 16:49 ` Marek Behún
2019-10-08 16:54 ` Miquel Raynal
@ 2019-10-08 16:58 ` Gregory CLEMENT
1 sibling, 0 replies; 4+ messages in thread
From: Gregory CLEMENT @ 2019-10-08 16:58 UTC (permalink / raw)
To: Marek Behún; +Cc: Andrew Lunn, linux-arm-kernel, Miquel Raynal
Hi Marek,
> Gregory,
>
> I communicated with Miquel when I first discovered this regression, see
> https://marc.info/?l=linux-usb&m=156966328310472&w=2
>
> In this mail Miquel is mentioning a patch
> (https://patchwork.kernel.org/patch/11067687/) which does the same for
> Armada 7k/8k. He says his patch should enter next release, but since on
> Mox this caused a regression, I would like it to be merged now.
I was not aware of this mail, but after reading I agree to apply it now.
>
> Anyway if you want to still wait for Miquels review to be sure, I am ok
> with that.
>
> What do you think about the "arm64: dts: armada-3720-turris-mox: add
> firmware node" which I sent last month?
I've applied a couple of minute agos.
Gregory
>
> Marek
>
> On Tue, 08 Oct 2019 18:37:03 +0200
> Gregory CLEMENT <gregory.clement@bootlin.com> wrote:
>
>> Hi Marek,
>>
>> > Update Turris Mox device tree to use the phy-supply property of the
>> > generic PHY framework instead of the legacy usb-phy property.
>> >
>> > This is needed since it caused a regression on Turris Mox since
>> > commit eb6c2eb6c7fb ("usb: host: xhci-plat: Prevent an
>> > abnormally...").
>>
>> I am not against this patch and if it the right fix for the
>> regression I will applied it on my mvebu/fixes branch to be part of
>> v5.4. However I'd like to have the feedback from Miquel on this one,
>> as for me it is not obvious that it is te correct fix.
>>
>> Gregory
>>
>> > Signed-off-by: Marek Behún <marek.behun@nic.cz>
>> > Fixes: eb6c2eb6c7fb ("usb: host: xhci-plat: Prevent an
>> > abnormally...") Cc: Miquel Raynal <miquel.raynal@bootlin.com>
>> > Cc: Gregory CLEMENT <gregory.clement@bootlin.com>
>> > Cc: Andrew Lunn <andrew@lunn.ch>
>> > ---
>> > .../boot/dts/marvell/armada-3720-turris-mox.dts | 13
>> > +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-)
>> >
>> > diff --git a/arch/arm64/boot/dts/marvell/armada-3720-turris-mox.dts
>> > b/arch/arm64/boot/dts/marvell/armada-3720-turris-mox.dts index
>> > d105986c6be1..5f350cc71a2f 100644 ---
>> > a/arch/arm64/boot/dts/marvell/armada-3720-turris-mox.dts +++
>> > b/arch/arm64/boot/dts/marvell/armada-3720-turris-mox.dts @@ -60,11
>> > +60,6 @@ gpio = <&gpiosb 0 GPIO_ACTIVE_HIGH>;
>> > };
>> >
>> > - usb3_phy: usb3-phy {
>> > - compatible = "usb-nop-xceiv";
>> > - vcc-supply = <&exp_usb3_vbus>;
>> > - };
>> > -
>> > vsdc_reg: vsdc-reg {
>> > compatible = "regulator-gpio";
>> > regulator-name = "vsdc";
>> > @@ -255,10 +250,16 @@
>> > status = "okay";
>> > };
>> >
>> > +&comphy2 {
>> > + connector {
>> > + compatible = "usb-a-connector";
>> > + phy-supply = <&exp_usb3_vbus>;
>> > + };
>> > +};
>> > +
>> > &usb3 {
>> > status = "okay";
>> > phys = <&comphy2 0>;
>> > - usb-phy = <&usb3_phy>;
>> > };
>> >
>> > &mdio {
>> > --
>> > 2.21.0
>> >
>>
>
--
Gregory Clement, Bootlin
Embedded Linux and Kernel engineering
http://bootlin.com
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2019-10-08 16:58 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20191008145944.21659-1-marek.behun@nic.cz>
2019-10-08 16:37 ` [PATCH mvebu-dt64 1/1] arm64: dts: armada-3720-turris-mox: convert usb-phy to phy-supply Gregory CLEMENT
2019-10-08 16:49 ` Marek Behún
2019-10-08 16:54 ` Miquel Raynal
2019-10-08 16:58 ` Gregory CLEMENT
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).