Re: [PATCH v3 02/15] mmc: host: omap_hsmmc: return on fatal errors from omap_hsmmc_reg_get

2015-08-28 Thread Tony Lindgren
n be obtained later and all other > > errors are fatal. > > > > Signed-off-by: Kishon Vijay Abraham I > > Tested-by: Tony Lindgren > > I bisected boot failures on Panda ES with multi_v7_defconfig down to > this commit on last night's -next build: >

Re: [PATCH v3 02/15] mmc: host: omap_hsmmc: return on fatal errors from omap_hsmmc_reg_get

2015-08-28 Thread Olof Johansson
> Signed-off-by: Kishon Vijay Abraham I > Tested-by: Tony Lindgren I bisected boot failures on Panda ES with multi_v7_defconfig down to this commit on last night's -next build: http://arm-soc.lixom.net/bootlogs/next/next-20150828/pandaes-arm-multi_v7_defconfig.html -Olof -- To unsubscrib

[PATCH v2] mmc: core: fix race condition in mmc_wait_data_done

2015-08-28 Thread Shawn Lin
From: Jialing Fu The following panic is captured in ker3.14, but the issue still exists in latest kernel. - [ 20.738217] c0 3136 (Compiler) Unable to handle kernel NULL pointer dereference at virtual address 0578 .. [

Re: [RESEND PATCH] mmc: core: fix race condition in mmc_wait_data_done

2015-08-28 Thread Shawn Lin
在 2015/8/28 18:22, Jialing Fu 写道: [...] Hi, ulf We find this bug on Intel-C3230RK platform for very small probability. Whereas I can easily reproduce this case if I add a mdelay(1) or longer delay as Jialing did. This patch seems useful to me. Should we push it forward? :) It seems like

RE: [RESEND PATCH] mmc: core: fix race condition in mmc_wait_data_done

2015-08-28 Thread Jialing Fu
[...] >>> Hi, ulf >>> >>> We find this bug on Intel-C3230RK platform for very small probability. >>> >>> Whereas I can easily reproduce this case if I add a mdelay(1) or >>> longer delay as Jialing did. >>> >>> This patch seems useful to me. Should we push it forward? :) >> >> >> It seems like

Re: [RESEND PATCH] mmc: core: fix race condition in mmc_wait_data_done

2015-08-28 Thread Ulf Hansson
[...] >>> Hi, ulf >>> >>> We find this bug on Intel-C3230RK platform for very small probability. >>> >>> Whereas I can easily reproduce this case if I add a mdelay(1) or longer >>> delay as Jialing did. >>> >>> This patch seems useful to me. Should we push it forward? :) >> >> >> It seems like a

Re: [RESEND PATCH] mmc: core: fix race condition in mmc_wait_data_done

2015-08-28 Thread Shawn Lin
在 2015/8/28 16:55, Ulf Hansson 写道: On 28 August 2015 at 05:25, Shawn Lin wrote: On 2015/8/28 11:13, Shawn Lin wrote: From: Jialing Fu The following panic is captured in ker3.14, but the issue still exists in latest kernel.

Re: [RESEND PATCH] mmc: core: fix race condition in mmc_wait_data_done

2015-08-28 Thread Ulf Hansson
On 28 August 2015 at 05:25, Shawn Lin wrote: > On 2015/8/28 11:13, Shawn Lin wrote: >> >> From: Jialing Fu >> >> The following panic is captured in ker3.14, but the issue still exists >> in latest kernel. >> - >> [ 20.738217] c

Re: [REGRESSION] dereference NULL pointer when removing mmc card

2015-08-28 Thread Shawn Lin
在 2015/8/27 22:35, Shawn Lin 写道: On 2015/8/27 14:45, Jaehoon Chung wrote: Hi, Igor. On 08/26/2015 06:34 PM, Jaehoon Chung wrote: On 08/26/2015 12:27 AM, Ulf Hansson wrote: On 22 August 2015 at 09:17, Igor Gnatenko wrote: mmc0: card 59b4 removed BUG: unable to handle kernel NULL pointer dere

Re: [PATCH] mmc: block: disable the reliable write If the card does not support CMD23

2015-08-28 Thread Ulf Hansson
- Chris and LKML, On 28 August 2015 at 02:38, LIYONG wrote: > Hi Uffe, > > Since Reliable Write depends on the CMD23. I think we need to disable > Reliable Write if the card does not support CMD23(due to performance impact > issue), so I add this patch. > > Do you think we still need to enable

Re: Re: [v4,1/6] mmc: dt: add DT binding for big endian controller

2015-08-28 Thread Ulf Hansson
On 27 August 2015 at 20:39, Li Leo wrote: >> > Or Maybe there is another method, use conditional compilation. The previous >> method would delete the MMC_SDHCI_BIG_ENDIAN_32BIT_BYTE_SWAPPER to >> use accessors defined in sdhci-esdhc.h, and add dependency of ARM. This >> method could use 'select i