From: Vinod Koul <vkoul@kernel.org>
To: Rob Herring <robh@kernel.org>
Cc: Konrad Dybcio <konradybcio@kernel.org>,
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 19:05:28 +0530 [thread overview]
Message-ID: <avXAoMcmDLeNuWED@parshuram> (raw)
In-Reply-To: <20261112125346.GA3821108-robh@kernel.org>
On 12-11-26, 06:53, Rob Herring wrote:
> +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.
If Konrad/Bjorn want to take this all together thur QC tree they can
have my ack
Acked-by: Vinod Koul <vkoul@kernel.org>
--
~Vinod
WARNING: multiple messages have this Message-ID (diff)
From: Vinod Koul <vkoul@kernel.org>
To: Rob Herring <robh@kernel.org>
Cc: Konrad Dybcio <konradybcio@kernel.org>,
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 19:05:28 +0530 [thread overview]
Message-ID: <avXAoMcmDLeNuWED@parshuram> (raw)
In-Reply-To: <20261112125346.GA3821108-robh@kernel.org>
On 12-11-26, 06:53, Rob Herring wrote:
> +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.
If Konrad/Bjorn want to take this all together thur QC tree they can
have my ack
Acked-by: Vinod Koul <vkoul@kernel.org>
--
~Vinod
--
linux-phy mailing list
linux-phy@lists.infradead.org
https://lists.infradead.org/mailman/listinfo/linux-phy
next prev parent reply other threads:[~2026-11-12 13:35 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
2026-11-12 12:53 ` Rob Herring
2026-11-12 13:35 ` Vinod Koul [this message]
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=avXAoMcmDLeNuWED@parshuram \
--to=vkoul@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=robh@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.