[PATCH v5 03/21] clk: starfive: Rename clk-starfive-jh7100.h to clk-starfive-jh71x0.h

Hal Feng posted 21 patches 2 years, 11 months ago
There is a newer version of this series
[PATCH v5 03/21] clk: starfive: Rename clk-starfive-jh7100.h to clk-starfive-jh71x0.h
Posted by Hal Feng 2 years, 11 months ago
From: Emil Renner Berthing <kernel@esmil.dk>

Rename clk-starfive-jh7100.h to clk-starfive-jh71x0.h for making
the code to be common.

Tested-by: Tommaso Merciai <tomm.merciai@gmail.com>
Reviewed-by: Conor Dooley <conor.dooley@microchip.com>
Signed-off-by: Emil Renner Berthing <kernel@esmil.dk>
Signed-off-by: Hal Feng <hal.feng@starfivetech.com>
---
 .../clk/starfive/{clk-starfive-jh7100.h => clk-starfive-jh71x0.h} | 0
 1 file changed, 0 insertions(+), 0 deletions(-)
 rename drivers/clk/starfive/{clk-starfive-jh7100.h => clk-starfive-jh71x0.h} (100%)

diff --git a/drivers/clk/starfive/clk-starfive-jh7100.h b/drivers/clk/starfive/clk-starfive-jh71x0.h
similarity index 100%
rename from drivers/clk/starfive/clk-starfive-jh7100.h
rename to drivers/clk/starfive/clk-starfive-jh71x0.h
-- 
2.38.1
Re: [PATCH v5 03/21] clk: starfive: Rename clk-starfive-jh7100.h to clk-starfive-jh71x0.h
Posted by Conor Dooley 2 years, 11 months ago
On Sat, Mar 11, 2023 at 05:07:15PM +0800, Hal Feng wrote:
> From: Emil Renner Berthing <kernel@esmil.dk>
> 
> Rename clk-starfive-jh7100.h to clk-starfive-jh71x0.h for making
> the code to be common.
> 
> Tested-by: Tommaso Merciai <tomm.merciai@gmail.com>
> Reviewed-by: Conor Dooley <conor.dooley@microchip.com>
> Signed-off-by: Emil Renner Berthing <kernel@esmil.dk>
> Signed-off-by: Hal Feng <hal.feng@starfivetech.com>
> ---
>  .../clk/starfive/{clk-starfive-jh7100.h => clk-starfive-jh71x0.h} | 0

So this patch has broken the build :(
https://patchwork.kernel.org/project/linux-riscv/patch/20230311090733.56918-4-hal.feng@starfivetech.com/

I would imagine it is because you can't rename the header without also
changing the users of the header.

Cheers,
Conor.
Re: [PATCH v5 03/21] clk: starfive: Rename clk-starfive-jh7100.h to clk-starfive-jh71x0.h
Posted by Hal Feng 2 years, 11 months ago
On Sun, 12 Mar 2023 14:00:23 +0000, Conor Dooley wrote:
> On Sat, Mar 11, 2023 at 05:07:15PM +0800, Hal Feng wrote:
>> From: Emil Renner Berthing <kernel@esmil.dk>
>> 
>> Rename clk-starfive-jh7100.h to clk-starfive-jh71x0.h for making
>> the code to be common.
>> 
>> Tested-by: Tommaso Merciai <tomm.merciai@gmail.com>
>> Reviewed-by: Conor Dooley <conor.dooley@microchip.com>
>> Signed-off-by: Emil Renner Berthing <kernel@esmil.dk>
>> Signed-off-by: Hal Feng <hal.feng@starfivetech.com>
>> ---
>>  .../clk/starfive/{clk-starfive-jh7100.h => clk-starfive-jh71x0.h} | 0
> 
> So this patch has broken the build :(
> https://patchwork.kernel.org/project/linux-riscv/patch/20230311090733.56918-4-hal.feng@starfivetech.com/
> 
> I would imagine it is because you can't rename the header without also
> changing the users of the header.

Will fix it later. Thanks for pointing it out.

Best regards,
Hal