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: <bfc27a11-9be1-2391-3d26-62aa29fac190@amlogic.com>
Date:   Tue, 16 May 2023 17:25:55 +0800
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101
 Thunderbird/102.10.1
Subject: Re: [PATCH V8 RESEND 0/4] Add S4 SoC PLL and Peripheral clock
Content-Language: en-US
To:     Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
        linux-clk@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
        linux-amlogic@lists.infradead.org, linux-kernel@vger.kernel.org,
        devicetree@vger.kernel.org, Rob Herring <robh+dt@kernel.org>,
        Neil Armstrong <neil.armstrong@linaro.org>,
        Jerome Brunet <jbrunet@baylibre.com>,
        Kevin Hilman <khilman@baylibre.com>,
        Michael Turquette <mturquette@baylibre.com>,
        Stephen Boyd <sboyd@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Conor Dooley <conor+dt@kernel.org>,
        Martin Blumenstingl <martin.blumenstingl@googlemail.com>
Cc:     kelvin.zhang@amlogic.com, qi.duan@amlogic.com
References: <20230516064736.10270-1-yu.tu@amlogic.com>
 <c4d30304-08d8-6a31-8f91-7840fe2922fb@linaro.org>
From:   Yu Tu <yu.tu@amlogic.com>
In-Reply-To: <c4d30304-08d8-6a31-8f91-7840fe2922fb@linaro.org>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?UHBPQUJZbnVVZGxwSW9mS3NQMmU0aWQ3R2QzdDhwMzlFTVpPTEowT3lhV1V1?=
 =?utf-8?B?djlVdWhtYUhBMmgxRjFZNUh3VDJoOEtKOTdGQUhleC9aOGVLbUk2V091d1h6?=
 =?utf-8?B?bXVBOVVYN2xNLzBXU1ROYU1KU3lhcUI1dEd5ZC8zTngzb1VBK0dQdDk1cU1R?=
 =?utf-8?B?dmsyWEx6blVFdmNxR2JNSDY0T25PVE5IblNzMmMrcG5DcHJjUi84OGE5dXNS?=
 =?utf-8?B?QzFORk5lQXNlS2pUakloYUxLcHdWdmNmR0xVUTczSVliejVEdlY3dzR0REVm?=
 =?utf-8?B?WVFnWGkzTVEyVzdUN0x5SkFvMWJNZTBmL3o4cHpSZ3I4d3dNK2lnTjRlM0JZ?=
 =?utf-8?B?ZTR6dmU2VlcwQjdSNkVuM0Y5UEphSGVEbkx6KzhyK0ZjWEdSNkZleGh0NUV6?=
 =?utf-8?B?c2dBVUwzT0pkaDVIeHplVmhqWW52MzkwditwQ0JpRGY1ZjdUbjFZQWVRMS9x?=
 =?utf-8?B?TXhYU0pSMEFVUDZNWVJkb08rT2Zuc2pJV2pxS1J2R3JtNmFCM2pxUHd0U0tt?=
 =?utf-8?B?R05FMjFaeGlnZ3h6SGRxSFVEY0ZPMnhLWm1xZTVyR3pGR0JCU2hHYiszKzJR?=
 =?utf-8?B?T01DUlVDZHByR08rdUpaaXp5VXFKbHFpajAxTmxyWFZCZ1pCb0dzbVNhSnpL?=
 =?utf-8?B?bndMdlBmeXNlend3YmJjZmpEYUlCY1k5blFMeUtxWDMzNFE2V0JPTEVlZ1dQ?=
 =?utf-8?B?bGx1QXgxbXFUV1Y5OHd2U1RodzhHL2oxVDFHalp1Q0RUZ3pxb3hNVk9SelZw?=
 =?utf-8?B?OERvSzF4VEloV0R6TjhLNlFETjlYVjJCMVlPNkxzQ3cwK0VMTTNxeEpiVlgx?=
 =?utf-8?B?RG95bWo3NE5QYmZXT2pQdFZzeS96dzc1N1RLRUg3Mkl3b1hHTzFqY3hVN240?=
 =?utf-8?B?VzV6NlhkcVRGR2RqWkVNTVBya3JHU2NQay9WRGN6cmxwSmlGUm9COEpNd2pa?=
 =?utf-8?B?STJPNDBnRDVLckpTaldiZTd6RGQ3cFAyMmlMVTRSNFVmWWdqME1TWHlhOHM1?=
 =?utf-8?B?T211OTJEMHVEbmdnKzczUmdwbUI5OWdCTnI1b1RVeTZjZ1pBUXFYOTVrSVUz?=
 =?utf-8?B?dGZyVEdmc2JPdlJ4S1dXenQzM1I2cWhmWUl2MG5YdW4zUC9HTzVWazVvWGdu?=
 =?utf-8?B?cEp2S0k4UGxmVUN2cUxiNHBsT2pKS3UrZmxXdTQwalA1OHRJMCtRNzhlKzl0?=
 =?utf-8?B?TW1oUVdYbUxUZDR3RHBYR3hDb2x5TytJc2FXVU1FdHdjSnZMSGlKcldyV3Nx?=
 =?utf-8?B?RE02R2FOSnlGTzVxTFRKeSsxd3p3blFpWlJyRzBDb1BrdElXMFJSb0VjVEIz?=
 =?utf-8?B?d1VNb0UzVWRqK2o5cUNsQm5abEVBakFCbExsNThPWWZHVWpRZWpEaVJ2WGUw?=
 =?utf-8?B?cUswN2RVRUtnaGxZbEZ3MjNZaENCOElDK2J4bEFHS1NkUUo5aWNTWmxKam90?=
 =?utf-8?B?TWhQVVBQMmxBR09mMWhMenp0MFlHbUQvd1JBb25FQUduMEdpdC9jdHBXK2Ni?=
 =?utf-8?B?VTRTTjE1OWZsNzhoeDd2QTBCNHNVUkpHb1JHTVR6WllaQi9CYys1bHUySlR0?=
 =?utf-8?B?MS9xSHdaOTBOQk5vdGhOcktlT3RmKy9yLzVwSXFGNXN4VGV2cDVYTW9wTUdF?=
 =?utf-8?B?WFJjQ1RmWWtsVnNHbTc3dEovL1hISUxQNVFETkVEN081a1c1NEw1WnJoZGRv?=
 =?utf-8?B?a1RqL2dvU29rNFUxV1k2R0lRSi9zbVFyd2hKc216WlRFSGpCWTQ2WEpBSzdo?=
 =?utf-8?B?eUwreU9pRXFzMENpMVRQS25LaDFXMUZNZTgvWGtEamRTNjE3N0tlQ1lNLzFI?=
 =?utf-8?B?RXJVMHZMRXV2VG95VFF1T3BPaU9kREJ4TElHeThoMnd5bXFvYXFvT0poWGx0?=
 =?utf-8?B?a2RNYnEyYVNmTWJJZDZ5dHdVTmgxZ3FWNE9kckVKOVBTbmYrN1I3Y043QzJ6?=
 =?utf-8?B?NnFOTEx5cXlidTduSUd0VEp0bGxORjFvQ01TMlZobXBtTG9VRXJZaTBJTDJT?=
 =?utf-8?B?UkhBQWFwUXlmVlFia2psZTd2NHR4VHdTMkhobkZmK2dyN09ibXFubjIrd2tS?=
 =?utf-8?B?OUJiNDZxY1J5TVdmZ05rUGpPWm5idjMycDNxU2l6Z0RLVlVhSldhV0ZzWnVT?=
 =?utf-8?Q?38jB0nQXV5y09ZkilMXTXsdwd?=
