On Fri, Oct 26, 2012 at 3:57 AM, Jaehoon Chung <jh80.chung@xxxxxxxxxxx> wrote: > I think right that dw_mci_setup_bus() is called into set_ios(). > Actually clock is enabled at dw_mci_setup_bus() in now. > Then didn't ensure to wait the 74clock. > To stable clock is relocated the position into dw_mci_set_ios(). > > Signed-off-by: Jaehoon Chung <jh80.chung@xxxxxxxxxxx> > Signed-off-by: Kyungmin Park <kyungmin.park@xxxxxxxxxxx> > --- > drivers/mmc/host/dw_mmc.c | 6 +++--- > 1 files changed, 3 insertions(+), 3 deletions(-) This patch looks ok in principle, but I have trouble understanding the commit message. Would it be possible to try and reword it a bit to make it clearer? Thanks, -- 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