lkml.org 
[lkml]   [2018]   [Mar]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v2 00/19] clk: meson: use regmap in clock controllers
Bonjour Jerome,

Excerpts from Jerome Brunet's message of February 12, 2018 6:58 am:
> This changeset is a rework of meson's clock controllers to use regmap
> instead of directly using io memory. It based clk-meson next/drivers
> and depends on few core clock patches, mainly to export generic clocks
> helpers: [0],[1]. The line count is pretty high but the changes are
> actually fairly simple and repetitive.
>
> This work has been triggered by the fact that the HHI register space on
> gxbb and axg provides more than just clocks. The display driver already
> uses a syscon for HHI on gxbb. This is why gxbb did not use
> devm_ioremap_resource() to map the registers, since it would have
> reserved the memory region, preventing another driver from re-mapping
> it. The cleaner solution is, of course, to use syscon to handle. The
> purpose of this changeset is to allow it. Even if meson8b does not need
> this ATM, there is real reason to leave it behind. It is actually easier
> to migrate it as well, so all meson clock drivers may support regmap
> only.
>
> The rework starts with a few easy clean-ups. The real deal starts with
> patch 5, which adds meson's clk_regmap. This will be used as common
> structure to implement all the controller clocks. Having this replaces
> the gxbb AO controller specific regmap gate. This structure will also be
> re-used in upcoming controllers, such as the axg's AO and audio
> controllers. Each clock type is then migrated, one at a time, to this
> new structure.
>
> While at it, the meson clock drivers have been cleaned-up a bit,
> removing the gate embedded in the mpll driver, simplifying the pll
> driver and removing the legacy cpu_clk of meson8b.
>
> The new code around the cpu clk of the meson8b is just re-implementation,
> using simple elements, of the old cpu_clk. As explained by Martin, the
> old cpu_clk would hang quite often when changing the rate of the cpu
> clock. Surprisingly, the new implementation improved the situation a bit,
> but still hangs from time to time. As this is not acceptable, the cpu
> clk subtree as been switched to a read-only mode, preventing any change
> of the cpu rate, until an acceptable solution is found.
>
> With this series applied, the clock controllers of the gxbb, gxl and axg
> SoC will try get regmap from their parent DT node. If this fails, they
> will fallback to mapping the register themselves. This fallback will be
> kept until platform DTs have changed so clock controllers is a child of
> the HHI system controller.

Looks OK to me. As we discussed off-list, can you send this (and the
other meson patches) as a PR based on top of the clk-helpers branch in
the clk git tree? That should satisfy the dependencies (I hope).

Best regards,
Mike

>
> Based on this changeset, more patches are coming. For those interested,
> the WIP is available here [2]
>
> Changes since v1 [3]:
> * Fixed a few typos in patches descriptions
> * Fixed cpu clock names on meson8b, as suggested by Martin
> * Switched cpu clock subtree to reead-only mode
> * Fix clk_regmap mux documentation, as reported by Yixun
>
> [0]: https://lkml.kernel.org/r/20180118110144.30619-1-jbrunet@baylibre.com
> [1]: https://lkml.kernel.org/r/20180122105759.12206-1-jbrunet@baylibre.com
> [2]: https://github.com/jeromebrunet/linux/tree/v4.17/meson/clk-regmap
> [1]: https://lkml.kernel.org/r/20180131180945.18025-1-jbrunet@baylibre.com
>
> Jerome Brunet (19):
> clk: meson: use dev pointer where possible
> clk: meson: use devm_of_clk_add_hw_provider
> clk: meson: only one loop index is necessary in probe
> clk: meson: remove obsolete comments
> clk: meson: add regmap clocks
> clk: meson: switch gxbb ao_clk to clk_regmap
> clk: meson: remove superseded aoclk_gate_regmap
> clk: meson: add regmap to the clock controllers
> clk: meson: migrate gates to clk_regmap
> clk: meson: migrate dividers to clk_regmap
> clk: meson: migrate muxes to clk_regmap
> clk: meson: add regmap helpers for parm
> clk: meson: migrate mplls clocks to clk_regmap
> clk: meson: migrate the audio divider clock to clk_regmap
> clk: meson: migrate plls clocks to clk_regmap
> clk: meson: split divider and gate part of mpll
> clk: meson: rework meson8b cpu clock
> clk: meson: remove obsolete cpu_clk
> clk: meson: use hhi syscon if available
>
> drivers/clk/meson/Kconfig | 9 +
> drivers/clk/meson/Makefile | 5 +-
> drivers/clk/meson/axg.c | 722 +++++++++--------
> drivers/clk/meson/axg.h | 6 +-
> drivers/clk/meson/clk-audio-divider.c | 63 +-
> drivers/clk/meson/clk-cpu.c | 178 -----
> drivers/clk/meson/clk-mpll.c | 132 +---
> drivers/clk/meson/clk-pll.c | 243 +++---
> drivers/clk/meson/clk-regmap.c | 166 ++++
> drivers/clk/meson/clk-regmap.h | 111 +++
> drivers/clk/meson/clkc.h | 93 +--
> drivers/clk/meson/gxbb-aoclk-regmap.c | 46 --
> drivers/clk/meson/gxbb-aoclk.c | 20 +-
> drivers/clk/meson/gxbb-aoclk.h | 11 -
> drivers/clk/meson/gxbb.c | 1402 ++++++++++++++++++---------------
> drivers/clk/meson/gxbb.h | 5 +-
> drivers/clk/meson/meson8b.c | 588 ++++++++------
> drivers/clk/meson/meson8b.h | 11 +-
> 18 files changed, 1978 insertions(+), 1833 deletions(-)
> delete mode 100644 drivers/clk/meson/clk-cpu.c
> create mode 100644 drivers/clk/meson/clk-regmap.c
> create mode 100644 drivers/clk/meson/clk-regmap.h
> delete mode 100644 drivers/clk/meson/gxbb-aoclk-regmap.c
>
> --
> 2.14.3
>
>

\
 
 \ /
  Last update: 2018-03-12 19:25    [W:0.105 / U:0.716 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site