1. Patchew
  2. linux
  3. View series

[PATCH] dt-bindings: interconnect: OSM L3: Add Eliza OSM L3 compatible

Abel Vesa posted 1 patch 1 month, 3 weeks ago 
Documentation/devicetree/bindings/interconnect/qcom,osm-l3.yaml | 1 +
1 file changed, 1 insertion(+)
[PATCH] dt-bindings: interconnect: OSM L3: Add Eliza OSM L3 compatible
Posted by Abel Vesa 1 month, 3 weeks ago 
Eliza, similarly to SDM845, uses OSM hardware for L3 scaling.
Document it.

Signed-off-by: Abel Vesa <abel.vesa@oss.qualcomm.com>
---
 Documentation/devicetree/bindings/interconnect/qcom,osm-l3.yaml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Documentation/devicetree/bindings/interconnect/qcom,osm-l3.yaml b/Documentation/devicetree/bindings/interconnect/qcom,osm-l3.yaml
index 4b9b98fbe8f2..6182599eb3c1 100644
--- a/Documentation/devicetree/bindings/interconnect/qcom,osm-l3.yaml
+++ b/Documentation/devicetree/bindings/interconnect/qcom,osm-l3.yaml
@@ -28,6 +28,7 @@ properties:
           - const: qcom,osm-l3
       - items:
           - enum:
+              - qcom,eliza-epss-l3
               - qcom,sa8775p-epss-l3
               - qcom,sc7280-epss-l3
               - qcom,sc8280xp-epss-l3

---
base-commit: 6de23f81a5e08be8fbf5e8d7e9febc72a5b5f27f
change-id: 20260222-eliza-bindings-interconnect-epss-l3-1f40101ed6de

Best regards,
--  
Abel Vesa <abel.vesa@oss.qualcomm.com>
Re: [PATCH] dt-bindings: interconnect: OSM L3: Add Eliza OSM L3 compatible
Posted by Krzysztof Kozlowski 1 month, 3 weeks ago 
On Mon, Feb 23, 2026 at 10:49:04AM +0200, Abel Vesa wrote:
> Eliza, similarly to SDM845, uses OSM hardware for L3 scaling.
> Document it.
> 
> Signed-off-by: Abel Vesa <abel.vesa@oss.qualcomm.com>
> ---
>  Documentation/devicetree/bindings/interconnect/qcom,osm-l3.yaml | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/Documentation/devicetree/bindings/interconnect/qcom,osm-l3.yaml b/Documentation/devicetree/bindings/interconnect/qcom,osm-l3.yaml
> index 4b9b98fbe8f2..6182599eb3c1 100644
> --- a/Documentation/devicetree/bindings/interconnect/qcom,osm-l3.yaml
> +++ b/Documentation/devicetree/bindings/interconnect/qcom,osm-l3.yaml
> @@ -28,6 +28,7 @@ properties:
>            - const: qcom,osm-l3
>        - items:
>            - enum:
> +              - qcom,eliza-epss-l3

I see EPSS, not OSM here. This is also list without SDM845, so I don't
understand explanation in commit msg at all.

>                - qcom,sa8775p-epss-l3
>                - qcom,sc7280-epss-l3
>                - qcom,sc8280xp-epss-l3

Best regards,
Krzysztof
Re: [PATCH] dt-bindings: interconnect: OSM L3: Add Eliza OSM L3 compatible
Posted by Abel Vesa 1 month, 3 weeks ago 
On 26-02-24 08:29:09, Krzysztof Kozlowski wrote:
> On Mon, Feb 23, 2026 at 10:49:04AM +0200, Abel Vesa wrote:
> > Eliza, similarly to SDM845, uses OSM hardware for L3 scaling.
> > Document it.
> > 
> > Signed-off-by: Abel Vesa <abel.vesa@oss.qualcomm.com>
> > ---
> >  Documentation/devicetree/bindings/interconnect/qcom,osm-l3.yaml | 1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/Documentation/devicetree/bindings/interconnect/qcom,osm-l3.yaml b/Documentation/devicetree/bindings/interconnect/qcom,osm-l3.yaml
> > index 4b9b98fbe8f2..6182599eb3c1 100644
> > --- a/Documentation/devicetree/bindings/interconnect/qcom,osm-l3.yaml
> > +++ b/Documentation/devicetree/bindings/interconnect/qcom,osm-l3.yaml
> > @@ -28,6 +28,7 @@ properties:
> >            - const: qcom,osm-l3
> >        - items:
> >            - enum:
> > +              - qcom,eliza-epss-l3
> 
> I see EPSS, not OSM here. This is also list without SDM845, so I don't
> understand explanation in commit msg at all.

So there are two issues here:

1. EPSS bindings are handled by the OSM schema. I believe this is by
design as they basically do the same thing. Interconnect providers for
L3 scaling. So the compatible is in the right schema.

2. The commit message should've said SM8650 instead. Will fix and
respin.
Re: [PATCH] dt-bindings: interconnect: OSM L3: Add Eliza OSM L3 compatible
Posted by Krzysztof Kozlowski 1 month, 3 weeks ago 
On 24/02/2026 09:10, Abel Vesa wrote:
> On 26-02-24 08:29:09, Krzysztof Kozlowski wrote:
>> On Mon, Feb 23, 2026 at 10:49:04AM +0200, Abel Vesa wrote:
>>> Eliza, similarly to SDM845, uses OSM hardware for L3 scaling.
>>> Document it.
>>>
>>> Signed-off-by: Abel Vesa <abel.vesa@oss.qualcomm.com>
>>> ---
>>>  Documentation/devicetree/bindings/interconnect/qcom,osm-l3.yaml | 1 +
>>>  1 file changed, 1 insertion(+)
>>>
>>> diff --git a/Documentation/devicetree/bindings/interconnect/qcom,osm-l3.yaml b/Documentation/devicetree/bindings/interconnect/qcom,osm-l3.yaml
>>> index 4b9b98fbe8f2..6182599eb3c1 100644
>>> --- a/Documentation/devicetree/bindings/interconnect/qcom,osm-l3.yaml
>>> +++ b/Documentation/devicetree/bindings/interconnect/qcom,osm-l3.yaml
>>> @@ -28,6 +28,7 @@ properties:
>>>            - const: qcom,osm-l3
>>>        - items:
>>>            - enum:
>>> +              - qcom,eliza-epss-l3
>>
>> I see EPSS, not OSM here. This is also list without SDM845, so I don't
>> understand explanation in commit msg at all.
> 
> So there are two issues here:
> 
> 1. EPSS bindings are handled by the OSM schema. I believe this is by
> design as they basically do the same thing. Interconnect providers for
> L3 scaling. So the compatible is in the right schema.

We do not talk here about schema or bindings at all. Look at your commit
msg:
"uses OSM hardware"
so again: EPSS or OSM.

> 
> 2. The commit message should've said SM8650 instead. Will fix and
> respin.
> 


Best regards,
Krzysztof
Re: [PATCH] dt-bindings: interconnect: OSM L3: Add Eliza OSM L3 compatible
Posted by Abel Vesa 1 month, 3 weeks ago 
On 26-02-24 09:14:01, Krzysztof Kozlowski wrote:
> On 24/02/2026 09:10, Abel Vesa wrote:
> > On 26-02-24 08:29:09, Krzysztof Kozlowski wrote:
> >> On Mon, Feb 23, 2026 at 10:49:04AM +0200, Abel Vesa wrote:
> >>> Eliza, similarly to SDM845, uses OSM hardware for L3 scaling.
> >>> Document it.
> >>>
> >>> Signed-off-by: Abel Vesa <abel.vesa@oss.qualcomm.com>
> >>> ---
> >>>  Documentation/devicetree/bindings/interconnect/qcom,osm-l3.yaml | 1 +
> >>>  1 file changed, 1 insertion(+)
> >>>
> >>> diff --git a/Documentation/devicetree/bindings/interconnect/qcom,osm-l3.yaml b/Documentation/devicetree/bindings/interconnect/qcom,osm-l3.yaml
> >>> index 4b9b98fbe8f2..6182599eb3c1 100644
> >>> --- a/Documentation/devicetree/bindings/interconnect/qcom,osm-l3.yaml
> >>> +++ b/Documentation/devicetree/bindings/interconnect/qcom,osm-l3.yaml
> >>> @@ -28,6 +28,7 @@ properties:
> >>>            - const: qcom,osm-l3
> >>>        - items:
> >>>            - enum:
> >>> +              - qcom,eliza-epss-l3
> >>
> >> I see EPSS, not OSM here. This is also list without SDM845, so I don't
> >> understand explanation in commit msg at all.
> > 
> > So there are two issues here:
> > 
> > 1. EPSS bindings are handled by the OSM schema. I believe this is by
> > design as they basically do the same thing. Interconnect providers for
> > L3 scaling. So the compatible is in the right schema.
> 
> We do not talk here about schema or bindings at all. Look at your commit
> msg:
> "uses OSM hardware"
> so again: EPSS or OSM.

They are basically the same thing.

Will use EPSS instead in the next respin.

Patchew 2.1-devel-ea86937

© 2016 - 2026 Red Hat, Inc.