On Fri, May 08, 2026 at 12:49:25AM +0200, Loic Poulain wrote: > Extend the qcm2290 CAMSS binding to describe CAMSS as a simple bus by > allowing child ISP nodes. Add the required address and size cells, as > well as ranges, and validate ISP subnodes against the existing > qcom,qcm2290-camss-ope schema. > > On qcm2290 the OPE (Offline Processing Engine) is a memory-to-memory > ISP (Image Signal Processor). > > The address-cells for the CAMSS bus is <2> (64-bit) as related > DMA/IOMMUs offer 36-bit addressing support. > > Signed-off-by: Loic Poulain <[email protected]> > --- > .../devicetree/bindings/media/qcom,qcm2290-camss.yaml | 13 > +++++++++++++ > 1 file changed, 13 insertions(+) > > diff --git a/Documentation/devicetree/bindings/media/qcom,qcm2290-camss.yaml > b/Documentation/devicetree/bindings/media/qcom,qcm2290-camss.yaml > index > 391d0f6f67ef5fdfea31dd3683477561516b1556..e70f4cd1348b8065ee9f0e4448185cfd8a8fb7de > 100644 > --- a/Documentation/devicetree/bindings/media/qcom,qcm2290-camss.yaml > +++ b/Documentation/devicetree/bindings/media/qcom,qcm2290-camss.yaml > @@ -52,6 +52,14 @@ properties: > - const: vfe1 > - const: vfe1_cphy_rx > > + "#address-cells": > + const: 2 > + > + "#size-cells": > + const: 2 > + > + ranges: true > + > interrupts: > maxItems: 8 > > @@ -117,6 +125,11 @@ properties: > required: > - data-lanes > > +patternProperties: > + "^isp@[0-9a-f]+$": > + $ref: /schemas/media/qcom,qcm2290-camss-ope.yaml > + unevaluatedProperties: false
The referenced schema will get applied twice. Instead, just define here what compatible must be present. IIRC, the QCom display bindings are structured like that if you need an example. Rob
