[PATCH] ARM: shmobile: r8a7778: tidyup SSI resource region

[PATCH] ARM: shmobile: r8a7778: tidyup SSI resource region

[Kuninori Morimoto]

From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>

Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
---
 arch/arm/boot/dts/r8a7778.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/r8a7778.dtsi b/arch/arm/boot/dts/r8a7778.dtsi
index 4b1fa9f..b317ec8 100644
--- a/arch/arm/boot/dts/r8a7778.dtsi
+++ b/arch/arm/boot/dts/r8a7778.dtsi
@@ -239,7 +239,7 @@
 		#sound-dai-cells = <1>;
 		compatible = "renesas,rcar_sound-r8a7778", "renesas,rcar_sound-gen1";
 		reg =	<0xffd90000 0x1000>,	/* SRU */
-			<0xffd91000 0x1240>,	/* SSI */
+			<0xffd91000 0x270>,	/* SSI */
 			<0xfffe0000 0x24>;	/* ADG */
 		clocks = <&mstp3_clks R8A7778_CLK_SSI8>,
 			<&mstp3_clks R8A7778_CLK_SSI7>,
-- 
1.9.1

Re: [PATCH] ARM: shmobile: r8a7778: tidyup SSI resource region

[Geert Uytterhoeven]

On Mon, Aug 24, 2015 at 10:27 AM, Kuninori Morimoto
<kuninori.morimoto.gx@renesas.com> wrote:
> --- a/arch/arm/boot/dts/r8a7778.dtsi
> +++ b/arch/arm/boot/dts/r8a7778.dtsi
> @@ -239,7 +239,7 @@
>                 #sound-dai-cells = <1>;
>                 compatible = "renesas,rcar_sound-r8a7778", "renesas,rcar_sound-gen1";
>                 reg =   <0xffd90000 0x1000>,    /* SRU */
> -                       <0xffd91000 0x1240>,    /* SSI */
> +                       <0xffd91000 0x270>,     /* SSI */

Why not 0x240?

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

Re: [PATCH] ARM: shmobile: r8a7778: tidyup SSI resource region

[Kuninori Morimoto]

Hi Geert

> > --- a/arch/arm/boot/dts/r8a7778.dtsi
> > +++ b/arch/arm/boot/dts/r8a7778.dtsi
> > @@ -239,7 +239,7 @@
> >                 #sound-dai-cells = <1>;
> >                 compatible = "renesas,rcar_sound-r8a7778", "renesas,rcar_sound-gen1";
> >                 reg =   <0xffd90000 0x1000>,    /* SRU */
> > -                       <0xffd91000 0x1240>,    /* SSI */
> > +                       <0xffd91000 0x270>,     /* SSI */
> 
> Why not 0x240?

0x240 can't care channel 9
I forgot why it used 0x1240, not 0x1270

Re: [PATCH] ARM: shmobile: r8a7778: tidyup SSI resource region

[Geert Uytterhoeven]

Hi Morimoto-san,

On Tue, Aug 25, 2015 at 1:35 AM, Kuninori Morimoto
<kuninori.morimoto.gx@renesas.com> wrote:
>> > --- a/arch/arm/boot/dts/r8a7778.dtsi
>> > +++ b/arch/arm/boot/dts/r8a7778.dtsi
>> > @@ -239,7 +239,7 @@
>> >                 #sound-dai-cells = <1>;
>> >                 compatible = "renesas,rcar_sound-r8a7778", "renesas,rcar_sound-gen1";
>> >                 reg =   <0xffd90000 0x1000>,    /* SRU */
>> > -                       <0xffd91000 0x1240>,    /* SSI */
>> > +                       <0xffd91000 0x270>,     /* SSI */
>>
>> Why not 0x240?
>
> 0x240 can't care channel 9
> I forgot why it used 0x1240, not 0x1270

According to my documentation, r8a7778 has channels 0-8, r8a7779 has 0-9.
So I'd expect 0x240 on r8a7778, and 0x280 on r8a7779.

Still, 10 * 0x40 = 0x280, not 0x270.

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

Re: [PATCH] ARM: shmobile: r8a7778: tidyup SSI resource region

[Kuninori Morimoto]

Hi Geert

> >> > --- a/arch/arm/boot/dts/r8a7778.dtsi
> >> > +++ b/arch/arm/boot/dts/r8a7778.dtsi
> >> > @@ -239,7 +239,7 @@
> >> >                 #sound-dai-cells = <1>;
> >> >                 compatible = "renesas,rcar_sound-r8a7778", "renesas,rcar_sound-gen1";
> >> >                 reg =   <0xffd90000 0x1000>,    /* SRU */
> >> > -                       <0xffd91000 0x1240>,    /* SSI */
> >> > +                       <0xffd91000 0x270>,     /* SSI */
> >>
> >> Why not 0x240?
> >
> > 0x240 can't care channel 9
> > I forgot why it used 0x1240, not 0x1270
> 
> According to my documentation, r8a7778 has channels 0-8, r8a7779 has 0-9.
> So I'd expect 0x240 on r8a7778, and 0x280 on r8a7779.
> 
> Still, 10 * 0x40 = 0x280, not 0x270.

Grr... my bad.
Thanks