[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <3453985c13aa51d068c0174e51e378fb@manjaro.org>
Date: Wed, 03 Sep 2025 10:24:55 +0200
From: Dragan Simic <dsimic@...jaro.org>
To: Sarthak Garg <quic_sartgarg@...cinc.com>
Cc: Ulf Hansson <ulf.hansson@...aro.org>, Rob Herring <robh@...nel.org>,
Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley
<conor+dt@...nel.org>, Bjorn Andersson <andersson@...nel.org>, Konrad Dybcio
<konradybcio@...nel.org>, Adrian Hunter <adrian.hunter@...el.com>,
linux-mmc@...r.kernel.org, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org, linux-arm-msm@...r.kernel.org,
quic_nguyenb@...cinc.com, quic_rampraka@...cinc.com,
quic_pragalla@...cinc.com, quic_sayalil@...cinc.com,
quic_nitirawa@...cinc.com, quic_bhaskarv@...cinc.com,
kernel@....qualcomm.com
Subject: Re: [PATCH V5 0/4] Add level shifter support for qualcomm SOC's
Hello Sarthak and Krzysztof,
On 2025-09-03 10:04, Sarthak Garg wrote:
> Add level shifter support for qualcomm SOC's.
>
> - Changed from v4
> - As suggested by Krzysztof Kozlowski Renamed the property from
> `max-sd-hs-frequency` to `max-sd-hs-hz` for clarity.
I just spotted this series, so I'm sorry for jumping in late. To me,
"max-sd-hs-frequency" was actually a better choice, because it goes
together with the already existing "max-frequency" property.
Yes, "max-sd-hs-hz" is obviously more clear, but to me, consistency
is more important. Just my $.02.
> - As suggested by Krzysztof Kozlowski remove min/max constraints
> and add default: 50000000 in dt-bindings.
> - As suggested by Konrad Dybcio moved max-sd-hs-hz property in the
> SoC dtsi.
> - Retained sdhci-caps-mask in sm8550.dtsi for now and will revisit
> its removal for future targets after thorough validation and
> testing
> from the beginning.
>
> - Changed from v3
> - As suggested by Krzysztof Kozlowski moved the property from the
> SoC-level DTS to the board-level DTS.
> - Revised the commit messages to clearly explain its
> board-specific.
>
> - Changed from v2
> - As suggested by Konrad Dybcio and Ulf Hansson redesigned logic
> to introduce a new DT property max-sd-hs-frequency and override
> the hs_max_dtr accordingly in sd.c file.
>
> - Changed from v1
> - As suggested by Krzysztof Kozlowski redesigned logic to use
> compatible property for adding this level shifter support.
> - Addressed Adrian Hunter comments on V1 with resepect to
> checkpatch.
> - Cleared the bits first and then set bits in
> sdhci_msm_execute_tuning as suggested by Adrian Hunter.
> - Upated the if condition logic in msm_set_clock_rate_for_bus_mode
> as suggested by Adrian Hunter.
>
> Sarthak Garg (4):
> mmc: sdhci-msm: Enable tuning for SDR50 mode for SD card
> dt-bindings: mmc: controller: Add max-sd-hs-frequency property
> mmc: core: Introduce a new flag max-sd-hs-hz
> arm64: dts: qcom: sm8550: Add max-sd-hs-hz property
>
> .../bindings/mmc/mmc-controller-common.yaml | 8 ++++++++
> arch/arm64/boot/dts/qcom/sm8550.dtsi | 1 +
> drivers/mmc/core/host.c | 2 ++
> drivers/mmc/core/sd.c | 2 +-
> drivers/mmc/host/sdhci-msm.c | 15 +++++++++++++++
> include/linux/mmc/host.h | 1 +
> 6 files changed, 28 insertions(+), 1 deletion(-)
Powered by blists - more mailing lists