[U-Boot] [PATCH v3 04/11] clk: Extend struct clk to provide information regarding clock rate

Peng Fan peng.fan at nxp.com
Fri Apr 26 02:23:14 UTC 2019


Hi Lukasz,

> Subject: [PATCH v3 04/11] clk: Extend struct clk to provide information
> regarding clock rate
> 
> This commit extends the struct clk to provide information regarding the clock
> rate.
> As a result the clock tree traversal is performed at most once, and further
> reads are using the cached value.

I am thinking recalc is needed for some platforms, for clks without
CLK_GET_RATE_NOCACHE, It is to use cached value. Since your test
mostly on i.MX6Q, so it should be fine for now.

Reviewed-by: Peng Fan <peng.fan at nxp.com>

> 
> Signed-off-by: Lukasz Majewski <lukma at denx.de>
> ---
> 
> Changes in v3: None
> 
>  include/clk.h | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/include/clk.h b/include/clk.h index f6fbcc6634..f29ba02da1
> 100644
> --- a/include/clk.h
> +++ b/include/clk.h
> @@ -39,6 +39,7 @@ struct udevice;
>   * other clock APIs to identify which clock signal to operate upon.
>   *
>   * @dev: The device which implements the clock signal.
> + * @rate: The clock rate (in HZ).
>   * @id: The clock signal ID within the provider.
>   * @data: An optional data field for scenarios where a single integer ID is
> not
>   *	  sufficient. If used, it can be populated through an .of_xlate op and
> @@ -54,6 +55,7 @@ struct udevice;
>   */
>  struct clk {
>  	struct udevice *dev;
> +	unsigned long rate;	/* in HZ */
>  	/*
>  	 * Written by of_xlate. In the future, we might add more fields here.
>  	 */
> --
> 2.11.0



More information about the U-Boot mailing list