On Mon, Jul 23, 2012 at 6:17 PM, Seungwon Jeon <tgih.jun@xxxxxxxxxxx> wrote: > I have followed this patch from v1. > I think it's a good point. > Looks good to me. Thanks for the review! Looking forward to your ack with the next version. :) > Above comment is correct, but this is not specific for Exynos5250. > Exynos5250 is just one of host controllers base on Synopsys's. > It'd be better to remove the part related to Exynos you mentioned . Done. Grant had requested me to quote exactly which manual I found the requirement in, but I think describing just that it was found in the description of the CLKENA register is enough. I've changed it. > Definitely, low power mode not power mode, right? Done. Good catch! -Doug -- 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