[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <r642meaclfme7thmb6spkna6xiuhbjuqw6x32jndjqhwqjtbbr@qkcziz3cyxb6>
Date: Tue, 27 Jan 2026 17:09:04 +0200
From: Dmitry Baryshkov <dmitry.baryshkov@....qualcomm.com>
To: Vikash Garodia <vikash.garodia@....qualcomm.com>
Cc: Dikshita Agarwal <dikshita.agarwal@....qualcomm.com>,
Abhinav Kumar <abhinav.kumar@...ux.dev>,
Bryan O'Donoghue <bod@...nel.org>,
Mauro Carvalho Chehab <mchehab@...nel.org>,
Rob Herring <robh@...nel.org>,
Krzysztof Kozlowski <krzk+dt@...nel.org>,
Conor Dooley <conor+dt@...nel.org>,
Saravana Kannan <saravanak@...nel.org>, Joerg Roedel <joro@...tes.org>,
Will Deacon <will@...nel.org>, Robin Murphy <robin.murphy@....com>,
Stefan Schmidt <stefan.schmidt@...aro.org>,
Hans Verkuil <hverkuil@...nel.org>,
Krzysztof Kozlowski <krzk@...nel.org>,
Vishnu Reddy <busanna.reddy@....qualcomm.com>,
Hans Verkuil <hverkuil+cisco@...nel.org>,
linux-arm-msm@...r.kernel.org, linux-media@...r.kernel.org,
devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
iommu@...ts.linux.dev, Bryan O'Donoghue <bryan.odonoghue@...aro.org>
Subject: Re: [PATCH 1/7] media: dt-bindings: qcom-kaanapali-iris: Add
kaanapali video codec binding
On Mon, Jan 26, 2026 at 05:55:44PM +0530, Vikash Garodia wrote:
> Kaanapali SOC brings in the new generation of video IP i.e iris4. When
> compared to previous generation, iris3x, it has,
> - separate power domains for stream and pixel processing hardware blocks
> (bse and vpp).
> - additional power domain for apv codec.
> - power domains for individual pipes (VPPx).
> - different clocks and reset lines.
>
> iommu-map include all the different stream-ids which can be possibly
> generated by vpu4 hardware.
It's not how it can be defined.
>
> Signed-off-by: Vikash Garodia <vikash.garodia@....qualcomm.com>
> ---
> .../bindings/media/qcom,kaanapali-iris.yaml | 234 +++++++++++++++++++++
> 1 file changed, 234 insertions(+)
>
> +
> + iommu-map: true
This is totally underspecifified.
> +
> + memory-region:
> + maxItems: 1
> +
> +
> + iommu-map = <0x100 &apps_smmu 0x1940 0x0 0x1>,
> + <0x100 &apps_smmu 0x1a20 0x0 0x1>,
> + <0x100 &apps_smmu 0x1944 0x0 0x1>,
> + <0x101 &apps_smmu 0x1943 0x0 0x1>,
> + <0x200 &apps_smmu 0x1941 0x0 0x1>,
> + <0x200 &apps_smmu 0x1a21 0x0 0x1>,
> + <0x201 &apps_smmu 0x1945 0x0 0x1>,
> + <0x202 &apps_smmu 0x1946 0x0 0x1>,
> + <0x300 &apps_smmu 0x1a22 0x0 0x1>;
#define the functions in the ABI, provide them in the bindings.
> +
--
With best wishes
Dmitry
Powered by blists - more mailing lists