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

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

 



On 4 June 2018 at 09:32, Wolfram Sang <wsa@xxxxxxxxxxxxx> wrote:
> 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?

Yes!

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



[Index of Archives]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux