Re: [PATCH v3 0/3] mmc: tmio: fix reset operation
Hi Ulf, Wolfram On 2018-11-19 14:33:58 +0100, Wolfram Sang wrote: > > > Sure, no problem. I drop this and the other series then. > > Thanks, Ulf! > Thanks for looking out for me here, I will fly home tomorrow so I hope to get a new versions of these series out late this week. -- Regards, Niklas Söderlund
Re: [PATCH v3 0/3] mmc: tmio: fix reset operation
> Sure, no problem. I drop this and the other series then. Thanks, Ulf! signature.asc Description: PGP signature
Re: [PATCH v3 0/3] mmc: tmio: fix reset operation
On 19 November 2018 at 13:14, Wolfram Sang wrote: > >> I noticed there were a minor comment from Yamada-san, however I >> decided to pick this as is and leave further improvements to be made >> on top. > > Can I vote for waiting until Niklas comes back from Plumbers and have a > proper V2 applied instead? Makes backporting easier. > Sure, no problem. I drop this and the other series then. Kind regards Uffe
Re: [PATCH v3 0/3] mmc: tmio: fix reset operation
> I noticed there were a minor comment from Yamada-san, however I > decided to pick this as is and leave further improvements to be made > on top. Can I vote for waiting until Niklas comes back from Plumbers and have a proper V2 applied instead? Makes backporting easier. signature.asc Description: PGP signature
Re: [PATCH v3 0/3] mmc: tmio: fix reset operation
On 1 November 2018 at 00:05, Niklas Söderlund wrote: > From: Niklas Söderlund > > Hi, > > While looking at the Renesas BSP kernel I found patches which improves > the state of the hardware at probe and after runtime resume. > > Patch 1/3 make sure the module clock is enabled after resuming before > register are accessed. Patch 2/3 is the real change in this series and > brings in reset of the vendor specific callback when resetting (SCC in > the Renesas case). While 3/3 simply make sure that the IRQ mask for > Renesas boards (Gen2 and later) are in a good shape after probe (and > reset). > > In addition to addressing the state after resuming it helped unbreak a > SD card I have which are very problematic on Koelsch. Without this > series inserting the card results in: > > sh_mobile_sdhi ee10.sd: timeout waiting for hardware interrupt (CMD19) > sh_mobile_sdhi ee10.sd: timeout waiting for hardware interrupt (CMD19) > sh_mobile_sdhi ee10.sd: timeout waiting for hardware interrupt (CMD19) > sh_mobile_sdhi ee10.sd: timeout waiting for hardware interrupt (CMD19) > sh_mobile_sdhi ee10.sd: timeout waiting for hardware interrupt (CMD19) > sh_mobile_sdhi ee10.sd: timeout waiting for hardware interrupt (CMD19) > sh_mobile_sdhi ee10.sd: timeout waiting for hardware interrupt (CMD19) > sh_mobile_sdhi ee10.sd: timeout waiting for hardware interrupt (CMD19) > sh_mobile_sdhi ee10.sd: timeout waiting for hardware interrupt (CMD19) > sh_mobile_sdhi ee10.sd: timeout waiting for hardware interrupt (CMD19) > sh_mobile_sdhi ee10.sd: Tuning procedure failed > mmc0: tuning execution failed: -5 > mmc0: error -5 whilst initialising SD card > sh_mobile_sdhi ee10.sd: timeout waiting for hardware interrupt (CMD19) > > While with this series applied (patch 2/3): > > sh_mobile_sdhi ee10.sd: timeout waiting for hardware interrupt (CMD19) > mmc0: new ultra high speed SDR50 SDHC card at address > mmcblk0: mmc0: SU04G 3.69 GiB > mmcblk0: p1 p2 > > Patch 1/3 was previously part of 2/3 but as it deals with a unrelated > issue it's here broken out to a separate patch. Patch 3/3 have once been > posted outside this series bit after comments from Wolfram it's brought > back as it now deepens on 2/3. > > Most changes in this series are based on similar work from Masaharu > Hayakawa but for this version all changes now differ quiet a lot from > his work. All mails sent to him bounce with a "Undelivered Mail > Returned to Sender" error. I therefor felt the need to claim authorship > as I don't want to post blame of my (potential) mistakes on some else. > > Niklas Söderlund (3): > mmc: tmio: enable module clock before resetting when resuming > mmc: tmio: fix reset operation > mmc: renesas_sdhi: add initial setting of interrupt mask register > > drivers/mmc/host/renesas_sdhi_core.c | 4 > drivers/mmc/host/tmio_mmc.h | 1 + > drivers/mmc/host/tmio_mmc_core.c | 26 +++--- > 3 files changed, 20 insertions(+), 11 deletions(-) > > -- > 2.19.1 > Applied for next, thanks! I noticed there were a minor comment from Yamada-san, however I decided to pick this as is and leave further improvements to be made on top. Kind regards Uffe
Re: [PATCH v3 0/3] mmc: tmio: fix reset operation
All patches tested on M3N, H2, and E2. Tested-by: Wolfram Sang signature.asc Description: PGP signature
Re: [PATCH v3 0/3] mmc: tmio: fix reset operation
So, we agreed on this series during our Renesas SDHI hackathon. Comments / testing from Yamada-san would be very welcome because we really don't want to cause regressions on his hardware. signature.asc Description: PGP signature
