Quoting Govind Singh (2018-10-12 02:40:40) > static const struct regmap_config gcc_qcs404_regmap_config = { > @@ -2669,7 +2699,7 @@ enum { > .fast_io = true, > }; > > -static const struct qcom_cc_desc gcc_qcs404_desc = { > +static struct qcom_cc_desc gcc_qcs404_desc = { > .config = &gcc_qcs404_regmap_config, > .clks = gcc_qcs404_clocks, > .num_clks = ARRAY_SIZE(gcc_qcs404_clocks), > @@ -2702,6 +2732,11 @@ static int gcc_qcs404_probe(struct platform_device *pdev) > clk_prepare_enable(apss_ahb_clk_src.clkr.hw.clk); > clk_prepare_enable(gpll0_ao_out_main.clkr.hw.clk); > > + if (of_property_read_bool(pdev->dev.of_node, "qcom,wcss-protected")) { Is this documented? And shouldn't it be inverted? If it isn't protected then we do this? > + gcc_qcs404_clocks[GCC_WCSS_Q6_AHB_CBCR_CLK] = &gcc_wdsp_q6ss_ahbs_clk.clkr; > + gcc_qcs404_clocks[GCC_WCSS_Q6_AXIM_CBCR_CLK] = &gcc_wdsp_q6ss_axim_clk.clkr; > + } > + > return qcom_cc_really_probe(pdev, &gcc_qcs404_desc, regmap); > } >