2020-10-22 11:26:57

by Jerome Brunet

[permalink] [raw]
Subject: [PATCH v2 0/3] clk: add api to get clk consumer from clk_hw

This patchset a call in CCF to get "struct clk*" from "struct clk_hw*"

Changes since v1: [0]
* Add a con_id string to help keep track of the consumer
* Add devm variant:
- Following our discussion on V1, I choose to have the dev as
argument as most devm function do. However, as Stephen pointed out
we don't expect this to differ from the one linked to clk_hw. In
this case a warning is thrown.
* Add a first usage of this in the amlogic clock driver.

[0]: https://lore.kernel.org/r/[email protected]

Jerome Brunet (3):
clk: avoid devm_clk_release name clash
clk: add api to get clk consumer from clk_hw
clk: meson: g12: drop use of __clk_lookup()

drivers/clk/clk.c | 73 +++++++++++++++++++++++++++++++++---
drivers/clk/meson/g12a.c | 68 ++++++++++++++++-----------------
include/linux/clk-provider.h | 5 +++
3 files changed, 104 insertions(+), 42 deletions(-)

--
2.25.4


2020-10-26 19:34:28

by Kevin Hilman

[permalink] [raw]
Subject: Re: [PATCH v2 0/3] clk: add api to get clk consumer from clk_hw

Jerome Brunet <[email protected]> writes:

> This patchset a call in CCF to get "struct clk*" from "struct clk_hw*"
>
> Changes since v1: [0]
> * Add a con_id string to help keep track of the consumer
> * Add devm variant:
> - Following our discussion on V1, I choose to have the dev as
> argument as most devm function do. However, as Stephen pointed out
> we don't expect this to differ from the one linked to clk_hw. In
> this case a warning is thrown.
> * Add a first usage of this in the amlogic clock driver.

Tested-by: Kevin Hilman <[email protected]>

Tested this on a couple g12a based platforms. This series also allows
me to build the g12 clock drivers as modules.

Kevin