[U-Boot] [PATCH v3 02/11] cmd: Do not show frequency for clocks which .get_rate() return error

[Peng Fan]

> Subject: [PATCH v3 02/11] cmd: Do not show frequency for clocks
> which .get_rate() return error
> 
> It may happen that some UCLASS_CLK clocks drivers work as a "managers", to
> call other, proper clocks. This situation is present in the iMX{6|8} clocks when
> supporting CONFIG_CLK (and CCF).
> 
> To avoid bogus output of "clk dump" we omit clocks which return error value -
> allowing reusing default implementation of this command.
> 
> Signed-off-by: Lukasz Majewski <lukma at denx.de>
> Reviewed-by: Simon Glass <sjg at chromium.org>
> ---
> 
> Changes in v3: None
> 
>  cmd/clk.c | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/cmd/clk.c b/cmd/clk.c
> index fd4231589c..2ea82176aa 100644
> --- a/cmd/clk.c
> +++ b/cmd/clk.c
> @@ -16,6 +16,7 @@ int __weak soc_clk_dump(void)
>  	struct udevice *dev;
>  	struct uclass *uc;
>  	struct clk clk;
> +	ulong rate;
>  	int ret;
> 
>  	/* Device addresses start at 1 */
> @@ -37,7 +38,9 @@ int __weak soc_clk_dump(void)
>  			continue;
>  		}
> 
> -		printf("%-30.30s : %lu Hz\n", dev->name, clk_get_rate(&clk));
> +		rate = clk_get_rate(&clk);
> +		if (!IS_ERR_VALUE(rate))
> +			printf("%-30.30s : %lu Hz\n", dev->name, rate);
> 
>  		clk_free(&clk);
>  	}

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

> --
> 2.11.0