On 11/24/2016 08:43 PM, Ulf Hansson wrote: > On 24 November 2016 at 12:17, Jaehoon Chung <jh80.chung@xxxxxxxxxxx> wrote: >> On 11/23/2016 09:31 PM, Jaehoon Chung wrote: >>> On 11/23/2016 08:50 PM, Jaehoon Chung wrote: >>>> On 11/23/2016 08:47 PM, Ulf Hansson wrote: >>>>> On 22 November 2016 at 17:13, Ulf Hansson <ulf.hansson@xxxxxxxxxx> wrote: >>>>>> On 22 November 2016 at 10:59, Jaehoon Chung <jh80.chung@xxxxxxxxxxx> wrote: >>>>>>> Dear Ulf, >>>>>>> >>>>>>> Could you pull these patches on your next branch? >>>>>>> If there is a problem, let me know, plz. >>>>>>> >>>>>>> The following changes since commit e2081b37d910c5e6c6925d9b4d0a7a6283f84ec5: >>>>>>> >>>>>>> Merge branch 'fixes' into next (2016-11-21 11:09:18 +0100) >>>>>>> >>>>>>> are available in the git repository at: >>>>>>> >>>>>>> https://github.com/jh80chung/dw-mmc.git for-ulf >>>>>>> >>>>>>> for you to fetch changes up to e25fd245b557482a8e0f7ab87841085f30706f3a: >>>>>>> >>>>>>> Documentation: synopsys-dw-mshc: remove the unused properties (2016-11-22 10:34:05 +0900) >>>>>>> >>>>>>> ---------------------------------------------------------------- >>>>>>> Colin Ian King (1): >>>>>>> mmc: dw_mmc: fix spelling mistake in dev_dbg message >>>>>>> >>>>>>> Jaehoon Chung (9): >>>>>>> mmc: dw_mmc: display the real register value on debugfs >>>>>>> mmc: dw_mmc: fix the debug message for checking card's present >>>>>>> mmc: dw_mmc: change the DW_MCI_FREQ_MIN from 400K to 100K >>>>>>> mmc: dw_mmc: use the hold register when send stop command >>>>>>> mmc: dw_mmc: call the dw_mci_prep_stop_abort() by default >>>>>>> mmc: dw_mmc: use the cookie's enum values for post/pre_req() >>>>>>> mmc: dw_mmc: remove the unnecessary mmc_data structure >>>>>>> mmc: dw_mmc: The "clock-freq-min-max" property was deprecated >>>>>>> Documentation: synopsys-dw-mshc: remove the unused properties >>>>>>> >>>>>>> Documentation/devicetree/bindings/mmc/synopsys-dw-mshc.txt | 8 ++------ >>>>>>> drivers/mmc/host/dw_mmc.c | 92 ++++++++++++++++++++++++++++++++++++++++++++------------------------------------------------ >>>>>>> include/linux/mmc/dw_mmc.h | 6 ++++++ >>>>>>> 3 files changed, 52 insertions(+), 54 deletions(-) >>>>>>> >>>>>>> Best Regards, >>>>>>> Jaehoon Chung >>>>>> >>>>>> Dear Jaehoon, thanks for the pull request. I have add queued it all up >>>>>> for my next branch! >>>>>> >>>>>> >>>>>> Kind regards >>>>>> Uffe >>>>> >>>>> Jaehoon, >>>>> >>>>> It seems like some of these patches breaks Exynos 5250-arndale. Can >>>>> you please have a look and see what you think? >>>>> https://kernelci.org/boot/all/job/ulfh/kernel/v4.9-rc5-78-g52eb5d4531e4/ >>>> >>>> I will check and reply. Thanks for noticing. >> >> Ulf, >> >> It's strange..My patches didn't have any dependency... >> There is no any log for dwmmc..and even it doesn't initialize dwmmc controller. >> >> Did you think that this failure is my patches problem? What is your thinking? > > Well, I don't know but indeed it seems a bit odd. Maybe there are some > interdependency that changes the behaviour for something outside > dw_mmc, which triggers the problems!? > > It could of course also be completely unrelated. > > I do have the Arndale board available. I will try it out within a > couple of days, to see if I can find something. If there is effect..it seems that only below patch for clock.. commit 38332b0fd6c06fcecc64bc238bbb7cf80ed6bc7b Author: Jaehoon Chung <jh80.chung@xxxxxxxxxxx> Date: Thu Nov 17 16:40:35 2016 +0900 mmc: dw_mmc: change the DW_MCI_FREQ_MIN from 400K to 100K If there is no property "clock-freq-min-max", mmc->f_min should be set to 400K by default. But Some SoC can be used 100K. When 100K is used, MMC core will try to check from 400K to 100K. otherwise timing? I wonder when the below patches is applied, what happen.. https://patchwork.kernel.org/patch/9445233/ https://patchwork.kernel.org/patch/9445209/ https://patchwork.kernel.org/patch/9445211/ I will also check this continuously..to find before PR for next. Best Regards, Jaehoon Chung > > [...] > > Kind regards > Uffe > -- > 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 > > > -- 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
