From: Neil Armstrong <neil.armstrong@linaro.org>
To: Vinod Koul <vkoul@kernel.org>
Cc: Kishon Vijay Abraham I <kishon@kernel.org>,
Rob Herring <robh+dt@kernel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
Martin Blumenstingl <martin.blumenstingl@googlemail.com>,
Rob Herring <robh@kernel.org>,
linux-phy@lists.infradead.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH] dt-bindings: phy: amlogic,g12a-usb3-pcie-phy: add missing optional phy-supply property
Date: Sat, 11 Feb 2023 14:49:12 +0100 [thread overview]
Message-ID: <f727dd7d-08a8-acdf-316c-d571e36a4b1e@linaro.org> (raw)
In-Reply-To: <Y+Z5xOlAeqO6JyIj@matsya>
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>
>
next prev parent reply other threads:[~2023-02-11 13:49 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-02-07 15:03 Neil Armstrong
2023-02-08 23:46 ` Rob Herring
2023-02-10 17:07 ` Vinod Koul
2023-02-11 13:49 ` Neil Armstrong [this message]
2023-02-14 13:57 ` Vinod Koul
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=f727dd7d-08a8-acdf-316c-d571e36a4b1e@linaro.org \
--to=neil.armstrong@linaro.org \
--cc=devicetree@vger.kernel.org \
--cc=kishon@kernel.org \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-phy@lists.infradead.org \
--cc=martin.blumenstingl@googlemail.com \
--cc=robh+dt@kernel.org \
--cc=robh@kernel.org \
--cc=vkoul@kernel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®