Re: [PATCH] mmc: dw_mmc: Add check for IDMAC configuration

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

 



On Tue, Jun 12, 2012 at 5:21 AM, Girish K S
<girish.shivananjappa@xxxxxxxxxx> wrote:
>
> In the Current dwmmc driver there is support for selecting IDMAC from
> the menu config option. If the support for IDMAC is enabled in the menu
> config and Hardware configuration register's DMA_INTERFACE field is 0.
> Still the driver will try to do the DMA initialization.
>
> The dw_mci_idmac_init function currently implemented returns only success
> indicating that the DMA initialization is always successful. The current
> patch will add a ciheck for existance of the DMA IP and allow the
> DMA initialization.
>
> Signed-off-by: Girish K S <girish.shivananjappa@xxxxxxxxxx>
> ---
>  drivers/mmc/host/dw_mmc.c |   11 ++++++++++-
>  1 files changed, 10 insertions(+), 1 deletions(-)

This looks ok in principle. I'm wondering if we should only allow
dma_support == 0x01 (DW_DMA) in the IDMAC case? It's not clear from
the TRM what each of the DMA values means and I only have a system
with external DMA support at the moment.
--
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