[PATCH/RFC] ARM: shmobile: r8a7791: Remove MSIOF address from device tree

devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed

* [PATCH/RFC] ARM: shmobile: r8a7791: Remove MSIOF address from device tree
@ 2015-03-15 14:43 Yoshihiro Kaneko
  2015-03-29 13:22 ` Yoshihiro Kaneko
  0 siblings, 1 reply; 5+ messages in thread
From: Yoshihiro Kaneko @ 2015-03-15 14:43 UTC (permalink / raw)
  To: linux-sh; +Cc: Simon Horman, Magnus Damm, linux-arm-kernel, devicetree

From: Ryo Kataoka <ryo.kataoka.wt@renesas.com>

MSIOF Base Address H'E6xx can be accessed by CPU and DMAC.
MSIOF Base Address H'E7xx for DMAC was removed from H/W manual.

Signed-off-by: Ryo Kataoka <ryo.kataoka.wt@renesas.com>
Signed-off-by: Yoshihiro Kaneko <ykaneko0929@gmail.com>
---

This patch is based on devel branch of Simon Horman's renesas tree.

 arch/arm/boot/dts/r8a7791.dtsi | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/arch/arm/boot/dts/r8a7791.dtsi b/arch/arm/boot/dts/r8a7791.dtsi
index 4696062..3e9f824 100644
--- a/arch/arm/boot/dts/r8a7791.dtsi
+++ b/arch/arm/boot/dts/r8a7791.dtsi
@@ -1288,7 +1288,7 @@
 
 	msiof0: spi@e6e20000 {
 		compatible = "renesas,msiof-r8a7791";
-		reg = <0 0xe6e20000 0 0x0064>, <0 0xe7e20000 0 0x0064>;
+		reg = <0 0xe6e20000 0 0x0064>;
 		interrupts = <0 156 IRQ_TYPE_LEVEL_HIGH>;
 		clocks = <&mstp0_clks R8A7791_CLK_MSIOF0>;
 		dmas = <&dmac0 0x51>, <&dmac0 0x52>;
@@ -1300,7 +1300,7 @@
 
 	msiof1: spi@e6e10000 {
 		compatible = "renesas,msiof-r8a7791";
-		reg = <0 0xe6e10000 0 0x0064>, <0 0xe7e10000 0 0x0064>;
+		reg = <0 0xe6e10000 0 0x0064>;
 		interrupts = <0 157 IRQ_TYPE_LEVEL_HIGH>;
 		clocks = <&mstp2_clks R8A7791_CLK_MSIOF1>;
 		dmas = <&dmac0 0x55>, <&dmac0 0x56>;
@@ -1312,7 +1312,7 @@
 
 	msiof2: spi@e6e00000 {
 		compatible = "renesas,msiof-r8a7791";
-		reg = <0 0xe6e00000 0 0x0064>, <0 0xe7e00000 0 0x0064>;
+		reg = <0 0xe6e00000 0 0x0064>;
 		interrupts = <0 158 IRQ_TYPE_LEVEL_HIGH>;
 		clocks = <&mstp2_clks R8A7791_CLK_MSIOF2>;
 		dmas = <&dmac0 0x41>, <&dmac0 0x42>;
-- 
1.9.1


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

* Re: [PATCH/RFC] ARM: shmobile: r8a7791: Remove MSIOF address from device tree
  2015-03-15 14:43 [PATCH/RFC] ARM: shmobile: r8a7791: Remove MSIOF address from device tree Yoshihiro Kaneko
@ 2015-03-29 13:22 ` Yoshihiro Kaneko
  2015-03-30  9:23   ` Geert Uytterhoeven
  0 siblings, 1 reply; 5+ messages in thread
From: Yoshihiro Kaneko @ 2015-03-29 13:22 UTC (permalink / raw)
  To: Linux-sh list
  Cc: Geert Uytterhoeven, Simon Horman, Magnus Damm,
	linux-arm-kernel@lists.infradead.org, devicetree

Hi Geert-san,

I'm wondering if you could find a moment to look over this patch.

Regards,
Kaneko

2015-03-15 23:43 GMT+09:00 Yoshihiro Kaneko <ykaneko0929@gmail.com>:
> From: Ryo Kataoka <ryo.kataoka.wt@renesas.com>
>
> MSIOF Base Address H'E6xx can be accessed by CPU and DMAC.
> MSIOF Base Address H'E7xx for DMAC was removed from H/W manual.
>
> Signed-off-by: Ryo Kataoka <ryo.kataoka.wt@renesas.com>
> Signed-off-by: Yoshihiro Kaneko <ykaneko0929@gmail.com>
> ---
>
> This patch is based on devel branch of Simon Horman's renesas tree.
>
>  arch/arm/boot/dts/r8a7791.dtsi | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/arch/arm/boot/dts/r8a7791.dtsi b/arch/arm/boot/dts/r8a7791.dtsi
> index 4696062..3e9f824 100644
> --- a/arch/arm/boot/dts/r8a7791.dtsi
> +++ b/arch/arm/boot/dts/r8a7791.dtsi
> @@ -1288,7 +1288,7 @@
>
>         msiof0: spi@e6e20000 {
>                 compatible = "renesas,msiof-r8a7791";
> -               reg = <0 0xe6e20000 0 0x0064>, <0 0xe7e20000 0 0x0064>;
> +               reg = <0 0xe6e20000 0 0x0064>;
>                 interrupts = <0 156 IRQ_TYPE_LEVEL_HIGH>;
>                 clocks = <&mstp0_clks R8A7791_CLK_MSIOF0>;
>                 dmas = <&dmac0 0x51>, <&dmac0 0x52>;
> @@ -1300,7 +1300,7 @@
>
>         msiof1: spi@e6e10000 {
>                 compatible = "renesas,msiof-r8a7791";
> -               reg = <0 0xe6e10000 0 0x0064>, <0 0xe7e10000 0 0x0064>;
> +               reg = <0 0xe6e10000 0 0x0064>;
>                 interrupts = <0 157 IRQ_TYPE_LEVEL_HIGH>;
>                 clocks = <&mstp2_clks R8A7791_CLK_MSIOF1>;
>                 dmas = <&dmac0 0x55>, <&dmac0 0x56>;
> @@ -1312,7 +1312,7 @@
>
>         msiof2: spi@e6e00000 {
>                 compatible = "renesas,msiof-r8a7791";
> -               reg = <0 0xe6e00000 0 0x0064>, <0 0xe7e00000 0 0x0064>;
> +               reg = <0 0xe6e00000 0 0x0064>;
>                 interrupts = <0 158 IRQ_TYPE_LEVEL_HIGH>;
>                 clocks = <&mstp2_clks R8A7791_CLK_MSIOF2>;
>                 dmas = <&dmac0 0x41>, <&dmac0 0x42>;
> --
> 1.9.1
>

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

* Re: [PATCH/RFC] ARM: shmobile: r8a7791: Remove MSIOF address from device tree
  2015-03-29 13:22 ` Yoshihiro Kaneko
