mbox series

[v2,0/4] clk: marvell: Move number of clocks to driver source

Message ID 20230812-mmp-nr-clks-v2-0-f9271bd7eaa5@skole.hr (mailing list archive)
Headers show
Series clk: marvell: Move number of clocks to driver source | expand

Message

Duje Mihanović Aug. 12, 2023, 10:02 a.m. UTC
Hi,

In v4 of my PXA1908 series, Conor suggested that the NR_CLKS variables
in dt bindings should be moved to the respective driver source files
because these are not used by the device trees and so are needlessly
added to the ABI. This series does that for the rest of the Marvell
PXA/MMP SoCs.

Link: https://lore.kernel.org/r/20230808-produce-thievish-3ce1b86a114b@spud/
Signed-off-by: Duje Mihanović <duje.mihanovic@skole.hr>
---
Changes in v2:
- Fix clk-audio.c compile error
- Reword mmp2 commit to match other commits
- Link to v1: https://lore.kernel.org/r/20230809-mmp-nr-clks-v1-0-5f3cdbbb89b8@skole.hr

---
Duje Mihanović (4):
      clk: mmp2: Move number of clocks to driver source
      clk: pxa168: Move number of clocks to driver source
      clk: pxa1928: Move number of clocks to driver source
      clk: pxa910: Move number of clocks to driver source

 drivers/clk/mmp/clk-audio.c                    | 6 ++++--
 drivers/clk/mmp/clk-of-mmp2.c                  | 4 +++-
 drivers/clk/mmp/clk-of-pxa168.c                | 4 +++-
 drivers/clk/mmp/clk-of-pxa1928.c               | 7 +++++--
 drivers/clk/mmp/clk-of-pxa910.c                | 4 +++-
 include/dt-bindings/clock/marvell,mmp2-audio.h | 1 -
 include/dt-bindings/clock/marvell,mmp2.h       | 1 -
 include/dt-bindings/clock/marvell,pxa168.h     | 1 -
 include/dt-bindings/clock/marvell,pxa1928.h    | 3 ---
 include/dt-bindings/clock/marvell,pxa910.h     | 1 -
 10 files changed, 18 insertions(+), 14 deletions(-)
---
base-commit: 52a93d39b17dc7eb98b6aa3edb93943248e03b2f
change-id: 20230809-mmp-nr-clks-7c80e416e6bf

Best regards,

Comments

Conor Dooley Aug. 13, 2023, 9:45 a.m. UTC | #1
On Sat, Aug 12, 2023 at 12:02:55PM +0200, Duje Mihanović wrote:
> Hi,
> 
> In v4 of my PXA1908 series, Conor suggested that the NR_CLKS variables
> in dt bindings should be moved to the respective driver source files
> because these are not used by the device trees and so are needlessly
> added to the ABI. This series does that for the rest of the Marvell
> PXA/MMP SoCs.
> 
> Link: https://lore.kernel.org/r/20230808-produce-thievish-3ce1b86a114b@spud/
> Signed-off-by: Duje Mihanović <duje.mihanovic@skole.hr>

Acked-by: Conor Dooley <conor.dooley@microchip.com>

Thanks,
conor.

> ---
> Changes in v2:
> - Fix clk-audio.c compile error
> - Reword mmp2 commit to match other commits
> - Link to v1: https://lore.kernel.org/r/20230809-mmp-nr-clks-v1-0-5f3cdbbb89b8@skole.hr
> 
> ---
> Duje Mihanović (4):
>       clk: mmp2: Move number of clocks to driver source
>       clk: pxa168: Move number of clocks to driver source
>       clk: pxa1928: Move number of clocks to driver source
>       clk: pxa910: Move number of clocks to driver source
> 
>  drivers/clk/mmp/clk-audio.c                    | 6 ++++--
>  drivers/clk/mmp/clk-of-mmp2.c                  | 4 +++-
>  drivers/clk/mmp/clk-of-pxa168.c                | 4 +++-
>  drivers/clk/mmp/clk-of-pxa1928.c               | 7 +++++--
>  drivers/clk/mmp/clk-of-pxa910.c                | 4 +++-
>  include/dt-bindings/clock/marvell,mmp2-audio.h | 1 -
>  include/dt-bindings/clock/marvell,mmp2.h       | 1 -
>  include/dt-bindings/clock/marvell,pxa168.h     | 1 -
>  include/dt-bindings/clock/marvell,pxa1928.h    | 3 ---
>  include/dt-bindings/clock/marvell,pxa910.h     | 1 -
>  10 files changed, 18 insertions(+), 14 deletions(-)
> ---
> base-commit: 52a93d39b17dc7eb98b6aa3edb93943248e03b2f
> change-id: 20230809-mmp-nr-clks-7c80e416e6bf
> 
> Best regards,
> -- 
> Duje Mihanović <duje.mihanovic@skole.hr>
> 
>