From: Konrad Dybcio <konrad.dybcio@linaro.org> To: Bartosz Golaszewski <brgl@bgdev.pl>, Andy Gross <agross@kernel.org>, Bjorn Andersson <andersson@kernel.org>, Rob Herring <robh+dt@kernel.org>, Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>, Dmitry Baryshkov <dmitry.baryshkov@linaro.org> Cc: linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-clk@vger.kernel.org, linux-arm-kernel@lists.infradead.org, iommu@lists.linux.dev, Shazad Hussain <quic_shazhuss@quicinc.com>, Stephen Boyd <sboyd@kernel.org>, Michael Turquette <mturquette@baylibre.com>, Bartosz Golaszewski <bartosz.golaszewski@linaro.org> Subject: Re: [PATCH 2/7] clk: qcom: add the GPUCC driver for sa8775p Date: Wed, 29 Mar 2023 13:30:49 +0200 [thread overview] Message-ID: <95b4bd56-b307-6c72-494f-e9b190df8d74@linaro.org> (raw) In-Reply-To: <20230328193632.226095-3-brgl@bgdev.pl> On 28.03.2023 21:36, Bartosz Golaszewski wrote: > From: Shazad Hussain <quic_shazhuss@quicinc.com> > > Add the clock driver for the Qualcomm Graphics Clock control module. > > Cc: Stephen Boyd <sboyd@kernel.org> > Cc: Michael Turquette <mturquette@baylibre.com> > Signed-off-by: Shazad Hussain <quic_shazhuss@quicinc.com> > [Bartosz: make ready for upstream] > Co-authored-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org> > Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org> > --- [...] > +/* Need to match the order of clocks in DT binding */ > +enum { > + DT_IFACE, I think it's never used? [...] > +static int gpu_cc_sa8775p_probe(struct platform_device *pdev) > +{ > + struct regmap *regmap; > + > + regmap = qcom_cc_map(pdev, &gpu_cc_sa8775p_desc); > + if (IS_ERR(regmap)) > + return PTR_ERR(regmap); > + > + clk_lucid_evo_pll_configure(&gpu_cc_pll0, regmap, &gpu_cc_pll0_config); > + clk_lucid_evo_pll_configure(&gpu_cc_pll1, regmap, &gpu_cc_pll1_config); > + > + /* > + * Keep the clocks always-ON > + * GPU_CC_CB_CLK > + */ > + regmap_update_bits(regmap, 0x93a4, BIT(0), BIT(0)); You set it as CRITICAL, this should be unnecessary now. Konrad > + > + return qcom_cc_really_probe(pdev, &gpu_cc_sa8775p_desc, regmap); > +} > + > +static struct platform_driver gpu_cc_sa8775p_driver = { > + .probe = gpu_cc_sa8775p_probe, > + .driver = { > + .name = "gpu_cc-sa8775p", > + .of_match_table = gpu_cc_sa8775p_match_table, > + }, > +}; > + > +static int __init gpu_cc_sa8775p_init(void) > +{ > + return platform_driver_register(&gpu_cc_sa8775p_driver); > +} > +subsys_initcall(gpu_cc_sa8775p_init); > + > +static void __exit gpu_cc_sa8775p_exit(void) > +{ > + platform_driver_unregister(&gpu_cc_sa8775p_driver); > +} > +module_exit(gpu_cc_sa8775p_exit); > + > +MODULE_DESCRIPTION("SA8775P GPUCC driver"); > +MODULE_LICENSE("GPL");
WARNING: multiple messages have this Message-ID (diff)
From: Konrad Dybcio <konrad.dybcio@linaro.org> To: Bartosz Golaszewski <brgl@bgdev.pl>, Andy Gross <agross@kernel.org>, Bjorn Andersson <andersson@kernel.org>, Rob Herring <robh+dt@kernel.org>, Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>, Dmitry Baryshkov <dmitry.baryshkov@linaro.org> Cc: linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-clk@vger.kernel.org, linux-arm-kernel@lists.infradead.org, iommu@lists.linux.dev, Shazad Hussain <quic_shazhuss@quicinc.com>, Stephen Boyd <sboyd@kernel.org>, Michael Turquette <mturquette@baylibre.com>, Bartosz Golaszewski <bartosz.golaszewski@linaro.org> Subject: Re: [PATCH 2/7] clk: qcom: add the GPUCC driver for sa8775p Date: Wed, 29 Mar 2023 13:30:49 +0200 [thread overview] Message-ID: <95b4bd56-b307-6c72-494f-e9b190df8d74@linaro.org> (raw) In-Reply-To: <20230328193632.226095-3-brgl@bgdev.pl> On 28.03.2023 21:36, Bartosz Golaszewski wrote: > From: Shazad Hussain <quic_shazhuss@quicinc.com> > > Add the clock driver for the Qualcomm Graphics Clock control module. > > Cc: Stephen Boyd <sboyd@kernel.org> > Cc: Michael Turquette <mturquette@baylibre.com> > Signed-off-by: Shazad Hussain <quic_shazhuss@quicinc.com> > [Bartosz: make ready for upstream] > Co-authored-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org> > Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org> > --- [...] > +/* Need to match the order of clocks in DT binding */ > +enum { > + DT_IFACE, I think it's never used? [...] > +static int gpu_cc_sa8775p_probe(struct platform_device *pdev) > +{ > + struct regmap *regmap; > + > + regmap = qcom_cc_map(pdev, &gpu_cc_sa8775p_desc); > + if (IS_ERR(regmap)) > + return PTR_ERR(regmap); > + > + clk_lucid_evo_pll_configure(&gpu_cc_pll0, regmap, &gpu_cc_pll0_config); > + clk_lucid_evo_pll_configure(&gpu_cc_pll1, regmap, &gpu_cc_pll1_config); > + > + /* > + * Keep the clocks always-ON > + * GPU_CC_CB_CLK > + */ > + regmap_update_bits(regmap, 0x93a4, BIT(0), BIT(0)); You set it as CRITICAL, this should be unnecessary now. Konrad > + > + return qcom_cc_really_probe(pdev, &gpu_cc_sa8775p_desc, regmap); > +} > + > +static struct platform_driver gpu_cc_sa8775p_driver = { > + .probe = gpu_cc_sa8775p_probe, > + .driver = { > + .name = "gpu_cc-sa8775p", > + .of_match_table = gpu_cc_sa8775p_match_table, > + }, > +}; > + > +static int __init gpu_cc_sa8775p_init(void) > +{ > + return platform_driver_register(&gpu_cc_sa8775p_driver); > +} > +subsys_initcall(gpu_cc_sa8775p_init); > + > +static void __exit gpu_cc_sa8775p_exit(void) > +{ > + platform_driver_unregister(&gpu_cc_sa8775p_driver); > +} > +module_exit(gpu_cc_sa8775p_exit); > + > +MODULE_DESCRIPTION("SA8775P GPUCC driver"); > +MODULE_LICENSE("GPL"); _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2023-03-29 11:30 UTC|newest] Thread overview: 40+ messages / expand[flat|nested] mbox.gz Atom feed top 2023-03-28 19:36 [PATCH 0/7] arm64: dts: qcom: sa8775p: add more IOMMUs Bartosz Golaszewski 2023-03-28 19:36 ` Bartosz Golaszewski 2023-03-28 19:36 ` [PATCH 1/7] dt-bindings: clock: qcom: describe the GPUCC clock for SA8775P Bartosz Golaszewski 2023-03-28 19:36 ` Bartosz Golaszewski 2023-03-29 8:30 ` Krzysztof Kozlowski 2023-03-29 8:30 ` Krzysztof Kozlowski 2023-03-28 19:36 ` [PATCH 2/7] clk: qcom: add the GPUCC driver for sa8775p Bartosz Golaszewski 2023-03-28 19:36 ` Bartosz Golaszewski 2023-03-29 2:15 ` Stephen Boyd 2023-03-29 2:15 ` Stephen Boyd 2023-04-06 11:20 ` Bartosz Golaszewski 2023-04-06 11:20 ` Bartosz Golaszewski 2023-04-10 19:09 ` Stephen Boyd 2023-04-10 19:09 ` Stephen Boyd 2023-03-29 11:30 ` Konrad Dybcio [this message] 2023-03-29 11:30 ` Konrad Dybcio 2023-03-28 19:36 ` [PATCH 3/7] arm64: defconfig: enable the SA8775P GPUCC driver Bartosz Golaszewski 2023-03-28 19:36 ` Bartosz Golaszewski 2023-03-29 8:33 ` Krzysztof Kozlowski 2023-03-29 8:33 ` Krzysztof Kozlowski 2023-04-06 13:28 ` Dmitry Baryshkov 2023-04-06 13:28 ` Dmitry Baryshkov 2023-03-28 19:36 ` [PATCH 4/7] dt-bindings: iommu: arm,smmu: enable clocks for sa8775p Bartosz Golaszewski 2023-03-28 19:36 ` Bartosz Golaszewski 2023-03-29 8:31 ` Krzysztof Kozlowski 2023-03-29 8:31 ` Krzysztof Kozlowski 2023-04-03 20:41 ` Rob Herring 2023-04-03 20:41 ` Rob Herring 2023-04-03 22:38 ` Konrad Dybcio 2023-04-03 22:38 ` Konrad Dybcio 2023-03-28 19:36 ` [PATCH 5/7] arm64: dts: qcom: sa8775p: add the pcie smmu node Bartosz Golaszewski 2023-03-28 19:36 ` Bartosz Golaszewski 2023-03-29 11:33 ` Konrad Dybcio 2023-03-29 11:33 ` Konrad Dybcio 2023-03-28 19:36 ` [PATCH 6/7] arm64: dts: qcom: sa8775p: add the GPU clock controller node Bartosz Golaszewski 2023-03-28 19:36 ` Bartosz Golaszewski 2023-03-29 11:31 ` Konrad Dybcio 2023-03-29 11:31 ` Konrad Dybcio 2023-03-28 19:36 ` [PATCH 7/7] arm64: dts: qcom: sa8775p: add the GPU IOMMU node Bartosz Golaszewski 2023-03-28 19:36 ` Bartosz Golaszewski
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=95b4bd56-b307-6c72-494f-e9b190df8d74@linaro.org \ --to=konrad.dybcio@linaro.org \ --cc=agross@kernel.org \ --cc=andersson@kernel.org \ --cc=bartosz.golaszewski@linaro.org \ --cc=brgl@bgdev.pl \ --cc=devicetree@vger.kernel.org \ --cc=dmitry.baryshkov@linaro.org \ --cc=iommu@lists.linux.dev \ --cc=krzysztof.kozlowski+dt@linaro.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-arm-msm@vger.kernel.org \ --cc=linux-clk@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=mturquette@baylibre.com \ --cc=quic_shazhuss@quicinc.com \ --cc=robh+dt@kernel.org \ --cc=sboyd@kernel.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.