Re: [PATCH v8 0/3] Introduce STM32MP257 clock driver

[Gabriel FERNANDEZ]

Hello Stephen,

Gentle reminder.

Thanks

Gabriel


On 1/11/24 08:15, gabriel.fernandez@xxxxxxxxxxx wrote:
> From: Gabriel Fernandez <gabriel.fernandez@xxxxxxxxxxx>
>
> 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