From b6488fc692ebd719e06e4323aa08e67541ad2d77 Mon Sep 17 00:00:00 2001 From: Mike Tipton Date: Wed, 31 Aug 2022 11:53:09 -0700 Subject: [PATCH] clk: qcom: camcc-pineapple: Remove BOOT_CRITICAL from some clocks The core_ahb and cpas_ahb clocks aren't critical during kernel boot. Only the drv_ahb and drv_xo clocks are, since those are the only ones required for the cam_rsc device to access its registers while probing. The core_ahb and cpas_ahb clocks require the titan_top_gdsc to be enabled, which isn't the case during camcc probe, so trying to enable them results in "stuck off" warnings. Change-Id: I95c19150f4812996737fffa29ac974f133bef92e Signed-off-by: Mike Tipton --- drivers/clk/qcom/camcc-pineapple.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/clk/qcom/camcc-pineapple.c b/drivers/clk/qcom/camcc-pineapple.c index 7227e06f39f2..fb8a952c20be 100644 --- a/drivers/clk/qcom/camcc-pineapple.c +++ b/drivers/clk/qcom/camcc-pineapple.c @@ -2318,7 +2318,6 @@ static struct clk_branch cam_cc_core_ahb_clk = { .clkr = { .enable_reg = 0x132d0, .enable_mask = BIT(0), - .flags = QCOM_CLK_BOOT_CRITICAL, .hw.init = &(const struct clk_init_data){ .name = "cam_cc_core_ahb_clk", .parent_hws = (const struct clk_hw*[]){ @@ -2337,7 +2336,6 @@ static struct clk_branch cam_cc_cpas_ahb_clk = { .clkr = { .enable_reg = 0x13220, .enable_mask = BIT(0), - .flags = QCOM_CLK_BOOT_CRITICAL, .hw.init = &(const struct clk_init_data){ .name = "cam_cc_cpas_ahb_clk", .parent_hws = (const struct clk_hw*[]){