Re: [PATCH 1/3] dt-bindings: clock: add binding for MA35D1 clock controller

From: MS10 YCHuang3 <ychuang3@nuvoton.com>
To: Krzysztof Kozlowski <krzk@kernel.org>,
	robh+dt@kernel.org, mturquette@baylibre.com, sboyd@kernel.org,
	arnd@arndb.de, olof@lixom.net
Cc: linux-kernel@vger.kernel.org, devicetree@vger.kernel.org,
	linux-clk@vger.kernel.org, soc@kernel.org,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 1/3] dt-bindings: clock: add binding for MA35D1 clock controller
Date: Tue, 8 Mar 2022 12:15:15 +0800	[thread overview]
Message-ID: <27bfea4c-40b7-9434-8078-317c1b087be1@nuvoton.com> (raw)
In-Reply-To: <81e8d2cb-c935-fb83-4a29-5ec9dfce16e8@kernel.org>

On 2022/3/7 下午 06:21, Krzysztof Kozlowski wrote:
> On 07/03/2022 10:19, Jacky Huang wrote:
>> Add the dt-bindings header for Nuvoton MA35D1, that gets shared
>> between the clock controller and clock references in the dts.
>>
>> Signed-off-by: Jacky Huang <ychuang3@nuvoton.com>
>> ---
>>   .../dt-bindings/clock/nuvoton,ma35d1-clk.h    | 262 ++++++++++++++++++
>>   1 file changed, 262 insertions(+)
>>   create mode 100644 include/dt-bindings/clock/nuvoton,ma35d1-clk.h
>>
>> diff --git a/include/dt-bindings/clock/nuvoton,ma35d1-clk.h b/include/dt-bindings/clock/nuvoton,ma35d1-clk.h
>> new file mode 100644
>> index 000000000000..3634e5edcac5
>> --- /dev/null
>> +++ b/include/dt-bindings/clock/nuvoton,ma35d1-clk.h
>> @@ -0,0 +1,262 @@
>> +/* SPDX-License-Identifier: GPL-2.0+ OR MIT */
>> +/*
>> + * Copyright (c) 2022 Nuvoton Technology Corporation.
>> + */
>> +
>> +#ifndef __DT_BINDINGS_MA35D1_CLK_H
> Please use header guard matching more parts of path, so:
> __DT_BINDINGS_CLOCK_ NUVOTON_MA35D1_CLK_H
>
>
>> +#define __DT_BINDINGS_MA35D1_CLK_H
>> +
> (...)
>
>> ________________________________
>> ________________________________
>>   The privileged confidential information contained in this email is intended for use only by the addressees as indicated by the original sender of this email. If you are not the addressee indicated in this email or are not responsible for delivery of the email to such a person, please kindly reply to the sender indicating this fact and delete all copies of it from your computer and network server immediately. Your cooperation is highly appreciated. It is advised that any unauthorized use of confidential information of Nuvoton is strictly prohibited; and any information in this email irrelevant to the official business of Nuvoton shall be deemed as neither given nor endorsed by Nuvoton.
> Since I was not intended recipient, I am going to NAK it and delete all
> copies.
>
> NAK. Cannot be included in Linux kernel.
>
> Deleted from my computer, but I cannot delete it from network servers.
> There are just too many of them...
>
> Best regards,
> Krzysztof

Dear Sir,

I didn't notice the additional message in the email, it was my mistake.

The problem with the mail server is still unresolved, and there may
still be additional messages in this reply.

We will fix it and re-send a clean patch series.

Thanks for your reminding.

Best Regards,

Jacky

________________________________
________________________________
 The privileged confidential information contained in this email is intended for use only by the addressees as indicated by the original sender of this email. If you are not the addressee indicated in this email or are not responsible for delivery of the email to such a person, please kindly reply to the sender indicating this fact and delete all copies of it from your computer and network server immediately. Your cooperation is highly appreciated. It is advised that any unauthorized use of confidential information of Nuvoton is strictly prohibited; and any information in this email irrelevant to the official business of Nuvoton shall be deemed as neither given nor endorsed by Nuvoton.