Re: [PATCH v2 02/12] dt-bindings: media: qcom,sm8550-iris: Add context bank subnodes
Dmitry Baryshkov <[email protected]> Thu, 6 Aug 2026 11:51:41 +0300
| Newsgroups | org.kernel.vger.linux-media,org.kernel.vger.linux-arm-msm,org.kernel.vger.linux-devicetree,org.kernel.vger.linux-kernel,org.kernel.vger.stable |
|---|---|
| Message-ID | <mbmibfbkba35xdytj7ukreqku5jwwbvfwqll2yv2vhy4ebnaf6@ntlhflnfznea> |
On Fri, Jul 31, 2026 at 03:36:51PM -0500, Rob Herring (Arm) wrote:
> On Fri, 31 Jul 2026 23:52:17 +0530, Vikash Garodia wrote:
> > The VPU issues DMA through several SMMU streams, and the hardware does
> > not give every stream the same addressable range. The non-pixel stream
> > is restricted to use 0-600MB of IOVA space, while the pixel stream can
> > address the full range:
> >
> > Fixes: 41661853ae8e ("arm64: dts: qcom: sm8550: add iris DT node")
> > Cc: [email protected]
> > Co-developed-by: Vishnu Reddy <[email protected]>
> > Signed-off-by: Vishnu Reddy <[email protected]>
> > Tested-by: Daniel J Blueman <[email protected]>
> > Signed-off-by: Vikash Garodia <[email protected]>
> > ---
> > .../devicetree/bindings/media/qcom,sm8550-iris.yaml | 18 +++++++++++++++---
> > 1 file changed, 15 insertions(+), 3 deletions(-)
> >
>
> My bot found errors running 'make dt_binding_check' on your patch:
>
> yamllint warnings/errors:
>
> dtschema/dtc warnings/errors:
> Documentation/devicetree/bindings/media/qcom,sm8550-iris.example.dts:67.13-24: Warning (dma_ranges_format): /example-0/video-codec@aa00000:dma-ranges: empty "dma-ranges" property but its #address-cells (2) differs from /example-0 (1)
> Documentation/devicetree/bindings/media/qcom,sm8550-iris.example.dts:67.13-24: Warning (dma_ranges_format): /example-0/video-codec@aa00000:dma-ranges: empty "dma-ranges" property but its #size-cells (2) differs from /example-0 (1)
Vikash, you promised to verify the examples. What has happened?
Also, does it make any sense to you? The device in the example has
32-bit addressing, now you state that _subdevices_ have 64-bit
addressing. Why? How are those 64 bits mapped to the parent 32 bits?
>
> doc reference errors (make refcheckdocs):
>
--
With best wishes
Dmitry