Message ID | 20250306-topic-dt_bindings_fixups-v1-3-0c84aceb0ef9@oss.qualcomm.com (mailing list archive) |
---|---|
State | New |
Headers | show |
Series | Various dt-bindings fixes | expand |
On Thu, Mar 06, 2025 at 07:11:15PM +0100, Konrad Dybcio wrote: > From: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com> > > MSM8916 seems to reuse the same hardware as MSM8974 and friends (for > whom this binding document was created). Add a new compatible for it. Ok, I should have read dt-bindigns before sending a comment. But this commit doesn't explain, why do you need an extra compat string. > > Signed-off-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com> > --- > Documentation/devicetree/bindings/power/qcom,kpss-acc-v2.yaml | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/power/qcom,kpss-acc-v2.yaml b/Documentation/devicetree/bindings/power/qcom,kpss-acc-v2.yaml > index 202a5d51ee88c7190805efe8f1bf493bdb69ec45..27dae49163fa0790ceb6fda8a5c674f739d4a41a 100644 > --- a/Documentation/devicetree/bindings/power/qcom,kpss-acc-v2.yaml > +++ b/Documentation/devicetree/bindings/power/qcom,kpss-acc-v2.yaml > @@ -18,7 +18,9 @@ description: > > properties: > compatible: > - const: qcom,kpss-acc-v2 > + enum: > + - qcom,msm8916-kpss-acc > + - qcom,kpss-acc-v2 > > reg: > items: > > -- > 2.48.1 >
On Thu, Mar 06, 2025 at 07:11:15PM +0100, Konrad Dybcio wrote: > From: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com> > > MSM8916 seems to reuse the same hardware as MSM8974 and friends (for > whom this binding document was created). Add a new compatible for it. > > Signed-off-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com> > --- > Documentation/devicetree/bindings/power/qcom,kpss-acc-v2.yaml | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/power/qcom,kpss-acc-v2.yaml b/Documentation/devicetree/bindings/power/qcom,kpss-acc-v2.yaml > index 202a5d51ee88c7190805efe8f1bf493bdb69ec45..27dae49163fa0790ceb6fda8a5c674f739d4a41a 100644 > --- a/Documentation/devicetree/bindings/power/qcom,kpss-acc-v2.yaml > +++ b/Documentation/devicetree/bindings/power/qcom,kpss-acc-v2.yaml > @@ -18,7 +18,9 @@ description: > > properties: > compatible: > - const: qcom,kpss-acc-v2 > + enum: > + - qcom,msm8916-kpss-acc Hm, MSM8916 doesn't have a *K*PSS (Krait Processor SubSystem), it has an *A*PSS (ARM Cortex – A53 SubSystem, or Application Processor SubSystem). I think this should be either qcom,msm8916-apss-acc, or you just keep the qcom,msm8916-acc we already use. I'm guessing ACC stands for "Application Clock Controller", so it would be unique enough already. There is actually a patch from Rayyan already with a R-b from Krzysztof. Maybe you, or whoever is responsible, can pick it up? https://lore.kernel.org/linux-arm-msm/20240710155226.130086-1-rayyan.ansari@linaro.org/ Thanks, Stephan
On 7.03.2025 12:07 PM, Stephan Gerhold wrote: > On Thu, Mar 06, 2025 at 07:11:15PM +0100, Konrad Dybcio wrote: >> From: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com> >> >> MSM8916 seems to reuse the same hardware as MSM8974 and friends (for >> whom this binding document was created). Add a new compatible for it. >> >> Signed-off-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com> >> --- >> Documentation/devicetree/bindings/power/qcom,kpss-acc-v2.yaml | 4 +++- >> 1 file changed, 3 insertions(+), 1 deletion(-) >> >> diff --git a/Documentation/devicetree/bindings/power/qcom,kpss-acc-v2.yaml b/Documentation/devicetree/bindings/power/qcom,kpss-acc-v2.yaml >> index 202a5d51ee88c7190805efe8f1bf493bdb69ec45..27dae49163fa0790ceb6fda8a5c674f739d4a41a 100644 >> --- a/Documentation/devicetree/bindings/power/qcom,kpss-acc-v2.yaml >> +++ b/Documentation/devicetree/bindings/power/qcom,kpss-acc-v2.yaml >> @@ -18,7 +18,9 @@ description: >> >> properties: >> compatible: >> - const: qcom,kpss-acc-v2 >> + enum: >> + - qcom,msm8916-kpss-acc > > Hm, MSM8916 doesn't have a *K*PSS (Krait Processor SubSystem), it has an > *A*PSS (ARM Cortex – A53 SubSystem, or Application Processor SubSystem). You would think that, but actually it's still called KPSS internally.. > > I think this should be either qcom,msm8916-apss-acc, or you just keep > the qcom,msm8916-acc we already use. I'm guessing ACC stands for > "Application Clock Controller", so it would be unique enough already. > > There is actually a patch from Rayyan already with a R-b from Krzysztof. > Maybe you, or whoever is responsible, can pick it up? > > https://lore.kernel.org/linux-arm-msm/20240710155226.130086-1-rayyan.ansari@linaro.org/ I'm not opposed to this either Looks like Sebastian Reichel and/or Ulf pick up power dt-bindings patches Konrad
On 06/03/2025 19:11, Konrad Dybcio wrote: > From: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com> > > MSM8916 seems to reuse the same hardware as MSM8974 and friends (for > whom this binding document was created). Add a new compatible for it. > > Signed-off-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com> > --- > Documentation/devicetree/bindings/power/qcom,kpss-acc-v2.yaml | 4 + Folks, please merge Ryan work instead... I keep carrying this patch for half a year: https://lore.kernel.org/all/20240710155226.130086-1-rayyan.ansari@linaro.org/ Best regards, Krzysztof
On Wed, Mar 12, 2025 at 04:12:14PM +0100, Krzysztof Kozlowski wrote: > On 06/03/2025 19:11, Konrad Dybcio wrote: > > From: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com> > > > > MSM8916 seems to reuse the same hardware as MSM8974 and friends (for > > whom this binding document was created). Add a new compatible for it. > > > > Signed-off-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com> > > --- > > Documentation/devicetree/bindings/power/qcom,kpss-acc-v2.yaml | 4 + > > > Folks, please merge Ryan work instead... I'm not sure which "folks" you're referring to and neither is get_maintainer (well it points to Rob and yourself). > I keep carrying this patch for > half a year: > Where are you carrying these? Are there any other patches on your list that have fallen through the cracks? > https://lore.kernel.org/all/20240710155226.130086-1-rayyan.ansari@linaro.org/ > I'm picking Ryan's patch through the Qualcomm tree... Regards, Bjorn > Best regards, > Krzysztof
On 13/03/2025 23:53, Bjorn Andersson wrote: >> > > Where are you carrying these? Are there any other patches on your list > that have fallen through the cracks? Since 4-5 years I have a branch on top of next for pending patches for Samsung (initially) and Qualcomm (since ~3 years) bindings, where I collect all my and other people's work related to 0-dbts-check wanrnings. The point is to avoid duplicated work. Around 2022-2023 the branch grew to and kept size of around 100-120 patches. I was rebasing it daily. I don't maintain that branch anymore, so for dtbs-check it has probably just 10 patches in total (remaining ~30-40 patches now is some other my work). I also rebase it once per month or two months. That Rayyan's patch is there since ~July/August last year, just like few things for IPQ, qcom,pm8941-pon, DWC3. https://github.com/krzk/linux/commits/pending/dt-bindings-qcom-new-and-fixes-for-warnings-linux-next/ https://github.com/krzk/linux/commit/f6761796e02330b92653b35ba4b1c20ff304dded https://github.com/krzk/linux/commit/fdd475359f378760085d3680423b291eca4bf808 https://github.com/krzk/linux/commit/855535ad2c4346b49f48c3fea6adbeea2262ffd8 https://github.com/krzk/linux/commit/f648bb95180f131241a5c7a7808c9f0865379c75 > >> https://lore.kernel.org/all/20240710155226.130086-1-rayyan.ansari@linaro.org/ >> > > I'm picking Ryan's patch through the Qualcomm tree... Thanks! That was my intention. Whenever subsystem maintainer skips such simple driver binding, please don't hesitate to pick it up. Best regards, Krzysztof
diff --git a/Documentation/devicetree/bindings/power/qcom,kpss-acc-v2.yaml b/Documentation/devicetree/bindings/power/qcom,kpss-acc-v2.yaml index 202a5d51ee88c7190805efe8f1bf493bdb69ec45..27dae49163fa0790ceb6fda8a5c674f739d4a41a 100644 --- a/Documentation/devicetree/bindings/power/qcom,kpss-acc-v2.yaml +++ b/Documentation/devicetree/bindings/power/qcom,kpss-acc-v2.yaml @@ -18,7 +18,9 @@ description: properties: compatible: - const: qcom,kpss-acc-v2 + enum: + - qcom,msm8916-kpss-acc + - qcom,kpss-acc-v2 reg: items: