From: Jeremy McNicoll <jeremymc@redhat.com>
To: linux-arm-msm@vger.kernel.org, linux-soc@vger.kernel.org,
devicetree@vger.kernel.org, linux-mmc@vger.kernel.org
Cc: andy.gross@linaro.org, sboyd@codeaurora.org, robh@kernel.org,
arnd@arndb.de, bjorn.andersson@linaro.org,
riteshh@codeaurora.org, git@kchr.de, ulf.hansson@linaro.org,
jszhang@marvell.com, jeremymc@redhat.com
Subject: [PATCH V2 0/4] Enable onboard SDHCI for Nexus 5X (msm8992)
Date: Mon, 16 Jan 2017 16:58:45 -0800 [thread overview]
Message-ID: <1484614729-26751-1-git-send-email-jeremymc@redhat.com> (raw)
V2->V1
------
* Removed SMD macro to define packet size
* Dropped "sdhci: dump vendor state and regs" as Ritesh has agreed to deal
with it as part of [https://lkml.org/lkml/2016/12/30/102] .
* Didn't include RobH's ACK as I stripped out the smd_rpm node from
msm8992-bullhead-rev-101.dts and created msm8994-smd-rpm.dtsi to share
between 8992 & 8994. All feedback from RobH was applied prior to making
this change.
All testing was performed with the inclusion of:
RESEND v1 -> http://www.spinics.net/lists/linux-arm-msm/msg25484.html
RESEND RFC -> http://www.spinics.net/lists/linux-arm-msm/msg25481.html
In V1 it was stated that
"Without the workaround (patch 5/5) mmc/sdhci didn't get detected 8/20
times. When including the afore mentioned workaround MMC detection is
100% (35 boots) ."
It turns out we weren't waiting long enough for detection. (see statement
below)
The following test results were obtained upon removing
the SDHCI workaround (patch 2/4):
-50 iterations
-100% successfully detected onboard storage
-2 times SDHCI detection took over 4+ min
-Sample size 1 (I only have 1 Nexus 5X)
General observations regarding workaround removal:
-detection of onboard storage is generally slower without
the workaround. No hard numbers to back this up just
a general observation. It may be because the author has
consumed far more caffeinated beverages compared to the
previous time V1.
-the failures observed in V1 were because my test script
did not wait long enough to declare failure. Previously
I was waiting 4 min to declare the SDHCI as missing.
This round I increased the duration to 10min.
NOTE to SDHCI maintainers:
"sdhci: Add quirk for delayed IRQ ACK" (patch 2/4) was
included, if there is no interest in including this change
feel free to drop it or provide guidance on how this type
of issue is generally dealt with.
Jeremy McNicoll (4):
clk: gcc: Updates for SDHCI enablement
sdhci: Add quirk for delayed IRQ ACK
arm64: dts: Enable SDHCI for Nexus 5X (msm8992)
dts: doc: rename rpm_requests to respect DT naming conventions
.../devicetree/bindings/clock/qcom,rpmcc.txt | 2 +-
.../bindings/regulator/qcom,smd-rpm-regulator.txt | 42 +++-
.../devicetree/bindings/soc/qcom/qcom,smd-rpm.txt | 6 +-
.../devicetree/bindings/soc/qcom/qcom,smd.txt | 2 +-
arch/arm/boot/dts/qcom-apq8074-dragonboard.dts | 2 +-
arch/arm/boot/dts/qcom-apq8084.dtsi | 2 +-
.../dts/qcom-msm8974-lge-nexus5-hammerhead.dts | 2 +-
.../boot/dts/qcom-msm8974-sony-xperia-honami.dts | 2 +-
arch/arm/boot/dts/qcom-msm8974.dtsi | 2 +-
arch/arm64/boot/dts/qcom/msm8916.dtsi | 2 +-
.../boot/dts/qcom/msm8992-bullhead-rev-101.dts | 2 +
arch/arm64/boot/dts/qcom/msm8992-pins.dtsi | 82 ++++++
arch/arm64/boot/dts/qcom/msm8992.dtsi | 153 ++++++++++++
arch/arm64/boot/dts/qcom/msm8994-smd-rpm.dtsi | 276 +++++++++++++++++++++
drivers/clk/qcom/gcc-msm8994.c | 108 ++++++--
drivers/mmc/host/sdhci-msm.c | 7 +
drivers/mmc/host/sdhci.c | 12 +-
drivers/mmc/host/sdhci.h | 2 +
drivers/regulator/qcom_smd-regulator.c | 49 ++++
include/dt-bindings/clock/qcom,gcc-msm8994.h | 32 ++-
20 files changed, 739 insertions(+), 48 deletions(-)
create mode 100644 arch/arm64/boot/dts/qcom/msm8994-smd-rpm.dtsi
--
2.6.1
next reply other threads:[~2017-01-17 0:58 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-01-17 0:58 Jeremy McNicoll [this message]
[not found] ` <1484614729-26751-1-git-send-email-jeremymc-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2017-01-17 0:58 ` [PATCH V2 1/4] clk: gcc: Updates for SDHCI enablement Jeremy McNicoll
2017-01-17 23:25 ` Stephen Boyd
[not found] ` <20170117232542.GV17126-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>
2017-01-18 20:14 ` Jeremy McNicoll
2017-01-17 0:58 ` [PATCH V2 2/4] sdhci: Add quirk for delayed IRQ ACK Jeremy McNicoll
2017-01-17 0:58 ` [PATCH V2 3/4] arm64: dts: Enable SDHCI for Nexus 5X (msm8992) Jeremy McNicoll
[not found] ` <1484614729-26751-4-git-send-email-jeremymc-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2017-01-18 19:02 ` Bjorn Andersson
2017-01-20 0:20 ` Jeremy McNicoll
2017-01-19 17:21 ` Rob Herring
2017-01-17 0:58 ` [PATCH V2 4/4] dts: doc: rename rpm_requests to respect DT naming conventions Jeremy McNicoll
2017-01-19 17:22 ` Rob Herring
2017-01-23 10:39 ` Jeremy McNicoll
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=1484614729-26751-1-git-send-email-jeremymc@redhat.com \
--to=jeremymc@redhat.com \
--cc=andy.gross@linaro.org \
--cc=arnd@arndb.de \
--cc=bjorn.andersson@linaro.org \
--cc=devicetree@vger.kernel.org \
--cc=git@kchr.de \
--cc=jszhang@marvell.com \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-mmc@vger.kernel.org \
--cc=linux-soc@vger.kernel.org \
--cc=riteshh@codeaurora.org \
--cc=robh@kernel.org \
--cc=sboyd@codeaurora.org \
--cc=ulf.hansson@linaro.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).