Messages in this thread | | | Date | Wed, 16 Nov 2022 17:06:17 -0600 | Subject | Re: [PATCH 1/2] media: dt-bindings: allwinner: video-engine: Fix number of IOMMU channels | From | Samuel Holland <> |
| |
On 11/16/22 13:31, Jernej Skrabec wrote: > Cedrus (video engine) on Allwinner H6 actually uses two IOMMU channel, > not just one. However, Cedrus on SoCs like D1 only uses one channel. > > Allow up to 2 IOMMU channels. > > Fixes: 62a8ccf3a248 ("arm64: dts: allwinner: h6: Fix Cedrus IOMMU usage") > Signed-off-by: Jernej Skrabec <jernej.skrabec@gmail.com> > --- > .../bindings/media/allwinner,sun4i-a10-video-engine.yaml | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/media/allwinner,sun4i-a10-video-engine.yaml b/Documentation/devicetree/bindings/media/allwinner,sun4i-a10-video-engine.yaml > index 541325f900a1..257bb372d166 100644 > --- a/Documentation/devicetree/bindings/media/allwinner,sun4i-a10-video-engine.yaml > +++ b/Documentation/devicetree/bindings/media/allwinner,sun4i-a10-video-engine.yaml > @@ -55,7 +55,7 @@ properties: > description: Phandle to the device SRAM > > iommus: > - maxItems: 1 > + maxItems: 2
You need to add minItems: 1, or else this will require two items.
Regards, Samuel
> > memory-region: > maxItems: 1
| |