diff mbox series

mmc: sdhci-acpi: Disable write protect detection on Toshiba Encore 2 WT8-B

Message ID 20210503092157.5689-1-hdegoede@redhat.com
State Superseded
Headers show
Series mmc: sdhci-acpi: Disable write protect detection on Toshiba Encore 2 WT8-B | expand

Commit Message

Hans de Goede May 3, 2021, 9:21 a.m. UTC
On the Toshiba Encore 2 WT8-B the  microSD slot always reports the card
being write-protected even though microSD cards do not have a write-protect
switch at all.

Add a new DMI_QUIRK_SD_NO_WRITE_PROTECT quirk entry to sdhci-acpi.c's
DMI quirk table for this.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
---
 drivers/mmc/host/sdhci-acpi.c | 11 +++++++++++
 1 file changed, 11 insertions(+)

Comments

Ulf Hansson May 11, 2021, 10:56 a.m. UTC | #1
On Mon, 3 May 2021 at 11:22, Hans de Goede <hdegoede@redhat.com> wrote:
>

> On the Toshiba Encore 2 WT8-B the  microSD slot always reports the card

> being write-protected even though microSD cards do not have a write-protect

> switch at all.

>

> Add a new DMI_QUIRK_SD_NO_WRITE_PROTECT quirk entry to sdhci-acpi.c's

> DMI quirk table for this.

>

> Signed-off-by: Hans de Goede <hdegoede@redhat.com>


Applied for next, thanks!

Should we tag this for stable?

Kind regards
Uffe


> ---

>  drivers/mmc/host/sdhci-acpi.c | 11 +++++++++++

>  1 file changed, 11 insertions(+)

>

> diff --git a/drivers/mmc/host/sdhci-acpi.c b/drivers/mmc/host/sdhci-acpi.c

> index b6574e7fd26b..9e4358d7a0a6 100644

> --- a/drivers/mmc/host/sdhci-acpi.c

> +++ b/drivers/mmc/host/sdhci-acpi.c

> @@ -820,6 +820,17 @@ static const struct dmi_system_id sdhci_acpi_quirks[] = {

>                 },

>                 .driver_data = (void *)DMI_QUIRK_SD_NO_WRITE_PROTECT,

>         },

> +       {

> +               /*

> +                * The Toshiba WT8-B's microSD slot always reports the card being

> +                * write-protected.

> +                */

> +               .matches = {

> +                       DMI_MATCH(DMI_SYS_VENDOR, "TOSHIBA"),

> +                       DMI_MATCH(DMI_PRODUCT_NAME, "TOSHIBA ENCORE 2 WT8-B"),

> +               },

> +               .driver_data = (void *)DMI_QUIRK_SD_NO_WRITE_PROTECT,

> +       },

>         {} /* Terminating entry */

>  };

>

> --

> 2.31.1

>
Hans de Goede May 11, 2021, 10:58 a.m. UTC | #2
Hi,

On 5/11/21 12:56 PM, Ulf Hansson wrote:
> On Mon, 3 May 2021 at 11:22, Hans de Goede <hdegoede@redhat.com> wrote:

>>

>> On the Toshiba Encore 2 WT8-B the  microSD slot always reports the card

>> being write-protected even though microSD cards do not have a write-protect

>> switch at all.

>>

>> Add a new DMI_QUIRK_SD_NO_WRITE_PROTECT quirk entry to sdhci-acpi.c's

>> DMI quirk table for this.

>>

>> Signed-off-by: Hans de Goede <hdegoede@redhat.com>

> 

> Applied for next, thanks!

> 

> Should we tag this for stable?


Yes please.

Thanks & Regards,

Hans




>> ---

>>  drivers/mmc/host/sdhci-acpi.c | 11 +++++++++++

>>  1 file changed, 11 insertions(+)

>>

>> diff --git a/drivers/mmc/host/sdhci-acpi.c b/drivers/mmc/host/sdhci-acpi.c

>> index b6574e7fd26b..9e4358d7a0a6 100644

>> --- a/drivers/mmc/host/sdhci-acpi.c

>> +++ b/drivers/mmc/host/sdhci-acpi.c

>> @@ -820,6 +820,17 @@ static const struct dmi_system_id sdhci_acpi_quirks[] = {

>>                 },

>>                 .driver_data = (void *)DMI_QUIRK_SD_NO_WRITE_PROTECT,

