* [PATCH] dt-bindings: phy: amlogic,g12a-usb3-pcie-phy: add missing optional phy-supply property
@ 2023-02-07 15:03 Neil Armstrong
2023-02-08 23:46 ` Rob Herring
2023-02-10 17:07 ` Vinod Koul
0 siblings, 2 replies; 5+ messages in thread
From: Neil Armstrong @ 2023-02-07 15:03 UTC (permalink / raw)
To: Vinod Koul, Kishon Vijay Abraham I, Rob Herring,
Krzysztof Kozlowski, Martin Blumenstingl
Cc: Rob Herring, linux-phy, devicetree, linux-kernel, Neil Armstrong
Add missing optional phy-supply property used to power up PHY regulators.
Fixes: 87a55485f2fc ("dt-bindings: phy: meson-g12a-usb3-pcie-phy: convert to yaml")
Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org>
---
.../devicetree/bindings/phy/amlogic,g12a-usb3-pcie-phy.yaml | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/Documentation/devicetree/bindings/phy/amlogic,g12a-usb3-pcie-phy.yaml b/Documentation/devicetree/bindings/phy/amlogic,g12a-usb3-pcie-phy.yaml
index 129d26e99776..3314711292d6 100644
--- a/Documentation/devicetree/bindings/phy/amlogic,g12a-usb3-pcie-phy.yaml
+++ b/Documentation/devicetree/bindings/phy/amlogic,g12a-usb3-pcie-phy.yaml
@@ -35,6 +35,11 @@ properties:
"#phy-cells":
const: 1
+ phy-supply:
+ description:
+ Phandle to a regulator that provides power to the PHY. This
+ regulator will be managed during the PHY power on/off sequence.
+
required:
- compatible
- reg
---
base-commit: 49a8133221c71b935f36a7c340c0271c2a9ee2db
change-id: 20230207-b4-amlogic-amlogic-g12a-usb3-pcie-phy-fix-b5daf54fb7d6
Best regards,
--
Neil Armstrong <neil.armstrong@linaro.org>
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] dt-bindings: phy: amlogic,g12a-usb3-pcie-phy: add missing optional phy-supply property
2023-02-07 15:03 [PATCH] dt-bindings: phy: amlogic,g12a-usb3-pcie-phy: add missing optional phy-supply property Neil Armstrong
@ 2023-02-08 23:46 ` Rob Herring
2023-02-10 17:07 ` Vinod Koul
1 sibling, 0 replies; 5+ messages in thread
From: Rob Herring @ 2023-02-08 23:46 UTC (permalink / raw)
To: Neil Armstrong
Cc: Martin Blumenstingl, Rob Herring, devicetree,
Krzysztof Kozlowski, Vinod Koul, linux-kernel,
Kishon Vijay Abraham I, linux-phy
On Tue, 07 Feb 2023 16:03:53 +0100, Neil Armstrong wrote:
> Add missing optional phy-supply property used to power up PHY regulators.
>
> Fixes: 87a55485f2fc ("dt-bindings: phy: meson-g12a-usb3-pcie-phy: convert to yaml")
> Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org>
> ---
> .../devicetree/bindings/phy/amlogic,g12a-usb3-pcie-phy.yaml | 5 +++++
> 1 file changed, 5 insertions(+)
>
Acked-by: Rob Herring <robh@kernel.org>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] dt-bindings: phy: amlogic,g12a-usb3-pcie-phy: add missing optional phy-supply property
2023-02-07 15:03 [PATCH] dt-bindings: phy: amlogic,g12a-usb3-pcie-phy: add missing optional phy-supply property Neil Armstrong
2023-02-08 23:46 ` Rob Herring
@ 2023-02-10 17:07 ` Vinod Koul
2023-02-11 13:49 ` Neil Armstrong
1 sibling, 1 reply; 5+ messages in thread
From: Vinod Koul @ 2023-02-10 17:07 UTC (permalink / raw)
To: Neil Armstrong
Cc: Kishon Vijay Abraham I, Rob Herring, Krzysztof Kozlowski,
Martin Blumenstingl, Rob Herring, linux-phy, devicetree,
linux-kernel
On 07-02-23, 16:03, Neil Armstrong wrote:
> Add missing optional phy-supply property used to power up PHY regulators.
>
> Fixes: 87a55485f2fc ("dt-bindings: phy: meson-g12a-usb3-pcie-phy: convert to yaml")
> Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org>
> ---
> .../devicetree/bindings/phy/amlogic,g12a-usb3-pcie-phy.yaml | 5 +++++
I dont have this file in my tree, what was this based on?
> 1 file changed, 5 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/phy/amlogic,g12a-usb3-pcie-phy.yaml b/Documentation/devicetree/bindings/phy/amlogic,g12a-usb3-pcie-phy.yaml
> index 129d26e99776..3314711292d6 100644
> --- a/Documentation/devicetree/bindings/phy/amlogic,g12a-usb3-pcie-phy.yaml
> +++ b/Documentation/devicetree/bindings/phy/amlogic,g12a-usb3-pcie-phy.yaml
> @@ -35,6 +35,11 @@ properties:
> "#phy-cells":
> const: 1
>
> + phy-supply:
> + description:
> + Phandle to a regulator that provides power to the PHY. This
> + regulator will be managed during the PHY power on/off sequence.
> +
> required:
> - compatible
> - reg
>
> ---
> base-commit: 49a8133221c71b935f36a7c340c0271c2a9ee2db
> change-id: 20230207-b4-amlogic-amlogic-g12a-usb3-pcie-phy-fix-b5daf54fb7d6
>
> Best regards,
> --
> Neil Armstrong <neil.armstrong@linaro.org>
--
~Vinod
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] dt-bindings: phy: amlogic,g12a-usb3-pcie-phy: add missing optional phy-supply property
2023-02-10 17:07 ` Vinod Koul
@ 2023-02-11 13:49 ` Neil Armstrong
2023-02-14 13:57 ` Vinod Koul
0 siblings, 1 reply; 5+ messages in thread
From: Neil Armstrong @ 2023-02-11 13:49 UTC (permalink / raw)
To: Vinod Koul
Cc: Kishon Vijay Abraham I, Rob Herring, Krzysztof Kozlowski,
Martin Blumenstingl, Rob Herring, linux-phy, devicetree,
linux-kernel
Le 10/02/2023 à 18:07, Vinod Koul a écrit :
> On 07-02-23, 16:03, Neil Armstrong wrote:
>> Add missing optional phy-supply property used to power up PHY regulators.
>>
>> Fixes: 87a55485f2fc ("dt-bindings: phy: meson-g12a-usb3-pcie-phy: convert to yaml")
>> Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org>
>> ---
>> .../devicetree/bindings/phy/amlogic,g12a-usb3-pcie-phy.yaml | 5 +++++
>
> I dont have this file in my tree, what was this based on?
It has been renamed in a previous patch by Heiner: https://lore.kernel.org/all/8d960029-e94d-224b-911f-03e5deb47ebc@gmail.com/
Neil
>
>> 1 file changed, 5 insertions(+)
>>
>> diff --git a/Documentation/devicetree/bindings/phy/amlogic,g12a-usb3-pcie-phy.yaml b/Documentation/devicetree/bindings/phy/amlogic,g12a-usb3-pcie-phy.yaml
>> index 129d26e99776..3314711292d6 100644
>> --- a/Documentation/devicetree/bindings/phy/amlogic,g12a-usb3-pcie-phy.yaml
>> +++ b/Documentation/devicetree/bindings/phy/amlogic,g12a-usb3-pcie-phy.yaml
>> @@ -35,6 +35,11 @@ properties:
>> "#phy-cells":
>> const: 1
>>
>> + phy-supply:
>> + description:
>> + Phandle to a regulator that provides power to the PHY. This
>> + regulator will be managed during the PHY power on/off sequence.
>> +
>> required:
>> - compatible
>> - reg
>>
>> ---
>> base-commit: 49a8133221c71b935f36a7c340c0271c2a9ee2db
>> change-id: 20230207-b4-amlogic-amlogic-g12a-usb3-pcie-phy-fix-b5daf54fb7d6
>>
>> Best regards,
>> --
>> Neil Armstrong <neil.armstrong@linaro.org>
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] dt-bindings: phy: amlogic,g12a-usb3-pcie-phy: add missing optional phy-supply property
2023-02-11 13:49 ` Neil Armstrong
@ 2023-02-14 13:57 ` Vinod Koul
0 siblings, 0 replies; 5+ messages in thread
From: Vinod Koul @ 2023-02-14 13:57 UTC (permalink / raw)
To: Neil Armstrong
Cc: Kishon Vijay Abraham I, Rob Herring, Krzysztof Kozlowski,
Martin Blumenstingl, Rob Herring, linux-phy, devicetree,
linux-kernel
On 11-02-23, 14:49, Neil Armstrong wrote:
> Le 10/02/2023 à 18:07, Vinod Koul a écrit :
> > On 07-02-23, 16:03, Neil Armstrong wrote:
> > > Add missing optional phy-supply property used to power up PHY regulators.
> > >
> > > Fixes: 87a55485f2fc ("dt-bindings: phy: meson-g12a-usb3-pcie-phy: convert to yaml")
> > > Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org>
> > > ---
> > > .../devicetree/bindings/phy/amlogic,g12a-usb3-pcie-phy.yaml | 5 +++++
> >
> > I dont have this file in my tree, what was this based on?
>
> It has been renamed in a previous patch by Heiner: https://lore.kernel.org/all/8d960029-e94d-224b-911f-03e5deb47ebc@gmail.com/
Thanks for letting me know, I have merged fixes which had above and
applied this
--
~Vinod
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2023-02-14 13:58 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-02-07 15:03 [PATCH] dt-bindings: phy: amlogic,g12a-usb3-pcie-phy: add missing optional phy-supply property Neil Armstrong
2023-02-08 23:46 ` Rob Herring
2023-02-10 17:07 ` Vinod Koul
2023-02-11 13:49 ` Neil Armstrong
2023-02-14 13:57 ` Vinod Koul
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).