* [PATCH v2] dt-bindings: spmi: qcom,x1e80100-spmi-pmic-arb: Document Eliza compatible
@ 2026-03-11 13:29 Abel Vesa
2026-03-11 13:43 ` Konrad Dybcio
0 siblings, 1 reply; 6+ messages in thread
From: Abel Vesa @ 2026-03-11 13:29 UTC (permalink / raw)
To: Stephen Boyd, Rob Herring, Krzysztof Kozlowski, Conor Dooley
Cc: linux-arm-msm, linux-kernel, devicetree, Abel Vesa
The SPMI multi-master Arbiter found on Eliza is version 7.2.0, yet
driver-wise, still compatible with the one featured on Hamoa (X1E80100),
which is 7.0.1.
So document the Eliza compatible and allow Hamoa one as fallback.
Signed-off-by: Abel Vesa <abel.vesa@oss.qualcomm.com>
---
Changes in v2:
- Instead of reworking the whole oneOf for compatibles,
add Eliza similar to sar2130p.
- Link to v1: https://patch.msgid.link/20260311-eliza-bindings-spmi-v1-1-1a20eec1179e@oss.qualcomm.com
---
.../devicetree/bindings/spmi/qcom,x1e80100-spmi-pmic-arb.yaml | 3 +++
1 file changed, 3 insertions(+)
diff --git a/Documentation/devicetree/bindings/spmi/qcom,x1e80100-spmi-pmic-arb.yaml b/Documentation/devicetree/bindings/spmi/qcom,x1e80100-spmi-pmic-arb.yaml
index 08369fdd2161..4152469b3880 100644
--- a/Documentation/devicetree/bindings/spmi/qcom,x1e80100-spmi-pmic-arb.yaml
+++ b/Documentation/devicetree/bindings/spmi/qcom,x1e80100-spmi-pmic-arb.yaml
@@ -23,6 +23,9 @@ allOf:
properties:
compatible:
oneOf:
+ - items:
+ - const: qcom,eliza-spmi-pmic-arb
+ - const: qcom,x1e80100-spmi-pmic-arb
- items:
- const: qcom,sar2130p-spmi-pmic-arb
- const: qcom,x1e80100-spmi-pmic-arb
---
base-commit: 7109a2155340cc7b21f27e832ece6df03592f2e8
change-id: 20260311-eliza-bindings-spmi-8d49b1a61d03
Best regards,
--
Abel Vesa <abel.vesa@oss.qualcomm.com>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2] dt-bindings: spmi: qcom,x1e80100-spmi-pmic-arb: Document Eliza compatible
2026-03-11 13:29 [PATCH v2] dt-bindings: spmi: qcom,x1e80100-spmi-pmic-arb: Document Eliza compatible Abel Vesa
@ 2026-03-11 13:43 ` Konrad Dybcio
2026-03-11 13:57 ` Abel Vesa
2026-03-13 8:34 ` Krzysztof Kozlowski
0 siblings, 2 replies; 6+ messages in thread
From: Konrad Dybcio @ 2026-03-11 13:43 UTC (permalink / raw)
To: Abel Vesa, Stephen Boyd, Rob Herring, Krzysztof Kozlowski, Conor Dooley
Cc: linux-arm-msm, linux-kernel, devicetree
On 3/11/26 2:29 PM, Abel Vesa wrote:
> The SPMI multi-master Arbiter found on Eliza is version 7.2.0, yet
> driver-wise, still compatible with the one featured on Hamoa (X1E80100),
> which is 7.0.1.
>
> So document the Eliza compatible and allow Hamoa one as fallback.
>
> Signed-off-by: Abel Vesa <abel.vesa@oss.qualcomm.com>
> ---
> Changes in v2:
> - Instead of reworking the whole oneOf for compatibles,
> add Eliza similar to sar2130p.
> - Link to v1: https://patch.msgid.link/20260311-eliza-bindings-spmi-v1-1-1a20eec1179e@oss.qualcomm.com
> ---
> .../devicetree/bindings/spmi/qcom,x1e80100-spmi-pmic-arb.yaml | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/spmi/qcom,x1e80100-spmi-pmic-arb.yaml b/Documentation/devicetree/bindings/spmi/qcom,x1e80100-spmi-pmic-arb.yaml
> index 08369fdd2161..4152469b3880 100644
> --- a/Documentation/devicetree/bindings/spmi/qcom,x1e80100-spmi-pmic-arb.yaml
> +++ b/Documentation/devicetree/bindings/spmi/qcom,x1e80100-spmi-pmic-arb.yaml
> @@ -23,6 +23,9 @@ allOf:
> properties:
> compatible:
> oneOf:
> + - items:
> + - const: qcom,eliza-spmi-pmic-arb
> + - const: qcom,x1e80100-spmi-pmic-arb
> - items:
> - const: qcom,sar2130p-spmi-pmic-arb
> - const: qcom,x1e80100-spmi-pmic-arb
I think Krzysztof's comment was meant to suggest you shouldn't add an
intermittent items: entry for a single const (lone-x1e compatible), instead
passing:
oneOf:
- items:
- enum:
- foo
- bar
- const: x1e
...
- const: x1e
Konrad
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2] dt-bindings: spmi: qcom,x1e80100-spmi-pmic-arb: Document Eliza compatible
2026-03-11 13:43 ` Konrad Dybcio
@ 2026-03-11 13:57 ` Abel Vesa
2026-03-11 14:01 ` Konrad Dybcio
2026-03-13 8:34 ` Krzysztof Kozlowski
1 sibling, 1 reply; 6+ messages in thread
From: Abel Vesa @ 2026-03-11 13:57 UTC (permalink / raw)
To: Konrad Dybcio
Cc: Stephen Boyd, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
linux-arm-msm, linux-kernel, devicetree
On 26-03-11 14:43:27, Konrad Dybcio wrote:
> On 3/11/26 2:29 PM, Abel Vesa wrote:
> > The SPMI multi-master Arbiter found on Eliza is version 7.2.0, yet
> > driver-wise, still compatible with the one featured on Hamoa (X1E80100),
> > which is 7.0.1.
> >
> > So document the Eliza compatible and allow Hamoa one as fallback.
> >
> > Signed-off-by: Abel Vesa <abel.vesa@oss.qualcomm.com>
> > ---
> > Changes in v2:
> > - Instead of reworking the whole oneOf for compatibles,
> > add Eliza similar to sar2130p.
> > - Link to v1: https://patch.msgid.link/20260311-eliza-bindings-spmi-v1-1-1a20eec1179e@oss.qualcomm.com
> > ---
> > .../devicetree/bindings/spmi/qcom,x1e80100-spmi-pmic-arb.yaml | 3 +++
> > 1 file changed, 3 insertions(+)
> >
> > diff --git a/Documentation/devicetree/bindings/spmi/qcom,x1e80100-spmi-pmic-arb.yaml b/Documentation/devicetree/bindings/spmi/qcom,x1e80100-spmi-pmic-arb.yaml
> > index 08369fdd2161..4152469b3880 100644
> > --- a/Documentation/devicetree/bindings/spmi/qcom,x1e80100-spmi-pmic-arb.yaml
> > +++ b/Documentation/devicetree/bindings/spmi/qcom,x1e80100-spmi-pmic-arb.yaml
> > @@ -23,6 +23,9 @@ allOf:
> > properties:
> > compatible:
> > oneOf:
> > + - items:
> > + - const: qcom,eliza-spmi-pmic-arb
> > + - const: qcom,x1e80100-spmi-pmic-arb
> > - items:
> > - const: qcom,sar2130p-spmi-pmic-arb
> > - const: qcom,x1e80100-spmi-pmic-arb
>
> I think Krzysztof's comment was meant to suggest you shouldn't add an
> intermittent items: entry for a single const (lone-x1e compatible), instead
> passing:
>
> oneOf:
> - items:
> - enum:
> - foo
> - bar
> - const: x1e
> ...
> - const: x1e
Oh, my bad then. You mean like this?
@@ -24,10 +24,9 @@ properties:
compatible:
oneOf:
- items:
- - const: qcom,eliza-spmi-pmic-arb
- - const: qcom,x1e80100-spmi-pmic-arb
- - items:
- - const: qcom,sar2130p-spmi-pmic-arb
+ - enum:
+ - qcom,eliza-spmi-pmic-arb
+ - qcom,sar2130p-spmi-pmic-arb
- const: qcom,x1e80100-spmi-pmic-arb
- const: qcom,x1e80100-spmi-pmic-arb
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2] dt-bindings: spmi: qcom,x1e80100-spmi-pmic-arb: Document Eliza compatible
2026-03-11 13:57 ` Abel Vesa
@ 2026-03-11 14:01 ` Konrad Dybcio
0 siblings, 0 replies; 6+ messages in thread
From: Konrad Dybcio @ 2026-03-11 14:01 UTC (permalink / raw)
To: Abel Vesa
Cc: Stephen Boyd, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
linux-arm-msm, linux-kernel, devicetree
On 3/11/26 2:57 PM, Abel Vesa wrote:
> On 26-03-11 14:43:27, Konrad Dybcio wrote:
>> On 3/11/26 2:29 PM, Abel Vesa wrote:
>>> The SPMI multi-master Arbiter found on Eliza is version 7.2.0, yet
>>> driver-wise, still compatible with the one featured on Hamoa (X1E80100),
>>> which is 7.0.1.
>>>
>>> So document the Eliza compatible and allow Hamoa one as fallback.
>>>
>>> Signed-off-by: Abel Vesa <abel.vesa@oss.qualcomm.com>
>>> ---
>>> Changes in v2:
>>> - Instead of reworking the whole oneOf for compatibles,
>>> add Eliza similar to sar2130p.
>>> - Link to v1: https://patch.msgid.link/20260311-eliza-bindings-spmi-v1-1-1a20eec1179e@oss.qualcomm.com
>>> ---
>>> .../devicetree/bindings/spmi/qcom,x1e80100-spmi-pmic-arb.yaml | 3 +++
>>> 1 file changed, 3 insertions(+)
>>>
>>> diff --git a/Documentation/devicetree/bindings/spmi/qcom,x1e80100-spmi-pmic-arb.yaml b/Documentation/devicetree/bindings/spmi/qcom,x1e80100-spmi-pmic-arb.yaml
>>> index 08369fdd2161..4152469b3880 100644
>>> --- a/Documentation/devicetree/bindings/spmi/qcom,x1e80100-spmi-pmic-arb.yaml
>>> +++ b/Documentation/devicetree/bindings/spmi/qcom,x1e80100-spmi-pmic-arb.yaml
>>> @@ -23,6 +23,9 @@ allOf:
>>> properties:
>>> compatible:
>>> oneOf:
>>> + - items:
>>> + - const: qcom,eliza-spmi-pmic-arb
>>> + - const: qcom,x1e80100-spmi-pmic-arb
>>> - items:
>>> - const: qcom,sar2130p-spmi-pmic-arb
>>> - const: qcom,x1e80100-spmi-pmic-arb
>>
>> I think Krzysztof's comment was meant to suggest you shouldn't add an
>> intermittent items: entry for a single const (lone-x1e compatible), instead
>> passing:
>>
>> oneOf:
>> - items:
>> - enum:
>> - foo
>> - bar
>> - const: x1e
>> ...
>> - const: x1e
>
> Oh, my bad then. You mean like this?
>
> @@ -24,10 +24,9 @@ properties:
> compatible:
> oneOf:
> - items:
> - - const: qcom,eliza-spmi-pmic-arb
> - - const: qcom,x1e80100-spmi-pmic-arb
> - - items:
> - - const: qcom,sar2130p-spmi-pmic-arb
> + - enum:
> + - qcom,eliza-spmi-pmic-arb
> + - qcom,sar2130p-spmi-pmic-arb
> - const: qcom,x1e80100-spmi-pmic-arb
> - const: qcom,x1e80100-spmi-pmic-arb
Yes, but let's wait for krzk's response
Konrad
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2] dt-bindings: spmi: qcom,x1e80100-spmi-pmic-arb: Document Eliza compatible
2026-03-11 13:43 ` Konrad Dybcio
2026-03-11 13:57 ` Abel Vesa
@ 2026-03-13 8:34 ` Krzysztof Kozlowski
2026-03-13 9:02 ` Abel Vesa
1 sibling, 1 reply; 6+ messages in thread
From: Krzysztof Kozlowski @ 2026-03-13 8:34 UTC (permalink / raw)
To: Konrad Dybcio
Cc: Abel Vesa, Stephen Boyd, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, linux-arm-msm, linux-kernel, devicetree
On Wed, Mar 11, 2026 at 02:43:27PM +0100, Konrad Dybcio wrote:
> On 3/11/26 2:29 PM, Abel Vesa wrote:
> > The SPMI multi-master Arbiter found on Eliza is version 7.2.0, yet
> > driver-wise, still compatible with the one featured on Hamoa (X1E80100),
> > which is 7.0.1.
> >
> > So document the Eliza compatible and allow Hamoa one as fallback.
> >
> > Signed-off-by: Abel Vesa <abel.vesa@oss.qualcomm.com>
> > ---
> > Changes in v2:
> > - Instead of reworking the whole oneOf for compatibles,
> > add Eliza similar to sar2130p.
> > - Link to v1: https://patch.msgid.link/20260311-eliza-bindings-spmi-v1-1-1a20eec1179e@oss.qualcomm.com
> > ---
> > .../devicetree/bindings/spmi/qcom,x1e80100-spmi-pmic-arb.yaml | 3 +++
> > 1 file changed, 3 insertions(+)
> >
> > diff --git a/Documentation/devicetree/bindings/spmi/qcom,x1e80100-spmi-pmic-arb.yaml b/Documentation/devicetree/bindings/spmi/qcom,x1e80100-spmi-pmic-arb.yaml
> > index 08369fdd2161..4152469b3880 100644
> > --- a/Documentation/devicetree/bindings/spmi/qcom,x1e80100-spmi-pmic-arb.yaml
> > +++ b/Documentation/devicetree/bindings/spmi/qcom,x1e80100-spmi-pmic-arb.yaml
> > @@ -23,6 +23,9 @@ allOf:
> > properties:
> > compatible:
> > oneOf:
> > + - items:
> > + - const: qcom,eliza-spmi-pmic-arb
> > + - const: qcom,x1e80100-spmi-pmic-arb
> > - items:
> > - const: qcom,sar2130p-spmi-pmic-arb
> > - const: qcom,x1e80100-spmi-pmic-arb
>
> I think Krzysztof's comment was meant to suggest you shouldn't add an
> intermittent items: entry for a single const (lone-x1e compatible), instead
> passing:
Yeah, you added unnecessary line "items". Your hunk was CHANGING a const
into items+const, so I asked not to change that. If I meant the enum,
then my comment would be under enum.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2] dt-bindings: spmi: qcom,x1e80100-spmi-pmic-arb: Document Eliza compatible
2026-03-13 8:34 ` Krzysztof Kozlowski
@ 2026-03-13 9:02 ` Abel Vesa
0 siblings, 0 replies; 6+ messages in thread
From: Abel Vesa @ 2026-03-13 9:02 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Konrad Dybcio, Stephen Boyd, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, linux-arm-msm, linux-kernel, devicetree
On 26-03-13 09:34:34, Krzysztof Kozlowski wrote:
> On Wed, Mar 11, 2026 at 02:43:27PM +0100, Konrad Dybcio wrote:
> > On 3/11/26 2:29 PM, Abel Vesa wrote:
> > > The SPMI multi-master Arbiter found on Eliza is version 7.2.0, yet
> > > driver-wise, still compatible with the one featured on Hamoa (X1E80100),
> > > which is 7.0.1.
> > >
> > > So document the Eliza compatible and allow Hamoa one as fallback.
> > >
> > > Signed-off-by: Abel Vesa <abel.vesa@oss.qualcomm.com>
> > > ---
> > > Changes in v2:
> > > - Instead of reworking the whole oneOf for compatibles,
> > > add Eliza similar to sar2130p.
> > > - Link to v1: https://patch.msgid.link/20260311-eliza-bindings-spmi-v1-1-1a20eec1179e@oss.qualcomm.com
> > > ---
> > > .../devicetree/bindings/spmi/qcom,x1e80100-spmi-pmic-arb.yaml | 3 +++
> > > 1 file changed, 3 insertions(+)
> > >
> > > diff --git a/Documentation/devicetree/bindings/spmi/qcom,x1e80100-spmi-pmic-arb.yaml b/Documentation/devicetree/bindings/spmi/qcom,x1e80100-spmi-pmic-arb.yaml
> > > index 08369fdd2161..4152469b3880 100644
> > > --- a/Documentation/devicetree/bindings/spmi/qcom,x1e80100-spmi-pmic-arb.yaml
> > > +++ b/Documentation/devicetree/bindings/spmi/qcom,x1e80100-spmi-pmic-arb.yaml
> > > @@ -23,6 +23,9 @@ allOf:
> > > properties:
> > > compatible:
> > > oneOf:
> > > + - items:
> > > + - const: qcom,eliza-spmi-pmic-arb
> > > + - const: qcom,x1e80100-spmi-pmic-arb
> > > - items:
> > > - const: qcom,sar2130p-spmi-pmic-arb
> > > - const: qcom,x1e80100-spmi-pmic-arb
> >
> > I think Krzysztof's comment was meant to suggest you shouldn't add an
> > intermittent items: entry for a single const (lone-x1e compatible), instead
> > passing:
>
> Yeah, you added unnecessary line "items". Your hunk was CHANGING a const
> into items+const, so I asked not to change that. If I meant the enum,
> then my comment would be under enum.
Oh, my bad then.
Will respin with v2 changes but the items dropped.
Thanks for reviewing.
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2026-03-13 9:02 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-03-11 13:29 [PATCH v2] dt-bindings: spmi: qcom,x1e80100-spmi-pmic-arb: Document Eliza compatible Abel Vesa
2026-03-11 13:43 ` Konrad Dybcio
2026-03-11 13:57 ` Abel Vesa
2026-03-11 14:01 ` Konrad Dybcio
2026-03-13 8:34 ` Krzysztof Kozlowski
2026-03-13 9:02 ` Abel Vesa
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®