RE: [PATCH v4 1/2] mmc: API for accessing host supported maximum segment count and size

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

 



Hi Ulf,

> From: Amitkumar Karwar [mailto:akarwar@xxxxxxxxxxx]
> Sent: Thursday, June 16, 2016 2:53 PM
> To: linux-wireless@xxxxxxxxxxxxxxx
> Cc: Nishant Sarmukadam; Wei-Ning Huang; linux-mmc@xxxxxxxxxxxxxxx; Cathy
> Luo; Xinming Hu; Amitkumar Karwar
> Subject: [PATCH v4 1/2] mmc: API for accessing host supported maximum
> segment count and size
> 
> From: Xinming Hu <huxm@xxxxxxxxxxx>
> 
> sdio device drivers need be able to get the host supported max_segs and
> max_seg_size, so that they know the buffer size to allocate while
> utilizing the scatter/gather DMA buffer list.
> 
> This patch provides API for this purpose.
> 
> Signed-off-by: Xinming Hu <huxm@xxxxxxxxxxx>
> Signed-off-by: Amitkumar Karwar <akarwar@xxxxxxxxxxx>
> ---
> v2: v2 was submitted with minor improvement like replacing BUG_ON() with
> WARN_ON()
> v3: Addressed below review comments from Ulf Hansson
>     a) In v3, patch has been split into two separate patches.
>     b) Patch 1/2 introduces an API to fetch max_seg_size and max_segs
>     c) Replaced WARN_ON() with proper error code when sg_ptr->length is
> invalid
>     d) Instead of duplicating the code in mmc_io_rw_extended(), extra
> bool parameter
>        has been added to this function and used it in new APIs for SG.
> v4: Removed WARN_ON() calls in newly added APIs. It's gets called in
> probe handler.
>     Caller already takes care of it(Shawn Lin).
> ---
>  drivers/mmc/core/sdio_io.c                          | 21
> +++++++++++++++++++++
>  .../wireless/broadcom/brcm80211/brcmfmac/bcmsdh.c   |  6 +++---
>  include/linux/mmc/sdio_func.h                       |  3 +++
>  3 files changed, 27 insertions(+), 3 deletions(-)
> 

Please let me know if you have any further review comments for this patch series.

Regards,
Amitkumar
--
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