Hi Girish, On Tue, Jun 12 2012, Will Newton wrote: > On Tue, Jun 12, 2012 at 10:58 AM, Girish K S > <girish.shivananja...@linaro.org> 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.shivananja...@linaro.org> >> --- >> drivers/mmc/host/dw_mmc.c | 15 +++++++++++++-- >> 1 files changed, 13 insertions(+), 2 deletions(-) > > Ok, this version looks better. > > Acked-by: Will Newton <will.new...@imgtec.com>
A quick point of patch etiquette: you used a subject line of [RESEND PATCH], but this isn't a resend of a patch. A resend is when you're resending a copy of a patch *without any modifications* because it was ignored, or you'd like more discussion, or you're adding new people to the e-mail thread. This is a v2 of a patch, because you made changes since the last patch. The right way to signify that is with [PATCH v2] in the Subject line, and a section just below the "---" after the Signed-off-by that explains what you changed. E.g.: ======= Subject: [PATCH v2] mmc: dw_mmc: Add check for IDMAC configuration 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.shivananja...@linaro.org> --- Changes since v1: * Protect against configuration mistakes by falling back to PIO if CONFIG_MMC_DW_IDMAC=y but the hardware has no IDMAC support. drivers/mmc/host/dw_mmc.c | 15 +++++++++++++-- 1 files changed, 13 insertions(+), 2 deletions(-) diff --git a/drivers/mmc/host/dw_mmc.c b/drivers/mmc/host/dw_mmc.c ======= I've applied the patch to mmc-next for 3.6 now. Thanks, - Chris. -- Chris Ball <c...@laptop.org> <http://printf.net/> One Laptop Per Child -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
