On Feb 18, 2011, at 9:17 AM, Nath, Arindam wrote: > Hi Philip, > > >> -----Original Message----- >> From: Philip Rakity [mailto:prakity@xxxxxxxxxxx] >> Sent: Friday, February 18, 2011 9:19 PM >> To: Nath, Arindam >> Cc: Zhangfei Gao; Chris Ball; linux-mmc@xxxxxxxxxxxxxxx; Su, Henry; Lu, >> Aaron; anath.amd@xxxxxxxxx; Nicolas Pitre >> Subject: [PATCH 01/12] mmc: sdhci: add support for SD Host Controller >> 3.0 >> >> >> Arindam, >> >> Could you look at making (Tuning) CMD19 part of the sdhc.ci driver? >> It flow is part of the Host Controller and Physical layer specs. > > There are two main reasons for exposing the callback of tuning procedure from sdhci.c: > > 1. The tuning procedure is part of UHS-I initialization sequence as described in Physical Layer Spec v3.01. Since the UHS-I init sequence takes places inside sd.c, so I thought it better to place it there instead. understand -- I think it should be in the driver as part of the init sequence. Once you have some code I can prototype this. We can do it offline. > 2. In future, if we need to add support for UHS-I sequence for sdio cards too, we can use the same callback from sdhci.c without any modifications. > SDIO is not a problem -- almost follows same code sequence. I have this almost working. > I am open to suggestions. > > Regards, > Arindam > -- 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