diff mbox series

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

Message ID 20230311090733.56918-4-hal.feng@starfivetech.com
State New
Headers
Series Basic clock, reset & device tree support for StarFive JH7110 RISC-V SoC |

Commit Message

Hal Feng March 11, 2023, 9:07 a.m. UTC 
  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%)

Comments

Conor Dooley March 12, 2023, 2 p.m. UTC | #1 
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.
Hal Feng March 13, 2023, 2:37 a.m. UTC | #2 
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
diff mbox series

Patch

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