* [PATCH v3 0/2] Add support for LPASS clock controller for SDM845 @ 2018-08-03 12:21 Taniya Das 2018-08-03 12:21 ` [PATCH v3 1/2] dt-bindings: clock: Introduce QCOM LPASS clock bindings Taniya Das 2018-08-03 12:21 ` [PATCH v3 2/2] clk: qcom: Add lpass clock controller driver for SDM845 Taniya Das 0 siblings, 2 replies; 10+ messages in thread From: Taniya Das @ 2018-08-03 12:21 UTC (permalink / raw) To: Stephen Boyd, Michael Turquette Cc: Andy Gross, David Brown, Rajendra Nayak, Amit Nischal, linux-arm-msm, linux-soc, linux-clk, linux-kernel, robh, Taniya Das [v3] * Add a device tree property to identify lpass protected GCC clocks. * Update the GCC driver code to register the lpass clocks when the flag is defined. * Add comment for clocks using the BRANCH_HALT_SKIP flag. * Use platform APIs instead of of_address_to_resource. * Replace devm_ioremap with devm_ioremap_resource. * Use fixed index for 'lpass_cc' & 'lpass_qdsp6ss' in probe. [v2] * Make gcc_lpass_sway_clk static. * Remove using child nodes and use reg-names to differentiate various domains of LPASS CC. Add support for the lpass clock controller found on SDM845 based devices. This would allow lpass peripheral loader drivers to control the clocks to bring the subsystem out of reset. Taniya Das (2): dt-bindings: clock: Introduce QCOM LPASS clock bindings clk: qcom: Add lpass clock controller driver for SDM845 .../devicetree/bindings/clock/qcom,gcc.txt | 2 + .../devicetree/bindings/clock/qcom,lpasscc.txt | 33 ++++ drivers/clk/qcom/Kconfig | 9 + drivers/clk/qcom/Makefile | 1 + drivers/clk/qcom/gcc-sdm845.c | 35 ++++ drivers/clk/qcom/lpasscc-sdm845.c | 189 +++++++++++++++++++++ include/dt-bindings/clock/qcom,gcc-sdm845.h | 2 + include/dt-bindings/clock/qcom,lpass-sdm845.h | 16 ++ 8 files changed, 287 insertions(+) create mode 100644 Documentation/devicetree/bindings/clock/qcom,lpasscc.txt create mode 100644 drivers/clk/qcom/lpasscc-sdm845.c create mode 100644 include/dt-bindings/clock/qcom,lpass-sdm845.h -- Qualcomm INDIA, on behalf of Qualcomm Innovation Center, Inc.is a member of the Code Aurora Forum, hosted by the Linux Foundation. ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH v3 1/2] dt-bindings: clock: Introduce QCOM LPASS clock bindings 2018-08-03 12:21 [PATCH v3 0/2] Add support for LPASS clock controller for SDM845 Taniya Das @ 2018-08-03 12:21 ` Taniya Das 2018-08-27 21:14 ` Stephen Boyd 2018-08-03 12:21 ` [PATCH v3 2/2] clk: qcom: Add lpass clock controller driver for SDM845 Taniya Das 1 sibling, 1 reply; 10+ messages in thread From: Taniya Das @ 2018-08-03 12:21 UTC (permalink / raw) To: Stephen Boyd, Michael Turquette Cc: Andy Gross, David Brown, Rajendra Nayak, Amit Nischal, linux-arm-msm, linux-soc, linux-clk, linux-kernel, robh, Taniya Das Add device tree bindings for Low Power Audio subsystem clock controller for Qualcomm Technology Inc's SDM845 SoCs. Signed-off-by: Taniya Das <tdas@codeaurora.org> --- .../devicetree/bindings/clock/qcom,gcc.txt | 2 ++ .../devicetree/bindings/clock/qcom,lpasscc.txt | 33 ++++++++++++++++++++++ include/dt-bindings/clock/qcom,gcc-sdm845.h | 2 ++ include/dt-bindings/clock/qcom,lpass-sdm845.h | 16 +++++++++++ 4 files changed, 53 insertions(+) create mode 100644 Documentation/devicetree/bindings/clock/qcom,lpasscc.txt create mode 100644 include/dt-bindings/clock/qcom,lpass-sdm845.h diff --git a/Documentation/devicetree/bindings/clock/qcom,gcc.txt b/Documentation/devicetree/bindings/clock/qcom,gcc.txt index 664ea1f..e452abc 100644 --- a/Documentation/devicetree/bindings/clock/qcom,gcc.txt +++ b/Documentation/devicetree/bindings/clock/qcom,gcc.txt @@ -32,6 +32,8 @@ be part of GCC and hence the TSENS properties can also be part of the GCC/clock-controller node. For more details on the TSENS properties please refer Documentation/devicetree/bindings/thermal/qcom-tsens.txt +- qcom,lpass-protected : Indicate GCC to be able to access the + lpass gcc clock branches. Example: clock-controller@900000 { diff --git a/Documentation/devicetree/bindings/clock/qcom,lpasscc.txt b/Documentation/devicetree/bindings/clock/qcom,lpasscc.txt new file mode 100644 index 0000000..062e413 --- /dev/null +++ b/Documentation/devicetree/bindings/clock/qcom,lpasscc.txt @@ -0,0 +1,33 @@ +Qualcomm LPASS Clock Controller Binding +----------------------------------------------- + +Required properties : +- compatible : shall contain "qcom,sdm845-lpasscc" +- #clock-cells : from common clock binding, shall contain 1. +- reg : shall contain base register address and size, + in the order + Index-0 maps to LPASS_CC register region + Index-1 maps to LPASS_QDSP6SS register region +- qcom,lpass-protected : Boolean property to indicate to GCC clock controller + for the lpass GCC clocks. + +Optional properties : +- reg-names : register names of LPASS domain + "lpass_cc", "lpass_qdsp6ss". + +Example: + +The below node has to be defined in the cases where the LPASS peripheral loader +would bring the subsystem out of reset. + + lpasscc: clock-controller { + compatible = "qcom,sdm845-lpasscc"; + reg = <0x17014000 0x1f004>, <0x17300000 0x200>; + reg-names = "lpass_cc", "lpass_qdsp6ss"; + #clock-cells = <1>; + }; + + gcc: clock-controller@100000 { + compatible = "qcom,gcc-sdm845"; + qcom,lpass-protected; + }; diff --git a/include/dt-bindings/clock/qcom,gcc-sdm845.h b/include/dt-bindings/clock/qcom,gcc-sdm845.h index f96fc2d..66c4267 100644 --- a/include/dt-bindings/clock/qcom,gcc-sdm845.h +++ b/include/dt-bindings/clock/qcom,gcc-sdm845.h @@ -194,6 +194,8 @@ #define GPLL4 184 #define GCC_CPUSS_DVM_BUS_CLK 185 #define GCC_CPUSS_GNOC_CLK 186 +#define GCC_LPASS_Q6_AXI_CLK 187 +#define GCC_LPASS_SWAY_CLK 188 /* GCC Resets */ #define GCC_MMSS_BCR 0 diff --git a/include/dt-bindings/clock/qcom,lpass-sdm845.h b/include/dt-bindings/clock/qcom,lpass-sdm845.h new file mode 100644 index 0000000..015968e --- /dev/null +++ b/include/dt-bindings/clock/qcom,lpass-sdm845.h @@ -0,0 +1,16 @@ +/* SPDX-License-Identifier: GPL-2.0 */ +/* + * Copyright (c) 2018, The Linux Foundation. All rights reserved. + */ + +#ifndef _DT_BINDINGS_CLK_SDM_LPASS_SDM845_H +#define _DT_BINDINGS_CLK_SDM_LPASS_SDM845_H + +#define LPASS_AUDIO_WRAPPER_AON_CLK 0 +#define LPASS_Q6SS_AHBM_AON_CLK 1 +#define LPASS_Q6SS_AHBS_AON_CLK 2 +#define LPASS_QDSP6SS_XO_CLK 3 +#define LPASS_QDSP6SS_SLEEP_CLK 4 +#define LPASS_QDSP6SS_CORE_CLK 5 + +#endif -- Qualcomm INDIA, on behalf of Qualcomm Innovation Center, Inc.is a member of the Code Aurora Forum, hosted by the Linux Foundation. ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH v3 1/2] dt-bindings: clock: Introduce QCOM LPASS clock bindings 2018-08-03 12:21 ` [PATCH v3 1/2] dt-bindings: clock: Introduce QCOM LPASS clock bindings Taniya Das @ 2018-08-27 21:14 ` Stephen Boyd 2018-09-05 18:26 ` Taniya Das 0 siblings, 1 reply; 10+ messages in thread From: Stephen Boyd @ 2018-08-27 21:14 UTC (permalink / raw) To: Michael Turquette, Taniya Das Cc: Andy Gross, David Brown, Rajendra Nayak, Amit Nischal, linux-arm-msm, linux-soc, linux-clk, linux-kernel, robh, Taniya Das Quoting Taniya Das (2018-08-03 05:21:13) > Add device tree bindings for Low Power Audio subsystem clock controller for > Qualcomm Technology Inc's SDM845 SoCs. > > Signed-off-by: Taniya Das <tdas@codeaurora.org> > --- > .../devicetree/bindings/clock/qcom,gcc.txt | 2 ++ > .../devicetree/bindings/clock/qcom,lpasscc.txt | 33 ++++++++++++++++++++++ > include/dt-bindings/clock/qcom,gcc-sdm845.h | 2 ++ > include/dt-bindings/clock/qcom,lpass-sdm845.h | 16 +++++++++++ > 4 files changed, 53 insertions(+) > create mode 100644 Documentation/devicetree/bindings/clock/qcom,lpasscc.txt > create mode 100644 include/dt-bindings/clock/qcom,lpass-sdm845.h > > diff --git a/Documentation/devicetree/bindings/clock/qcom,gcc.txt b/Documentation/devicetree/bindings/clock/qcom,gcc.txt > index 664ea1f..e452abc 100644 > --- a/Documentation/devicetree/bindings/clock/qcom,gcc.txt > +++ b/Documentation/devicetree/bindings/clock/qcom,gcc.txt > @@ -32,6 +32,8 @@ be part of GCC and hence the TSENS properties can also be > part of the GCC/clock-controller node. > For more details on the TSENS properties please refer > Documentation/devicetree/bindings/thermal/qcom-tsens.txt > +- qcom,lpass-protected : Indicate GCC to be able to access the > + lpass gcc clock branches. This doesn't parse well for me. Maybe something like: 'Indicate that the LPASS clock branches within GCC are unusable due to firmware access control restrictions'? > > Example: > clock-controller@900000 { > diff --git a/Documentation/devicetree/bindings/clock/qcom,lpasscc.txt b/Documentation/devicetree/bindings/clock/qcom,lpasscc.txt > new file mode 100644 > index 0000000..062e413 > --- /dev/null > +++ b/Documentation/devicetree/bindings/clock/qcom,lpasscc.txt > @@ -0,0 +1,33 @@ > +Qualcomm LPASS Clock Controller Binding > +----------------------------------------------- > + > +Required properties : > +- compatible : shall contain "qcom,sdm845-lpasscc" > +- #clock-cells : from common clock binding, shall contain 1. > +- reg : shall contain base register address and size, > + in the order > + Index-0 maps to LPASS_CC register region > + Index-1 maps to LPASS_QDSP6SS register region > +- qcom,lpass-protected : Boolean property to indicate to GCC clock controller > + for the lpass GCC clocks. Why is this here? > + > +Optional properties : > +- reg-names : register names of LPASS domain > + "lpass_cc", "lpass_qdsp6ss". > + > +Example: > + > +The below node has to be defined in the cases where the LPASS peripheral loader > +would bring the subsystem out of reset. > + > + lpasscc: clock-controller { > + compatible = "qcom,sdm845-lpasscc"; > + reg = <0x17014000 0x1f004>, <0x17300000 0x200>; > + reg-names = "lpass_cc", "lpass_qdsp6ss"; > + #clock-cells = <1>; > + }; > + > + gcc: clock-controller@100000 { > + compatible = "qcom,gcc-sdm845"; > + qcom,lpass-protected; > + }; ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v3 1/2] dt-bindings: clock: Introduce QCOM LPASS clock bindings 2018-08-27 21:14 ` Stephen Boyd @ 2018-09-05 18:26 ` Taniya Das 2018-09-06 1:53 ` Stephen Boyd 0 siblings, 1 reply; 10+ messages in thread From: Taniya Das @ 2018-09-05 18:26 UTC (permalink / raw) To: Stephen Boyd, Michael Turquette Cc: Andy Gross, David Brown, Rajendra Nayak, Amit Nischal, linux-arm-msm, linux-soc, linux-clk, linux-kernel, robh Hello Stephen, Thanks for the review comments. On 8/28/2018 2:44 AM, Stephen Boyd wrote: > Quoting Taniya Das (2018-08-03 05:21:13) >> Add device tree bindings for Low Power Audio subsystem clock controller for >> Qualcomm Technology Inc's SDM845 SoCs. >> >> Signed-off-by: Taniya Das <tdas@codeaurora.org> >> --- >> .../devicetree/bindings/clock/qcom,gcc.txt | 2 ++ >> .../devicetree/bindings/clock/qcom,lpasscc.txt | 33 ++++++++++++++++++++++ >> include/dt-bindings/clock/qcom,gcc-sdm845.h | 2 ++ >> include/dt-bindings/clock/qcom,lpass-sdm845.h | 16 +++++++++++ >> 4 files changed, 53 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/clock/qcom,lpasscc.txt >> create mode 100644 include/dt-bindings/clock/qcom,lpass-sdm845.h >> >> diff --git a/Documentation/devicetree/bindings/clock/qcom,gcc.txt b/Documentation/devicetree/bindings/clock/qcom,gcc.txt >> index 664ea1f..e452abc 100644 >> --- a/Documentation/devicetree/bindings/clock/qcom,gcc.txt >> +++ b/Documentation/devicetree/bindings/clock/qcom,gcc.txt >> @@ -32,6 +32,8 @@ be part of GCC and hence the TSENS properties can also be >> part of the GCC/clock-controller node. >> For more details on the TSENS properties please refer >> Documentation/devicetree/bindings/thermal/qcom-tsens.txt >> +- qcom,lpass-protected : Indicate GCC to be able to access the >> + lpass gcc clock branches. > > This doesn't parse well for me. Maybe something like: > > 'Indicate that the LPASS clock branches within GCC are unusable due to > firmware access control restrictions'? > Sure, will update in the next series. >> >> Example: >> clock-controller@900000 { >> diff --git a/Documentation/devicetree/bindings/clock/qcom,lpasscc.txt b/Documentation/devicetree/bindings/clock/qcom,lpasscc.txt >> new file mode 100644 >> index 0000000..062e413 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/clock/qcom,lpasscc.txt >> @@ -0,0 +1,33 @@ >> +Qualcomm LPASS Clock Controller Binding >> +----------------------------------------------- >> + >> +Required properties : >> +- compatible : shall contain "qcom,sdm845-lpasscc" >> +- #clock-cells : from common clock binding, shall contain 1. >> +- reg : shall contain base register address and size, >> + in the order >> + Index-0 maps to LPASS_CC register region >> + Index-1 maps to LPASS_QDSP6SS register region >> +- qcom,lpass-protected : Boolean property to indicate to GCC clock controller >> + for the lpass GCC clocks. > > Why is this here? > Yes, I kept it to make sure it is marked in GCC clock driver too. May be should remove it. >> + >> +Optional properties : >> +- reg-names : register names of LPASS domain >> + "lpass_cc", "lpass_qdsp6ss". >> + >> +Example: >> + >> +The below node has to be defined in the cases where the LPASS peripheral loader >> +would bring the subsystem out of reset. >> + >> + lpasscc: clock-controller { >> + compatible = "qcom,sdm845-lpasscc"; >> + reg = <0x17014000 0x1f004>, <0x17300000 0x200>; >> + reg-names = "lpass_cc", "lpass_qdsp6ss"; >> + #clock-cells = <1>; >> + }; >> + >> + gcc: clock-controller@100000 { >> + compatible = "qcom,gcc-sdm845"; >> + qcom,lpass-protected; >> + }; -- QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation. -- ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v3 1/2] dt-bindings: clock: Introduce QCOM LPASS clock bindings 2018-09-05 18:26 ` Taniya Das @ 2018-09-06 1:53 ` Stephen Boyd 0 siblings, 0 replies; 10+ messages in thread From: Stephen Boyd @ 2018-09-06 1:53 UTC (permalink / raw) To: Michael Turquette, Taniya Das Cc: Andy Gross, David Brown, Rajendra Nayak, Amit Nischal, linux-arm-msm, linux-soc, linux-clk, linux-kernel, robh Quoting Taniya Das (2018-09-05 11:26:36) > > On 8/28/2018 2:44 AM, Stephen Boyd wrote: > > Quoting Taniya Das (2018-08-03 05:21:13) > > >> > >> Example: > >> clock-controller@900000 { > >> diff --git a/Documentation/devicetree/bindings/clock/qcom,lpasscc.txt b/Documentation/devicetree/bindings/clock/qcom,lpasscc.txt > >> new file mode 100644 > >> index 0000000..062e413 > >> --- /dev/null > >> +++ b/Documentation/devicetree/bindings/clock/qcom,lpasscc.txt > >> @@ -0,0 +1,33 @@ > >> +Qualcomm LPASS Clock Controller Binding > >> +----------------------------------------------- > >> + > >> +Required properties : > >> +- compatible : shall contain "qcom,sdm845-lpasscc" > >> +- #clock-cells : from common clock binding, shall contain 1. > >> +- reg : shall contain base register address and size, > >> + in the order > >> + Index-0 maps to LPASS_CC register region > >> + Index-1 maps to LPASS_QDSP6SS register region > >> +- qcom,lpass-protected : Boolean property to indicate to GCC clock controller > >> + for the lpass GCC clocks. > > > > Why is this here? > > > > Yes, I kept it to make sure it is marked in GCC clock driver too. May be > should remove it. > It isn't a property of the LPASS clock controller though so I don't see why it is here. ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH v3 2/2] clk: qcom: Add lpass clock controller driver for SDM845 2018-08-03 12:21 [PATCH v3 0/2] Add support for LPASS clock controller for SDM845 Taniya Das 2018-08-03 12:21 ` [PATCH v3 1/2] dt-bindings: clock: Introduce QCOM LPASS clock bindings Taniya Das @ 2018-08-03 12:21 ` Taniya Das 2018-08-27 21:11 ` Stephen Boyd 1 sibling, 1 reply; 10+ messages in thread From: Taniya Das @ 2018-08-03 12:21 UTC (permalink / raw) To: Stephen Boyd, Michael Turquette Cc: Andy Gross, David Brown, Rajendra Nayak, Amit Nischal, linux-arm-msm, linux-soc, linux-clk, linux-kernel, robh, Taniya Das Add support for the lpass clock controller found on SDM845 based devices. This would allow lpass peripheral loader drivers to control the clocks to bring the subsystem out of reset. LPASS clocks present on the global clock controller would be registered with the clock framework based on the device tree flag. Signed-off-by: Taniya Das <tdas@codeaurora.org> --- drivers/clk/qcom/Kconfig | 9 ++ drivers/clk/qcom/Makefile | 1 + drivers/clk/qcom/gcc-sdm845.c | 35 +++++++ drivers/clk/qcom/lpasscc-sdm845.c | 189 ++++++++++++++++++++++++++++++++++++++ 4 files changed, 234 insertions(+) create mode 100644 drivers/clk/qcom/lpasscc-sdm845.c diff --git a/drivers/clk/qcom/Kconfig b/drivers/clk/qcom/Kconfig index 2b69cf2..7bd940d 100644 --- a/drivers/clk/qcom/Kconfig +++ b/drivers/clk/qcom/Kconfig @@ -254,6 +254,15 @@ config SDM_VIDEOCC_845 Say Y if you want to support video devices and functionality such as video encode and decode. +config SDM_LPASSCC_845 + tristate "SDM845 LPASS Clock Controller" + depends on COMMON_CLK_QCOM + select SDM_GCC_845 + help + Support for the LPASS clock controller on SDM845 devices. + Say Y if you want to use the LPASS branch clocks of the LPASS clock + controller to reset the LPASS subsystem. + config SPMI_PMIC_CLKDIV tristate "SPMI PMIC clkdiv Support" depends on (COMMON_CLK_QCOM && SPMI) || COMPILE_TEST diff --git a/drivers/clk/qcom/Makefile b/drivers/clk/qcom/Makefile index 599ab91..df2bd1f 100644 --- a/drivers/clk/qcom/Makefile +++ b/drivers/clk/qcom/Makefile @@ -40,5 +40,6 @@ obj-$(CONFIG_QCOM_CLK_RPM) += clk-rpm.o obj-$(CONFIG_QCOM_CLK_RPMH) += clk-rpmh.o obj-$(CONFIG_QCOM_CLK_SMD_RPM) += clk-smd-rpm.o obj-$(CONFIG_SDM_GCC_845) += gcc-sdm845.o +obj-$(CONFIG_SDM_LPASSCC_845) += lpasscc-sdm845.o obj-$(CONFIG_SDM_VIDEOCC_845) += videocc-sdm845.o obj-$(CONFIG_SPMI_PMIC_CLKDIV) += clk-spmi-pmic-div.o diff --git a/drivers/clk/qcom/gcc-sdm845.c b/drivers/clk/qcom/gcc-sdm845.c index 0f694ed..068cf53 100644 --- a/drivers/clk/qcom/gcc-sdm845.c +++ b/drivers/clk/qcom/gcc-sdm845.c @@ -3086,6 +3086,32 @@ enum { }, }; +static struct clk_branch gcc_lpass_q6_axi_clk = { + .halt_reg = 0x47000, + .halt_check = BRANCH_HALT, + .clkr = { + .enable_reg = 0x47000, + .enable_mask = BIT(0), + .hw.init = &(struct clk_init_data){ + .name = "gcc_lpass_q6_axi_clk", + .ops = &clk_branch2_ops, + }, + }, +}; + +static struct clk_branch gcc_lpass_sway_clk = { + .halt_reg = 0x47008, + .halt_check = BRANCH_HALT, + .clkr = { + .enable_reg = 0x47008, + .enable_mask = BIT(0), + .hw.init = &(struct clk_init_data){ + .name = "gcc_lpass_sway_clk", + .ops = &clk_branch2_ops, + }, + }, +}; + static struct gdsc pcie_0_gdsc = { .gdscr = 0x6b004, .pd = { @@ -3383,6 +3409,8 @@ enum { [GPLL4] = &gpll4.clkr, [GCC_CPUSS_DVM_BUS_CLK] = &gcc_cpuss_dvm_bus_clk.clkr, [GCC_CPUSS_GNOC_CLK] = &gcc_cpuss_gnoc_clk.clkr, + [GCC_LPASS_Q6_AXI_CLK] = NULL, + [GCC_LPASS_SWAY_CLK] = NULL, }; static const struct qcom_reset_map gcc_sdm845_resets[] = { @@ -3472,6 +3500,13 @@ static int gcc_sdm845_probe(struct platform_device *pdev) regmap_update_bits(regmap, 0x09ffc, 0x3, 0x3); regmap_update_bits(regmap, 0x71028, 0x3, 0x3); + if (of_property_read_bool(pdev->dev.of_node, "qcom,lpass-protected")) { + gcc_sdm845_clocks[GCC_LPASS_Q6_AXI_CLK] = + &gcc_lpass_q6_axi_clk.clkr; + gcc_sdm845_clocks[GCC_LPASS_SWAY_CLK] = + &gcc_lpass_sway_clk.clkr; + } + return qcom_cc_really_probe(pdev, &gcc_sdm845_desc, regmap); } diff --git a/drivers/clk/qcom/lpasscc-sdm845.c b/drivers/clk/qcom/lpasscc-sdm845.c new file mode 100644 index 0000000..6f387f9 --- /dev/null +++ b/drivers/clk/qcom/lpasscc-sdm845.c @@ -0,0 +1,189 @@ +// SPDX-License-Identifier: GPL-2.0 +/* + * Copyright (c) 2018, The Linux Foundation. All rights reserved. + */ + +#include <linux/bitops.h> +#include <linux/err.h> +#include <linux/platform_device.h> +#include <linux/module.h> +#include <linux/of_address.h> +#include <linux/regmap.h> + +#include <dt-bindings/clock/qcom,lpass-sdm845.h> + +#include "clk-regmap.h" +#include "clk-branch.h" +#include "common.h" + +static struct clk_branch lpass_audio_wrapper_aon_clk = { + .halt_reg = 0x098, + .halt_check = BRANCH_VOTED, + .clkr = { + .enable_reg = 0x098, + .enable_mask = BIT(0), + .hw.init = &(struct clk_init_data){ + .name = "lpass_audio_wrapper_aon_clk", + .ops = &clk_branch2_ops, + }, + }, +}; + +static struct clk_branch lpass_q6ss_ahbm_aon_clk = { + .halt_reg = 0x12000, + .halt_check = BRANCH_VOTED, + .clkr = { + .enable_reg = 0x12000, + .enable_mask = BIT(0), + .hw.init = &(struct clk_init_data){ + .name = "lpass_q6ss_ahbm_aon_clk", + .ops = &clk_branch2_ops, + }, + }, +}; + +static struct clk_branch lpass_q6ss_ahbs_aon_clk = { + .halt_reg = 0x1f000, + .halt_check = BRANCH_VOTED, + .clkr = { + .enable_reg = 0x1f000, + .enable_mask = BIT(0), + .hw.init = &(struct clk_init_data){ + .name = "lpass_q6ss_ahbs_aon_clk", + .ops = &clk_branch2_ops, + }, + }, +}; + +/* CLK_OFF would not toggle until LPASS is not out of reset */ +static struct clk_branch lpass_qdsp6ss_core_clk = { + .halt_reg = 0x20, + .halt_check = BRANCH_HALT_SKIP, + .clkr = { + .enable_reg = 0x20, + .enable_mask = BIT(0), + .hw.init = &(struct clk_init_data){ + .name = "lpass_qdsp6ss_core_clk", + .ops = &clk_branch2_ops, + }, + }, +}; + +/* CLK_OFF would not toggle until LPASS is not out of reset */ +static struct clk_branch lpass_qdsp6ss_xo_clk = { + .halt_reg = 0x38, + .halt_check = BRANCH_HALT_SKIP, + .clkr = { + .enable_reg = 0x38, + .enable_mask = BIT(0), + .hw.init = &(struct clk_init_data){ + .name = "lpass_qdsp6ss_xo_clk", + .ops = &clk_branch2_ops, + }, + }, +}; + +/* CLK_OFF would not toggle until LPASS is not out of reset */ +static struct clk_branch lpass_qdsp6ss_sleep_clk = { + .halt_reg = 0x3c, + .halt_check = BRANCH_HALT_SKIP, + .clkr = { + .enable_reg = 0x3c, + .enable_mask = BIT(0), + .hw.init = &(struct clk_init_data){ + .name = "lpass_qdsp6ss_sleep_clk", + .ops = &clk_branch2_ops, + }, + }, +}; + +static struct regmap_config lpass_regmap_config = { + .reg_bits = 32, + .reg_stride = 4, + .val_bits = 32, + .fast_io = true, +}; + +static struct clk_regmap *lpass_cc_sdm845_clocks[] = { + [LPASS_AUDIO_WRAPPER_AON_CLK] = &lpass_audio_wrapper_aon_clk.clkr, + [LPASS_Q6SS_AHBM_AON_CLK] = &lpass_q6ss_ahbm_aon_clk.clkr, + [LPASS_Q6SS_AHBS_AON_CLK] = &lpass_q6ss_ahbs_aon_clk.clkr, +}; + +static const struct qcom_cc_desc lpass_cc_sdm845_desc = { + .config = &lpass_regmap_config, + .clks = lpass_cc_sdm845_clocks, + .num_clks = ARRAY_SIZE(lpass_cc_sdm845_clocks), +}; + +static struct clk_regmap *lpass_qdsp6ss_sdm845_clocks[] = { + [LPASS_QDSP6SS_XO_CLK] = &lpass_qdsp6ss_xo_clk.clkr, + [LPASS_QDSP6SS_SLEEP_CLK] = &lpass_qdsp6ss_sleep_clk.clkr, + [LPASS_QDSP6SS_CORE_CLK] = &lpass_qdsp6ss_core_clk.clkr, +}; + +static const struct qcom_cc_desc lpass_qdsp6ss_sdm845_desc = { + .config = &lpass_regmap_config, + .clks = lpass_qdsp6ss_sdm845_clocks, + .num_clks = ARRAY_SIZE(lpass_qdsp6ss_sdm845_clocks), +}; + +static int lpass_clocks_sdm845_probe(struct platform_device *pdev, int index, + const struct qcom_cc_desc *desc) +{ + struct regmap *regmap; + struct resource *res; + void __iomem *base; + + res = platform_get_resource(pdev, IORESOURCE_MEM, index); + base = devm_ioremap_resource(&pdev->dev, res); + if (IS_ERR(base)) + return -ENOMEM; + + regmap = devm_regmap_init_mmio(&pdev->dev, base, desc->config); + if (IS_ERR(regmap)) + return PTR_ERR(regmap); + + return qcom_cc_really_probe(pdev, desc, regmap); +} + +/* LPASS CC clock controller */ +static const struct of_device_id lpass_cc_sdm845_match_table[] = { + { .compatible = "qcom,sdm845-lpasscc" }, + { } +}; +MODULE_DEVICE_TABLE(of, lpass_cc_sdm845_match_table); + +static int lpass_cc_sdm845_probe(struct platform_device *pdev) +{ + const struct qcom_cc_desc *desc; + int ret; + + lpass_regmap_config.name = "lpass_cc"; + desc = &lpass_cc_sdm845_desc; + + ret = lpass_clocks_sdm845_probe(pdev, 0, desc); + if (ret) + return ret; + + lpass_regmap_config.name = "lpass_qdsp6ss"; + desc = &lpass_qdsp6ss_sdm845_desc; + + return lpass_clocks_sdm845_probe(pdev, 1, desc); +} + +static struct platform_driver lpass_cc_sdm845_driver = { + .probe = lpass_cc_sdm845_probe, + .driver = { + .name = "sdm845-lpasscc", + .of_match_table = lpass_cc_sdm845_match_table, + }, +}; + +static int __init lpass_cc_sdm845_init(void) +{ + return platform_driver_register(&lpass_cc_sdm845_driver); +} +subsys_initcall(lpass_cc_sdm845_init); + +MODULE_LICENSE("GPL v2"); -- Qualcomm INDIA, on behalf of Qualcomm Innovation Center, Inc.is a member of the Code Aurora Forum, hosted by the Linux Foundation. ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH v3 2/2] clk: qcom: Add lpass clock controller driver for SDM845 2018-08-03 12:21 ` [PATCH v3 2/2] clk: qcom: Add lpass clock controller driver for SDM845 Taniya Das @ 2018-08-27 21:11 ` Stephen Boyd 2018-09-05 18:26 ` Taniya Das 0 siblings, 1 reply; 10+ messages in thread From: Stephen Boyd @ 2018-08-27 21:11 UTC (permalink / raw) To: Michael Turquette, Taniya Das Cc: Andy Gross, David Brown, Rajendra Nayak, Amit Nischal, linux-arm-msm, linux-soc, linux-clk, linux-kernel, robh, Taniya Das Quoting Taniya Das (2018-08-03 05:21:14) > diff --git a/drivers/clk/qcom/Kconfig b/drivers/clk/qcom/Kconfig > index 2b69cf2..7bd940d 100644 > --- a/drivers/clk/qcom/Kconfig > +++ b/drivers/clk/qcom/Kconfig > @@ -254,6 +254,15 @@ config SDM_VIDEOCC_845 > Say Y if you want to support video devices and functionality such as > video encode and decode. > > +config SDM_LPASSCC_845 > + tristate "SDM845 LPASS Clock Controller" Spell out the acronym? So "SDM845 Low Power Audio Subsystem (LPASS) Clock Controller"? > + depends on COMMON_CLK_QCOM > + select SDM_GCC_845 > + help > + Support for the LPASS clock controller on SDM845 devices. > + Say Y if you want to use the LPASS branch clocks of the LPASS clock > + controller to reset the LPASS subsystem. > + > config SPMI_PMIC_CLKDIV > tristate "SPMI PMIC clkdiv Support" > depends on (COMMON_CLK_QCOM && SPMI) || COMPILE_TEST > diff --git a/drivers/clk/qcom/gcc-sdm845.c b/drivers/clk/qcom/gcc-sdm845.c > index 0f694ed..068cf53 100644 > --- a/drivers/clk/qcom/gcc-sdm845.c > +++ b/drivers/clk/qcom/gcc-sdm845.c > @@ -3086,6 +3086,32 @@ enum { > }, > }; > > +static struct clk_branch gcc_lpass_q6_axi_clk = { > + .halt_reg = 0x47000, > + .halt_check = BRANCH_HALT, > + .clkr = { > + .enable_reg = 0x47000, > + .enable_mask = BIT(0), > + .hw.init = &(struct clk_init_data){ > + .name = "gcc_lpass_q6_axi_clk", > + .ops = &clk_branch2_ops, > + }, > + }, > +}; > + > +static struct clk_branch gcc_lpass_sway_clk = { > + .halt_reg = 0x47008, > + .halt_check = BRANCH_HALT, > + .clkr = { > + .enable_reg = 0x47008, > + .enable_mask = BIT(0), > + .hw.init = &(struct clk_init_data){ > + .name = "gcc_lpass_sway_clk", > + .ops = &clk_branch2_ops, > + }, > + }, > +}; > + > static struct gdsc pcie_0_gdsc = { > .gdscr = 0x6b004, > .pd = { > @@ -3383,6 +3409,8 @@ enum { > [GPLL4] = &gpll4.clkr, > [GCC_CPUSS_DVM_BUS_CLK] = &gcc_cpuss_dvm_bus_clk.clkr, > [GCC_CPUSS_GNOC_CLK] = &gcc_cpuss_gnoc_clk.clkr, > + [GCC_LPASS_Q6_AXI_CLK] = NULL, > + [GCC_LPASS_SWAY_CLK] = NULL, > }; > > static const struct qcom_reset_map gcc_sdm845_resets[] = { > diff --git a/drivers/clk/qcom/lpasscc-sdm845.c b/drivers/clk/qcom/lpasscc-sdm845.c > new file mode 100644 > index 0000000..6f387f9 > --- /dev/null > +++ b/drivers/clk/qcom/lpasscc-sdm845.c [...] > + > +/* CLK_OFF would not toggle until LPASS is not out of reset */ Can we change the branch ops to check for out of reset or not? Do the clks even work when LPASS isn't out of reset? Why would the clk APIs even be called on here if it hadn't taken LPASS out of reset? > +static struct clk_branch lpass_qdsp6ss_sleep_clk = { > + .halt_reg = 0x3c, > + .halt_check = BRANCH_HALT_SKIP, > + .clkr = { > + .enable_reg = 0x3c, > + .enable_mask = BIT(0), > + .hw.init = &(struct clk_init_data){ > + .name = "lpass_qdsp6ss_sleep_clk", > + .ops = &clk_branch2_ops, > + }, > + }, > +}; > + > +static struct regmap_config lpass_regmap_config = { > + .reg_bits = 32, > + .reg_stride = 4, > + .val_bits = 32, > + .fast_io = true, > +}; > + > +static struct clk_regmap *lpass_cc_sdm845_clocks[] = { > + [LPASS_AUDIO_WRAPPER_AON_CLK] = &lpass_audio_wrapper_aon_clk.clkr, > + [LPASS_Q6SS_AHBM_AON_CLK] = &lpass_q6ss_ahbm_aon_clk.clkr, > + [LPASS_Q6SS_AHBS_AON_CLK] = &lpass_q6ss_ahbs_aon_clk.clkr, > +}; > + > +static const struct qcom_cc_desc lpass_cc_sdm845_desc = { > + .config = &lpass_regmap_config, > + .clks = lpass_cc_sdm845_clocks, > + .num_clks = ARRAY_SIZE(lpass_cc_sdm845_clocks), > +}; > + > +static struct clk_regmap *lpass_qdsp6ss_sdm845_clocks[] = { > + [LPASS_QDSP6SS_XO_CLK] = &lpass_qdsp6ss_xo_clk.clkr, > + [LPASS_QDSP6SS_SLEEP_CLK] = &lpass_qdsp6ss_sleep_clk.clkr, > + [LPASS_QDSP6SS_CORE_CLK] = &lpass_qdsp6ss_core_clk.clkr, > +}; > + > +static const struct qcom_cc_desc lpass_qdsp6ss_sdm845_desc = { > + .config = &lpass_regmap_config, > + .clks = lpass_qdsp6ss_sdm845_clocks, > + .num_clks = ARRAY_SIZE(lpass_qdsp6ss_sdm845_clocks), > +}; > + > +static int lpass_clocks_sdm845_probe(struct platform_device *pdev, int index, > + const struct qcom_cc_desc *desc) > +{ > + struct regmap *regmap; > + struct resource *res; > + void __iomem *base; > + > + res = platform_get_resource(pdev, IORESOURCE_MEM, index); > + base = devm_ioremap_resource(&pdev->dev, res); > + if (IS_ERR(base)) > + return -ENOMEM; return PTR_ERR(base)? > + > + regmap = devm_regmap_init_mmio(&pdev->dev, base, desc->config); > + if (IS_ERR(regmap)) > + return PTR_ERR(regmap); > + > + return qcom_cc_really_probe(pdev, desc, regmap); > +} > + > +/* LPASS CC clock controller */ > +static const struct of_device_id lpass_cc_sdm845_match_table[] = { > + { .compatible = "qcom,sdm845-lpasscc" }, > + { } > +}; > +MODULE_DEVICE_TABLE(of, lpass_cc_sdm845_match_table); > + > +static int lpass_cc_sdm845_probe(struct platform_device *pdev) > +{ > + const struct qcom_cc_desc *desc; > + int ret; > + > + lpass_regmap_config.name = "lpass_cc"; > + desc = &lpass_cc_sdm845_desc; > + > + ret = lpass_clocks_sdm845_probe(pdev, 0, desc); > + if (ret) > + return ret; > + > + lpass_regmap_config.name = "lpass_qdsp6ss"; > + desc = &lpass_qdsp6ss_sdm845_desc; > + > + return lpass_clocks_sdm845_probe(pdev, 1, desc); > +} > + > +static struct platform_driver lpass_cc_sdm845_driver = { > + .probe = lpass_cc_sdm845_probe, > + .driver = { > + .name = "sdm845-lpasscc", > + .of_match_table = lpass_cc_sdm845_match_table, > + }, > +}; > + > +static int __init lpass_cc_sdm845_init(void) > +{ > + return platform_driver_register(&lpass_cc_sdm845_driver); > +} > +subsys_initcall(lpass_cc_sdm845_init); Also add module_exit() path. > + > +MODULE_LICENSE("GPL v2"); ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v3 2/2] clk: qcom: Add lpass clock controller driver for SDM845 2018-08-27 21:11 ` Stephen Boyd @ 2018-09-05 18:26 ` Taniya Das 2018-09-06 1:51 ` Stephen Boyd 0 siblings, 1 reply; 10+ messages in thread From: Taniya Das @ 2018-09-05 18:26 UTC (permalink / raw) To: Stephen Boyd, Michael Turquette Cc: Andy Gross, David Brown, Rajendra Nayak, Amit Nischal, linux-arm-msm, linux-soc, linux-clk, linux-kernel, robh Hello Stephen, Thanks for the review comments. On 8/28/2018 2:41 AM, Stephen Boyd wrote: > Quoting Taniya Das (2018-08-03 05:21:14) >> diff --git a/drivers/clk/qcom/Kconfig b/drivers/clk/qcom/Kconfig >> index 2b69cf2..7bd940d 100644 >> --- a/drivers/clk/qcom/Kconfig >> +++ b/drivers/clk/qcom/Kconfig >> @@ -254,6 +254,15 @@ config SDM_VIDEOCC_845 >> Say Y if you want to support video devices and functionality such as >> video encode and decode. >> >> +config SDM_LPASSCC_845 >> + tristate "SDM845 LPASS Clock Controller" > > Spell out the acronym? So "SDM845 Low Power Audio Subsystem (LPASS) > Clock Controller"? > Sure will update in the next patch. >> + depends on COMMON_CLK_QCOM >> + select SDM_GCC_845 >> + help >> + Support for the LPASS clock controller on SDM845 devices. >> + Say Y if you want to use the LPASS branch clocks of the LPASS clock >> + controller to reset the LPASS subsystem. >> + >> config SPMI_PMIC_CLKDIV >> tristate "SPMI PMIC clkdiv Support" >> depends on (COMMON_CLK_QCOM && SPMI) || COMPILE_TEST >> diff --git a/drivers/clk/qcom/gcc-sdm845.c b/drivers/clk/qcom/gcc-sdm845.c >> index 0f694ed..068cf53 100644 >> --- a/drivers/clk/qcom/gcc-sdm845.c >> +++ b/drivers/clk/qcom/gcc-sdm845.c >> @@ -3086,6 +3086,32 @@ enum { >> }, >> }; >> >> +static struct clk_branch gcc_lpass_q6_axi_clk = { >> + .halt_reg = 0x47000, >> + .halt_check = BRANCH_HALT, >> + .clkr = { >> + .enable_reg = 0x47000, >> + .enable_mask = BIT(0), >> + .hw.init = &(struct clk_init_data){ >> + .name = "gcc_lpass_q6_axi_clk", >> + .ops = &clk_branch2_ops, >> + }, >> + }, >> +}; >> + >> +static struct clk_branch gcc_lpass_sway_clk = { >> + .halt_reg = 0x47008, >> + .halt_check = BRANCH_HALT, >> + .clkr = { >> + .enable_reg = 0x47008, >> + .enable_mask = BIT(0), >> + .hw.init = &(struct clk_init_data){ >> + .name = "gcc_lpass_sway_clk", >> + .ops = &clk_branch2_ops, >> + }, >> + }, >> +}; >> + >> static struct gdsc pcie_0_gdsc = { >> .gdscr = 0x6b004, >> .pd = { >> @@ -3383,6 +3409,8 @@ enum { >> [GPLL4] = &gpll4.clkr, >> [GCC_CPUSS_DVM_BUS_CLK] = &gcc_cpuss_dvm_bus_clk.clkr, >> [GCC_CPUSS_GNOC_CLK] = &gcc_cpuss_gnoc_clk.clkr, >> + [GCC_LPASS_Q6_AXI_CLK] = NULL, >> + [GCC_LPASS_SWAY_CLK] = NULL, >> }; >> >> static const struct qcom_reset_map gcc_sdm845_resets[] = { >> diff --git a/drivers/clk/qcom/lpasscc-sdm845.c b/drivers/clk/qcom/lpasscc-sdm845.c >> new file mode 100644 >> index 0000000..6f387f9 >> --- /dev/null >> +++ b/drivers/clk/qcom/lpasscc-sdm845.c > [...] >> + >> +/* CLK_OFF would not toggle until LPASS is not out of reset */ > > Can we change the branch ops to check for out of reset or not? Do the > clks even work when LPASS isn't out of reset? Why would the clk APIs > even be called on here if it hadn't taken LPASS out of reset? > The branches need to be turned ON before the LPASS is out of reset. But we would not be able to check the CLK_ENABLE bit. >> +static struct clk_branch lpass_qdsp6ss_sleep_clk = { >> + .halt_reg = 0x3c, >> + .halt_check = BRANCH_HALT_SKIP, >> + .clkr = { >> + .enable_reg = 0x3c, >> + .enable_mask = BIT(0), >> + .hw.init = &(struct clk_init_data){ >> + .name = "lpass_qdsp6ss_sleep_clk", >> + .ops = &clk_branch2_ops, >> + }, >> + }, >> +}; >> + >> +static struct regmap_config lpass_regmap_config = { >> + .reg_bits = 32, >> + .reg_stride = 4, >> + .val_bits = 32, >> + .fast_io = true, >> +}; >> + >> +static struct clk_regmap *lpass_cc_sdm845_clocks[] = { >> + [LPASS_AUDIO_WRAPPER_AON_CLK] = &lpass_audio_wrapper_aon_clk.clkr, >> + [LPASS_Q6SS_AHBM_AON_CLK] = &lpass_q6ss_ahbm_aon_clk.clkr, >> + [LPASS_Q6SS_AHBS_AON_CLK] = &lpass_q6ss_ahbs_aon_clk.clkr, >> +}; >> + >> +static const struct qcom_cc_desc lpass_cc_sdm845_desc = { >> + .config = &lpass_regmap_config, >> + .clks = lpass_cc_sdm845_clocks, >> + .num_clks = ARRAY_SIZE(lpass_cc_sdm845_clocks), >> +}; >> + >> +static struct clk_regmap *lpass_qdsp6ss_sdm845_clocks[] = { >> + [LPASS_QDSP6SS_XO_CLK] = &lpass_qdsp6ss_xo_clk.clkr, >> + [LPASS_QDSP6SS_SLEEP_CLK] = &lpass_qdsp6ss_sleep_clk.clkr, >> + [LPASS_QDSP6SS_CORE_CLK] = &lpass_qdsp6ss_core_clk.clkr, >> +}; >> + >> +static const struct qcom_cc_desc lpass_qdsp6ss_sdm845_desc = { >> + .config = &lpass_regmap_config, >> + .clks = lpass_qdsp6ss_sdm845_clocks, >> + .num_clks = ARRAY_SIZE(lpass_qdsp6ss_sdm845_clocks), >> +}; >> + >> +static int lpass_clocks_sdm845_probe(struct platform_device *pdev, int index, >> + const struct qcom_cc_desc *desc) >> +{ >> + struct regmap *regmap; >> + struct resource *res; >> + void __iomem *base; >> + >> + res = platform_get_resource(pdev, IORESOURCE_MEM, index); >> + base = devm_ioremap_resource(&pdev->dev, res); >> + if (IS_ERR(base)) >> + return -ENOMEM; > > return PTR_ERR(base)? > Sure will update it. >> + >> + regmap = devm_regmap_init_mmio(&pdev->dev, base, desc->config); >> + if (IS_ERR(regmap)) >> + return PTR_ERR(regmap); >> + >> + return qcom_cc_really_probe(pdev, desc, regmap); >> +} >> + >> +/* LPASS CC clock controller */ >> +static const struct of_device_id lpass_cc_sdm845_match_table[] = { >> + { .compatible = "qcom,sdm845-lpasscc" }, >> + { } >> +}; >> +MODULE_DEVICE_TABLE(of, lpass_cc_sdm845_match_table); >> + >> +static int lpass_cc_sdm845_probe(struct platform_device *pdev) >> +{ >> + const struct qcom_cc_desc *desc; >> + int ret; >> + >> + lpass_regmap_config.name = "lpass_cc"; >> + desc = &lpass_cc_sdm845_desc; >> + >> + ret = lpass_clocks_sdm845_probe(pdev, 0, desc); >> + if (ret) >> + return ret; >> + >> + lpass_regmap_config.name = "lpass_qdsp6ss"; >> + desc = &lpass_qdsp6ss_sdm845_desc; >> + >> + return lpass_clocks_sdm845_probe(pdev, 1, desc); >> +} >> + >> +static struct platform_driver lpass_cc_sdm845_driver = { >> + .probe = lpass_cc_sdm845_probe, >> + .driver = { >> + .name = "sdm845-lpasscc", >> + .of_match_table = lpass_cc_sdm845_match_table, >> + }, >> +}; >> + >> +static int __init lpass_cc_sdm845_init(void) >> +{ >> + return platform_driver_register(&lpass_cc_sdm845_driver); >> +} >> +subsys_initcall(lpass_cc_sdm845_init); > > Also add module_exit() path. > Will update in the next patch. >> + >> +MODULE_LICENSE("GPL v2"); -- QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation. -- ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v3 2/2] clk: qcom: Add lpass clock controller driver for SDM845 2018-09-05 18:26 ` Taniya Das @ 2018-09-06 1:51 ` Stephen Boyd 2018-09-06 11:31 ` Taniya Das 0 siblings, 1 reply; 10+ messages in thread From: Stephen Boyd @ 2018-09-06 1:51 UTC (permalink / raw) To: Michael Turquette, Taniya Das Cc: Andy Gross, David Brown, Rajendra Nayak, Amit Nischal, linux-arm-msm, linux-soc, linux-clk, linux-kernel, robh Quoting Taniya Das (2018-09-05 11:26:10) > On 8/28/2018 2:41 AM, Stephen Boyd wrote: > > Quoting Taniya Das (2018-08-03 05:21:14) > >> diff --git a/drivers/clk/qcom/lpasscc-sdm845.c b/drivers/clk/qcom/lpasscc-sdm845.c > >> new file mode 100644 > >> index 0000000..6f387f9 > >> --- /dev/null > >> +++ b/drivers/clk/qcom/lpasscc-sdm845.c > > [...] > >> + > >> +/* CLK_OFF would not toggle until LPASS is not out of reset */ > > > > Can we change the branch ops to check for out of reset or not? Do the > > clks even work when LPASS isn't out of reset? Why would the clk APIs > > even be called on here if it hadn't taken LPASS out of reset? > > > > The branches need to be turned ON before the LPASS is out of reset. > But we would not be able to check the CLK_ENABLE bit. Ok. Are the branches actually outputting any clk frequency when LPASS is in reset? It sounds like the hardware is broken and we can't ever check the halt bits? ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v3 2/2] clk: qcom: Add lpass clock controller driver for SDM845 2018-09-06 1:51 ` Stephen Boyd @ 2018-09-06 11:31 ` Taniya Das 0 siblings, 0 replies; 10+ messages in thread From: Taniya Das @ 2018-09-06 11:31 UTC (permalink / raw) To: Stephen Boyd, Michael Turquette Cc: Andy Gross, David Brown, Rajendra Nayak, Amit Nischal, linux-arm-msm, linux-soc, linux-clk, linux-kernel, robh On 9/6/2018 7:21 AM, Stephen Boyd wrote: > Quoting Taniya Das (2018-09-05 11:26:10) >> On 8/28/2018 2:41 AM, Stephen Boyd wrote: >>> Quoting Taniya Das (2018-08-03 05:21:14) >>>> diff --git a/drivers/clk/qcom/lpasscc-sdm845.c b/drivers/clk/qcom/lpasscc-sdm845.c >>>> new file mode 100644 >>>> index 0000000..6f387f9 >>>> --- /dev/null >>>> +++ b/drivers/clk/qcom/lpasscc-sdm845.c >>> [...] >>>> + >>>> +/* CLK_OFF would not toggle until LPASS is not out of reset */ >>> >>> Can we change the branch ops to check for out of reset or not? Do the >>> clks even work when LPASS isn't out of reset? Why would the clk APIs >>> even be called on here if it hadn't taken LPASS out of reset? >>> >> >> The branches need to be turned ON before the LPASS is out of reset. >> But we would not be able to check the CLK_ENABLE bit. > > Ok. Are the branches actually outputting any clk frequency when LPASS is > in reset? It sounds like the hardware is broken and we can't ever check > the halt bits? > Yes, they would output when the reset is pulled for LPASS. These are as per the steps to be followed to bring the LPASS out of reset. -- QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation. -- ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2018-09-06 11:31 UTC | newest] Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2018-08-03 12:21 [PATCH v3 0/2] Add support for LPASS clock controller for SDM845 Taniya Das 2018-08-03 12:21 ` [PATCH v3 1/2] dt-bindings: clock: Introduce QCOM LPASS clock bindings Taniya Das 2018-08-27 21:14 ` Stephen Boyd 2018-09-05 18:26 ` Taniya Das 2018-09-06 1:53 ` Stephen Boyd 2018-08-03 12:21 ` [PATCH v3 2/2] clk: qcom: Add lpass clock controller driver for SDM845 Taniya Das 2018-08-27 21:11 ` Stephen Boyd 2018-09-05 18:26 ` Taniya Das 2018-09-06 1:51 ` Stephen Boyd 2018-09-06 11:31 ` Taniya Das
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).