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 Kevin Liu <keyuan.liu@xxxxxxxxx>:
> 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.
>
Johan and Ulf,

I just sent the v2 patchset which include your review comments.
Please review and integrate based on this version.

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