Hi Kevin and Ulf, 2012/12/14 Kevin Liu <keyuan.liu@xxxxxxxxx>: > 2012/12/14 Ulf Hansson <ulf.hansson@xxxxxxxxxx>: >> Hi Kevin, >> >> Thanks for looking into this for SDHCI. >> >> On 14 December 2012 11:53, Kevin Liu <kliu5@xxxxxxxxxxx> wrote: >>> >>> This patchset is based on Johan's below patch series submitted recently: >>> mmc: core: Add mmc_power_cycle >>> mmc: core: Add card_busy to host_ops >>> mmc: core: Fixup signal voltage switch >>> >>> This patchset mainly update sdhci.c to co-work with Johan's above patches. >>> The first two patches are some update for the core stack with Johan's patch. >>> The third patch add the card_busy function and change the voltage_switch function in sdhci driver to work with the new code. >>> >>> Kevin Liu (3): >>> mmc: core: enhance the code for signal voltage setting >>> mmc: core: cycle power the card in voltage switch rather than mmc_sd_get_cid >> >> Regarding you patches on the core layer. I think it would make sense >> to instead of patching on top of Johan patchset, if needed, it is >> better that you put review comment directly on Johans patches. Would >> that be possible for you? >> > > Sure, it's better to integrate related change into one patch. > I can update my patch, then Johan can just integrate my patch if he agree. > Or Johan change his patch directly if needed. I'll try to have an updated patchset by Monday next week which integrates your changes and adds your Signed-off-by if that's OK with you. Thank you for testing this stuff! Kind regards, Johan -- 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