linux-kernel - Re: [PATCH 2/2] riscv: dts: sifive: fu540-c000: align dma node name with dtschema

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <03e28a55-d3bd-f3e1-f418-557306d65505@microchip.com>
Date:   Tue, 19 Apr 2022 10:45:38 +0000
From:   <Conor.Dooley@...rochip.com>
To:     <krzysztof.kozlowski@...aro.org>, <green.wan@...ive.com>,
        <vkoul@...nel.org>, <robh+dt@...nel.org>, <krzk+dt@...nel.org>,
        <paul.walmsley@...ive.com>, <palmer@...belt.com>,
        <aou@...s.berkeley.edu>, <geert@...ux-m68k.org>,
        <alexandre.ghiti@...onical.com>, <palmer@...ive.com>,
        <dmaengine@...r.kernel.org>, <devicetree@...r.kernel.org>,
        <linux-riscv@...ts.infradead.org>, <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 2/2] riscv: dts: sifive: fu540-c000: align dma node name
 with dtschema

On 19/04/2022 10:09, Krzysztof Kozlowski wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
> 
> On 18/03/2022 17:20, Krzysztof Kozlowski wrote:
>> Fixes dtbs_check warnings like:
>>
>>    dma@...0000: $nodename:0: 'dma@...0000' does not match '^dma-controller(@.*)?$'
>>
>> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@...onical.com>
>> ---
>>   arch/riscv/boot/dts/sifive/fu540-c000.dtsi | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> Any comments here?

Not sure that this one is actually needed Krzysztof, Zong Li has a fix
for this in his series of fixes for the sifive pdma:
https://lore.kernel.org/linux-riscv/edd72c0cca1ebceddc032ff6ec2284e3f48c5ad3.1648461096.git.zong.li@sifive.com/

Maybe you could add your review to his version?