[PATCH v3 1/5] clk: rk3399: Set empty for vopl assigned-clocks

[Kever Yang]

On 2020/4/2 下午7:41, Jagan Teki wrote:
> During vidconsole probe, the device probe will try to
> check whether the assigned clocks on that video console
> node is initialized or not? and return an error if not.
>
> But, unlike Linux U-Boot won't require to handle these
> vopl assigned-clocks since core clocks are enough to
> handle the video out to process.
>
> So, mark them as empty in set_rate to satisfy clk_set_defaults
> so-that probe happened properly.
>
> Signed-off-by: Jagan Teki <jagan at amarulasolutions.com>



Reviewed-by: Kever Yang <kever.yang at rock-chips.com>

Thanks,
- Kever
> ---
> Changes for v3:
> - new patch
>
>   drivers/clk/rockchip/clk_rk3399.c | 7 +++++++
>   1 file changed, 7 insertions(+)
>
> diff --git a/drivers/clk/rockchip/clk_rk3399.c b/drivers/clk/rockchip/clk_rk3399.c
> index 865b80cc0f..1f62376595 100644
> --- a/drivers/clk/rockchip/clk_rk3399.c
> +++ b/drivers/clk/rockchip/clk_rk3399.c
> @@ -994,6 +994,13 @@ static ulong rk3399_clk_set_rate(struct clk *clk, ulong rate)
>   	case DCLK_VOP1:
>   		ret = rk3399_vop_set_clk(priv->cru, clk->id, rate);
>   		break;
> +	case ACLK_VOP1:
> +	case HCLK_VOP1:
> +		/**
> +		 * assigned-clocks handling won't require for vopl, so
> +		 * return 0 to satisfy clk_set_defaults during device probe.
> +		 */
> +		return 0;
>   	case SCLK_DDRCLK:
>   		ret = rk3399_ddr_set_clk(priv->cru, rate);
>   		break;