Re: [PATCH 0/2] mmc: factor out getting dma direction from host drivers

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

 



Am 24.03.2017 um 02:14 schrieb Shawn Lin:
> Hi Heiner,
> 
> On 2017/3/24 5:17, Heiner Kallweit wrote:
>> Several host drivers contain identical code for determining the
>> DMA direction. Therefore factor this out to the core.
> 
> I like this patchset, at least it removes more lines than insertions :)
> 
> Only a small nit,
> 
> s3cmci_prepare_dma seems need some more cleanup as the rw variable
> isn't useful personally. Anyway,
> 
rw is still used for setting conf.direction, therefore I decided to
keep it (at least for now, however this variable might be a victim of
some later, more general refactoring).

However I found few more places where the new function can be applied.
So I'll send a v2 anyway.

> Reviewed-by: Shawn Lin <shawn.lin@xxxxxxxxxxxxxx>
> 
Thanks for the quick review.

> 
> 
>>
>> Heiner Kallweit (2):
>>   mmc: core: add mmc_get_dma_dir
>>   mmc: use new core function mmc_get_dma_dir
>>
>>  drivers/mmc/host/android-goldfish.c | 10 ++--------
>>  drivers/mmc/host/atmel-mci.c        |  6 ++----
>>  drivers/mmc/host/davinci_mmc.c      | 10 +++-------
>>  drivers/mmc/host/dw_mmc.c           | 14 +++-----------
>>  drivers/mmc/host/jz4740_mmc.c       |  9 ++-------
>>  drivers/mmc/host/mmc_spi.c          |  5 +----
>>  drivers/mmc/host/mtk-sd.c           |  8 ++------
>>  drivers/mmc/host/mvsdio.c           | 11 +++++------
>>  drivers/mmc/host/omap_hsmmc.c       | 17 ++++-------------
>>  drivers/mmc/host/s3cmci.c           |  4 ++--
>>  drivers/mmc/host/sdhci.c            |  9 +++------
>>  drivers/mmc/host/sunxi-mmc.c        | 14 +++-----------
>>  include/linux/mmc/host.h            |  6 ++++++
>>  13 files changed, 38 insertions(+), 85 deletions(-)
>>
> 
> 

--
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