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]

 



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


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

  Powered by Linux