Skip to content

FROMLIST: arm64: dts: qcom: monaco: Enable SDHCI storage support#338

Open
mchunara007 wants to merge 3 commits intoqualcomm-linux:qcom-6.18.yfrom
mchunara007:monaco_sdcc
Open

FROMLIST: arm64: dts: qcom: monaco: Enable SDHCI storage support#338
mchunara007 wants to merge 3 commits intoqualcomm-linux:qcom-6.18.yfrom
mchunara007:monaco_sdcc

Conversation

@mchunara007
Copy link

This series enables SDHCI storage support for both SD Card and eMMC on the
Qualcomm Monaco EVK platform.

The Monaco SoC shares the SDHCI controller between SD Card and eMMC use
cases. Previously, the common SoC dtsi unconditionally enabled the
'supports-cqe' property. This causes regression for SD cards, resulting
in timeouts and initialization failures during the probe sequence, as
the driver attempts to enable Command Queueing (CQE) logic incompatible
with the SD protocol.

To resolve this and enable full storage support, this series:

  1. Moves the 'supports-cqe' property out of the common SoC dtsi. It is
    now only enabled in the specific eMMC configuration where it is
    supported.
  2. Adds a device tree overlay to enable SD Card support (SDR/DDR modes).
  3. Adds a device tree overlay to enable eMMC support. This configuration
    also explicitly disables the UFS controller to prevent power leakage,
    as the VCC regulator is shared between the UFS and eMMC rails on this
    platform.

Validated on Qualcomm Monaco EVK with both SD Card and eMMC modules.

… board DT

The Monaco SoC SDHC controller supports both eMMC and SD cards. However,
the 'supports-cqe' property (Command Queue Engine) is specific to eMMC
and conflicts with SD card operation.

Remove 'supports-cqe' from the SoC device tree to ensure compatibility
with SD cards. Simultaneously, add the property explicitly to the
qcs8300-ride board device tree, as this board uses the controller in
eMMC mode.

This ensures the SoC definition remains generic while enabling features
correctly at the board level.

Signed-off-by: Monish Chunara <[email protected]>
…overlay

The monaco EVK board supports either eMMC or SD-card, but only one
can be active at a time.

Enable the SD Host Controller Interface (SDHCI) on the monaco EVK board
to support SD Card for storage via a device tree overlay. This allows
eMMC support to be enabled through a separate overlay when required.

Signed-off-by: Monish Chunara <[email protected]>
…a overlay

Enable the SDHCI controller for eMMC functionality on the Monaco EVK
using a device tree overlay. Configure the corresponding VCC regulator
parameters to support the eMMC interface.

Additionally, disable the UFS controller node. The VCC regulator is
shared between the UFS and eMMC subsystems; leaving UFS enabled while
operating eMMC results in power leakage on the shared rail.

Signed-off-by: Monish Chunara <[email protected]>
Copy link

@shashim-quic shashim-quic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add Link: tag to commit log pointing to lore. Refer other commits how they add.
If you are using b4 then -l option automatically adds it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants