| Message ID | 1516206496-16612-12-git-send-email-yamada.masahiro@socionext.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
2018-01-18 1:28 GMT+09:00 Masahiro Yamada <yamada.masahiro@socionext.com>: > This property is equivalent to "disable-wp" defined in > Documentation/devicetree/bindings/mmc/tmio_mmc.txt This is mistake. "disable-wp" is defined in Documentation/devicetree/bindings/mmc/mmc.txt > The TMIO MMC core calls mmc_of_parse(), and it sets > MMC_CAP2_NO_WRITE_PROTECT if "disable-wp" property is present. > > We do not need a vendor-specific property to do the same thing. > > Let's remove the description from the dt-binding to prevent new boards > from using it. > > I am keeping the driver code for existing DT files, but added > comments that this is deprecated. > > Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com> > --- > > Changes in v3: > - newly added > > Changes in v2: None > > Documentation/devicetree/bindings/mmc/tmio_mmc.txt | 1 - > drivers/mmc/host/tmio_mmc_core.c | 5 +++++ > 2 files changed, 5 insertions(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/mmc/tmio_mmc.txt b/Documentation/devicetree/bindings/mmc/tmio_mmc.txt > index 3c67624..cb7d40e 100644 > --- a/Documentation/devicetree/bindings/mmc/tmio_mmc.txt > +++ b/Documentation/devicetree/bindings/mmc/tmio_mmc.txt > @@ -49,7 +49,6 @@ Required properties: > 2: R7S72100 > > Optional properties: > -- toshiba,mmc-wrprotect-disable: write-protect detection is unavailable > - pinctrl-names: should be "default", "state_uhs" > - pinctrl-0: should contain default/high speed pin ctrl > - pinctrl-1: should contain uhs mode pin ctrl > diff --git a/drivers/mmc/host/tmio_mmc_core.c b/drivers/mmc/host/tmio_mmc_core.c > index 7ad3433c..f30ac69 100644 > --- a/drivers/mmc/host/tmio_mmc_core.c > +++ b/drivers/mmc/host/tmio_mmc_core.c > @@ -1134,6 +1134,11 @@ static void tmio_mmc_of_parse(struct platform_device *pdev, > if (!np) > return; > > + /* > + * DEPRECATED: > + * For new platforms, please use "disable-wp" instead of > + * "toshiba,mmc-wrprotect-disable" > + */ > if (of_get_property(np, "toshiba,mmc-wrprotect-disable", NULL)) > mmc->caps2 |= MMC_CAP2_NO_WRITE_PROTECT; > } > -- > 2.7.4 > > -- > To unsubscribe from this list: send the line "unsubscribe devicetree" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html
On Thu, Jan 18, 2018 at 10:58:36AM +0900, Masahiro Yamada wrote: > 2018-01-18 1:28 GMT+09:00 Masahiro Yamada <yamada.masahiro@socionext.com>: > > This property is equivalent to "disable-wp" defined in > > Documentation/devicetree/bindings/mmc/tmio_mmc.txt > > This is mistake. > > "disable-wp" is defined in > > Documentation/devicetree/bindings/mmc/mmc.txt > > > > > > The TMIO MMC core calls mmc_of_parse(), and it sets > > MMC_CAP2_NO_WRITE_PROTECT if "disable-wp" property is present. > > > > We do not need a vendor-specific property to do the same thing. > > > > Let's remove the description from the dt-binding to prevent new boards > > from using it. > > > > I am keeping the driver code for existing DT files, but added > > comments that this is deprecated. > > > > Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com> > > --- > > > > Changes in v3: > > - newly added > > > > Changes in v2: None > > > > Documentation/devicetree/bindings/mmc/tmio_mmc.txt | 1 - Other than the above, Acked-by: Rob Herring <robh@kernel.org> > > drivers/mmc/host/tmio_mmc_core.c | 5 +++++ > > 2 files changed, 5 insertions(+), 1 deletion(-) -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Thu, Jan 18, 2018 at 10:58:36AM +0900, Masahiro Yamada wrote: > 2018-01-18 1:28 GMT+09:00 Masahiro Yamada <yamada.masahiro@socionext.com>: > > This property is equivalent to "disable-wp" defined in > > Documentation/devicetree/bindings/mmc/tmio_mmc.txt > > This is mistake. > > "disable-wp" is defined in > > Documentation/devicetree/bindings/mmc/mmc.txt With that fixed: Reviewed-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
2018-02-08 4:32 GMT+09:00 Wolfram Sang <wsa@the-dreams.de>: > On Thu, Jan 18, 2018 at 10:58:36AM +0900, Masahiro Yamada wrote: >> 2018-01-18 1:28 GMT+09:00 Masahiro Yamada <yamada.masahiro@socionext.com>: >> > This property is equivalent to "disable-wp" defined in >> > Documentation/devicetree/bindings/mmc/tmio_mmc.txt >> >> This is mistake. >> >> "disable-wp" is defined in >> >> Documentation/devicetree/bindings/mmc/mmc.txt > > With that fixed: > > Reviewed-by: Wolfram Sang <wsa+renesas@sang-engineering.com> > BTW, the subject of the applied version is wrong. In my original post: https://patchwork.kernel.org/patch/10170007/ "toshiba,mmc-wrprotect-disable" No space after the vendor prefix. But, applied commit (788778b0d21a6d5cd5) I see a space, like "toshiba, mmc-wrprotect-disable" I think this is a bug of patchwork. I have seen various bugs that break the patch format. I filed some bug reports for patchwork, but not sure what happened.
diff --git a/Documentation/devicetree/bindings/mmc/tmio_mmc.txt b/Documentation/devicetree/bindings/mmc/tmio_mmc.txt index 3c67624..cb7d40e 100644 --- a/Documentation/devicetree/bindings/mmc/tmio_mmc.txt +++ b/Documentation/devicetree/bindings/mmc/tmio_mmc.txt @@ -49,7 +49,6 @@ Required properties: 2: R7S72100 Optional properties: -- toshiba,mmc-wrprotect-disable: write-protect detection is unavailable - pinctrl-names: should be "default", "state_uhs" - pinctrl-0: should contain default/high speed pin ctrl - pinctrl-1: should contain uhs mode pin ctrl diff --git a/drivers/mmc/host/tmio_mmc_core.c b/drivers/mmc/host/tmio_mmc_core.c index 7ad3433c..f30ac69 100644 --- a/drivers/mmc/host/tmio_mmc_core.c +++ b/drivers/mmc/host/tmio_mmc_core.c @@ -1134,6 +1134,11 @@ static void tmio_mmc_of_parse(struct platform_device *pdev, if (!np) return; + /* + * DEPRECATED: + * For new platforms, please use "disable-wp" instead of + * "toshiba,mmc-wrprotect-disable" + */ if (of_get_property(np, "toshiba,mmc-wrprotect-disable", NULL)) mmc->caps2 |= MMC_CAP2_NO_WRITE_PROTECT; }
This property is equivalent to "disable-wp" defined in Documentation/devicetree/bindings/mmc/tmio_mmc.txt The TMIO MMC core calls mmc_of_parse(), and it sets MMC_CAP2_NO_WRITE_PROTECT if "disable-wp" property is present. We do not need a vendor-specific property to do the same thing. Let's remove the description from the dt-binding to prevent new boards from using it. I am keeping the driver code for existing DT files, but added comments that this is deprecated. Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com> --- Changes in v3: - newly added Changes in v2: None Documentation/devicetree/bindings/mmc/tmio_mmc.txt | 1 - drivers/mmc/host/tmio_mmc_core.c | 5 +++++ 2 files changed, 5 insertions(+), 1 deletion(-)