From: Georgi Djakov <djakov@kernel.org>
To: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
Andy Gross <agross@kernel.org>,
Bjorn Andersson <bjorn.andersson@linaro.org>,
linux-arm-msm@vger.kernel.org, linux-pm@vger.kernel.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH] interconnect: qcom: use icc_sync_state
Date: Wed, 27 Apr 2022 18:34:53 +0300 [thread overview]
Message-ID: <4769c796-6edd-c23a-ee2a-ce54495548f7@kernel.org> (raw)
In-Reply-To: <20220427145616.523557-1-krzysztof.kozlowski@linaro.org>
On 27.04.22 17:56, Krzysztof Kozlowski wrote:
> Use icc_sync_state for interconnect providers, so that the bandwidth
> request doesn't need to stay on maximum value.
Did you test this? In general, we should not enable this on boards that
do not have full interconnect scaling support in consumer drivers yet.
Some of the interconnects could be enabled by default by the bootloader
and usually later during boot the consumer drivers request the bandwidth
that they need. But if the requests are missing, the interconnects
without bandwidth users will be disabled when we reach sync state. So
this may (or not) cause issues...
Thanks,
Georgi
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
> drivers/interconnect/qcom/msm8916.c | 1 +
> drivers/interconnect/qcom/qcm2290.c | 1 +
> drivers/interconnect/qcom/qcs404.c | 1 +
> drivers/interconnect/qcom/sdm660.c | 1 +
> drivers/interconnect/qcom/sm8150.c | 1 +
> drivers/interconnect/qcom/sm8250.c | 1 +
> drivers/interconnect/qcom/sm8350.c | 1 +
> drivers/interconnect/qcom/sm8450.c | 1 +
> 8 files changed, 8 insertions(+)
>
> diff --git a/drivers/interconnect/qcom/msm8916.c b/drivers/interconnect/qcom/msm8916.c
> index 2f397a7c3322..811370fcd211 100644
> --- a/drivers/interconnect/qcom/msm8916.c
> +++ b/drivers/interconnect/qcom/msm8916.c
> @@ -1347,6 +1347,7 @@ static struct platform_driver msm8916_noc_driver = {
> .driver = {
> .name = "qnoc-msm8916",
> .of_match_table = msm8916_noc_of_match,
> + .sync_state = icc_sync_state,
> },
> };
> module_platform_driver(msm8916_noc_driver);
> diff --git a/drivers/interconnect/qcom/qcm2290.c b/drivers/interconnect/qcom/qcm2290.c
> index 74404e0b2080..6cf75da91428 100644
> --- a/drivers/interconnect/qcom/qcm2290.c
> +++ b/drivers/interconnect/qcom/qcm2290.c
> @@ -1355,6 +1355,7 @@ static struct platform_driver qcm2290_noc_driver = {
> .driver = {
> .name = "qnoc-qcm2290",
> .of_match_table = qcm2290_noc_of_match,
> + .sync_state = icc_sync_state,
> },
> };
> module_platform_driver(qcm2290_noc_driver);
> diff --git a/drivers/interconnect/qcom/qcs404.c b/drivers/interconnect/qcom/qcs404.c
> index 416c8bff8efa..d82f9add4933 100644
> --- a/drivers/interconnect/qcom/qcs404.c
> +++ b/drivers/interconnect/qcom/qcs404.c
> @@ -1086,6 +1086,7 @@ static struct platform_driver qcs404_noc_driver = {
> .driver = {
> .name = "qnoc-qcs404",
> .of_match_table = qcs404_noc_of_match,
> + .sync_state = icc_sync_state,
> },
> };
> module_platform_driver(qcs404_noc_driver);
> diff --git a/drivers/interconnect/qcom/sdm660.c b/drivers/interconnect/qcom/sdm660.c
> index 274a7139fe1a..706b49a4bb70 100644
> --- a/drivers/interconnect/qcom/sdm660.c
> +++ b/drivers/interconnect/qcom/sdm660.c
> @@ -1716,6 +1716,7 @@ static struct platform_driver sdm660_noc_driver = {
> .driver = {
> .name = "qnoc-sdm660",
> .of_match_table = sdm660_noc_of_match,
> + .sync_state = icc_sync_state,
> },
> };
> module_platform_driver(sdm660_noc_driver);
> diff --git a/drivers/interconnect/qcom/sm8150.c b/drivers/interconnect/qcom/sm8150.c
> index 745e3c36a61a..2a85f53802b5 100644
> --- a/drivers/interconnect/qcom/sm8150.c
> +++ b/drivers/interconnect/qcom/sm8150.c
> @@ -535,6 +535,7 @@ static struct platform_driver qnoc_driver = {
> .driver = {
> .name = "qnoc-sm8150",
> .of_match_table = qnoc_of_match,
> + .sync_state = icc_sync_state,
> },
> };
> module_platform_driver(qnoc_driver);
> diff --git a/drivers/interconnect/qcom/sm8250.c b/drivers/interconnect/qcom/sm8250.c
> index aa707582ea01..8dfb5dea562a 100644
> --- a/drivers/interconnect/qcom/sm8250.c
> +++ b/drivers/interconnect/qcom/sm8250.c
> @@ -551,6 +551,7 @@ static struct platform_driver qnoc_driver = {
> .driver = {
> .name = "qnoc-sm8250",
> .of_match_table = qnoc_of_match,
> + .sync_state = icc_sync_state,
> },
> };
> module_platform_driver(qnoc_driver);
> diff --git a/drivers/interconnect/qcom/sm8350.c b/drivers/interconnect/qcom/sm8350.c
> index c79f93a1ac73..3e26a2175b28 100644
> --- a/drivers/interconnect/qcom/sm8350.c
> +++ b/drivers/interconnect/qcom/sm8350.c
> @@ -531,6 +531,7 @@ static struct platform_driver qnoc_driver = {
> .driver = {
> .name = "qnoc-sm8350",
> .of_match_table = qnoc_of_match,
> + .sync_state = icc_sync_state,
> },
> };
> module_platform_driver(qnoc_driver);
> diff --git a/drivers/interconnect/qcom/sm8450.c b/drivers/interconnect/qcom/sm8450.c
> index 8d99ee6421df..d573018a6324 100644
> --- a/drivers/interconnect/qcom/sm8450.c
> +++ b/drivers/interconnect/qcom/sm8450.c
> @@ -1968,6 +1968,7 @@ static struct platform_driver qnoc_driver = {
> .driver = {
> .name = "qnoc-sm8450",
> .of_match_table = qnoc_of_match,
> + .sync_state = icc_sync_state,
> },
> };
>
next prev parent reply other threads:[~2022-04-27 15:35 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-04-27 14:56 [PATCH] interconnect: qcom: use icc_sync_state Krzysztof Kozlowski
2022-04-27 15:34 ` Georgi Djakov [this message]
2022-04-28 10:23 ` Krzysztof Kozlowski
2022-04-28 11:20 ` Georgi Djakov
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=4769c796-6edd-c23a-ee2a-ce54495548f7@kernel.org \
--to=djakov@kernel.org \
--cc=agross@kernel.org \
--cc=bjorn.andersson@linaro.org \
--cc=krzysztof.kozlowski@linaro.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pm@vger.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: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox