linux-arm-msm.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Georgi Djakov <georgi.djakov@linaro.org>
To: Pradeep P V K <ppvk@codeaurora.org>,
	adrian.hunter@intel.com, ulf.hansson@linaro.org,
	robh+dt@kernel.org
Cc: asutoshd@codeaurora.org, vbadigan@codeaurora.org,
	stummala@codeaurora.org, sayalil@codeaurora.org,
	rampraka@codeaurora.org, linux-mmc@vger.kernel.org,
	linux-kernel@vger.kernel.org, linux-arm-msm@vger.kernel.org,
	devicetree@vger.kernel.org,
	Subhash Jadavani <subhashj@codeaurora.org>,
	Andy Gross <agross@kernel.org>
Subject: Re: [RFC 1/2] mmc: sdhci-msm: Add support for bus bandwidth voting
Date: Thu, 12 Sep 2019 15:56:08 +0300	[thread overview]
Message-ID: <616c7a8c-a1cf-2043-4ea4-f452ee90f083@linaro.org> (raw)
In-Reply-To: <1567774037-2344-2-git-send-email-ppvk@codeaurora.org>

Hi Pradeep,

Thanks for the patch!

On 9/6/19 15:47, Pradeep P V K wrote:
> Vote for the MSM bus bandwidth required by SDHC driver
> based on the clock frequency and bus width of the card.
> Otherwise,the system clocks may run at minimum clock speed
> and thus affecting the performance.
> 
> This change is based on Georgi Djakov [RFC]
> (https://lkml.org/lkml/2018/10/11/499)

I am just wondering whether do we really need to predefine the bandwidth values
in DT? Can't we use the computations from the above patch or is there any
problem with that approach?

Thanks,
Georgi

  parent reply	other threads:[~2019-09-12 12:56 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-09-06 12:47 [RFC 0/2] Add Support for SDHC bus bandwidth voting Pradeep P V K
2019-09-06 12:47 ` [RFC 1/2] mmc: sdhci-msm: Add support for " Pradeep P V K
2019-09-06 12:51   ` ppvk
2019-09-06 21:02     ` Stephen Boyd
2019-09-12 15:01       ` ppvk
2019-09-12 12:56   ` Georgi Djakov [this message]
2019-09-25  6:24     ` ppvk
2019-09-12 16:45   ` Bjorn Andersson
2019-09-25 11:27     ` ppvk
2019-10-03 10:00       ` Ulf Hansson
2019-09-06 12:47 ` [RFC 2/2] dt-bindings: mmc: sdhci-msm: Add Bus BW vote supported strings Pradeep P V K
2019-09-13 14:36   ` Rob Herring
2019-09-30 14:32     ` ppvk

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=616c7a8c-a1cf-2043-4ea4-f452ee90f083@linaro.org \
    --to=georgi.djakov@linaro.org \
    --cc=adrian.hunter@intel.com \
    --cc=agross@kernel.org \
    --cc=asutoshd@codeaurora.org \
    --cc=devicetree@vger.kernel.org \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mmc@vger.kernel.org \
    --cc=ppvk@codeaurora.org \
    --cc=rampraka@codeaurora.org \
    --cc=robh+dt@kernel.org \
    --cc=sayalil@codeaurora.org \
    --cc=stummala@codeaurora.org \
    --cc=subhashj@codeaurora.org \
    --cc=ulf.hansson@linaro.org \
    --cc=vbadigan@codeaurora.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;
as well as URLs for NNTP newsgroup(s).