From: Jacky Huang <ychuang570808@gmail.com>
To: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
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>
Subject: Re: [PATCH v8 05/11] dt-bindings: arm: Add initial bindings for Nuvoton platform
Date: Tue, 2 May 2023 14:40:44 +0800 [thread overview]
Message-ID: <10ee47df-df1f-5c44-7d2b-d367ab7a6f29@gmail.com> (raw)
In-Reply-To: <34d70271-ff07-aea8-bcb7-81d39a55167c@linaro.org>
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@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
>
Okay, I think I should combine the patch 11 (for MAINTAINER) into patch 5.
Best Regards,
Jacky Huang
next prev parent reply other threads:[~2023-05-02 6:40 UTC|newest]
Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-04-25 10:24 [PATCH v8 00/11] Introduce Nuvoton ma35d1 SoC Jacky Huang
2023-04-25 10:24 ` [PATCH v8 01/11] arm64: Kconfig.platforms: Add config for Nuvoton MA35 platform Jacky Huang
2023-04-25 10:24 ` [PATCH v8 02/11] arm64: defconfig: Add support for Nuvoton MA35 family SoCs Jacky Huang
2023-04-25 10:24 ` [PATCH v8 03/11] dt-bindings: clock: nuvoton: add binding for ma35d1 clock controller Jacky Huang
2023-04-25 10:24 ` [PATCH v8 04/11] dt-bindings: reset: nuvoton: Document ma35d1 reset control Jacky Huang
2023-04-25 10:24 ` [PATCH v8 05/11] dt-bindings: arm: Add initial bindings for Nuvoton platform Jacky Huang
2023-04-25 19:02 ` Rob Herring
2023-05-01 9:50 ` Krzysztof Kozlowski
2023-05-02 1:31 ` Jacky Huang
2023-05-02 6:31 ` Krzysztof Kozlowski
2023-05-02 6:40 ` Jacky Huang [this message]
2023-05-02 6:46 ` Krzysztof Kozlowski
2023-05-02 6:49 ` Jacky Huang
2023-04-25 10:24 ` [PATCH v8 06/11] dt-bindings: serial: Document ma35d1 uart controller Jacky Huang
2023-04-25 10:24 ` [PATCH v8 07/11] arm64: dts: nuvoton: Add initial ma35d1 device tree Jacky Huang
2023-04-25 10:24 ` [PATCH v8 08/11] clk: nuvoton: Add clock driver for ma35d1 clock controller Jacky Huang
2023-04-25 11:30 ` Ilpo Järvinen
2023-04-26 4:55 ` Jacky Huang
2023-04-25 10:24 ` [PATCH v8 09/11] reset: Add Nuvoton ma35d1 reset driver support Jacky Huang
2023-04-25 10:57 ` Ilpo Järvinen
2023-04-26 0:44 ` Jacky Huang
2023-04-26 7:39 ` Philipp Zabel
2023-04-25 10:24 ` [PATCH v8 10/11] tty: serial: Add Nuvoton ma35d1 serial " Jacky Huang
2023-04-25 10:38 ` Christophe JAILLET
2023-04-27 7:22 ` Jacky Huang
2023-04-25 14:01 ` kernel test robot
2023-04-25 10:24 ` [PATCH v8 11/11] MAINTAINERS: Add entry for NUVOTON MA35 Jacky Huang
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=10ee47df-df1f-5c44-7d2b-d367ab7a6f29@gmail.com \
--to=ychuang570808@gmail.com \
--cc=arnd@arndb.de \
--cc=catalin.marinas@arm.com \
--cc=devicetree@vger.kernel.org \
--cc=gregkh@linuxfoundation.org \
--cc=jirislaby@kernel.org \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=krzysztof.kozlowski@linaro.org \
--cc=lee@kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-clk@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-serial@vger.kernel.org \
--cc=mjchen@nuvoton.com \
--cc=mturquette@baylibre.com \
--cc=p.zabel@pengutronix.de \
--cc=robh+dt@kernel.org \
--cc=sboyd@kernel.org \
--cc=schung@nuvoton.com \
--cc=tmaimon77@gmail.com \
--cc=will@kernel.org \
--cc=ychuang3@nuvoton.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox