Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <34d70271-ff07-aea8-bcb7-81d39a55167c@linaro.org>
Date:   Tue, 2 May 2023 08:31:07 +0200
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.10.1
Subject: Re: [PATCH v8 05/11] dt-bindings: arm: Add initial bindings for
 Nuvoton platform
To:     Jacky Huang <ychuang570808@gmail.com>, robh+dt@kernel.org,
        krzysztof.kozlowski+dt@linaro.org, lee@kernel.org,
        mturquette@baylibre.com, sboyd@kernel.org, p.zabel@pengutronix.de,
        gregkh@linuxfoundation.org, jirislaby@kernel.org,
        tmaimon77@gmail.com, catalin.marinas@arm.com, will@kernel.org
Cc:     devicetree@vger.kernel.org, linux-clk@vger.kernel.org,
        linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
        linux-serial@vger.kernel.org, arnd@arndb.de, schung@nuvoton.com,
        mjchen@nuvoton.com, Jacky Huang <ychuang3@nuvoton.com>
References: <20230425102418.185783-1-ychuang570808@gmail.com>
 <20230425102418.185783-6-ychuang570808@gmail.com>
 <2b79021d-3793-5f2d-8659-5d0d8fc78017@linaro.org>
 <ded0c68a-e0b3-b106-e24d-7d9087c6ca4c@gmail.com>
Content-Language: en-US
From:   Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
In-Reply-To: <ded0c68a-e0b3-b106-e24d-7d9087c6ca4c@gmail.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
Precedence: bulk

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@nuvoton.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@nuvoton.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