[PATCH 0/4 v2] ARM: shmobile: r8a779x: add SDHI DMA support

[PATCH 0/4 v2] ARM: shmobile: r8a779x: add SDHI DMA support

[Kuninori Morimoto]

Hi Simon

These are v2 of SDHI DMA support for r8a7790/r8a7791.
These are based on Ulf's
git.linaro.org/people/ulf.hansson/mmc.git tmio

Kuninori Morimoto (4):
      ARM: shmobile: r8a7790: tidyup SDHI register size on DTSI
      ARM: shmobile: r8a7791: tidyup SDHI register size on DTSI
      ARM: shmobile: r8a7790: add SDHI DMA channels on DTSI
      ARM: shmobile: r8a7791: add SDHI DMA channels on DTSI

 arch/arm/boot/dts/r8a7790.dtsi |   12 ++++++++++--
 arch/arm/boot/dts/r8a7791.dtsi |    8 +++++++-
 2 files changed, 17 insertions(+), 3 deletions(-)


Best regards
---
Kuninori Morimoto

Re: [PATCH 0/4 v2] ARM: shmobile: r8a779x: add SDHI DMA support

[Laurent Pinchart]

Hi Morimoto-san,

Thank you for the patches.

On Wednesday 21 January 2015 04:26:03 Kuninori Morimoto wrote:
> Hi Simon
> 
> These are v2 of SDHI DMA support for r8a7790/r8a7791.
> These are based on Ulf's
> git.linaro.org/people/ulf.hansson/mmc.git tmio
> 
> Kuninori Morimoto (4):
>       ARM: shmobile: r8a7790: tidyup SDHI register size on DTSI
>       ARM: shmobile: r8a7791: tidyup SDHI register size on DTSI
>       ARM: shmobile: r8a7790: add SDHI DMA channels on DTSI
>       ARM: shmobile: r8a7791: add SDHI DMA channels on DTSI

I've already submitted patches 3 and 4 a few months ago, as

ARM: shmobile: r8a7790: Reference DMA channels in SDHI DT nodes
ARM: shmobile: r8a7790: Reference DMA channels in SDHI DT nodes

You had tested them back then, and reported a performance regression. Has that 
been fixed ?

>  arch/arm/boot/dts/r8a7790.dtsi |   12 ++++++++++--
>  arch/arm/boot/dts/r8a7791.dtsi |    8 +++++++-
>  2 files changed, 17 insertions(+), 3 deletions(-)

-- 
Regards,

Laurent Pinchart

Re: [PATCH 0/4 v2] ARM: shmobile: r8a779x: add SDHI DMA support

[Kuninori Morimoto]

Hi Laurent

> > These are v2 of SDHI DMA support for r8a7790/r8a7791.
> > These are based on Ulf's
> > git.linaro.org/people/ulf.hansson/mmc.git tmio
> > 
> > Kuninori Morimoto (4):
> >       ARM: shmobile: r8a7790: tidyup SDHI register size on DTSI
> >       ARM: shmobile: r8a7791: tidyup SDHI register size on DTSI
> >       ARM: shmobile: r8a7790: add SDHI DMA channels on DTSI
> >       ARM: shmobile: r8a7791: add SDHI DMA channels on DTSI
> 
> I've already submitted patches 3 and 4 a few months ago, as
> 
> ARM: shmobile: r8a7790: Reference DMA channels in SDHI DT nodes
> ARM: shmobile: r8a7790: Reference DMA channels in SDHI DT nodes
> 
> You had tested them back then, and reported a performance regression. Has that 
> been fixed ?

I think it was block device issue, and was fixed in upstream.
I forgot to add your Signed-off-by on these patches.
Will send v3 patch soon

Best regards
---
Kuninori Morimoto

Re: [PATCH 0/4 v2] ARM: shmobile: r8a779x: add SDHI DMA support

[Laurent Pinchart]

Hi Morimoto-san,

On Thursday 22 January 2015 00:28:44 Kuninori Morimoto wrote:
> Hi Laurent
> 
> > > These are v2 of SDHI DMA support for r8a7790/r8a7791.
> > > These are based on Ulf's
> > > git.linaro.org/people/ulf.hansson/mmc.git tmio
> > > 
> > > Kuninori Morimoto (4):
> > >       ARM: shmobile: r8a7790: tidyup SDHI register size on DTSI
> > >       ARM: shmobile: r8a7791: tidyup SDHI register size on DTSI
> > >       ARM: shmobile: r8a7790: add SDHI DMA channels on DTSI
> > >       ARM: shmobile: r8a7791: add SDHI DMA channels on DTSI
> > 
> > I've already submitted patches 3 and 4 a few months ago, as
> > 
> > ARM: shmobile: r8a7790: Reference DMA channels in SDHI DT nodes
> > ARM: shmobile: r8a7790: Reference DMA channels in SDHI DT nodes
> > 
> > You had tested them back then, and reported a performance regression. Has
> > that been fixed ?
> 
> I think it was block device issue, and was fixed in upstream.

That's nice to know.

> I forgot to add your Signed-off-by on these patches.
> Will send v3 patch soon

Thank you.

-- 
Regards,

Laurent Pinchart