| Message ID | 20241226-x1e80100-qcp-sdhc-v5-0-0b28f2e13c85@linaro.org (mailing list archive) |
|---|---|
| Headers | show |
| Series | arm64: dts: qcom: x1e80100: Describe SDCs and enable support on QCP | expand |
On Thu, Dec 26, 2024 at 01:47:37PM +0200, Abel Vesa wrote: > The X1E80100 has two SDHC controllers (called SDC2 and SDC4). > Describe both of them and enable the SDC2 on QCP. This brings > SD card support for the microSD port on QCP. > > The SDC4 is described but there is no device outthere yet that makes > use of it, AFAIK. > > Signed-off-by: Abel Vesa <abel.vesa@linaro.org> Seems we hit a race condition, I fixed up the icc tags while applying v4, but I didn't adjust the status. Could you please double check what I did pick up and send an incremental patch with the status update? Thanks, Bjorn > --- > Changes in v5: > - Switched the interconnect paths tags to QCOM_ICC_TAG_ALWAYS and > QCOM_ICC_TAG_ACTIVE_ONLY, as Konrad suggested. > - Actually enabled the sdhc on QCP (status = "okay" was missing). > - Rebased to fix conflicts due to smb2360 nodes which were already > merged. > - Link to v4: https://lore.kernel.org/r/20241212-x1e80100-qcp-sdhc-v4-0-a74c48ee68a3@linaro.org > > Changes in v4: > - Squashed the pinconf for SDC2 into the patch that describes the > controllers. > - Reworded the commit messages a bit. > - Link to v3: https://lore.kernel.org/r/20241022-x1e80100-qcp-sdhc-v3-0-46c401e32cbf@linaro.org > > Changes in v3: > - Reordered the default and sleep pinconfs. Also the bias and > drive-strength properties. As per Konrad's suggestion. > - Link to v2: https://lore.kernel.org/r/20241014-x1e80100-qcp-sdhc-v2-0-868e70a825e0@linaro.org > > Changes in v2: > - rebased on next-20241011 > - dropped the bindings schema update patch > - dropped the sdhci-caps-mask properties from both > controllers as SDR104/SDR50 are actually supported > - Link to v1: https://lore.kernel.org/r/20241008-x1e80100-qcp-sdhc-v1-0-dfef4c92ae31@linaro.org > > --- > Abel Vesa (2): > arm64: dts: qcom: x1e80100: Describe the SDHC controllers > arm64: dts: qcom: x1e80100-qcp: Enable SD card support > > arch/arm64/boot/dts/qcom/x1e80100-qcp.dts | 21 +++++ > arch/arm64/boot/dts/qcom/x1e80100.dtsi | 146 ++++++++++++++++++++++++++++++ > 2 files changed, 167 insertions(+) > --- > base-commit: 8155b4ef3466f0e289e8fcc9e6e62f3f4dceeac2 > change-id: 20241007-x1e80100-qcp-sdhc-15c716dad946 > > Best regards, > -- > Abel Vesa <abel.vesa@linaro.org> >
On Thu, 26 Dec 2024 13:47:37 +0200, Abel Vesa wrote: > The X1E80100 has two SDHC controllers (called SDC2 and SDC4). > Describe both of them and enable the SDC2 on QCP. This brings > SD card support for the microSD port on QCP. > > The SDC4 is described but there is no device outthere yet that makes > use of it, AFAIK. > > [...] Applied, thanks! [1/2] arm64: dts: qcom: x1e80100: Describe the SDHC controllers commit: ffb21c1e19b17f3b2f5f56c70e379ef7c96afad5 [2/2] arm64: dts: qcom: x1e80100-qcp: Enable SD card support commit: ab8f487d2f8905641541c27c7929363ee538b0f8 Best regards,
The X1E80100 has two SDHC controllers (called SDC2 and SDC4). Describe both of them and enable the SDC2 on QCP. This brings SD card support for the microSD port on QCP. The SDC4 is described but there is no device outthere yet that makes use of it, AFAIK. Signed-off-by: Abel Vesa <abel.vesa@linaro.org> --- Changes in v5: - Switched the interconnect paths tags to QCOM_ICC_TAG_ALWAYS and QCOM_ICC_TAG_ACTIVE_ONLY, as Konrad suggested. - Actually enabled the sdhc on QCP (status = "okay" was missing). - Rebased to fix conflicts due to smb2360 nodes which were already merged. - Link to v4: https://lore.kernel.org/r/20241212-x1e80100-qcp-sdhc-v4-0-a74c48ee68a3@linaro.org Changes in v4: - Squashed the pinconf for SDC2 into the patch that describes the controllers. - Reworded the commit messages a bit. - Link to v3: https://lore.kernel.org/r/20241022-x1e80100-qcp-sdhc-v3-0-46c401e32cbf@linaro.org Changes in v3: - Reordered the default and sleep pinconfs. Also the bias and drive-strength properties. As per Konrad's suggestion. - Link to v2: https://lore.kernel.org/r/20241014-x1e80100-qcp-sdhc-v2-0-868e70a825e0@linaro.org Changes in v2: - rebased on next-20241011 - dropped the bindings schema update patch - dropped the sdhci-caps-mask properties from both controllers as SDR104/SDR50 are actually supported - Link to v1: https://lore.kernel.org/r/20241008-x1e80100-qcp-sdhc-v1-0-dfef4c92ae31@linaro.org --- Abel Vesa (2): arm64: dts: qcom: x1e80100: Describe the SDHC controllers arm64: dts: qcom: x1e80100-qcp: Enable SD card support arch/arm64/boot/dts/qcom/x1e80100-qcp.dts | 21 +++++ arch/arm64/boot/dts/qcom/x1e80100.dtsi | 146 ++++++++++++++++++++++++++++++ 2 files changed, 167 insertions(+) --- base-commit: 8155b4ef3466f0e289e8fcc9e6e62f3f4dceeac2 change-id: 20241007-x1e80100-qcp-sdhc-15c716dad946 Best regards,