On Sat, Jan 11, 2025 at 04:14:27PM +0000, Raviteja Laggyshetty wrote: > Add Epoch Subsystem (EPSS) L3 interconnect provider binding on > SA8775P SoCs. 1. And why is this not compatible with sm8250? There was lengthy discussion and no outcome of it managed to get to commit msg. Really, so we are going to repeat everything again and you will not get any acks. You have entire commit msg to explain things but instead you repeat what the patch does. We can read the diff for that. 2. Binding *ALWAYS* comes before the user. > > Signed-off-by: Raviteja Laggyshetty <quic_rlaggysh@xxxxxxxxxxx> > --- > 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 21dae0b92819..94f7f283787a 100644 > --- a/Documentation/devicetree/bindings/interconnect/qcom,osm-l3.yaml > +++ b/Documentation/devicetree/bindings/interconnect/qcom,osm-l3.yaml > @@ -33,6 +33,7 @@ properties: > - qcom,sm6375-cpucp-l3 > - qcom,sm8250-epss-l3 > - qcom,sm8350-epss-l3 > + - qcom,sa8775p-epss-l3 > - const: qcom,epss-l3 Your driver suggests this is not really true - it is not compatible with qcom,epss-l3. Maybe it is, maybe not, no clue, commit explains nothing. Best regards, Krzysztof
