[v8,0/3] Introduce STM32MP257 clock driver

Message

Gabriel FERNANDEZ Jan. 11, 2024, 7:15 a.m. UTC

From: Gabriel Fernandez <[e-mail]>

v8:
  - use .index of clk_parent_data struct to define a parent
  - remove unnecessary dependency check with SCMI clock driver
  - convert to platform device APIs
  - convert to devm_of_clk_add_hw_provider()
  - convert single value enum to a define

v7: base on next-20231219
  - These patches below are applied to clk-next:
      clk: stm32mp1: move stm32mp1 clock driver into stm32 directory
      clk: stm32mp1: use stm32mp13 reset driver
      dt-bindings: stm32: add clocks and reset binding for stm32mp25
  - remove unnecessary includes
  - migrate clock parents to struct clk_parent_data and remove
    CLK_STM32_XXX() macros  to have a more readble code
  - use platform device APIs (devm_of_iomap() instead of_iomap())
  - move content of stm32mp25_rcc_init() to stm32mp25_rcc_clocks_probe()
  - simply get_clock_deps()
  - add const to stm32mp25_data struct
  - remove ck_icn_p_serc clock (will be integrate later with security
    management)

v6:
  - remove useless defines in drivers/clk/stm32/stm32mp25_rcc.h

v5:
  - Fix sparse warnings: was not declared. Should it be static?
    drivers/clk/stm32/clk-stm32mp13.c:1516:29: symbol 'stm32mp13_reset_data'
    drivers/clk/stm32/clk-stm32mp1.c:2148:29: symbol 'stm32mp1_reset_data'
    drivers/clk/stm32/clk-stm32mp25.c:1003:5: symbol 'stm32mp25_cpt_gate'
    drivers/clk/stm32/clk-stm32mp25.c:1005:29: symbol 'stm32mp25_clock_data'
    drivers/clk/stm32/clk-stm32mp25.c:1011:29: symbol 'stm32mp25_reset_data'

v4:
  - use GPL-2.0-only OR BSD-2-Clause for clock and reset binding files
  - use quotes ' for #clock-cells and #reset-cells in YAML documentation
  - reset binding start now to 0 instead 1
  - improve management of reset lines that are not managed

v3:
  - from Rob Herring change clock item description in YAML documentation
v2:
  - rework reset binding (use ID witch start from 0)
  - rework reset driver to manage STM32MP13 / STM32MP15 / STM32MP25
  - rework YAML documentation

Gabriel Fernandez (3):
  clk: stm32mp13: use platform device APIs
  clk: stm32: introduce clocks for STM32MP257 platform
  arm64: dts: st: add rcc support in stm32mp251

 arch/arm64/boot/dts/st/stm32mp251.dtsi |  132 +-
 drivers/clk/stm32/Kconfig              |    7 +
 drivers/clk/stm32/Makefile             |    1 +
 drivers/clk/stm32/clk-stm32-core.c     |   11 +-
 drivers/clk/stm32/clk-stm32mp13.c      |   72 +-
 drivers/clk/stm32/clk-stm32mp25.c      | 1876 ++++++++++++++++++++++++
 drivers/clk/stm32/reset-stm32.c        |   59 +-
 drivers/clk/stm32/reset-stm32.h        |    7 +
 drivers/clk/stm32/stm32mp25_rcc.h      |  712 +++++++++
 9 files changed, 2765 insertions(+), 112 deletions(-)
 create mode 100644 drivers/clk/stm32/clk-stm32mp25.c
 create mode 100644 drivers/clk/stm32/stm32mp25_rcc.h

Comments

Gabriel FERNANDEZ Jan. 25, 2024, 2:51 p.m. UTC | #1

Hello Stephen,

Gentle reminder.

Thanks

Gabriel


On 1/11/24 08:15, [e-mail] wrote:
> From: Gabriel Fernandez <[e-mail]>
>
> v8:
>    - use .index of clk_parent_data struct to define a parent
>    - remove unnecessary dependency check with SCMI clock driver
>    - convert to platform device APIs
>    - convert to devm_of_clk_add_hw_provider()
>    - convert single value enum to a define
>
> v7: base on next-20231219
>    - These patches below are applied to clk-next:
>        clk: stm32mp1: move stm32mp1 clock driver into stm32 directory
>        clk: stm32mp1: use stm32mp13 reset driver
>        dt-bindings: stm32: add clocks and reset binding for stm32mp25
>    - remove unnecessary includes
>    - migrate clock parents to struct clk_parent_data and remove
>      CLK_STM32_XXX() macros  to have a more readble code
>    - use platform device APIs (devm_of_iomap() instead of_iomap())
>    - move content of stm32mp25_rcc_init() to stm32mp25_rcc_clocks_probe()
>    - simply get_clock_deps()
>    - add const to stm32mp25_data struct
>    - remove ck_icn_p_serc clock (will be integrate later with security
>      management)
>
> v6:
>    - remove useless defines in drivers/clk/stm32/stm32mp25_rcc.h
>
> v5:
>    - Fix sparse warnings: was not declared. Should it be static?
>      drivers/clk/stm32/clk-stm32mp13.c:1516:29: symbol 'stm32mp13_reset_data'
>      drivers/clk/stm32/clk-stm32mp1.c:2148:29: symbol 'stm32mp1_reset_data'
>      drivers/clk/stm32/clk-stm32mp25.c:1003:5: symbol 'stm32mp25_cpt_gate'
>      drivers/clk/stm32/clk-stm32mp25.c:1005:29: symbol 'stm32mp25_clock_data'
>      drivers/clk/stm32/clk-stm32mp25.c:1011:29: symbol 'stm32mp25_reset_data'
>
> v4:
>    - use GPL-2.0-only OR BSD-2-Clause for clock and reset binding files
>    - use quotes ' for #clock-cells and #reset-cells in YAML documentation
>    - reset binding start now to 0 instead 1
>    - improve management of reset lines that are not managed
>
> v3:
>    - from Rob Herring change clock item description in YAML documentation
> v2:
>    - rework reset binding (use ID witch start from 0)
>    - rework reset driver to manage STM32MP13 / STM32MP15 / STM32MP25
>    - rework YAML documentation
>
> Gabriel Fernandez (3):
>    clk: stm32mp13: use platform device APIs
>    clk: stm32: introduce clocks for STM32MP257 platform
>    arm64: dts: st: add rcc support in stm32mp251
>
>   arch/arm64/boot/dts/st/stm32mp251.dtsi |  132 +-
>   drivers/clk/stm32/Kconfig              |    7 +
>   drivers/clk/stm32/Makefile             |    1 +
>   drivers/clk/stm32/clk-stm32-core.c     |   11 +-
>   drivers/clk/stm32/clk-stm32mp13.c      |   72 +-
>   drivers/clk/stm32/clk-stm32mp25.c      | 1876 ++++++++++++++++++++++++
>   drivers/clk/stm32/reset-stm32.c        |   59 +-
>   drivers/clk/stm32/reset-stm32.h        |    7 +
>   drivers/clk/stm32/stm32mp25_rcc.h      |  712 +++++++++
>   9 files changed, 2765 insertions(+), 112 deletions(-)
>   create mode 100644 drivers/clk/stm32/clk-stm32mp25.c
>   create mode 100644 drivers/clk/stm32/stm32mp25_rcc.h
>