Re: [PATCH v1 14/30] clk: starfive: Factor out common clock driver code

From: Stephen Boyd
Date: Fri Sep 30 2022 - 17:43:46 EST


Quoting Hal Feng (2022-09-29 10:54:59)
> From: Emil Renner Berthing <kernel@xxxxxxxx>
>
> The clock control registers on the StarFive SoCs work identically,
> so factor out the code then drivers for different SoCs can share
> it without depending on each other. No functional change.

Sounds great! No functional change! But to verify that it is pretty
hard. Can you generate the patch with `git format-patch -M -C` and not
rename anything initially? I hope that will allow us to see that really
nothing has changed except code is moved from one file to another. Then
the next patch can be the sed command to rename to starfive.

As the patch is right now, I'm not particularly interested in going
through 700 lines to make sure nothing really changed.