@ 2015-03-30  9:23   ` Geert Uytterhoeven
       [not found]     ` <CAMuHMdV201bh8-QrJgsO4KNSMW-d9aVyMczHdyQO0Y_pg4sukw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
  0 siblings, 1 reply; 5+ messages in thread
From: Geert Uytterhoeven @ 2015-03-30  9:23 UTC (permalink / raw)
  To: Yoshihiro Kaneko
  Cc: Linux-sh list, Simon Horman, Magnus Damm,
	linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org

Hi Kaneko-san,

On Sun, Mar 29, 2015 at 3:22 PM, Yoshihiro Kaneko <ykaneko0929@gmail.com> wrote:
> I'm wondering if you could find a moment to look over this patch.

In light of the discussion thread w.r.t. to the driver changes (I
don't recall you
replying, did I miss that?), I think the DTS changes for r8a779[01] are fine.
However, please also update the example (for r8a7791) in
Documentation/devicetree/bindings/spi/sh-msiof.txt.

Thanks!

> 2015-03-15 23:43 GMT+09:00 Yoshihiro Kaneko <ykaneko0929@gmail.com>:
>> From: Ryo Kataoka <ryo.kataoka.wt@renesas.com>
>>
>> MSIOF Base Address H'E6xx can be accessed by CPU and DMAC.
>> MSIOF Base Address H'E7xx for DMAC was removed from H/W manual.
>>
>> Signed-off-by: Ryo Kataoka <ryo.kataoka.wt@renesas.com>
>> Signed-off-by: Yoshihiro Kaneko <ykaneko0929@gmail.com>
>> ---
>>
>> This patch is based on devel branch of Simon Horman's renesas tree.
>>
>>  arch/arm/boot/dts/r8a7791.dtsi | 6 +++---
>>  1 file changed, 3 insertions(+), 3 deletions(-)
>>
>> diff --git a/arch/arm/boot/dts/r8a7791.dtsi b/arch/arm/boot/dts/r8a7791.dtsi
>> index 4696062..3e9f824 100644
>> --- a/arch/arm/boot/dts/r8a7791.dtsi
>> +++ b/arch/arm/boot/dts/r8a7791.dtsi
>> @@ -1288,7 +1288,7 @@
>>
>>         msiof0: spi@e6e20000 {
>>                 compatible = "renesas,msiof-r8a7791";
>> -               reg = <0 0xe6e20000 0 0x0064>, <0 0xe7e20000 0 0x0064>;
>> +               reg = <0 0xe6e20000 0 0x0064>;
>>                 interrupts = <0 156 IRQ_TYPE_LEVEL_HIGH>;
>>                 clocks = <&mstp0_clks R8A7791_CLK_MSIOF0>;
>>                 dmas = <&dmac0 0x51>, <&dmac0 0x52>;

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

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

[parent not found: <CAMuHMdV201bh8-QrJgsO4KNSMW-d9aVyMczHdyQO0Y_pg4sukw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>]

* Re: [PATCH/RFC] ARM: shmobile: r8a7791: Remove MSIOF address from device tree
       [not found]     ` <CAMuHMdV201bh8-QrJgsO4KNSMW-d9aVyMczHdyQO0Y_pg4sukw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
@ 2015-04-02 16:35       ` Yoshihiro Kaneko
  2015-04-06  0:48         ` Simon Horman
  0 siblings, 1 reply; 5+ messages in thread
From: Yoshihiro Kaneko @ 2015-04-02 16:35 UTC (permalink / raw)
  To: Geert Uytterhoeven
  Cc: Linux-sh list, Simon Horman, Magnus Damm,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
	devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org

Hi Geert-san,

Thanks for your reply.

It seems that you and Simon concluded that the driver change wasn't
necessary. As I don't have access to the relevant documentation I'm happy
to defer to you two on that matter.

With that in mind I'll refresh this patch as you request.

I'm assuming that the r8a7790 version of this patch is fine as-is.
I'll re-post that one unchanged other than dropping the RFC tag.

Thanks,
Kaneko

2015-03-30 18:23 GMT+09:00 Geert Uytterhoeven <geert-Td1EMuHUCqxL1ZNQvxDV9g@public.gmane.org>:
> Hi Kaneko-san,
>
> On Sun, Mar 29, 2015 at 3:22 PM, Yoshihiro Kaneko <ykaneko0929-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:
>> I'm wondering if you could find a moment to look over this patch.
>
> In light of the discussion thread w.r.t. to the driver changes (I
> don't recall you
> replying, did I miss that?), I think the DTS changes for r8a779[01] are fine.
> However, please also update the example (for r8a7791) in
> Documentation/devicetree/bindings/spi/sh-msiof.txt.
>
> Thanks!
>
>> 2015-03-15 23:43 GMT+09:00 Yoshihiro Kaneko <ykaneko0929-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>:
>>> From: Ryo Kataoka <ryo.kataoka.wt-zM6kxYcvzFBBDgjK7y7TUQ@public.gmane.org>
>>>
>>> MSIOF Base Address H'E6xx can be accessed by CPU and DMAC.
>>> MSIOF Base Address H'E7xx for DMAC was removed from H/W manual.
>>>
>>> Signed-off-by: Ryo Kataoka <ryo.kataoka.wt-zM6kxYcvzFBBDgjK7y7TUQ@public.gmane.org>
>>> Signed-off-by: Yoshihiro Kaneko <ykaneko0929-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
>>> ---
>>>
>>> This patch is based on devel branch of Simon Horman's renesas tree.
>>>
>>>  arch/arm/boot/dts/r8a7791.dtsi | 6 +++---
>>>  1 file changed, 3 insertions(+), 3 deletions(-)
>>>
>>> diff --git a/arch/arm/boot/dts/r8a7791.dtsi b/arch/arm/boot/dts/r8a7791.dtsi
>>> index 4696062..3e9f824 100644
>>> --- a/arch/arm/boot/dts/r8a7791.dtsi
>>> +++ b/arch/arm/boot/dts/r8a7791.dtsi
>>> @@ -1288,7 +1288,7 @@
>>>
>>>         msiof0: spi@e6e20000 {
>>>                 compatible = "renesas,msiof-r8a7791";
>>> -               reg = <0 0xe6e20000 0 0x0064>, <0 0xe7e20000 0 0x0064>;
>>> +               reg = <0 0xe6e20000 0 0x0064>;
>>>                 interrupts = <0 156 IRQ_TYPE_LEVEL_HIGH>;
>>>                 clocks = <&mstp0_clks R8A7791_CLK_MSIOF0>;
>>>                 dmas = <&dmac0 0x51>, <&dmac0 0x52>;
>
> Gr{oetje,eeting}s,
>
>                         Geert
>
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert-Td1EMuHUCqxL1ZNQvxDV9g@public.gmane.org
>
> In personal conversations with technical people, I call myself a hacker. But
> when I'm talking to journalists I just say "programmer" or something like that.
>                                 -- Linus Torvalds
--
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] 5+ messages in thread

* Re: [PATCH/RFC] ARM: shmobile: r8a7791: Remove MSIOF address from device tree
  2015-04-02 16:35       ` Yoshihiro Kaneko
@ 2015-04-06  0:48         ` Simon Horman
  0 siblings, 0 replies; 5+ messages in thread
From: Simon Horman @ 2015-04-06  0:48 UTC (permalink / raw)
  To: Yoshihiro Kaneko
  Cc: Geert Uytterhoeven, Linux-sh list, Magnus Damm,
	linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org

Hi Kaneko-san,

On Fri, Apr 03, 2015 at 01:35:23AM +0900, Yoshihiro Kaneko wrote:
> Hi Geert-san,
> 
> Thanks for your reply.
> 
> It seems that you and Simon concluded that the driver change wasn't
> necessary. As I don't have access to the relevant documentation I'm happy
> to defer to you two on that matter.
> 
> With that in mind I'll refresh this patch as you request.
> 
> I'm assuming that the r8a7790 version of this patch is fine as-is.
> I'll re-post that one unchanged other than dropping the RFC tag.

Thanks for your attention to this matter. As noted elsewhere I have
queued up your reposted/updated DT patches. And I believe we should
not pursue the driver patch any further at this time. So for now I think
this case is closed.

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

end of thread, other threads:[~2015-04-06  0:48 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-03-15 14:43 [PATCH/RFC] ARM: shmobile: r8a7791: Remove MSIOF address from device tree Yoshihiro Kaneko
2015-03-29 13:22 ` Yoshihiro Kaneko
2015-03-30  9:23   ` Geert Uytterhoeven
     [not found]     ` <CAMuHMdV201bh8-QrJgsO4KNSMW-d9aVyMczHdyQO0Y_pg4sukw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2015-04-02 16:35       ` Yoshihiro Kaneko
2015-04-06  0:48         ` Simon Horman

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).