Re: [PATCH v3 0/5] Introduce "non-pixel" sub node within iris video node

Dmitry Baryshkov
2025-10-08T03:10:34Z

On Tue, Oct 07, 2025 at 11:10:28PM +0100, Bryan O'Donoghue wrote:
> On 07/10/2025 20:40, Dmitry Baryshkov wrote:
> > > Surely we could do
> > >
> > > #iommu-cells = <4>;
> > Because #iommu-cells is a part of the apps_smmu: device rather than a
> > part ofthe iris/venus/GPU/display/etc.
>
> What's to stop us extending the definition for qcom platforms ?
>
> Rather than jumping through hoops in drivers, we can just encode the data in
> the DT.

The IOMMU driver or a ARM SMMU device itself has nothing to do with the
way individual devices use it. As such, I don't think we should be
extending #iommu-cells in the SMMU device.

--
With best wishes
Dmitry