On 4/28/26 8:35 AM, Krzysztof Kozlowski wrote: > 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.
I can't find data for this platform, but on e.g. Hamoa, most camera components can address 36b Konrad
