Re: [PATCH v2] clk: qcom: gcc-sdm660: Replace usage of parent_names

[Marijn Suijten]

Hi Bjorn,

On 8/25/21 10:45 PM, Bjorn Andersson wrote:
> Using parent_data and parent_hws, instead of parent_names, does protect
> against some cases of incompletely defined clock trees. While it turns
> out that the bug being chased this time was totally unrelated, this
> patch converts the SDM660 GCC driver to avoid such issues.
>
> The "xo" fixed_factor clock is unused within the gcc driver, but
> referenced from the DSI PHY. So it's left in place until the DSI driver
> is updated.
>
> Tested-by: Marijn Suijten <marijn.suijten@xxxxxxxxxxxxxx>
> Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@xxxxxxxxxxxxxx>
> Reviewed-by: Marijn Suijten <marijn.suijten@xxxxxxxxxxxxxx>
> Signed-off-by: Bjorn Andersson <bjorn.andersson@xxxxxxxxxx>
> ---
>
> Changes since v1:
> - Added "xo" back to avoid regression in the DSI PHY


Tested again on the XA2 Ultra just to be sure, all is good now. Thanks!

- Marijn