Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1512292 > unrolled thread

[PATCH 0/2] mmc: dw_mmc: fix the wrong operation for reset controller

Back to article view | Back to linux.kernel

Contents

  [PATCH 0/2] mmc: dw_mmc: fix the wrong operation for reset controller Jaehoon Chung <jh80.chung@samsung.com> - 2016-10-31 04:00 +0100
    Re: [PATCH 0/2] mmc: dw_mmc: fix the wrong operation for reset controller John Stultz <john.stultz@linaro.org> - 2016-11-01 23:00 +0100
      Re: [PATCH 0/2] mmc: dw_mmc: fix the wrong operation for reset  controller Jaehoon Chung <jh80.chung@samsung.com> - 2016-11-03 06:20 +0100

#1512292 — [PATCH 0/2] mmc: dw_mmc: fix the wrong operation for reset controller

This patch adds the "reset-names" as reset controller for dwmmc controller.
Linaro guys had reported the issue about booting stuck.

Some SoCs are enabled the CONFIG_RESET_CONTROLLER.
then dwmmc controller are waiting for getting reset controller.
But if doesn't define "resets" property in device-tree, it should be stuck.

If use the reset-names as reset controller for dwmmc controller,
it's more stable than now.
This commit is related with the below commit.

Fixes: d6786fefe816 ("mmc: dw_mmc: add reset support to dwmmc host controller")

Jaehoon Chung (2):
  Documentation: synopsys-dw-mshc: add binding for reset-names
  mmc: dw_mmc: add the "reset" as name of reset controller

 Documentation/devicetree/bindings/mmc/synopsys-dw-mshc.txt | 5 +++++
 drivers/mmc/host/dw_mmc.c                                  | 2 +-
 2 files changed, 6 insertions(+), 1 deletion(-)

-- 
2.10.1

[toc] | [next] | [standalone]

#1513535

On Sun, Oct 30, 2016 at 7:49 PM, Jaehoon Chung <jh80.chung@samsung.com> wrote:
> This patch adds the "reset-names" as reset controller for dwmmc controller.
> Linaro guys had reported the issue about booting stuck.
>
> Some SoCs are enabled the CONFIG_RESET_CONTROLLER.
> then dwmmc controller are waiting for getting reset controller.
> But if doesn't define "resets" property in device-tree, it should be stuck.
>
> If use the reset-names as reset controller for dwmmc controller,
> it's more stable than now.
> This commit is related with the below commit.
>
> Fixes: d6786fefe816 ("mmc: dw_mmc: add reset support to dwmmc host controller")

So I wasn't hit by the issue, as I don't tend to boot from SD on my
hikey, but I did run these patches and I'm not seeing any issues with
my Android environment.

So for what its worth:
Acked-by: John Stultz <john.stultz@linaro.org>

thanks
-john

[toc] | [prev] | [next] | [standalone]

#1514293 — Re: [PATCH 0/2] mmc: dw_mmc: fix the wrong operation for reset controller

Hi,

On 11/02/2016 06:56 AM, John Stultz wrote:
> On Sun, Oct 30, 2016 at 7:49 PM, Jaehoon Chung <jh80.chung@samsung.com> wrote:
>> This patch adds the "reset-names" as reset controller for dwmmc controller.
>> Linaro guys had reported the issue about booting stuck.
>>
>> Some SoCs are enabled the CONFIG_RESET_CONTROLLER.
>> then dwmmc controller are waiting for getting reset controller.
>> But if doesn't define "resets" property in device-tree, it should be stuck.
>>
>> If use the reset-names as reset controller for dwmmc controller,
>> it's more stable than now.
>> This commit is related with the below commit.
>>
>> Fixes: d6786fefe816 ("mmc: dw_mmc: add reset support to dwmmc host controller")
> 
> So I wasn't hit by the issue, as I don't tend to boot from SD on my
> hikey, but I did run these patches and I'm not seeing any issues with
> my Android environment.
> 
> So for what its worth:
> Acked-by: John Stultz <john.stultz@linaro.org>

Applied on my repository (fixes branch).

Best Regards,
Jaehoon Chung

> 
> thanks
> -john
> 
> 
> 

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web