Re: [PATCH] dt-bindings: mmc: renesas,sdhi: Add iommus property

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Geert,

Thank you for the review.

On Tue, Aug 30, 2022 at 8:06 AM Geert Uytterhoeven <geert@xxxxxxxxxxxxxx> wrote:
>
> Hi Prabhakar,
>
> Thanks for your patch!
>
> On Mon, Aug 29, 2022 at 11:43 PM Lad Prabhakar
> <prabhakar.mahadev-lad.rj@xxxxxxxxxxxxxx> wrote:
> > The SDHI block on Renesas R-Car and RZ/G2 SoC's make use of IOMMU.
>
> blocks ... SoCs
>
> > This patch fixes the below dtbs_check warnings:
> > arch/arm64/boot/dts/renesas/r8a774e1-hihope-rzg2h-ex-idk-1110wr.dtb: mmc@ee100000: Unevaluated properties are not allowed ('iommus' was unexpected)
> >         From schema: Documentation/devicetree/bindings/mmc/renesas,sdhi.yaml
> >
> > Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@xxxxxxxxxxxxxx>
>
> Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx>
>
> > --- a/Documentation/devicetree/bindings/mmc/renesas,sdhi.yaml
> > +++ b/Documentation/devicetree/bindings/mmc/renesas,sdhi.yaml
> > @@ -77,6 +77,9 @@ properties:
> >
> >    clock-names: true
> >
> > +  iommus:
> > +    maxItems: 1
>
> I would insert iommus below dma-names, as iommus belongs to dmas,
> logically (yes I know dmas is not used on R-Car Gen3 and RZ/G2, as
> SDHI has its own DMA controller).
>
Ok, I will move it after dmas along with an updated commit message and
send a v2.

Cheers,
Prabhakar



[Index of Archives]     [Linux Samsung SOC]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux