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

[Yoshihiro Kaneko]

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 at linux-m68k.org>:
> Hi Kaneko-san,
>
> On Sun, Mar 29, 2015 at 3:22 PM, Yoshihiro Kaneko <ykaneko0929 at 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 at gmail.com>:
>>> From: Ryo Kataoka <ryo.kataoka.wt at 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 at renesas.com>
>>> Signed-off-by: Yoshihiro Kaneko <ykaneko0929 at 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 at 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 at 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