Re: [PATCH v2 0/9] r8a7790: add UHS-I (SDR50) support to Lager
On Tue, Apr 05, 2016 at 01:19:20PM +0200, Wolfram Sang wrote: > > > > Do you want me to resend or could you fix it before applying? > > > > Patch 1->7 is applied for next (amended patch7), thanks! > > Thanks! Simon, can you pick up patches 8+9 now? Thanks, done!
Re: [PATCH v2 0/9] r8a7790: add UHS-I (SDR50) support to Lager
> > Do you want me to resend or could you fix it before applying? > > Patch 1->7 is applied for next (amended patch7), thanks! Thanks! Simon, can you pick up patches 8+9 now? Next series coming soon... ;) signature.asc Description: PGP signature
Re: [PATCH v2 0/9] r8a7790: add UHS-I (SDR50) support to Lager
On 4 April 2016 at 17:21, Wolfram Sang wrote: > >> The documentation of mmc is quite poor. There are both code and >> specification that needs to be understand, so some more help from >> kernel doc could clearly help. > > I agree. > >> Anyway, I noticed that patch 6 helps to improve some documentation, >> that's great - thanks! > > You're welcome. It wasn't too much work and maybe it helps someone with > debugging. I found some bug reports on the web which sounded somewhat > similar. > >> I only had one minor comment on patch 7, the rest looks good to me! > > Do you want me to resend or could you fix it before applying? Patch 1->7 is applied for next (amended patch7), thanks! Kind regards Uffe
Re: [PATCH v2 0/9] r8a7790: add UHS-I (SDR50) support to Lager
> The documentation of mmc is quite poor. There are both code and > specification that needs to be understand, so some more help from > kernel doc could clearly help. I agree. > Anyway, I noticed that patch 6 helps to improve some documentation, > that's great - thanks! You're welcome. It wasn't too much work and maybe it helps someone with debugging. I found some bug reports on the web which sounded somewhat similar. > I only had one minor comment on patch 7, the rest looks good to me! Do you want me to resend or could you fix it before applying? Thanks for the review! Wolfram signature.asc Description: PGP signature
Re: [PATCH v2 0/9] r8a7790: add UHS-I (SDR50) support to Lager
On 1 April 2016 at 17:44, Wolfram Sang wrote: > So, here is v2 of the series adding SDR50 support to the Renesas Lager board. > Change since v1: > > * rebased to renesas-drivers based on v4.6-rc1 > * dropped pinctrl patch which was already picked up > * directly populate start_signal_voltage_switch() in mmc_ops > instead of using a wrapper function > * use generic DT pinctrl bindings instead of vendor specific ones > (e.g. "groups" instead of "renesas,groups") > > The RFC series worked fine with my Transcend card, but failed to switch > voltages on a SanDisk and Samsung card. This bug hunting resulted in patches > 4-6 newly added: The clock really has to be disabled on ios->clock == 0, > setting frequency to 0Hz doesn't work. I wonder if this isn't true for some > more controllers? Is this known already? The documentation of mmc is quite poor. There are both code and specification that needs to be understand, so some more help from kernel doc could clearly help. Anyway, I noticed that patch 6 helps to improve some documentation, that's great - thanks! > > My "copy-large-files-around"-setup showed now 30MB/s while it had 19MB/s > without SDR50. Frankly, I hoped for a little more, but let's start with this > initial support and do the tuning incrementally I'd say. > > Patches 1-7 should go via Ulf. After those patches went in, Simon can take 8+9 > to tie it all together. > > Please comment, test, apply... I only had one minor comment on patch 7, the rest looks good to me! [...] Kind regards Uffe