linux-kernel - Re: [PATCH v8 05/11] dt-bindings: arm: Add initial bindings for Nuvoton platform

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]

Message-ID: <6da2239a-32e7-c8c0-6be9-55f11b66ca1b@gmail.com>
Date:   Tue, 2 May 2023 14:49:37 +0800
From:   Jacky Huang <ychuang570808@...il.com>
To:     Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>,
        robh+dt@...nel.org, krzysztof.kozlowski+dt@...aro.org,
        lee@...nel.org, mturquette@...libre.com, sboyd@...nel.org,
        p.zabel@...gutronix.de, gregkh@...uxfoundation.org,
        jirislaby@...nel.org, tmaimon77@...il.com, catalin.marinas@....com,
        will@...nel.org
Cc:     devicetree@...r.kernel.org, linux-clk@...r.kernel.org,
        linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
        linux-serial@...r.kernel.org, arnd@...db.de, schung@...oton.com,
        mjchen@...oton.com, Jacky Huang <ychuang3@...oton.com>
Subject: Re: [PATCH v8 05/11] dt-bindings: arm: Add initial bindings for
 Nuvoton platform

Dear Krzysztof,


On 2023/5/2 下午 02:46, Krzysztof Kozlowski wrote:
> On 02/05/2023 08:40, Jacky Huang wrote:
>> Dear Krzysztof,
>>
>>
>>
>> On 2023/5/2 下午 02:31, Krzysztof Kozlowski wrote:
>>> On 02/05/2023 03:31, Jacky Huang wrote:
>>>> Dear Krzysztof,
>>>>
>>>>
>>>> On 2023/5/1 下午 05:50, Krzysztof Kozlowski wrote:
>>>>> On 25/04/2023 12:24, Jacky Huang wrote:
>>>>>> From: Jacky Huang <ychuang3@...oton.com>
>>>>>>
>>>>>> Move 'nuvoton,npcm-gcr.yaml' from 'arm/npcm' to 'soc/nuvoton'.
>>>>>> Rename the '/arm/npcm' directory to 'arm/nuvoton'. Additionally, add
>>>>>> bindings for ARMv8-based Nuvoton SoCs and platform boards, and include
>>>>>> the initial bindings for ma35d1 series development boards.
>>>>>>
>>>>>> Signed-off-by: Jacky Huang <ychuang3@...oton.com>
>>>>>> ---
>>>>>>     .../bindings/arm/nuvoton/nuvoton,ma35d1.yaml  | 30 +++++++++++++++++++
>>>>>>     .../npcm.yaml => nuvoton/nuvoton,npcm.yaml}   |  2 +-
>>>>>>     .../nuvoton/nuvoton,npcm-gcr.yaml}            |  2 +-
>>>>>>     3 files changed, 32 insertions(+), 2 deletions(-)
>>>>>>     create mode 100644 Documentation/devicetree/bindings/arm/nuvoton/nuvoton,ma35d1.yaml
>>>>> I don't see any improvements here. Path in maintainers is still broken.
>>>>>
>>>>> Best regards,
>>>>> Krzysztof
>>>>>
>>>> Does this line cover
>>>> 'Documentation/devicetree/bindings/arm/nuvoton/nuvoton,ma35d1.yaml'?
>>>> ==>  F:    Documentation/devicetree/bindings/*/*/*ma35*
>>>>
>>>> I ran 'get_maintainer.pl' on this patch, and it was able to find
>>>> maintainers and looked good.
>>>> I also ran 'make htmldocs' and did not see any warnings or errors
>>>> related to the .yaml files
>>>> in this patch. However, I am still unsure where the problem lies. Can
>>>> you suggest any
>>>> tools I can use to check for errors? I would greatly appreciate any
>>>> guidance you can provide.
>>> Apply the patches 1 to 5 and then check if all paths from maintainers
>>> are correct. I believe the path
>>> Documentation/devicetree/bindings/arm/npcm/ is not.
>>>
>>> I think I told the same last time with explanation that this patch must
>>> correct path.
>>>
>>> Best regards,
>>> Krzysztof
>>>
>> Okay, I think I should combine the patch 11 (for MAINTAINER)  into patch 5.
> I don't know what's there. The path which is changed (invalidated) here,
> should be updated here.
>
> Best regards,
> Krzysztof
>

I got it and learn the rule. Thank you very much.
I will fix it.

Best Regards,
Jacky Huang