Re: [PATCH v13 2/7] mmc: k1: add sdhci platform driver

[Yixun Lan <dlan@kernel.org>]

Hi Peng Fan, Jaehoon Chung,

On 12:40 Wed 16 Sep     , Yixun Lan wrote:
> Hi Peng Fan, Jaehoon Chung,
> 
> On 20:36 Sat 12 Sep     , Eric Chung wrote:
> > Add SDHCI platform driver support for SpacemiT K1 SoC. This driver
> > implements the necessary platform-specific operations for the SDHCI
> > controller, enabling MMC/SD card functionality on K1-based platforms.
> > 
> > Signed-off-by: Eric Chung <eric.chung@riscstar.com>
> > 
> > ---
> > v13:
> > - Sync definition values from upstream mmc driver.
> > v9:
> > - Disable CMD18 (multiple-block read) for SD cards in SPL.
> > v6:
> > - Remove ".data = 0" in spacemit_sdhci_ids[].
> > v5:
> > - Remove MMC_CAP_CMD23 as the CMD23 support is now a separate series.
> > - Use tabs for macro definition alignment.
> > - Use read_poll_timeout().
> > - Set pinctrl state based on selected timing mode.
> > - Add the definition of SDHC_DLL_REG1_HS400_VAL.
> > - Remove the condition based on quirks.
> > v4:
> > - Add bulk release operations on reset and clock.
> > v3:
> > - Enable CMD23 in capability.
> > v2:
> > - Enable ADMA mode support.
> > - Use CMD23 for multi-block read/write.
> > - Move ASR/AIB register into pinctrl driver.
> > - Correct pinctrl state from "fast" to "uhs".
> > - Migrate tuning support from the spacemit linux driver.
> > ---
> >  drivers/mmc/Kconfig          |   7 +
> >  drivers/mmc/Makefile         |   1 +
> >  drivers/mmc/spacemit_sdhci.c | 684 +++++++++++++++++++++++++++++++++++++++++++
> >  3 files changed, 692 insertions(+)
> > 
> 
> Ping mmc maintainer for this patch, would it be Ok if I take this patch via
> spacemit's custodian tree [1], if yes, then I'd like to grant an Acked-by
> 
> Or, if you guys prefer to take it with mmc tree, then I could handle the rest..
> 
> Btw, FYI, I've already pushed for CI testing which looks good [2]
> 
Ping again, otherwise I'd go ahead and proceed via spacemit custodian
tree

> Thank you
> 
> Link: https://git.u-boot-project.org/u-boot/custodians/u-boot-spacemit.git [1]
> Link: https://git.u-boot-project.org/u-boot/custodians/u-boot-spacemit/-/pipelines/1284 [2] 
> 

-- 
Yixun Lan (dlan)