On 06/03/2025 08:55, Jagadeesh Kona wrote:
+
+static struct qcom_clk_cfg video_cc_sm8450_clocks_cfg[] = {
+ { .offset = 0x80e4, .mask = BIT(0) }, /* VIDEO_CC_AHB_CLK */
+ { .offset = 0x8114, .mask = BIT(0) }, /* VIDEO_CC_XO_CLK */
+ { .offset = 0x8130, .mask = BIT(0) }, /* VIDEO_CC_SLEEP_CLK */
+};
- /* Keep some clocks always-on */
- qcom_branch_set_clk_en(regmap, 0x80e4); /* VIDEO_CC_AHB_CLK */
- qcom_branch_set_clk_en(regmap, 0x8130); /* VIDEO_CC_SLEEP_CLK */
- qcom_branch_set_clk_en(regmap, 0x8114); /* VIDEO_CC_XO_CLK */
Dmitry already said this but, this change feels like a loss of fidelity.
Please find a way to use the existing helper functions.
---
bod