>>         },

>> +       {

>> +               /*

>> +                * The Toshiba WT8-B's microSD slot always reports the card being

>> +                * write-protected.

>> +                */

>> +               .matches = {

>> +                       DMI_MATCH(DMI_SYS_VENDOR, "TOSHIBA"),

>> +                       DMI_MATCH(DMI_PRODUCT_NAME, "TOSHIBA ENCORE 2 WT8-B"),

>> +               },

>> +               .driver_data = (void *)DMI_QUIRK_SD_NO_WRITE_PROTECT,

>> +       },

>>         {} /* Terminating entry */

>>  };

>>

>> --

>> 2.31.1

>>

>
Ulf Hansson May 11, 2021, 11:02 a.m. UTC | #3
On Tue, 11 May 2021 at 12:58, Hans de Goede <hdegoede@redhat.com> wrote:
>

> Hi,

>

> On 5/11/21 12:56 PM, Ulf Hansson wrote:

> > On Mon, 3 May 2021 at 11:22, Hans de Goede <hdegoede@redhat.com> wrote:

> >>

> >> On the Toshiba Encore 2 WT8-B the  microSD slot always reports the card

> >> being write-protected even though microSD cards do not have a write-protect

> >> switch at all.

> >>

> >> Add a new DMI_QUIRK_SD_NO_WRITE_PROTECT quirk entry to sdhci-acpi.c's

> >> DMI quirk table for this.

> >>

> >> Signed-off-by: Hans de Goede <hdegoede@redhat.com>

> >

> > Applied for next, thanks!

> >

> > Should we tag this for stable?

>

> Yes please.


Done, thanks!

Kind regards
Uffe

>

> Thanks & Regards,

>

> Hans

>

>

>

>

> >> ---

> >>  drivers/mmc/host/sdhci-acpi.c | 11 +++++++++++

> >>  1 file changed, 11 insertions(+)

> >>

> >> diff --git a/drivers/mmc/host/sdhci-acpi.c b/drivers/mmc/host/sdhci-acpi.c

> >> index b6574e7fd26b..9e4358d7a0a6 100644

> >> --- a/drivers/mmc/host/sdhci-acpi.c

> >> +++ b/drivers/mmc/host/sdhci-acpi.c

> >> @@ -820,6 +820,17 @@ static const struct dmi_system_id sdhci_acpi_quirks[] = {

> >>                 },

> >>                 .driver_data = (void *)DMI_QUIRK_SD_NO_WRITE_PROTECT,

> >>         },

> >> +       {

> >> +               /*

> >> +                * The Toshiba WT8-B's microSD slot always reports the card being

> >> +                * write-protected.

> >> +                */

> >> +               .matches = {

> >> +                       DMI_MATCH(DMI_SYS_VENDOR, "TOSHIBA"),

> >> +                       DMI_MATCH(DMI_PRODUCT_NAME, "TOSHIBA ENCORE 2 WT8-B"),

> >> +               },

> >> +               .driver_data = (void *)DMI_QUIRK_SD_NO_WRITE_PROTECT,

> >> +       },

> >>         {} /* Terminating entry */

> >>  };

> >>

> >> --

> >> 2.31.1

> >>

> >

>
diff mbox series

Patch

diff --git a/drivers/mmc/host/sdhci-acpi.c b/drivers/mmc/host/sdhci-acpi.c
index b6574e7fd26b..9e4358d7a0a6 100644
--- a/drivers/mmc/host/sdhci-acpi.c
+++ b/drivers/mmc/host/sdhci-acpi.c
@@ -820,6 +820,17 @@  static const struct dmi_system_id sdhci_acpi_quirks[] = {
 		},
 		.driver_data = (void *)DMI_QUIRK_SD_NO_WRITE_PROTECT,
 	},
+	{
+		/*
+		 * The Toshiba WT8-B's microSD slot always reports the card being
+		 * write-protected.
+		 */
+		.matches = {
+			DMI_MATCH(DMI_SYS_VENDOR, "TOSHIBA"),
+			DMI_MATCH(DMI_PRODUCT_NAME, "TOSHIBA ENCORE 2 WT8-B"),
+		},
+		.driver_data = (void *)DMI_QUIRK_SD_NO_WRITE_PROTECT,
+	},
 	{} /* Terminating entry */
 };