Re: [PATCH] clk: qcom: gcc-msm8939: Add missing CSI2 related clocks

[Bryan O'Donoghue]

On 28/10/2023 13:10, Vincent Knecht wrote:
> When adding in the indexes for this clock-controller we missed
> GCC_CAMSS_CSI2_AHB_CLK, GCC_CAMSS_CSI2_CLK, GCC_CAMSS_CSI2PHY_CLK,
> GCC_CAMSS_CSI2PIX_CLK and GCC_CAMSS_CSI2RDI_CLK.
>
> Add them in now.
>
> Signed-off-by: Vincent Knecht <vincent.knecht@xxxxxxxxxx>
> ---
> No fixes tag because camss is a not-yet-enabled feature for msm8939.
>
> Also didn't rename ftbl_gcc_camss_csi0_1_clk now that csi2 uses it
> to avoid not-required-churn... should it be done anyway ?

An aesthetic choice with no right answer but, downstream calls it

ftbl_gcc_camss_csi0_1_2_clk,

maybe that's reasonable.

Assuming Krz's comments acted upon.

Reviewed-by: Bryan O'Donoghue <bryan.odonoghue@xxxxxxxxxx>