devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v3 00/16] mmc: tmio: another batch of TMIO MMC fixes and cleanups
@ 2018-01-17 16:28 Masahiro Yamada
       [not found] ` <1516206496-16612-1-git-send-email-yamada.masahiro-uWyLwvC0a2jby3iVrkZq2A@public.gmane.org>
                   ` (2 more replies)
  0 siblings, 3 replies; 19+ messages in thread
From: Masahiro Yamada @ 2018-01-17 16:28 UTC (permalink / raw)
  To: linux-mmc, Wolfram Sang
  Cc: Ulf Magnusson, Geert Uytterhoeven, Simon Horman,
	Yoshihiro Shimoda, linux-renesas-soc, Masahiro Yamada,
	devicetree, linux-sh, Yoshinori Sato, Rob Herring, linux-kernel,
	Rob Herring, Lee Jones, Rich Felker, Mark Rutland, Ulf Hansson


In the previous batch (https://lkml.org/lkml/2017/11/24/428)
I sent 22 patches for TMIO MMC.

14 patches were applied, the rest is under discussion.

This series consists of 16 patches.

Patch 1-4:  repost from v2.
            (Wolfram gave Reviewed by, so I moved them to the head)

Patch 5-11: new patches  (cleanups of WP flag)

Patch 12: updated per Wolfram

Patch 13: repost  (almost reviewed)

Patch 14: updated

Patch 15: new

Patch 16: repost (almost reviewed)


Masahiro Yamada (16):
  mmc: tmio: ioremap memory resource in tmio_mmc_host_alloc()
  mmc: tmio: move clk_enable/disable out of tmio_mmc_host_probe()
  mmc: tmio: move {tmio_}mmc_of_parse() to tmio_mmc_host_alloc()
  mmc: tmio: remove dma_ops from tmio_mmc_host_probe() argument
  mmc: slot-gpio: add a helper to check capability of GPIO WP detection
  mmc: tmio: refactor .get_ro hook
  mmc: renesas_sdhi: use MMC_CAP2_NO_WRITE_PROTECT instead of TMIO own
    flag
  sh: kfr2r09: use MMC_CAP2_NO_WRITE_PROTECT instead of TMIO own flag
  mmc: tmio: use MMC_CAP2_NO_WRITE_PROTECT instead of TMIO own flag
  mmc: tmio: remove TMIO_MMC_WRPROTECT_DISABLE
  mmc: tmio: deprecate "toshiba,mmc-wrprotect-disable" DT property
  mmc: tmio: support IP-builtin card detection logic
  mmc: tmio: fix never-detected card insertion bug
  mmc: tmio: move TMIO_MASK_{READOP,WRITEOP} handling to correct place
  mmc: tmio: clear force_pio flag before starting data transfer
  mmc: tmio: remove useless TMIO_MASK_CMD handling in
    tmio_mmc_host_probe()

 Documentation/devicetree/bindings/mmc/tmio_mmc.txt |   1 -
 arch/sh/boards/mach-kfr2r09/setup.c                |   2 +-
 drivers/mmc/core/slot-gpio.c                       |   8 ++
 drivers/mmc/host/renesas_sdhi_core.c               |  18 ++-
 drivers/mmc/host/renesas_sdhi_internal_dmac.c      |  12 +-
 drivers/mmc/host/renesas_sdhi_sys_dmac.c           |  20 ++-
 drivers/mmc/host/tmio_mmc.c                        |  11 +-
 drivers/mmc/host/tmio_mmc.h                        |   7 +-
 drivers/mmc/host/tmio_mmc_core.c                   | 139 +++++++++++----------
 include/linux/mfd/tmio.h                           |   1 -
 include/linux/mmc/slot-gpio.h                      |   1 +
 11 files changed, 120 insertions(+), 100 deletions(-)

-- 
2.7.4

^ permalink raw reply	[flat|nested] 19+ messages in thread

* [PATCH v3 11/16] mmc: tmio: deprecate "toshiba,mmc-wrprotect-disable" DT property
       [not found] ` <1516206496-16612-1-git-send-email-yamada.masahiro-uWyLwvC0a2jby3iVrkZq2A@public.gmane.org>
@ 2018-01-17 16:28   ` Masahiro Yamada
  2018-01-18  1:58     ` Masahiro Yamada
  0 siblings, 1 reply; 19+ messages in thread
From: Masahiro Yamada @ 2018-01-17 16:28 UTC (permalink / raw)
  To: linux-mmc-u79uwXL29TY76Z2rM5mHXA, Wolfram Sang
  Cc: Ulf Magnusson, Geert Uytterhoeven, Simon Horman,
	Yoshihiro Shimoda, linux-renesas-soc-u79uwXL29TY76Z2rM5mHXA,
	Masahiro Yamada, devicetree-u79uwXL29TY76Z2rM5mHXA, Rob Herring,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA, Rob Herring, Mark Rutland,
	Ulf Hansson

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-uWyLwvC0a2jby3iVrkZq2A@public.gmane.org>
---

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-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply related	[flat|nested] 19+ messages in thread

* Re: [PATCH v3 11/16] mmc: tmio: deprecate "toshiba,mmc-wrprotect-disable" DT property
  2018-01-17 16:28   ` [PATCH v3 11/16] mmc: tmio: deprecate "toshiba,mmc-wrprotect-disable" DT property Masahiro Yamada
@ 2018-01-18  1:58     ` Masahiro Yamada
       [not found]       ` <CAK7LNAQUJw-fjuJO68Kvskn8BRYW3rZj4YMtbo-Oxfv7c=6Ggg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
  0 siblings, 1 reply; 19+ messages in thread
From: Masahiro Yamada @ 2018-01-18  1:58 UTC (permalink / raw)
  To: linux-mmc, Wolfram Sang
  Cc: Ulf Magnusson, Geert Uytterhoeven, Simon Horman,
	Yoshihiro Shimoda, Linux-Renesas, Masahiro Yamada, devicetree,
	Rob Herring, Linux Kernel Mailing List, Rob Herring,
	Mark Rutland, Ulf Hansson

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



-- 
Best Regards
Masahiro Yamada

^ permalink raw reply	[flat|nested] 19+ messages in thread

* Re: [PATCH v3 00/16] mmc: tmio: another batch of TMIO MMC fixes and cleanups
  2018-01-17 16:28 [PATCH v3 00/16] mmc: tmio: another batch of TMIO MMC fixes and cleanups Masahiro Yamada
       [not found] ` <1516206496-16612-1-git-send-email-yamada.masahiro-uWyLwvC0a2jby3iVrkZq2A@public.gmane.org>
@ 2018-01-18  8:13 ` Ulf Hansson
  2018-02-07 19:11   ` Wolfram Sang
  2018-03-04 22:42 ` Wolfram Sang
  2 siblings, 1 reply; 19+ messages in thread
From: Ulf Hansson @ 2018-01-18  8:13 UTC (permalink / raw)
  To: Masahiro Yamada
  Cc: linux-mmc, Wolfram Sang, Ulf Magnusson, Geert Uytterhoeven,
	Simon Horman, Yoshihiro Shimoda, Linux-Renesas, devicetree,
	Linux-sh list, Yoshinori Sato, Rob Herring,
	Linux Kernel Mailing List, Rob Herring, Lee Jones, Rich Felker,
	Mark Rutland

On 17 January 2018 at 17:28, Masahiro Yamada
<yamada.masahiro@socionext.com> wrote:
>
> In the previous batch (https://lkml.org/lkml/2017/11/24/428)
> I sent 22 patches for TMIO MMC.
>
> 14 patches were applied, the rest is under discussion.
>
> This series consists of 16 patches.
>
> Patch 1-4:  repost from v2.
>             (Wolfram gave Reviewed by, so I moved them to the head)

Applied for next.

>
> Patch 5-11: new patches  (cleanups of WP flag)

I picked patch5 and patch6, as those seemed trivial. Unless there is a
rc9, let's aim for 4.17 for the rest.

[...]

Kind regards
Uffe

^ permalink raw reply	[flat|nested] 19+ messages in thread

* Re: [PATCH v3 11/16] mmc: tmio: deprecate "toshiba,mmc-wrprotect-disable" DT property
       [not found]       ` <CAK7LNAQUJw-fjuJO68Kvskn8BRYW3rZj4YMtbo-Oxfv7c=6Ggg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
@ 2018-01-29 17:18         ` Rob Herring
  2018-02-07 19:32         ` Wolfram Sang
  1 sibling, 0 replies; 19+ messages in thread
From: Rob Herring @ 2018-01-29 17:18 UTC (permalink / raw)
  To: Masahiro Yamada
  Cc: linux-mmc, Wolfram Sang, Ulf Magnusson, Geert Uytterhoeven,
	Simon Horman, Yoshihiro Shimoda, Linux-Renesas,
	devicetree-u79uwXL29TY76Z2rM5mHXA, Linux Kernel Mailing List,
	Mark Rutland, Ulf Hansson

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-uWyLwvC0a2jby3iVrkZq2A@public.gmane.org>:
> > 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-uWyLwvC0a2jby3iVrkZq2A@public.gmane.org>
> > ---
> >
> > 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-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.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 devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 19+ messages in thread

* Re: [PATCH v3 00/16] mmc: tmio: another batch of TMIO MMC fixes and cleanups
  2018-01-18  8:13 ` [PATCH v3 00/16] mmc: tmio: another batch of TMIO MMC fixes and cleanups Ulf Hansson
@ 2018-02-07 19:11   ` Wolfram Sang
  2018-02-14  9:36     ` Ulf Hansson
  0 siblings, 1 reply; 19+ messages in thread
From: Wolfram Sang @ 2018-02-07 19:11 UTC (permalink / raw)
  To: Ulf Hansson
  Cc: Masahiro Yamada, linux-mmc, Wolfram Sang, Ulf Magnusson,
	Geert Uytterhoeven, Simon Horman, Yoshihiro Shimoda,
	Linux-Renesas, devicetree, Linux-sh list, Yoshinori Sato,
	Rob Herring, Linux Kernel Mailing List, Rob Herring, Lee Jones,
	Rich Felker, Mark Rutland

[-- Attachment #1: Type: text/plain, Size: 420 bytes --]


> I picked patch5 and patch6, as those seemed trivial. Unless there is a
> rc9, let's aim for 4.17 for the rest.

I can't find the branch where you picked these patches? I wanted to use
it to apply the rest of the patches. They don't seem to fit on v4.15
or current linus/master. Can you share this branch?

I will do reviewing now "visually". But please wait until I tested them
before applying.

Thanks,

   Wolfram


[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply	[flat|nested] 19+ messages in thread

* Re: [PATCH v3 11/16] mmc: tmio: deprecate "toshiba,mmc-wrprotect-disable" DT property
       [not found]       ` <CAK7LNAQUJw-fjuJO68Kvskn8BRYW3rZj4YMtbo-Oxfv7c=6Ggg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
  2018-01-29 17:18         ` Rob Herring
@ 2018-02-07 19:32         ` Wolfram Sang
  2018-03-18  2:46           ` Masahiro Yamada
  1 sibling, 1 reply; 19+ messages in thread
From: Wolfram Sang @ 2018-02-07 19:32 UTC (permalink / raw)
  To: Masahiro Yamada
  Cc: linux-mmc, Wolfram Sang, Ulf Magnusson, Geert Uytterhoeven,
	Simon Horman, Yoshihiro Shimoda, Linux-Renesas,
	devicetree-u79uwXL29TY76Z2rM5mHXA, Rob Herring,
	Linux Kernel Mailing List, Rob Herring, Mark Rutland,
	Ulf Hansson

[-- Attachment #1: Type: text/plain, Size: 512 bytes --]

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-uWyLwvC0a2jby3iVrkZq2A@public.gmane.org>:
> > 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-jBu1N2QxHDJrcw3mvpCnnVaTQe2KTcn/@public.gmane.org>


[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply	[flat|nested] 19+ messages in thread

* Re: [PATCH v3 00/16] mmc: tmio: another batch of TMIO MMC fixes and cleanups
  2018-02-07 19:11   ` Wolfram Sang
@ 2018-02-14  9:36     ` Ulf Hansson
  2018-02-14  9:43       ` Masahiro Yamada
       [not found]       ` <CAPDyKFoxp7p0atQTV=PoQ7Bwt0fOs2aEq1KJHPtrq+zA3eFYgw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
  0 siblings, 2 replies; 19+ messages in thread
From: Ulf Hansson @ 2018-02-14  9:36 UTC (permalink / raw)
  To: Wolfram Sang
  Cc: Masahiro Yamada, linux-mmc, Wolfram Sang, Ulf Magnusson,
	Geert Uytterhoeven, Simon Horman, Yoshihiro Shimoda,
	Linux-Renesas, devicetree, Linux-sh list, Yoshinori Sato,
	Rob Herring, Linux Kernel Mailing List, Rob Herring, Lee Jones,
	Rich Felker, Mark Rutland

On 7 February 2018 at 20:11, Wolfram Sang <wsa@the-dreams.de> wrote:
>
>> I picked patch5 and patch6, as those seemed trivial. Unless there is a
>> rc9, let's aim for 4.17 for the rest.
>
> I can't find the branch where you picked these patches? I wanted to use
> it to apply the rest of the patches. They don't seem to fit on v4.15
> or current linus/master. Can you share this branch?

You need to check again. :-)

Patches up and to patch6 is already in Linus' tree v.16rc1, those I
carried for quite a while on my next branch before sending them in the
PR to Linus.

>
> I will do reviewing now "visually". But please wait until I tested them
> before applying.

I am eager to apply those patches you already have reviewed (to
patch14), as those mostly seemed rather trivial and should be nice
cleanups.

Do you mind me going ahead, then you can continue and review/test the rest?

Kind regards
Uffe

^ permalink raw reply	[flat|nested] 19+ messages in thread

* Re: [PATCH v3 00/16] mmc: tmio: another batch of TMIO MMC fixes and cleanups
  2018-02-14  9:36     ` Ulf Hansson
@ 2018-02-14  9:43       ` Masahiro Yamada
  2018-02-14  9:46         ` Ulf Hansson
       [not found]       ` <CAPDyKFoxp7p0atQTV=PoQ7Bwt0fOs2aEq1KJHPtrq+zA3eFYgw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
  1 sibling, 1 reply; 19+ messages in thread
From: Masahiro Yamada @ 2018-02-14  9:43 UTC (permalink / raw)
  To: Ulf Hansson
  Cc: Wolfram Sang, linux-mmc, Wolfram Sang, Ulf Magnusson,
	Geert Uytterhoeven, Simon Horman, Yoshihiro Shimoda,
	Linux-Renesas, devicetree, Linux-sh list, Yoshinori Sato,
	Rob Herring, Linux Kernel Mailing List, Rob Herring, Lee Jones,
	Rich Felker, Mark Rutland

Hi Ulf,


2018-02-14 18:36 GMT+09:00 Ulf Hansson <ulf.hansson@linaro.org>:
> On 7 February 2018 at 20:11, Wolfram Sang <wsa@the-dreams.de> wrote:
>>
>>> I picked patch5 and patch6, as those seemed trivial. Unless there is a
>>> rc9, let's aim for 4.17 for the rest.
>>
>> I can't find the branch where you picked these patches? I wanted to use
>> it to apply the rest of the patches. They don't seem to fit on v4.15
>> or current linus/master. Can you share this branch?
>
> You need to check again. :-)
>
> Patches up and to patch6 is already in Linus' tree v.16rc1, those I
> carried for quite a while on my next branch before sending them in the
> PR to Linus.
>
>>
>> I will do reviewing now "visually". But please wait until I tested them
>> before applying.
>
> I am eager to apply those patches you already have reviewed (to
> patch14), as those mostly seemed rather trivial and should be nice
> cleanups.
>
> Do you mind me going ahead, then you can continue and review/test the rest?
>


If you move forward, please fix-up the commit log of the patch 11.
(https://patchwork.kernel.org/patch/10170007/)


Documentation/devicetree/bindings/mmc/tmio_mmc.txt

  ->

Documentation/devicetree/bindings/mmc/mmc.txt



Thanks.


-- 
Best Regards
Masahiro Yamada

^ permalink raw reply	[flat|nested] 19+ messages in thread

* Re: [PATCH v3 00/16] mmc: tmio: another batch of TMIO MMC fixes and cleanups
  2018-02-14  9:43       ` Masahiro Yamada
@ 2018-02-14  9:46         ` Ulf Hansson
  0 siblings, 0 replies; 19+ messages in thread
From: Ulf Hansson @ 2018-02-14  9:46 UTC (permalink / raw)
  To: Masahiro Yamada
  Cc: Wolfram Sang, linux-mmc, Wolfram Sang, Ulf Magnusson,
	Geert Uytterhoeven, Simon Horman, Yoshihiro Shimoda,
	Linux-Renesas, devicetree, Linux-sh list, Yoshinori Sato,
	Rob Herring, Linux Kernel Mailing List, Rob Herring, Lee Jones,
	Rich Felker, Mark Rutland

On 14 February 2018 at 10:43, Masahiro Yamada
<yamada.masahiro@socionext.com> wrote:
> Hi Ulf,
>
>
> 2018-02-14 18:36 GMT+09:00 Ulf Hansson <ulf.hansson@linaro.org>:
>> On 7 February 2018 at 20:11, Wolfram Sang <wsa@the-dreams.de> wrote:
>>>
>>>> I picked patch5 and patch6, as those seemed trivial. Unless there is a
>>>> rc9, let's aim for 4.17 for the rest.
>>>
>>> I can't find the branch where you picked these patches? I wanted to use
>>> it to apply the rest of the patches. They don't seem to fit on v4.15
>>> or current linus/master. Can you share this branch?
>>
>> You need to check again. :-)
>>
>> Patches up and to patch6 is already in Linus' tree v.16rc1, those I
>> carried for quite a while on my next branch before sending them in the
>> PR to Linus.
>>
>>>
>>> I will do reviewing now "visually". But please wait until I tested them
>>> before applying.
>>
>> I am eager to apply those patches you already have reviewed (to
>> patch14), as those mostly seemed rather trivial and should be nice
>> cleanups.
>>
>> Do you mind me going ahead, then you can continue and review/test the rest?
>>
>
>
> If you move forward, please fix-up the commit log of the patch 11.
> (https://patchwork.kernel.org/patch/10170007/)
>
>
> Documentation/devicetree/bindings/mmc/tmio_mmc.txt
>
>   ->
>
> Documentation/devicetree/bindings/mmc/mmc.txt

Thanks for the reminder. Yes I can amend the patch, no worries.

Kind regards
Uffe

^ permalink raw reply	[flat|nested] 19+ messages in thread

* Re: [PATCH v3 00/16] mmc: tmio: another batch of TMIO MMC fixes and cleanups
       [not found]       ` <CAPDyKFoxp7p0atQTV=PoQ7Bwt0fOs2aEq1KJHPtrq+zA3eFYgw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
@ 2018-02-14 10:23         ` Wolfram Sang
  2018-02-14 10:39           ` Ulf Hansson
  0 siblings, 1 reply; 19+ messages in thread
From: Wolfram Sang @ 2018-02-14 10:23 UTC (permalink / raw)
  To: Ulf Hansson
  Cc: Masahiro Yamada, linux-mmc-u79uwXL29TY76Z2rM5mHXA, Wolfram Sang,
	Ulf Magnusson, Geert Uytterhoeven, Simon Horman,
	Yoshihiro Shimoda, Linux-Renesas,
	devicetree-u79uwXL29TY76Z2rM5mHXA, Linux-sh list, Yoshinori Sato,
	Rob Herring, Linux Kernel Mailing List, Rob Herring, Lee Jones,
	Rich Felker, Mark Rutland

[-- Attachment #1: Type: text/plain, Size: 409 bytes --]


> You need to check again. :-)

Seems like it. Sorry for the noise.

> I am eager to apply those patches you already have reviewed (to
> patch14), as those mostly seemed rather trivial and should be nice
> cleanups.
> 
> Do you mind me going ahead, then you can continue and review/test the rest?

Yes, we can work incremently from there. I'll send patches if I discover
something unexpected.


[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply	[flat|nested] 19+ messages in thread

* Re: [PATCH v3 00/16] mmc: tmio: another batch of TMIO MMC fixes and cleanups
  2018-02-14 10:23         ` Wolfram Sang
@ 2018-02-14 10:39           ` Ulf Hansson
  0 siblings, 0 replies; 19+ messages in thread
From: Ulf Hansson @ 2018-02-14 10:39 UTC (permalink / raw)
  To: Wolfram Sang, Masahiro Yamada
  Cc: linux-mmc, Wolfram Sang, Ulf Magnusson, Geert Uytterhoeven,
	Simon Horman, Yoshihiro Shimoda, Linux-Renesas, devicetree,
	Linux-sh list, Yoshinori Sato, Rob Herring,
	Linux Kernel Mailing List, Rob Herring, Lee Jones, Rich Felker,
	Mark Rutland

On 14 February 2018 at 11:23, Wolfram Sang <wsa@the-dreams.de> wrote:
>
>> You need to check again. :-)
>
> Seems like it. Sorry for the noise.
>
>> I am eager to apply those patches you already have reviewed (to
>> patch14), as those mostly seemed rather trivial and should be nice
>> cleanups.
>>
>> Do you mind me going ahead, then you can continue and review/test the rest?
>
> Yes, we can work incremently from there. I'll send patches if I discover
> something unexpected.
>

Great! So, applied patches 7->14.

Thanks and kind regards
Uffe

^ permalink raw reply	[flat|nested] 19+ messages in thread

* Re: [PATCH v3 00/16] mmc: tmio: another batch of TMIO MMC fixes and cleanups
  2018-01-17 16:28 [PATCH v3 00/16] mmc: tmio: another batch of TMIO MMC fixes and cleanups Masahiro Yamada
       [not found] ` <1516206496-16612-1-git-send-email-yamada.masahiro-uWyLwvC0a2jby3iVrkZq2A@public.gmane.org>
  2018-01-18  8:13 ` [PATCH v3 00/16] mmc: tmio: another batch of TMIO MMC fixes and cleanups Ulf Hansson
@ 2018-03-04 22:42 ` Wolfram Sang
  2018-03-05  8:45   ` Ulf Hansson
  2 siblings, 1 reply; 19+ messages in thread
From: Wolfram Sang @ 2018-03-04 22:42 UTC (permalink / raw)
  To: Masahiro Yamada
  Cc: linux-mmc, Wolfram Sang, Ulf Magnusson, Geert Uytterhoeven,
	Simon Horman, Yoshihiro Shimoda, linux-renesas-soc, devicetree,
	linux-sh, Yoshinori Sato, Rob Herring, linux-kernel, Rob Herring,
	Lee Jones, Rich Felker, Mark Rutland, Ulf Hansson

[-- Attachment #1: Type: text/plain, Size: 510 bytes --]

On Thu, Jan 18, 2018 at 01:28:00AM +0900, Masahiro Yamada wrote:
> 
> In the previous batch (https://lkml.org/lkml/2017/11/24/428)
> I sent 22 patches for TMIO MMC.
> 
> 14 patches were applied, the rest is under discussion.
> 
> This series consists of 16 patches.

Which I tested now, so we can add

Tested-by: Wolfram Sang <wsa+renesas@sang-engineering.com>

to the patches except for

> Patch 12: updated per Wolfram

this one which needs a fix. I will send it as an incremental patch.


[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply	[flat|nested] 19+ messages in thread

* Re: [PATCH v3 00/16] mmc: tmio: another batch of TMIO MMC fixes and cleanups
  2018-03-04 22:42 ` Wolfram Sang
@ 2018-03-05  8:45   ` Ulf Hansson
  2018-03-05  9:22     ` Wolfram Sang
  0 siblings, 1 reply; 19+ messages in thread
From: Ulf Hansson @ 2018-03-05  8:45 UTC (permalink / raw)
  To: Wolfram Sang, Masahiro Yamada
  Cc: linux-mmc, Wolfram Sang, Ulf Magnusson, Geert Uytterhoeven,
	Simon Horman, Yoshihiro Shimoda, Linux-Renesas, devicetree,
	Linux-sh list, Yoshinori Sato, Rob Herring,
	Linux Kernel Mailing List, Rob Herring, Lee Jones, Rich Felker,
	Mark Rutland

On 4 March 2018 at 23:42, Wolfram Sang <wsa@the-dreams.de> wrote:
> On Thu, Jan 18, 2018 at 01:28:00AM +0900, Masahiro Yamada wrote:
>>
>> In the previous batch (https://lkml.org/lkml/2017/11/24/428)
>> I sent 22 patches for TMIO MMC.
>>
>> 14 patches were applied, the rest is under discussion.
>>
>> This series consists of 16 patches.
>
> Which I tested now, so we can add
>
> Tested-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
>
> to the patches except for
>
>> Patch 12: updated per Wolfram
>
> this one which needs a fix. I will send it as an incremental patch.
>

Yamada-san, Wolfram,

I have added Wolframs tags to the already applied patches and applied
that last two, 15 and 16.

They are all present in my next branch. The fix that Wolfram sent on
top, is added immediately after the commit it fixes.

Kind regards
Uffe

^ permalink raw reply	[flat|nested] 19+ messages in thread

* Re: [PATCH v3 00/16] mmc: tmio: another batch of TMIO MMC fixes and cleanups
  2018-03-05  8:45   ` Ulf Hansson
@ 2018-03-05  9:22     ` Wolfram Sang
  2018-03-05  9:34       ` Masahiro Yamada
  0 siblings, 1 reply; 19+ messages in thread
From: Wolfram Sang @ 2018-03-05  9:22 UTC (permalink / raw)
  To: Ulf Hansson
  Cc: Masahiro Yamada, linux-mmc, Wolfram Sang, Ulf Magnusson,
	Geert Uytterhoeven, Simon Horman, Yoshihiro Shimoda,
	Linux-Renesas, devicetree, Linux-sh list, Yoshinori Sato,
	Rob Herring, Linux Kernel Mailing List, Rob Herring, Lee Jones,
	Rich Felker, Mark Rutland

[-- Attachment #1: Type: text/plain, Size: 414 bytes --]


> I have added Wolframs tags to the already applied patches and applied
> that last two, 15 and 16.

Thanks. Please note that patch 14 misses my Rev tag. (I gave it
yesterday but the patch was accidently applied before. One round too
early, so to say).

> They are all present in my next branch. The fix that Wolfram sent on
> top, is added immediately after the commit it fixes.

Could we also squash it, then?


[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply	[flat|nested] 19+ messages in thread

* Re: [PATCH v3 00/16] mmc: tmio: another batch of TMIO MMC fixes and cleanups
  2018-03-05  9:22     ` Wolfram Sang
@ 2018-03-05  9:34       ` Masahiro Yamada
  2018-03-05  9:39         ` Wolfram Sang
  2018-03-05 12:06         ` Ulf Hansson
  0 siblings, 2 replies; 19+ messages in thread
From: Masahiro Yamada @ 2018-03-05  9:34 UTC (permalink / raw)
  To: Wolfram Sang
  Cc: Ulf Hansson, linux-mmc, Wolfram Sang, Ulf Magnusson,
	Geert Uytterhoeven, Simon Horman, Yoshihiro Shimoda,
	Linux-Renesas, devicetree, Linux-sh list, Yoshinori Sato,
	Rob Herring, Linux Kernel Mailing List, Rob Herring, Lee Jones,
	Rich Felker, Mark Rutland

Hi Ulf, Wolfram,

2018-03-05 18:22 GMT+09:00 Wolfram Sang <wsa@the-dreams.de>:
>
>> I have added Wolframs tags to the already applied patches and applied
>> that last two, 15 and 16.
>
> Thanks. Please note that patch 14 misses my Rev tag. (I gave it
> yesterday but the patch was accidently applied before. One round too
> early, so to say).
>
>> They are all present in my next branch. The fix that Wolfram sent on
>> top, is added immediately after the commit it fixes.
>
> Could we also squash it, then?
>

If Wolfram is fine with squashing,
that would be cleaner.

In that case, can you add Tested-by
to patch 12?




-- 
Best Regards
Masahiro Yamada

^ permalink raw reply	[flat|nested] 19+ messages in thread

* Re: [PATCH v3 00/16] mmc: tmio: another batch of TMIO MMC fixes and cleanups
  2018-03-05  9:34       ` Masahiro Yamada
@ 2018-03-05  9:39         ` Wolfram Sang
  2018-03-05 12:06         ` Ulf Hansson
  1 sibling, 0 replies; 19+ messages in thread
From: Wolfram Sang @ 2018-03-05  9:39 UTC (permalink / raw)
  To: Masahiro Yamada
  Cc: Ulf Hansson, linux-mmc, Wolfram Sang, Ulf Magnusson,
	Geert Uytterhoeven, Simon Horman, Yoshihiro Shimoda,
	Linux-Renesas, devicetree, Linux-sh list, Yoshinori Sato,
	Rob Herring, Linux Kernel Mailing List, Rob Herring, Lee Jones,
	Rich Felker, Mark Rutland

[-- Attachment #1: Type: text/plain, Size: 338 bytes --]


> >> They are all present in my next branch. The fix that Wolfram sent on
> >> top, is added immediately after the commit it fixes.
> >
> > Could we also squash it, then?
> >
> 
> If Wolfram is fine with squashing,
> that would be cleaner.
> 
> In that case, can you add Tested-by
> to patch 12?

Yup, sounds best to me.


[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply	[flat|nested] 19+ messages in thread

* Re: [PATCH v3 00/16] mmc: tmio: another batch of TMIO MMC fixes and cleanups
  2018-03-05  9:34       ` Masahiro Yamada
  2018-03-05  9:39         ` Wolfram Sang
@ 2018-03-05 12:06         ` Ulf Hansson
  1 sibling, 0 replies; 19+ messages in thread
From: Ulf Hansson @ 2018-03-05 12:06 UTC (permalink / raw)
  To: Masahiro Yamada, Wolfram Sang
  Cc: linux-mmc, Wolfram Sang, Ulf Magnusson, Geert Uytterhoeven,
	Simon Horman, Yoshihiro Shimoda, Linux-Renesas, devicetree,
	Linux-sh list, Yoshinori Sato, Rob Herring,
	Linux Kernel Mailing List, Rob Herring, Lee Jones, Rich Felker,
	Mark Rutland

On 5 March 2018 at 10:34, Masahiro Yamada <yamada.masahiro@socionext.com> wrote:
> Hi Ulf, Wolfram,
>
> 2018-03-05 18:22 GMT+09:00 Wolfram Sang <wsa@the-dreams.de>:
>>
>>> I have added Wolframs tags to the already applied patches and applied
>>> that last two, 15 and 16.
>>
>> Thanks. Please note that patch 14 misses my Rev tag. (I gave it
>> yesterday but the patch was accidently applied before. One round too
>> early, so to say).

Added rev. tag.

>>
>>> They are all present in my next branch. The fix that Wolfram sent on
>>> top, is added immediately after the commit it fixes.
>>
>> Could we also squash it, then?
>>
>
> If Wolfram is fine with squashing,
> that would be cleaner.
>

Done.

> In that case, can you add Tested-by
> to patch 12?

Done.

Kind regards
Uffe

^ permalink raw reply	[flat|nested] 19+ messages in thread

* Re: [PATCH v3 11/16] mmc: tmio: deprecate "toshiba,mmc-wrprotect-disable" DT property
  2018-02-07 19:32         ` Wolfram Sang
@ 2018-03-18  2:46           ` Masahiro Yamada
  0 siblings, 0 replies; 19+ messages in thread
From: Masahiro Yamada @ 2018-03-18  2:46 UTC (permalink / raw)
  To: Wolfram Sang, linux-mmc, Ulf Hansson
  Cc: Wolfram Sang, Ulf Magnusson, Geert Uytterhoeven, Simon Horman,
	Yoshihiro Shimoda, Linux-Renesas, devicetree, Rob Herring,
	Linux Kernel Mailing List, Rob Herring, Mark Rutland

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.

-- 
Best Regards
Masahiro Yamada

^ permalink raw reply	[flat|nested] 19+ messages in thread

end of thread, other threads:[~2018-03-18  2:46 UTC | newest]

Thread overview: 19+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-01-17 16:28 [PATCH v3 00/16] mmc: tmio: another batch of TMIO MMC fixes and cleanups Masahiro Yamada
     [not found] ` <1516206496-16612-1-git-send-email-yamada.masahiro-uWyLwvC0a2jby3iVrkZq2A@public.gmane.org>
2018-01-17 16:28   ` [PATCH v3 11/16] mmc: tmio: deprecate "toshiba,mmc-wrprotect-disable" DT property Masahiro Yamada
2018-01-18  1:58     ` Masahiro Yamada
     [not found]       ` <CAK7LNAQUJw-fjuJO68Kvskn8BRYW3rZj4YMtbo-Oxfv7c=6Ggg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2018-01-29 17:18         ` Rob Herring
2018-02-07 19:32         ` Wolfram Sang
2018-03-18  2:46           ` Masahiro Yamada
2018-01-18  8:13 ` [PATCH v3 00/16] mmc: tmio: another batch of TMIO MMC fixes and cleanups Ulf Hansson
2018-02-07 19:11   ` Wolfram Sang
2018-02-14  9:36     ` Ulf Hansson
2018-02-14  9:43       ` Masahiro Yamada
2018-02-14  9:46         ` Ulf Hansson
     [not found]       ` <CAPDyKFoxp7p0atQTV=PoQ7Bwt0fOs2aEq1KJHPtrq+zA3eFYgw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2018-02-14 10:23         ` Wolfram Sang
2018-02-14 10:39           ` Ulf Hansson
2018-03-04 22:42 ` Wolfram Sang
2018-03-05  8:45   ` Ulf Hansson
2018-03-05  9:22     ` Wolfram Sang
2018-03-05  9:34       ` Masahiro Yamada
2018-03-05  9:39         ` Wolfram Sang
2018-03-05 12:06         ` Ulf Hansson

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).