[PATCH] mmc: rtsx: Revert "mmc: rtsx: add support for pre_req and post_req"

Ulf Hansson ulf.hansson at linaro.org
Mon Apr 28 14:11:30 UTC 2014


On 28 April 2014 14:29, Lee Jones <lee.jones at linaro.org> wrote:
>> >> From: Micky Ching <micky_ching at realsil.com.cn>
>> >>
>> >> This reverts commit c42deffd5b53c9e583d83c7964854ede2f12410d.
>> >
>> > Why was this patch even merged without an MFD Ack?
>> >
>> >> commit <mmc: rtsx: add support for pre_req and post_req> did use
>> >> mutex_unlock() in tasklet, but mutex_unlock() can't used in
>> >> tasklet(atomic context). The driver need use mutex to avoid concurrency,
>> >> so we can't use tasklet here, the patch need to be removed.
>> >>
>> >> The spinlock host->lock and pcr->lock may deadlock, one way to solve the
>> >> deadlock is remove host->lock in sd_isr_done_transfer(), but if using
>> >> workqueue the we can avoid using the spinlock and also avoid the problem.
>> >>
>> >> Signed-off-by: Micky Ching <micky_ching at realsil.com.cn>
>> >> ---
>> >>  drivers/mfd/rtsx_pcr.c            |  132 ++++--------
>> >>  drivers/mmc/host/rtsx_pci_sdmmc.c |  418 ++++++-------------------------------
>> >>  include/linux/mfd/rtsx_common.h   |    1 -
>> >>  include/linux/mfd/rtsx_pci.h      |    6 -
>> >>  4 files changed, 109 insertions(+), 448 deletions(-)
>> >
>> > This patch should be sent to the -rcs as soon as possible:
>>
>> Do you mind helping out doing it here Lee? Not sure how busy Chris is
>> at the moment.
>
> I don't mind. I just need a maintainer Ack from one of you guys.

Thanks for helping out Lee. Please hold on until we sorted out exactly
what should be reverted, it seems like some additional patch should be
removed/reverted as well.

Kind regards
Uffe

>
>> >
>> > Acked-by: Lee Jones <lee.jones at linaro.org>
>
> --
> Lee Jones
> Linaro STMicroelectronics Landing Team Lead
> Linaro.org │ Open source software for ARM SoCs
> Follow Linaro: Facebook | Twitter | Blog


More information about the devel mailing list