[PATCH 10/22] clk: mediatek: Add MT8195 ipesys clock support

Chen-Yu Tsai wenst at chromium.org
Tue Jul 6 02:11:38 PDT 2021


On Thu, Jun 17, 2021 at 7:08 AM Chun-Jie Chen
<chun-jie.chen at mediatek.com> wrote:
>
> Add MT8195 ipesys clock provider

It would be better if you could provide some more context for the patch,
such as what hardware subsystem / group does this belong to, what do
these clocks feed into? And in the case of supporting multiple clock
controllers in one driver, what led you to group them in the same
driver?

This applies to all your patches. The single statement "Add xxx clock
provider" is simply repeating what the subject already says.

The code looks OK.

Thanks
ChenYu

> Signed-off-by: Chun-Jie Chen <chun-jie.chen at mediatek.com>



More information about the Linux-mediatek mailing list