Re: [PATCH 0/13] dw_mmc cleanup

[Shawn Lin <shawn.lin@rock-chips.com>]

在 2025/11/20 星期四 19:10, Ulf Hansson 写道:
> On Thu, 20 Nov 2025 at 11:29, Shawn Lin <shawn.lin@rock-chips.com> wrote:
>>
>> Hi Ulf & Jaehoon
>>
>> As you can see, dw_mmc is likely one of the most complex and difficult-to-read
>> host driver at present. It maintains various states and bottom-half scheduling,
>> containing a significant amount of redundant code—including a multi-slot mechanism
>> that has been unsupported for over a decade.
>>
>> Jaehoon attempted to remove cur_slot more than ten years ago, but the driver still
>> retains the slot structure and the associated queue mechanism designed to support
>> multiple slots. This has made the already complex code even harder to read and maintain.
>>
>> The first four patches aim to eliminate some of the redundant code, while the remaining
>> patches are intended to ultimately remove the dw_mci_slot variable. To facilitate review
>> and minimize the risk of regression, each patch is designed to accomplish a single,
>> clear objective.
>>
>> This series have been tested on the RK3588S EVB1 platform.
> 
> Thanks for working on this! I had something similar on my todo list
> for dw_mmc for years, but never reached to actually work on it.
> Anyway, I will review the code shortly.

Thanks for helping review it. I found patch 8，12 and 13 need to be 
improved a bit after another round of self-reviewing, but they are
trival formatting issues. Will update onced get reviewed.

> 
> That said, would it be possible for you to step in and help Jaehoon
> with the maintenance going forward?
> 

As I do have other follow-up patches to improve this driver, it would be
good for me to keep an eye on changes of dw_mmc. So yes, I am OK
to step in and help review dw_mmc patches.


> Kind regards
> Uffe
> 
>>
>>
>>
>> Shawn Lin (13):
>>    mmc: dw_mmc: Remove unused struct dma_pdata
>>    mmc: dw_mmc: add dw_mci_prepare_desc() for both of 32bit and 64bit DMA
>>    mmc: dw_mmc: Remove vqmmc_enabled from struct dw_mci
>>    mmc: dw_mmc: Remove unused header files and keep alphabetical order
>>    mmc: dw_mmc: Move struct mmc_host from struct dw_mci_slot to struct
>>      dw_mci
>>    mmc: dw_mmc: Let glue drivers to use struct dw_mci as possible
>>    mmc: dw_mmc: Move flags from struct dw_mci_slot to struct dw_mci
>>    mmc: dw_mmc: Remove id from dw_mci_slot
>>    mmc: dw_mmc: Remove sdio_id from struct dw_mci_slot
>>    mmc: dw_mmc: Move clock rate stuff from struct dw_mci_slot to struct
>>      dw_mci
>>    mmc: dw_mmc: Remove mrq from struct dw_mci_slot
>>    mmc: dw_mmc: Remove queue from dw_mci
>>    mmc: dw_mmc: Remove struct dw_mci_slot
>>
>>   drivers/mmc/host/dw_mmc-exynos.c      |   9 +-
>>   drivers/mmc/host/dw_mmc-hi3798cv200.c |   6 +-
>>   drivers/mmc/host/dw_mmc-hi3798mv200.c |  17 +-
>>   drivers/mmc/host/dw_mmc-k3.c          |  21 +-
>>   drivers/mmc/host/dw_mmc-pltfm.c       |   2 +-
>>   drivers/mmc/host/dw_mmc-rockchip.c    |   9 +-
>>   drivers/mmc/host/dw_mmc-starfive.c    |   5 +-
>>   drivers/mmc/host/dw_mmc.c             | 606 ++++++++++++++--------------------
>>   drivers/mmc/host/dw_mmc.h             |  83 ++---
>>   9 files changed, 297 insertions(+), 461 deletions(-)
>>
>> --
>> 2.7.4
>>
>