linux-kernel - Re: [PATCH] arm64: dts: xilinx: Align IOMMU nodename 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: Windows password security audit tool. GUI, reports in PDF.

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

Date:   Mon, 17 Aug 2020 12:31:25 +0200
From:   Michal Simek <michal.simek@...inx.com>
To:     Krzysztof Kozlowski <krzk@...nel.org>,
        Michal Simek <michal.simek@...inx.com>
Cc:     Rob Herring <robh+dt@...nel.org>,
        Nava kishore Manne <nava.manne@...inx.com>,
        Rajan Vaja <rajan.vaja@...inx.com>,
        Kalyani Akula <kalyani.akula@...inx.com>,
        Manish Narani <manish.narani@...inx.com>,
        devicetree@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH] arm64: dts: xilinx: Align IOMMU nodename with dtschema

Hi,

On 16. 08. 20 13:21, Krzysztof Kozlowski wrote:
> On Wed, Jul 08, 2020 at 02:46:55PM +0200, Michal Simek wrote:
>>
>>
>> On 29. 06. 20 10:17, Krzysztof Kozlowski wrote:
>>> Fix dtschema validator warnings like:
>>>     smmu@...00000: $nodename:0: 'smmu@...00000' does not match '^iommu@[0-9a-f]*'
>>>
>>> Signed-off-by: Krzysztof Kozlowski <krzk@...nel.org>
>>> ---
>>>  arch/arm64/boot/dts/xilinx/zynqmp.dtsi | 2 +-
>>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/arch/arm64/boot/dts/xilinx/zynqmp.dtsi b/arch/arm64/boot/dts/xilinx/zynqmp.dtsi
>>> index 9174ddc76bdc..2b82206eba02 100644
>>> --- a/arch/arm64/boot/dts/xilinx/zynqmp.dtsi
>>> +++ b/arch/arm64/boot/dts/xilinx/zynqmp.dtsi
>>> @@ -601,7 +601,7 @@
>>>  			power-domains = <&zynqmp_firmware PD_SD_1>;
>>>  		};
>>>  
>>> -		smmu: smmu@...00000 {
>>> +		smmu: iommu@...00000 {
>>>  			compatible = "arm,mmu-500";
>>>  			reg = <0x0 0xfd800000 0x0 0x20000>;
>>>  			status = "disabled";
>>>
>>
>> Applied.
> 
> Hi Michal,
> 
> What happened with this patch? I cannot find it in Linus' nor in
> next/master trees.

Laurent sent one patch before yours which depend on binding doc merged
from different tree. Here is the link.
https://lore.kernel.org/lkml/20200629120054.29338-4-laurent.pinchart@ideasonboard.com/

That's why I didn't send my PR for rc1.
But your patch is listed here.
https://github.com/Xilinx/linux-xlnx/commits/zynqmp/dt

I will rebase it based on rc1 and retest and send PR to arm-soc guys soon.

Thanks,
Michal