Re: [PATCH 0/6] mmc: dw_mmc: remove the multiple slot concept

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 06/08/2017 10:52 PM, Ulf Hansson wrote:
> On 5 June 2017 at 06:41, Jaehoon Chung <jh80.chung@xxxxxxxxxxx> wrote:
>> DWMMC IP is provided the multiple slot.
>> It's only used the one slot per an IP in mmc subsystem.
>> If we can consider only one slot, then dwmmc controller codes can be more simpler than now.
>> This patchset is the first step to remove the multiple slot.
>>
>> In future work,
>>    - Remove the entire slot structure.
>>    - Move to slot's members into dwmci host.
>>
>> Jaehoon Chung (6):
>>   mmc: dw_mmc: deprecated the "num-slots" property
>>   mmc: dw_mmc: remove the loop about finding slots
>>   mmc: dw_mmc: change the array of slots
>>   mmc: dw_mmc: remove the 'id' arguments about functions relevant to
>>     slot
>>   mmc: dw_mmc: use the 'slot' instead of 'cur_slot'
>>   mmc: dw_mmc: remove the unnecessary slot variable
>>
>>  drivers/mmc/host/dw_mmc-exynos.c |   4 +-
>>  drivers/mmc/host/dw_mmc.c        | 165 +++++++++++++--------------------------
>>  drivers/mmc/host/dw_mmc.h        |   7 +-
>>  3 files changed, 57 insertions(+), 119 deletions(-)
>>
>> --
>> 2.10.2
>>
> 
> This looks good to me, however I couldn't apply them cleanly on my next branch.
> 
> Perhaps you have something additional for dw_mmc in you local tree?
> Then never mind, I can pick these via your regular pull request.

Will send PR..Thanks!

Best Regards,
Jaehoon Chung

> 
> Kind regards
> Uffe
> --
> To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 
> 
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux