From: Rob Herring <robh@kernel.org>
To: Vinod Koul <vkoul@kernel.org>, Konrad Dybcio <konradybcio@kernel.org>
Cc: Neil Armstrong <neil.armstrong@linaro.org>,
Manivannan Sadhasivam <mani@kernel.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Conor Dooley <conor+dt@kernel.org>,
Wesley Cheng <quic_wcheng@quicinc.com>,
Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>,
linux-arm-msm@vger.kernel.org, linux-phy@lists.infradead.org,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v2] dt-bindings: phy: qcom: Relax required supplies
Date: Thu, 12 Nov 2026 06:53:46 -0600 [thread overview]
Message-ID: <20261112125346.GA3821108-robh@kernel.org> (raw)
In-Reply-To: <avWbmsjkYo2LMJKs@parshuram>
+Konrad
On Thu, Nov 12, 2026 at 04:27:30PM +0530, Vinod Koul wrote:
> On 20-10-26, 21:24, Rob Herring (Arm) wrote:
> > For the QUSB2 PHY binding, keep the three regulator supplies required
> > except for legacy IPQ6018 nodes that do not describe the board rails.
> >
> > This relaxes the requirements for the IPQ5210 as well as if it is
> > compatible with IPQ6018, then it shouldn't have more required
> > properties either.
>
> Hey Rob,
>
> This fails for phy/next I see vdd-supply has been moved around..
The change in question[1] should not be applied until the .dts changes
in the series are applied. It introduces a bunch of warnings. Can the
QCom folks stop doing that! Coordinate applying the series or take
the binding and dts changes in 1 tree. Given the patch series is 2+
months old, I'd be surprised if the rest gets applied.
This patch OTOH fixes warnings.
Rob
[1] https://lore.kernel.org/all/qusb2phy-v2-1-4c3f980a4560@oss.qualcomm.com
--
linux-phy mailing list
linux-phy@lists.infradead.org
https://lists.infradead.org/mailman/listinfo/linux-phy
WARNING: multiple messages have this Message-ID (diff)
From: Rob Herring <robh@kernel.org>
To: Vinod Koul <vkoul@kernel.org>, Konrad Dybcio <konradybcio@kernel.org>
Cc: Neil Armstrong <neil.armstrong@linaro.org>,
Manivannan Sadhasivam <mani@kernel.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Conor Dooley <conor+dt@kernel.org>,
Wesley Cheng <quic_wcheng@quicinc.com>,
Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>,
linux-arm-msm@vger.kernel.org, linux-phy@lists.infradead.org,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v2] dt-bindings: phy: qcom: Relax required supplies
Date: Thu, 12 Nov 2026 06:53:46 -0600 [thread overview]
Message-ID: <20261112125346.GA3821108-robh@kernel.org> (raw)
In-Reply-To: <avWbmsjkYo2LMJKs@parshuram>
+Konrad
On Thu, Nov 12, 2026 at 04:27:30PM +0530, Vinod Koul wrote:
> On 20-10-26, 21:24, Rob Herring (Arm) wrote:
> > For the QUSB2 PHY binding, keep the three regulator supplies required
> > except for legacy IPQ6018 nodes that do not describe the board rails.
> >
> > This relaxes the requirements for the IPQ5210 as well as if it is
> > compatible with IPQ6018, then it shouldn't have more required
> > properties either.
>
> Hey Rob,
>
> This fails for phy/next I see vdd-supply has been moved around..
The change in question[1] should not be applied until the .dts changes
in the series are applied. It introduces a bunch of warnings. Can the
QCom folks stop doing that! Coordinate applying the series or take
the binding and dts changes in 1 tree. Given the patch series is 2+
months old, I'd be surprised if the rest gets applied.
This patch OTOH fixes warnings.
Rob
[1] https://lore.kernel.org/all/qusb2phy-v2-1-4c3f980a4560@oss.qualcomm.com
next prev parent reply other threads:[~2026-11-12 12:53 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-10-21 2:24 [PATCH v2] dt-bindings: phy: qcom: Relax required supplies Rob Herring (Arm)
2026-10-21 2:24 ` Rob Herring (Arm)
2026-11-12 10:57 ` Vinod Koul
2026-11-12 10:57 ` Vinod Koul
2026-11-12 12:53 ` Rob Herring [this message]
2026-11-12 12:53 ` Rob Herring
2026-11-12 13:35 ` Vinod Koul
2026-11-12 13:35 ` Vinod Koul
2026-11-12 18:29 ` Bjorn Andersson
2026-11-12 18:29 ` Bjorn Andersson
2026-11-12 19:22 ` Rob Herring
2026-11-12 19:22 ` Rob Herring
2026-11-12 20:13 ` Krzysztof Kozlowski
2026-11-12 20:13 ` Krzysztof Kozlowski
2026-11-12 21:17 ` Rob Herring
2026-11-12 21:17 ` Rob Herring
2026-11-12 23:41 ` Bjorn Andersson
2026-11-12 23:41 ` Bjorn Andersson
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=20261112125346.GA3821108-robh@kernel.org \
--to=robh@kernel.org \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=konradybcio@kernel.org \
--cc=krzk+dt@kernel.org \
--cc=krzysztof.kozlowski@oss.qualcomm.com \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-phy@lists.infradead.org \
--cc=mani@kernel.org \
--cc=neil.armstrong@linaro.org \
--cc=quic_wcheng@quicinc.com \
--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 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.