All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Philippe Mathieu-Daudé" <philmd@oss.qualcomm.com>
To: Bin Meng <bin.meng@processmission.com>, QEMU <qemu-devel@nongnu.org>
Cc: "Bin Meng" <bmeng.cn@gmail.com>,
	"Philippe Mathieu-Daudé" <philmd@mailo.com>,
	qemu-block@nongnu.org
Subject: Re: [PATCH 06/10] hw/sd: sdhci: Honor i.MX uSDHC vendor clock gates
Date: Sat, 15 Aug 2026 16:15:00 +0200	[thread overview]
Message-ID: <fca795b7-052f-4bc5-900c-d77b8cbae14e@oss.qualcomm.com> (raw)
In-Reply-To: <20260812013619.2134092-7-bin.meng@processmission.com>

On 12/8/26 03:36, Bin Meng wrote:
> U-Boot's i.MX uSDHC driver enables IPGEN, HCKEN, PEREN and CKEN in
> VEND_SPEC rather than the standard SDHCI clock-control fields. QEMU
> only checks the standard fields before issuing a command. It therefore
> silently drops U-Boot's MMC commands even though the controller clocks
> are enabled. U-Boot eventually times out waiting for command completion
> and cannot load the kernel and device tree from the SD card.
> 
> Accept the complete vendor clock-gate set as another valid clock source
> for i.MX uSDHC. Also report SDSTB when the vendor IP and host clocks
> are enabled, matching the state U-Boot polls while changing the clock.
> 
> Keep this behavior behind an i.MX uSDHC quirk because the shared eSDHC
> paths also serve controllers which use the standard SDHCI fields.
> 
> Reference: IMX6ULRM (Rev 2), section 56.8.10, 56.8.12 and 56.8.26
> https://www.nxp.com/webapp/Download?colCode=IMX6ULRM
> 
> Signed-off-by: Bin Meng <bin.meng@processmission.com>
> ---
> 
>   hw/sd/sdhci.c         | 27 ++++++++++++++++++++++++---
>   include/hw/sd/sdhci.h |  7 ++++++-
>   2 files changed, 30 insertions(+), 4 deletions(-)

Reviewed-by: Philippe Mathieu-Daudé <philmd@oss.qualcomm.com>


  reply	other threads:[~2026-08-15 14:15 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-08-12  1:36 [PATCH 00/10] hw/arm: Enable U-Boot boot on MCIMX6UL-EVK Bin Meng
2026-08-12  1:36 ` [PATCH 01/10] hw/arm: fsl-imx6ul: Add SCU compatibility window Bin Meng
2026-08-12  1:36 ` [PATCH 02/10] hw/misc: imx6ul_ccm: Update PMU_MISC0 reset value Bin Meng
2026-08-12  1:36 ` [PATCH 03/10] hw/arm: fsl-imx6ul: Map early firmware register placeholders Bin Meng
2026-08-12  1:36 ` [PATCH 04/10] hw/arm: fsl-imx6ul: Add a minimal MMDC geometry model Bin Meng
2026-08-12  1:36 ` [PATCH 05/10] hw/sd: sdhci: Use a QEMU-local no-busy IRQ quirk bit Bin Meng
2026-08-15 14:12   ` Philippe Mathieu-Daudé
2026-08-12  1:36 ` [PATCH 06/10] hw/sd: sdhci: Honor i.MX uSDHC vendor clock gates Bin Meng
2026-08-15 14:15   ` Philippe Mathieu-Daudé [this message]
2026-08-12  1:36 ` [PATCH 07/10] hw/sd: sdhci: Preserve uSDHC status enables for U-Boot Bin Meng
2026-08-12  1:36 ` [PATCH 08/10] hw/sd: sdhci: Skip SDMA boundary stops for i.MX uSDHC Bin Meng
2026-08-12  1:36 ` [PATCH 09/10] docs/system/arm: Document MCIMX6UL-EVK Buildroot boot Bin Meng
2026-08-12  1:36 ` [PATCH 10/10] tests/functional/arm: Add MCIMX6UL-EVK boot tests Bin Meng

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=fca795b7-052f-4bc5-900c-d77b8cbae14e@oss.qualcomm.com \
    --to=philmd@oss.qualcomm.com \
    --cc=bin.meng@processmission.com \
    --cc=bmeng.cn@gmail.com \
    --cc=philmd@mailo.com \
    --cc=qemu-block@nongnu.org \
    --cc=qemu-devel@nongnu.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.