[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <28be175a-8e9b-937a-de35-bcc218006fe9@opensource.wdc.com>
Date: Fri, 26 Nov 2021 09:01:28 +0900
From: Damien Le Moal <damien.lemoal@...nsource.wdc.com>
To: Geert Uytterhoeven <geert@...ux-m68k.org>,
Eugeniy Paltsev <Eugeniy.Paltsev@...opsys.com>,
Vinod Koul <vkoul@...nel.org>,
Rob Herring <robh+dt@...nel.org>,
Heng Sia <jee.heng.sia@...el.com>
Cc: "dmaengine@...r.kernel.org" <dmaengine@...r.kernel.org>,
"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
"linux-riscv@...ts.infradead.org" <linux-riscv@...ts.infradead.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] dt-bindings: dma: snps,dw-axi-dmac: Document optional
reset
On 2021/11/26 0:20, Geert Uytterhoeven wrote:
> "make dtbs_check":
>
> Documentation/devicetree/bindings/dma/snps,dw-axi-dmac.yaml
> arch/riscv/boot/dts/canaan/sipeed_maix_bit.dt.yaml: dma-controller@...00000: 'resets' does not match any of the regexes: 'pinctrl-[0-9]+'
> From schema: Documentation/devicetree/bindings/dma/snps,dw-axi-dmac.yaml
>
> The Synopsys DesignWare AXI DMA Controller on the Canaan K210 SoC
> exposes its reset signal.
>
> Signed-off-by: Geert Uytterhoeven <geert@...ux-m68k.org>
> ---
> Documentation/devicetree/bindings/dma/snps,dw-axi-dmac.yaml | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/dma/snps,dw-axi-dmac.yaml b/Documentation/devicetree/bindings/dma/snps,dw-axi-dmac.yaml
> index 90d9274e5464e396..06ddffaada295171 100644
> --- a/Documentation/devicetree/bindings/dma/snps,dw-axi-dmac.yaml
> +++ b/Documentation/devicetree/bindings/dma/snps,dw-axi-dmac.yaml
> @@ -53,6 +53,9 @@ properties:
> minimum: 1
> maximum: 8
>
> + resets:
> + maxItems: 1
> +
> snps,dma-masters:
> description: |
> Number of AXI masters supported by the hardware.
>
Looks good to me.
Reviewed-by: Damien Le Moal <damien.lemoal@...nsource.wdc.com>
--
Damien Le Moal
Western Digital Research
Powered by blists - more mailing lists