| Message ID | 20200709145145.4055619-1-dianders@chromium.org (mailing list archive) |
|---|---|
| Headers | show |
| Series | spi: spi-qcom-qspi: Avoid some per-transfer overhead | expand |
On 7/9/2020 8:21 PM, Douglas Anderson wrote: > Not to be confused with the similar series I posed for the _other_ > Qualcomm SPI controller (spi-geni-qcom) [1], this one avoids the > overhead on the Quad SPI controller. > > It's based atop the current Qualcomm tree including Rajendra's ("spi: > spi-qcom-qspi: Use OPP API to set clk/perf state"). As discussed in > individual patches, these could ideally land through the Qualcomm tree > with Mark's Ack. > > Measuring: > * Before OPP / Interconnect patches reading all flash takes: ~3.4 seconds > * After OPP / Interconnect patches reading all flash takes: ~4.7 seconds > * After this patch reading all flash takes: ~3.3 seconds > > [1] https://lore.kernel.org/r/20200702004509.2333554-1-dianders@chromium.org > [2] https://lore.kernel.org/r/1593769293-6354-2-git-send-email-rnayak@codeaurora.org > > Changes in v2: > - Return error from runtime resume if dev_pm_opp_set_rate() fails. > > Douglas Anderson (2): > spi: spi-qcom-qspi: Avoid clock setting if not needed > spi: spi-qcom-qspi: Set an autosuspend delay of 250 ms > > drivers/spi/spi-qcom-qspi.c | 43 ++++++++++++++++++++++++++++--------- > 1 file changed, 33 insertions(+), 10 deletions(-) Reviewed-by: Akash Asthana <akashast@codeaurora.org>
Not to be confused with the similar series I posed for the _other_ Qualcomm SPI controller (spi-geni-qcom) [1], this one avoids the overhead on the Quad SPI controller. It's based atop the current Qualcomm tree including Rajendra's ("spi: spi-qcom-qspi: Use OPP API to set clk/perf state"). As discussed in individual patches, these could ideally land through the Qualcomm tree with Mark's Ack. Measuring: * Before OPP / Interconnect patches reading all flash takes: ~3.4 seconds * After OPP / Interconnect patches reading all flash takes: ~4.7 seconds * After this patch reading all flash takes: ~3.3 seconds [1] https://lore.kernel.org/r/20200702004509.2333554-1-dianders@chromium.org [2] https://lore.kernel.org/r/1593769293-6354-2-git-send-email-rnayak@codeaurora.org Changes in v2: - Return error from runtime resume if dev_pm_opp_set_rate() fails. Douglas Anderson (2): spi: spi-qcom-qspi: Avoid clock setting if not needed spi: spi-qcom-qspi: Set an autosuspend delay of 250 ms drivers/spi/spi-qcom-qspi.c | 43 ++++++++++++++++++++++++++++--------- 1 file changed, 33 insertions(+), 10 deletions(-)