lkml.org 
[lkml]   [2023]   [Jan]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH 2/2] clk: loongson1: Refactor to add devicetree support
From
On 13/01/2023 12:07, Keguang Zhang wrote:
> This patch refactors Loongson-1 clock driver.
> - Use CLK_OF_DECLARE() to declare the "early clocks"
> required by of_clk_init()
> - Merge clk-loongson1b.c and clk-loongson1c.c into one driver
> because most of the differences between them will be moved to DT
> - Add set_rate callback for ls1x_clk_divider
> - Update the Kconfig/Makefile accordingly
> - Update copyright
>
> Signed-off-by: Keguang Zhang <keguang.zhang@gmail.com>
> ---
> drivers/clk/Makefile | 2 +-
> drivers/clk/clk-loongson1.c | 348 +++++++++++++++++++++++++

No, this is not a refactor. This is removal and re-add. NAK.

One change per commit, this is unreviewable.

> drivers/clk/loongson1/Makefile | 4 -
> drivers/clk/loongson1/clk-loongson1b.c | 118 ---------
> drivers/clk/loongson1/clk-loongson1c.c | 95 -------

This is not explained at all. You are pushing some crappy vendor code
here instead of merging with upstream code.

Otherwise, explain the drop of directory. Why Loongson should be special?

Best regards,
Krzysztof

\
 
 \ /
  Last update: 2023-03-26 23:39    [W:0.127 / U:0.156 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site