* [PATCH 0/2] Add support for LPASS clock controller for SDM845 @ 2018-06-14 7:39 Taniya Das 2018-06-14 7:39 ` [PATCH 1/2] dt-bindings: clock: Introduce QCOM LPASS clock bindings Taniya Das 2018-06-14 7:39 ` [PATCH 2/2] clk: qcom: Add lpass clock controller driver for SDM845 Taniya Das 0 siblings, 2 replies; 8+ messages in thread From: Taniya Das @ 2018-06-14 7:39 UTC (permalink / raw) To: Stephen Boyd, Michael Turquette, robh Cc: Andy Gross, David Brown, Rajendra Nayak, Amit Nischal, linux-arm-msm, linux-soc, linux-clk, linux-kernel, devicetree, Rohit Kumar, 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. Taniya Das (2): dt-bindings: clock: Introduce QCOM LPASS clock bindings clk: qcom: Add lpass clock controller driver for SDM845 .../devicetree/bindings/clock/qcom,lpasscc.txt | 46 ++++ drivers/clk/qcom/Kconfig | 9 + drivers/clk/qcom/Makefile | 1 + drivers/clk/qcom/lpasscc-sdm845.c | 248 +++++++++++++++++++++ include/dt-bindings/clock/qcom,lpass-sdm845.h | 18 ++ 5 files changed, 322 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] 8+ messages in thread
* [PATCH 1/2] dt-bindings: clock: Introduce QCOM LPASS clock bindings 2018-06-14 7:39 [PATCH 0/2] Add support for LPASS clock controller for SDM845 Taniya Das @ 2018-06-14 7:39 ` Taniya Das 2018-06-26 19:53 ` Rob Herring 2018-06-14 7:39 ` [PATCH 2/2] clk: qcom: Add lpass clock controller driver for SDM845 Taniya Das 1 sibling, 1 reply; 8+ messages in thread From: Taniya Das @ 2018-06-14 7:39 UTC (permalink / raw) To: Stephen Boyd, Michael Turquette, robh Cc: Andy Gross, David Brown, Rajendra Nayak, Amit Nischal, linux-arm-msm, linux-soc, linux-clk, linux-kernel, devicetree, Rohit Kumar, 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,lpasscc.txt | 46 ++++++++++++++++++++++ include/dt-bindings/clock/qcom,lpass-sdm845.h | 18 +++++++++ 2 files changed, 64 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,lpasscc.txt b/Documentation/devicetree/bindings/clock/qcom,lpasscc.txt new file mode 100644 index 0000000..16cabc4 --- /dev/null +++ b/Documentation/devicetree/bindings/clock/qcom,lpasscc.txt @@ -0,0 +1,46 @@ +Qualcomm LPASS Clock Controller Binding +----------------------------------------------- + +Required properties : +- compatible : shall contain "qcom,sdm845-lpasscc" +- #clock-cells : from common clock binding, shall contain 1. + +Note that #address-cells, #size-cells, and ranges shall be present to ensure +the lpasscc can address the various lpass cc registers. + +Child Node Properties : +The Low Pass Audio clock controller would need to define the following child +nodes with the properties. +- compatible : shall contain all of the below for clocks in each LPASS domain + "qcom,sdm845-lpass-gcc", + "qcom,sdm845-lpass-cc", + "qcom,sdm845-lpass-qdsp6ss" +- reg : shall contain base register address and size, + +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"; + #clock-cells = <1>; + #address-cells = <1>; + #size-cells = <1>; + ranges; + + lpass_gcc@100000 { + compatible = "qcom,sdm845-lpass-gcc"; + reg = <0x00147000 0x20>; + }; + + lpass@17014000 { + compatible = "qcom,sdm845-lpass-cc"; + reg = <0x17014000 0x1f004>; + }; + + lpass_q6@17300020 { + compatible = "qcom,sdm845-lpass-qdsp6ss"; + reg = <0x17300020 0x20>; + }; + }; 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..b9d816e --- /dev/null +++ b/include/dt-bindings/clock/qcom,lpass-sdm845.h @@ -0,0 +1,18 @@ +/* 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 GCC_LPASS_Q6_AXI_CLK 0 +#define GCC_LPASS_SWAY_CLK 1 +#define LPASS_AUDIO_WRAPPER_AON_CLK 2 +#define LPASS_Q6SS_AHBM_AON_CLK 3 +#define LPASS_Q6SS_AHBS_AON_CLK 4 +#define LPASS_QDSP6SS_XO_CLK 5 +#define LPASS_QDSP6SS_SLEEP_CLK 6 +#define LPASS_QDSP6SS_CORE_CLK 7 + +#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] 8+ messages in thread
* Re: [PATCH 1/2] dt-bindings: clock: Introduce QCOM LPASS clock bindings 2018-06-14 7:39 ` [PATCH 1/2] dt-bindings: clock: Introduce QCOM LPASS clock bindings Taniya Das @ 2018-06-26 19:53 ` Rob Herring 0 siblings, 0 replies; 8+ messages in thread From: Rob Herring @ 2018-06-26 19:53 UTC (permalink / raw) To: Taniya Das Cc: Stephen Boyd, Michael Turquette, Andy Gross, David Brown, Rajendra Nayak, Amit Nischal, linux-arm-msm, linux-soc, linux-clk, linux-kernel, devicetree, Rohit Kumar On Thu, Jun 14, 2018 at 01:09:02PM +0530, Taniya Das wrote: > 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,lpasscc.txt | 46 ++++++++++++++++++++++ > include/dt-bindings/clock/qcom,lpass-sdm845.h | 18 +++++++++ > 2 files changed, 64 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,lpasscc.txt b/Documentation/devicetree/bindings/clock/qcom,lpasscc.txt > new file mode 100644 > index 0000000..16cabc4 > --- /dev/null > +++ b/Documentation/devicetree/bindings/clock/qcom,lpasscc.txt > @@ -0,0 +1,46 @@ > +Qualcomm LPASS Clock Controller Binding > +----------------------------------------------- > + > +Required properties : > +- compatible : shall contain "qcom,sdm845-lpasscc" > +- #clock-cells : from common clock binding, shall contain 1. > + > +Note that #address-cells, #size-cells, and ranges shall be present to ensure > +the lpasscc can address the various lpass cc registers. > + > +Child Node Properties : > +The Low Pass Audio clock controller would need to define the following child > +nodes with the properties. > +- compatible : shall contain all of the below for clocks in each LPASS domain > + "qcom,sdm845-lpass-gcc", > + "qcom,sdm845-lpass-cc", > + "qcom,sdm845-lpass-qdsp6ss" > +- reg : shall contain base register address and size, I can't really see any reason to have these child nodes. Just put 3 entries in reg in the parent. Rob ^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH 2/2] clk: qcom: Add lpass clock controller driver for SDM845 2018-06-14 7:39 [PATCH 0/2] Add support for LPASS clock controller for SDM845 Taniya Das 2018-06-14 7:39 ` [PATCH 1/2] dt-bindings: clock: Introduce QCOM LPASS clock bindings Taniya Das @ 2018-06-14 7:39 ` Taniya Das 2018-06-14 10:38 ` kbuild test robot ` (2 more replies) 1 sibling, 3 replies; 8+ messages in thread From: Taniya Das @ 2018-06-14 7:39 UTC (permalink / raw) To: Stephen Boyd, Michael Turquette, robh Cc: Andy Gross, David Brown, Rajendra Nayak, Amit Nischal, linux-arm-msm, linux-soc, linux-clk, linux-kernel, devicetree, Rohit Kumar, 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. Signed-off-by: Taniya Das <tdas@codeaurora.org> --- drivers/clk/qcom/Kconfig | 9 ++ drivers/clk/qcom/Makefile | 1 + drivers/clk/qcom/lpasscc-sdm845.c | 252 ++++++++++++++++++++++++++++++++++++++ 3 files changed, 262 insertions(+) create mode 100644 drivers/clk/qcom/lpasscc-sdm845.c diff --git a/drivers/clk/qcom/Kconfig b/drivers/clk/qcom/Kconfig index 9c3480d..06b3f2e 100644 --- a/drivers/clk/qcom/Kconfig +++ b/drivers/clk/qcom/Kconfig @@ -245,6 +245,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 762c011..799b9d7 100644 --- a/drivers/clk/qcom/Makefile +++ b/drivers/clk/qcom/Makefile @@ -39,5 +39,6 @@ obj-$(CONFIG_QCOM_CLK_APCS_MSM8916) += apcs-msm8916.o obj-$(CONFIG_QCOM_CLK_RPM) += clk-rpm.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/lpasscc-sdm845.c b/drivers/clk/qcom/lpasscc-sdm845.c new file mode 100644 index 0000000..d04a4c8 --- /dev/null +++ b/drivers/clk/qcom/lpasscc-sdm845.c @@ -0,0 +1,252 @@ +// 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 gcc_lpass_q6_axi_clk = { + .halt_reg = 0x0, + .halt_check = BRANCH_HALT, + .clkr = { + .enable_reg = 0x0, + .enable_mask = BIT(0), + .hw.init = &(struct clk_init_data){ + .name = "gcc_lpass_q6_axi_clk", + .ops = &clk_branch2_ops, + }, + }, +}; + +struct clk_branch gcc_lpass_sway_clk = { + .halt_reg = 0x8, + .halt_check = BRANCH_HALT, + .clkr = { + .enable_reg = 0x8, + .enable_mask = BIT(0), + .hw.init = &(struct clk_init_data){ + .name = "gcc_lpass_sway_clk", + .ops = &clk_branch2_ops, + }, + }, +}; + +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, + }, + }, +}; + +static struct clk_branch lpass_qdsp6ss_xo_clk = { + .halt_reg = 0x18, + .halt_check = BRANCH_HALT_SKIP, + .clkr = { + .enable_reg = 0x18, + .enable_mask = BIT(0), + .hw.init = &(struct clk_init_data){ + .name = "lpass_qdsp6ss_xo_clk", + .ops = &clk_branch2_ops, + }, + }, +}; + +static struct clk_branch lpass_qdsp6ss_sleep_clk = { + .halt_reg = 0x1c, + .halt_check = BRANCH_HALT_SKIP, + .clkr = { + .enable_reg = 0x1c, + .enable_mask = BIT(0), + .hw.init = &(struct clk_init_data){ + .name = "lpass_qdsp6ss_sleep_clk", + .ops = &clk_branch2_ops, + }, + }, +}; + +static struct clk_branch lpass_qdsp6ss_core_clk = { + .halt_reg = 0x0, + .halt_check = BRANCH_HALT_SKIP, + .clkr = { + .enable_reg = 0x0, + .enable_mask = BIT(0), + .hw.init = &(struct clk_init_data){ + .name = "lpass_qdsp6ss_core_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_gcc_sdm845_clocks[] = { + [GCC_LPASS_Q6_AXI_CLK] = &gcc_lpass_q6_axi_clk.clkr, + [GCC_LPASS_SWAY_CLK] = &gcc_lpass_sway_clk.clkr, +}; + +static const struct qcom_cc_desc lpass_gcc_sdm845_desc = { + .config = &lpass_regmap_config, + .clks = lpass_gcc_sdm845_clocks, + .num_clks = ARRAY_SIZE(lpass_gcc_sdm845_clocks), +}; + +static const struct of_device_id lpass_gcc_sdm845_match_table[] = { + { .compatible = "qcom,sdm845-lpass-gcc" }, + { } +}; + +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 const struct of_device_id lpasscc_sdm845_match_table[] = { + { .compatible = "qcom,sdm845-lpass-cc" }, + { } +}; + +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 const struct of_device_id lpass_qdsp6_sdm845_match_table[] = { + { .compatible = "qcom,sdm845-lpass-qdsp6ss" }, + { } +}; + +static int lpass_clocks_sdm845_probe(struct platform_device *pdev, + struct device_node *np, + const struct qcom_cc_desc *desc) +{ + struct regmap *regmap; + struct resource res; + void __iomem *base; + + if (of_address_to_resource(np, 0, &res)) + return -ENOMEM; + + base = devm_ioremap(&pdev->dev, res.start, resource_size(&res)); + if (IS_ERR(base)) + return -ENOMEM; + + regmap = devm_regmap_init_mmio(&pdev->dev, base, desc->config); + if (!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) +{ + struct device_node *cp; + const struct qcom_cc_desc *desc; + int ret; + + for_each_available_child_of_node(pdev->dev.of_node, cp) { + if (of_match_node(lpass_gcc_sdm845_match_table, cp)) { + lpass_regmap_config.name = "lpass_gcc"; + desc = &lpass_gcc_sdm845_desc; + } else if (of_match_node(lpasscc_sdm845_match_table, cp)) { + lpass_regmap_config.name = "lpass_cc"; + desc = &lpass_cc_sdm845_desc; + } else if (of_match_node(lpass_qdsp6_sdm845_match_table, cp)) { + lpass_regmap_config.name = "lpass_qdsp6ss"; + desc = &lpass_qdsp6ss_sdm845_desc; + } else { + dev_err(&pdev->dev, "LPASS child node not defined\n"); + return -EINVAL; + } + + ret = lpass_clocks_sdm845_probe(pdev, cp, desc); + if (ret) + return ret; + } + + return 0; +} + +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] 8+ messages in thread
* Re: [PATCH 2/2] clk: qcom: Add lpass clock controller driver for SDM845 2018-06-14 7:39 ` [PATCH 2/2] clk: qcom: Add lpass clock controller driver for SDM845 Taniya Das @ 2018-06-14 10:38 ` kbuild test robot 2018-06-14 10:38 ` [RFC PATCH] clk: qcom: gcc_lpass_sway_clk can be static kbuild test robot 2018-06-18 14:40 ` [PATCH 2/2] clk: qcom: Add lpass clock controller driver for SDM845 Dan Carpenter 2 siblings, 0 replies; 8+ messages in thread From: kbuild test robot @ 2018-06-14 10:38 UTC (permalink / raw) To: Taniya Das Cc: kbuild-all, Stephen Boyd, Michael Turquette, robh, Andy Gross, David Brown, Rajendra Nayak, Amit Nischal, linux-arm-msm, linux-soc, linux-clk, linux-kernel, devicetree, Rohit Kumar, Taniya Das Hi Taniya, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on clk/clk-next] [also build test WARNING on next-20180614] [cannot apply to v4.17] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Taniya-Das/Add-support-for-LPASS-clock-controller-for-SDM845/20180614-155144 base: https://git.kernel.org/pub/scm/linux/kernel/git/clk/linux.git clk-next reproduce: # apt-get install sparse make ARCH=x86_64 allmodconfig make C=1 CF=-D__CHECK_ENDIAN__ sparse warnings: (new ones prefixed by >>) >> drivers/clk/qcom/lpasscc-sdm845.c:32:19: sparse: symbol 'gcc_lpass_sway_clk' was not declared. Should it be static? Please review and possibly fold the followup patch. --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation ^ permalink raw reply [flat|nested] 8+ messages in thread
* [RFC PATCH] clk: qcom: gcc_lpass_sway_clk can be static 2018-06-14 7:39 ` [PATCH 2/2] clk: qcom: Add lpass clock controller driver for SDM845 Taniya Das 2018-06-14 10:38 ` kbuild test robot @ 2018-06-14 10:38 ` kbuild test robot 2018-06-18 14:40 ` [PATCH 2/2] clk: qcom: Add lpass clock controller driver for SDM845 Dan Carpenter 2 siblings, 0 replies; 8+ messages in thread From: kbuild test robot @ 2018-06-14 10:38 UTC (permalink / raw) To: Taniya Das Cc: kbuild-all, Stephen Boyd, Michael Turquette, robh, Andy Gross, David Brown, Rajendra Nayak, Amit Nischal, linux-arm-msm, linux-soc, linux-clk, linux-kernel, devicetree, Rohit Kumar, Taniya Das Fixes: 035d2520712d ("clk: qcom: Add lpass clock controller driver for SDM845") Signed-off-by: kbuild test robot <fengguang.wu@intel.com> --- lpasscc-sdm845.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/clk/qcom/lpasscc-sdm845.c b/drivers/clk/qcom/lpasscc-sdm845.c index d04a4c8..5b9ad76 100644 --- a/drivers/clk/qcom/lpasscc-sdm845.c +++ b/drivers/clk/qcom/lpasscc-sdm845.c @@ -29,7 +29,7 @@ static struct clk_branch gcc_lpass_q6_axi_clk = { }, }; -struct clk_branch gcc_lpass_sway_clk = { +static struct clk_branch gcc_lpass_sway_clk = { .halt_reg = 0x8, .halt_check = BRANCH_HALT, .clkr = { ^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH 2/2] clk: qcom: Add lpass clock controller driver for SDM845 2018-06-14 7:39 ` [PATCH 2/2] clk: qcom: Add lpass clock controller driver for SDM845 Taniya Das 2018-06-14 10:38 ` kbuild test robot 2018-06-14 10:38 ` [RFC PATCH] clk: qcom: gcc_lpass_sway_clk can be static kbuild test robot @ 2018-06-18 14:40 ` Dan Carpenter 2 siblings, 0 replies; 8+ messages in thread From: Dan Carpenter @ 2018-06-18 14:40 UTC (permalink / raw) To: kbuild, Taniya Das Cc: kbuild-all, Stephen Boyd, Michael Turquette, robh, Andy Gross, David Brown, Rajendra Nayak, Amit Nischal, linux-arm-msm, linux-soc, linux-clk, linux-kernel, devicetree, Rohit Kumar, Taniya Das Hi Taniya, Thank you for the patch! Perhaps something to improve: url: https://github.com/0day-ci/linux/commits/Taniya-Das/Add-support-for-LPASS-clock-controller-for-SDM845/20180614-155144 base: https://git.kernel.org/pub/scm/linux/kernel/git/clk/linux.git clk-next smatch warnings: drivers/clk/qcom/lpasscc-sdm845.c:197 lpass_clocks_sdm845_probe() warn: passing zero to 'PTR_ERR' # https://github.com/0day-ci/linux/commit/035d2520712de752be68d10a37650b65b6aee154 git remote add linux-review https://github.com/0day-ci/linux git remote update linux-review git checkout 035d2520712de752be68d10a37650b65b6aee154 vim +/PTR_ERR +197 drivers/clk/qcom/lpasscc-sdm845.c 035d2520 Taniya Das 2018-06-14 179 035d2520 Taniya Das 2018-06-14 180 static int lpass_clocks_sdm845_probe(struct platform_device *pdev, 035d2520 Taniya Das 2018-06-14 181 struct device_node *np, 035d2520 Taniya Das 2018-06-14 182 const struct qcom_cc_desc *desc) 035d2520 Taniya Das 2018-06-14 183 { 035d2520 Taniya Das 2018-06-14 184 struct regmap *regmap; 035d2520 Taniya Das 2018-06-14 185 struct resource res; 035d2520 Taniya Das 2018-06-14 186 void __iomem *base; 035d2520 Taniya Das 2018-06-14 187 035d2520 Taniya Das 2018-06-14 188 if (of_address_to_resource(np, 0, &res)) 035d2520 Taniya Das 2018-06-14 189 return -ENOMEM; 035d2520 Taniya Das 2018-06-14 190 035d2520 Taniya Das 2018-06-14 191 base = devm_ioremap(&pdev->dev, res.start, resource_size(&res)); 035d2520 Taniya Das 2018-06-14 192 if (IS_ERR(base)) 035d2520 Taniya Das 2018-06-14 193 return -ENOMEM; 035d2520 Taniya Das 2018-06-14 194 035d2520 Taniya Das 2018-06-14 195 regmap = devm_regmap_init_mmio(&pdev->dev, base, desc->config); 035d2520 Taniya Das 2018-06-14 196 if (!regmap) 035d2520 Taniya Das 2018-06-14 @197 return PTR_ERR(regmap); ^^^^^^^^^^^^^^^ 035d2520 Taniya Das 2018-06-14 198 035d2520 Taniya Das 2018-06-14 199 return qcom_cc_really_probe(pdev, desc, regmap); 035d2520 Taniya Das 2018-06-14 200 } 035d2520 Taniya Das 2018-06-14 201 --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation ^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH v4 0/2] Add support for LPASS clock controller for SDM845 @ 2018-09-11 17:00 Taniya Das 2018-09-11 17:00 ` [PATCH 2/2] clk: qcom: Add lpass clock controller driver " Taniya Das 0 siblings, 1 reply; 8+ messages in thread From: Taniya Das @ 2018-09-11 17:00 UTC (permalink / raw) To: Stephen Boyd, Michael Turquette Cc: Andy Gross, David Brown, Rajendra Nayak, linux-arm-msm, linux-soc, linux-clk, linux-kernel, devicetree, robh, Taniya Das [v4] * Update the description in GCC Documentation binding for 'qcom,lpass-protected'. * Remove 'qcom,lpass-protected' from LPASS Documentation binding. * Update KConfig to use Low Power Audio Subsystem. * Add module_exit() and also update return value for devm_ioremap_resource failure. [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 | 31 ++++ drivers/clk/qcom/Kconfig | 9 + drivers/clk/qcom/Makefile | 1 + drivers/clk/qcom/gcc-sdm845.c | 35 ++++ drivers/clk/qcom/lpasscc-sdm845.c | 195 +++++++++++++++++++++ include/dt-bindings/clock/qcom,gcc-sdm845.h | 2 + include/dt-bindings/clock/qcom,lpass-sdm845.h | 16 ++ 8 files changed, 291 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] 8+ messages in thread
* [PATCH 2/2] clk: qcom: Add lpass clock controller driver for SDM845 2018-09-11 17:00 [PATCH v4 0/2] Add support for LPASS clock controller " Taniya Das @ 2018-09-11 17:00 ` Taniya Das 0 siblings, 0 replies; 8+ messages in thread From: Taniya Das @ 2018-09-11 17:00 UTC (permalink / raw) To: Stephen Boyd, Michael Turquette Cc: Andy Gross, David Brown, Rajendra Nayak, linux-arm-msm, linux-soc, linux-clk, linux-kernel, devicetree, 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 | 195 ++++++++++++++++++++++++++++++++++++++ 4 files changed, 240 insertions(+) create mode 100644 drivers/clk/qcom/lpasscc-sdm845.c diff --git a/drivers/clk/qcom/Kconfig b/drivers/clk/qcom/Kconfig index 5b181b1..747ffb4 100644 --- a/drivers/clk/qcom/Kconfig +++ b/drivers/clk/qcom/Kconfig @@ -272,6 +272,15 @@ config SDM_DISPCC_845 Say Y if you want to support display devices and functionality such as splash screen. +config SDM_LPASSCC_845 + tristate "SDM845 Low Power Audio Subsystem (LPAAS) 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 935f142..53a5283 100644 --- a/drivers/clk/qcom/Makefile +++ b/drivers/clk/qcom/Makefile @@ -42,5 +42,6 @@ obj-$(CONFIG_QCOM_CLK_SMD_RPM) += clk-smd-rpm.o obj-$(CONFIG_SDM_CAMCC_845) += camcc-sdm845.o obj-$(CONFIG_SDM_DISPCC_845) += dispcc-sdm845.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 08d593e..fa8c37c 100644 --- a/drivers/clk/qcom/gcc-sdm845.c +++ b/drivers/clk/qcom/gcc-sdm845.c @@ -3169,6 +3169,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 = { @@ -3469,6 +3495,8 @@ enum { [GCC_QSPI_CORE_CLK_SRC] = &gcc_qspi_core_clk_src.clkr, [GCC_QSPI_CORE_CLK] = &gcc_qspi_core_clk.clkr, [GCC_QSPI_CNOC_PERIPH_AHB_CLK] = &gcc_qspi_cnoc_periph_ahb_clk.clkr, + [GCC_LPASS_Q6_AXI_CLK] = NULL, + [GCC_LPASS_SWAY_CLK] = NULL, }; static const struct qcom_reset_map gcc_sdm845_resets[] = { @@ -3583,6 +3611,13 @@ static int gcc_sdm845_probe(struct platform_device *pdev) if (ret) return ret; + 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..a8ddb49 --- /dev/null +++ b/drivers/clk/qcom/lpasscc-sdm845.c @@ -0,0 +1,195 @@ +// 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 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); + +static void __exit lpass_cc_sdm845_exit(void) +{ + platform_driver_unregister(&lpass_cc_sdm845_driver); +} +module_exit(lpass_cc_sdm845_exit); + +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] 8+ messages in thread
end of thread, other threads:[~2018-09-11 17:00 UTC | newest] Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2018-06-14 7:39 [PATCH 0/2] Add support for LPASS clock controller for SDM845 Taniya Das 2018-06-14 7:39 ` [PATCH 1/2] dt-bindings: clock: Introduce QCOM LPASS clock bindings Taniya Das 2018-06-26 19:53 ` Rob Herring 2018-06-14 7:39 ` [PATCH 2/2] clk: qcom: Add lpass clock controller driver for SDM845 Taniya Das 2018-06-14 10:38 ` kbuild test robot 2018-06-14 10:38 ` [RFC PATCH] clk: qcom: gcc_lpass_sway_clk can be static kbuild test robot 2018-06-18 14:40 ` [PATCH 2/2] clk: qcom: Add lpass clock controller driver for SDM845 Dan Carpenter 2018-09-11 17:00 [PATCH v4 0/2] Add support for LPASS clock controller " Taniya Das 2018-09-11 17:00 ` [PATCH 2/2] clk: qcom: Add lpass clock controller driver " 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).