Re: [PATCH 2/2] clk: mediatek: mt8365: remove the max clock number defines

From: Krzysztof Kozlowski
Date: Wed May 17 2023 - 04:41:45 EST


On 17/05/2023 10:28, Alexandre Mergnat wrote:
> Since we swap to the generic MTK probe function [1], the *_NR_CLK
> defines used to know the number of defined clocks are no longer
> necessary. This patch remove all *_NR_CLK.

Please do not use "This commit/patch", but imperative mood. See longer
explanation here:
https://elixir.bootlin.com/linux/v5.17.1/source/Documentation/process/submitting-patches.rst#L95

>
> [1]: Commit ffe91cb28f6a ("clk: mediatek: mt8365: Convert to
> mtk_clk_simple_{probe,remove}()")
>
> Signed-off-by: Alexandre Mergnat <amergnat@xxxxxxxxxxxx>


Please use subject prefixes matching the subsystem. You can get them for
example with `git log --oneline -- DIRECTORY_OR_FILE` on the directory
your patch is touching.

And what about other users of bindings?

Best regards,
Krzysztof