Re: [PATCH 0/3] mmc: Signal voltage switch procedure for UHS mode (sdhci)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



2012/12/14 Johan Rudholm <jrudholm@xxxxxxxxx>:
> 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!
>

I'm ok.

Thanks
Kevin
--
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


[Index of Archives]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux