Messages in this thread | | | Date | Thu, 3 Aug 2023 11:51:30 +0530 | From | Vinod Koul <> | Subject | Re: [RESEND PATCH v2] clk: qcom: gcc-sm8250: Fix gcc_sdcc2_apps_clk_src |
| |
On 02-08-23, 14:04, Patrick Whewell wrote: > GPLL9 is not on by default, which causes a "gcc_sdcc2_apps_clk_src: rcg > didn't update its configuration" error when booting. Set .flags = > CLK_OPS_PARENT_ENABLE to fix the error.
Reviewed-by: Vinod Koul <vkoul@kernel.org>
> > Fixes: 3e5770921a88 ("clk: qcom: gcc: Add global clock controller driver for SM8250") > Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org> > Reviewed-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org> > Signed-off-by: Patrick Whewell <patrick.whewell@sightlineapplications.com> > --- > Changes in v2: > - Updated commit messaage to mention GPLL9 not being on by default > being the cause of error > - Moved Signed-off-by line to the bottom. > > drivers/clk/qcom/gcc-sm8250.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/clk/qcom/gcc-sm8250.c b/drivers/clk/qcom/gcc-sm8250.c > index b6cf4bc88d4d..d3c75bb55946 100644 > --- a/drivers/clk/qcom/gcc-sm8250.c > +++ b/drivers/clk/qcom/gcc-sm8250.c > @@ -721,6 +721,7 @@ static struct clk_rcg2 gcc_sdcc2_apps_clk_src = { > .name = "gcc_sdcc2_apps_clk_src", > .parent_data = gcc_parent_data_4, > .num_parents = ARRAY_SIZE(gcc_parent_data_4), > + .flags = CLK_OPS_PARENT_ENABLE, > .ops = &clk_rcg2_floor_ops, > }, > }; > -- > 2.25.1
-- ~Vinod
| |