Re: [PATCH] mmc: renesas_sdhi: really fix WP logic regressions

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Jun 04, 2018 at 08:44:42AM +0200, Ulf Hansson wrote:
> On 1 June 2018 at 13:00, Wolfram Sang <wsa+renesas@xxxxxxxxxxxxxxxxxxxx> wrote:
> > This reverts commit e060d376cc61 ("mmc: renesas_sdhi: fix WP detection")
> > and adds some code to really fix the regressions.
> >
> > It was missed so far that Renesas R-Car instantiations of SDHI chose to
> > disable internal WP and used the existence of "wp-gpios" to en/disable
> > WP at all.
> >
> > With the first refactoring by Yamada-san with commit  ("mmc:
> > renesas_sdhi: use MMC_CAP2_NO_WRITE_PROTECT instead of TMIO own flag"),
> > WP was always disabled even when GPIOs were present. With Wolfram's
> > first fix which gets now reverted, GPIOs were honored. But when not
> > available, the fallback was to internal WP and not to disabled WP. This
> > caused wrong WP status on uSD card slots.
> >
> > Restore the old behaviour now. By default, WP is disabled. When a GPIO
> > is found, the GPIO re-enables WP. We will think about possible better
> > ways to handle this in the future.
> >
> > Tested on a previously regressing Renesas Lager board (H2) and a still
> > working Renesas Salvator-X board (M3-W).
> >
> > Reported-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@xxxxxxxxxxx>
> > Signed-off-by: Wolfram Sang <wsa+renesas@xxxxxxxxxxxxxxxxxxxx>
> 
> Unfortunate this didn't make it for 4.17, however I have applied it for 4.18.

With stable attached?

Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [Linux Samsung SOC]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux