On Mon, Apr 27, 2026 at 02:43:38PM +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). > > 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..d8b356028e24c3c5b2e9b7f20e220db7d491ad68 > 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':
Please use consistent quotes, either ' or " > + const: 2 This means the child will use 64-bit addressing, but does it need that? Or do you need that to have 36-bit DMA addressing? If there are true no needs for above, usually recommendation is to use narrowe address space for children, so only 32-bit. > + > + '#size-cells': > + const: 2 Best regards, Krzysztof