X-OriginatorOrg: amlogic.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 5e5a5a8c-7b69-4fc2-cc60-08db55efa86f
X-MS-Exchange-CrossTenant-AuthSource: SG2PR03MB6730.apcprd03.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 16 May 2023 09:26:41.5778
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 0df2add9-25ca-4b3a-acb4-c99ddf0b1114
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: RJmU/FfIzdIx4EQXZ2MYH1Yz734xvdEXKFdsVlJ9Z46ineOtPKH4GMsK2nisUhMtcn9g1w3E6ZOO7XJGvFe+8w==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: TYZPR03MB5535
X-Spam-Status: No, score=-4.8 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,RCVD_IN_DNSWL_NONE,
        RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS,T_SCC_BODY_TEXT_LINE,
        URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


On 2023/5/16 17:20, Krzysztof Kozlowski wrote:
> [ EXTERNAL EMAIL ]
> 
> On 16/05/2023 08:47, Yu Tu wrote:
>> 1. Add S4 SoC PLL and Peripheral clock controller dt-bindings.
>> 2. Add PLL and Peripheral clock controller driver for S4 SOC.
>>
>> Yu Tu (4):
>>    dt-bindings: clock: document Amlogic S4 SoC PLL clock controller
>>    dt-bindings: clock: document Amlogic S4 SoC peripherals clock
>>      controller
>>    clk: meson: S4: add support for Amlogic S4 SoC PLL clock driver
>>    clk: meson: s4: add support for Amlogic S4 SoC peripheral clock
>>      controller
>>
>> V7 -> V8:
>> 1. Change patch 0001/0002 dt-bindings title description, remove "meson".
>> Suggested by Dmitry, Neil.
> 
> So you only removed one word from title and description? This does not
> justify dropping review tag.

Yes. I just remove one work.

> 
> You already got a comment for this - add the tag.

I wonder what I should do next?
Add the tag and RESEND it again?

> 
> Best regards,
> Krzysztof
>