* [PATCH 0/3] R8A7778/BOCK-W: HPB-DMAC support for SDHI0 @ 2013-06-29 22:43 ` Sergei Shtylyov 0 siblings, 0 replies; 30+ messages in thread From: Sergei Shtylyov @ 2013-06-29 22:43 UTC (permalink / raw) To: linux-arm-kernel Hello. Here's the set of 3 patches against the Simon Horman's 'renesas.git' repo, 'renesas-next-20130628' tag. Here we add the HPB-DMAC platform code working on the R8A7778 and enable DMA for SDHI0 on BOCK-W. The patchset depends on the R-Car HPB-DMAC driver patch posted earlier. [1/3] ARM: shmobile: use common platform header for HPB-DMAC [2/3] ARM: shmobile: r8a7778: add HPB-DMAC support [3/3] ARM: shmobile: BOCK-W: enable DMA for SDHI0 WBR, Sergei ^ permalink raw reply [flat|nested] 30+ messages in thread
* [PATCH 0/3] R8A7778/BOCK-W: HPB-DMAC support for SDHI0 @ 2013-06-29 22:43 ` Sergei Shtylyov 0 siblings, 0 replies; 30+ messages in thread From: Sergei Shtylyov @ 2013-06-29 22:43 UTC (permalink / raw) To: linux-arm-kernel Hello. Here's the set of 3 patches against the Simon Horman's 'renesas.git' repo, 'renesas-next-20130628' tag. Here we add the HPB-DMAC platform code working on the R8A7778 and enable DMA for SDHI0 on BOCK-W. The patchset depends on the R-Car HPB-DMAC driver patch posted earlier. [1/3] ARM: shmobile: use common platform header for HPB-DMAC [2/3] ARM: shmobile: r8a7778: add HPB-DMAC support [3/3] ARM: shmobile: BOCK-W: enable DMA for SDHI0 WBR, Sergei ^ permalink raw reply [flat|nested] 30+ messages in thread
* [PATCH 1/3] ARM: shmobile: use common platform header for HPB-DMAC 2013-06-29 22:43 ` Sergei Shtylyov @ 2013-06-29 22:45 ` Sergei Shtylyov -1 siblings, 0 replies; 30+ messages in thread From: Sergei Shtylyov @ 2013-06-29 22:45 UTC (permalink / raw) To: linux-arm-kernel From: Max Filippov <max.filippov@cogentembedded.com> Use previously empty <mach/dma.h> to declare HPB-DMA slave IDs. Signed-off-by: Max Filippov <max.filippov@cogentembedded.com> [Sergei: changed the guard macro name, fixed comment, extended copyright.] Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> --- arch/arm/mach-shmobile/include/mach/dma.h | 25 ++++++++++++++++++++++++- 1 file changed, 24 insertions(+), 1 deletion(-) Index: renesas/arch/arm/mach-shmobile/include/mach/dma.h =================================--- renesas.orig/arch/arm/mach-shmobile/include/mach/dma.h +++ renesas/arch/arm/mach-shmobile/include/mach/dma.h @@ -1 +1,24 @@ -/* empty */ +/* + * Copyright (C) 2011-2013 Renesas Electronics Corporation + * Copyright (C) 2013 Cogent Embedded, Inc. + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License version 2 + * as published by the Free Software Foundation. + */ + +#ifndef __MACH_DMA_H +#define __MACH_DMA_H + +/* HPB-DMA slave IDs */ +enum { + HPBDMA_SLAVE_DUMMY, + HPBDMA_SLAVE_SDHI0_TX, + HPBDMA_SLAVE_SDHI0_RX, + HPBDMA_SLAVE_SSI0_TX_ST, + HPBDMA_SLAVE_SSI0_TX_MN, + HPBDMA_SLAVE_SSI1_RX_ST, + HPBDMA_SLAVE_SSI1_RX_MN, +}; + +#endif /* __MACH_DMA_H */ ^ permalink raw reply [flat|nested] 30+ messages in thread
* [PATCH 1/3] ARM: shmobile: use common platform header for HPB-DMAC @ 2013-06-29 22:45 ` Sergei Shtylyov 0 siblings, 0 replies; 30+ messages in thread From: Sergei Shtylyov @ 2013-06-29 22:45 UTC (permalink / raw) To: linux-arm-kernel From: Max Filippov <max.filippov@cogentembedded.com> Use previously empty <mach/dma.h> to declare HPB-DMA slave IDs. Signed-off-by: Max Filippov <max.filippov@cogentembedded.com> [Sergei: changed the guard macro name, fixed comment, extended copyright.] Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> --- arch/arm/mach-shmobile/include/mach/dma.h | 25 ++++++++++++++++++++++++- 1 file changed, 24 insertions(+), 1 deletion(-) Index: renesas/arch/arm/mach-shmobile/include/mach/dma.h =================================================================== --- renesas.orig/arch/arm/mach-shmobile/include/mach/dma.h +++ renesas/arch/arm/mach-shmobile/include/mach/dma.h @@ -1 +1,24 @@ -/* empty */ +/* + * Copyright (C) 2011-2013 Renesas Electronics Corporation + * Copyright (C) 2013 Cogent Embedded, Inc. + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License version 2 + * as published by the Free Software Foundation. + */ + +#ifndef __MACH_DMA_H +#define __MACH_DMA_H + +/* HPB-DMA slave IDs */ +enum { + HPBDMA_SLAVE_DUMMY, + HPBDMA_SLAVE_SDHI0_TX, + HPBDMA_SLAVE_SDHI0_RX, + HPBDMA_SLAVE_SSI0_TX_ST, + HPBDMA_SLAVE_SSI0_TX_MN, + HPBDMA_SLAVE_SSI1_RX_ST, + HPBDMA_SLAVE_SSI1_RX_MN, +}; + +#endif /* __MACH_DMA_H */ ^ permalink raw reply [flat|nested] 30+ messages in thread
* Re: [PATCH 1/3] ARM: shmobile: use common platform header for HPB-DMAC 2013-06-29 22:45 ` Sergei Shtylyov @ 2013-07-01 4:11 ` Kuninori Morimoto -1 siblings, 0 replies; 30+ messages in thread From: Kuninori Morimoto @ 2013-07-01 4:11 UTC (permalink / raw) To: linux-arm-kernel Hi > From: Max Filippov <max.filippov@cogentembedded.com> > > Use previously empty <mach/dma.h> to declare HPB-DMA slave IDs. > > Signed-off-by: Max Filippov <max.filippov@cogentembedded.com> > [Sergei: changed the guard macro name, fixed comment, extended copyright.] > Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> > > --- > arch/arm/mach-shmobile/include/mach/dma.h | 25 ++++++++++++++++++++++++- > 1 file changed, 24 insertions(+), 1 deletion(-) > > Index: renesas/arch/arm/mach-shmobile/include/mach/dma.h > =================================> --- renesas.orig/arch/arm/mach-shmobile/include/mach/dma.h > +++ renesas/arch/arm/mach-shmobile/include/mach/dma.h > @@ -1 +1,24 @@ > -/* empty */ > +/* > + * Copyright (C) 2011-2013 Renesas Electronics Corporation > + * Copyright (C) 2013 Cogent Embedded, Inc. > + * > + * This program is free software; you can redistribute it and/or modify > + * it under the terms of the GNU General Public License version 2 > + * as published by the Free Software Foundation. > + */ > + > +#ifndef __MACH_DMA_H > +#define __MACH_DMA_H > + > +/* HPB-DMA slave IDs */ > +enum { > + HPBDMA_SLAVE_DUMMY, > + HPBDMA_SLAVE_SDHI0_TX, > + HPBDMA_SLAVE_SDHI0_RX, > + HPBDMA_SLAVE_SSI0_TX_ST, > + HPBDMA_SLAVE_SSI0_TX_MN, > + HPBDMA_SLAVE_SSI1_RX_ST, > + HPBDMA_SLAVE_SSI1_RX_MN, > +}; it should be in r8a7778.h Best regards --- Kuninori Morimoto ^ permalink raw reply [flat|nested] 30+ messages in thread
* [PATCH 1/3] ARM: shmobile: use common platform header for HPB-DMAC @ 2013-07-01 4:11 ` Kuninori Morimoto 0 siblings, 0 replies; 30+ messages in thread From: Kuninori Morimoto @ 2013-07-01 4:11 UTC (permalink / raw) To: linux-arm-kernel Hi > From: Max Filippov <max.filippov@cogentembedded.com> > > Use previously empty <mach/dma.h> to declare HPB-DMA slave IDs. > > Signed-off-by: Max Filippov <max.filippov@cogentembedded.com> > [Sergei: changed the guard macro name, fixed comment, extended copyright.] > Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> > > --- > arch/arm/mach-shmobile/include/mach/dma.h | 25 ++++++++++++++++++++++++- > 1 file changed, 24 insertions(+), 1 deletion(-) > > Index: renesas/arch/arm/mach-shmobile/include/mach/dma.h > =================================================================== > --- renesas.orig/arch/arm/mach-shmobile/include/mach/dma.h > +++ renesas/arch/arm/mach-shmobile/include/mach/dma.h > @@ -1 +1,24 @@ > -/* empty */ > +/* > + * Copyright (C) 2011-2013 Renesas Electronics Corporation > + * Copyright (C) 2013 Cogent Embedded, Inc. > + * > + * This program is free software; you can redistribute it and/or modify > + * it under the terms of the GNU General Public License version 2 > + * as published by the Free Software Foundation. > + */ > + > +#ifndef __MACH_DMA_H > +#define __MACH_DMA_H > + > +/* HPB-DMA slave IDs */ > +enum { > + HPBDMA_SLAVE_DUMMY, > + HPBDMA_SLAVE_SDHI0_TX, > + HPBDMA_SLAVE_SDHI0_RX, > + HPBDMA_SLAVE_SSI0_TX_ST, > + HPBDMA_SLAVE_SSI0_TX_MN, > + HPBDMA_SLAVE_SSI1_RX_ST, > + HPBDMA_SLAVE_SSI1_RX_MN, > +}; it should be in r8a7778.h Best regards --- Kuninori Morimoto ^ permalink raw reply [flat|nested] 30+ messages in thread
* Re: [PATCH 1/3] ARM: shmobile: use common platform header for HPB-DMAC 2013-07-01 4:11 ` Kuninori Morimoto @ 2013-07-01 12:11 ` Sergei Shtylyov -1 siblings, 0 replies; 30+ messages in thread From: Sergei Shtylyov @ 2013-07-01 12:11 UTC (permalink / raw) To: linux-arm-kernel Hello. On 01-07-2013 8:11, Kuninori Morimoto wrote: >> From: Max Filippov <max.filippov@cogentembedded.com> >> Use previously empty <mach/dma.h> to declare HPB-DMA slave IDs. >> Signed-off-by: Max Filippov <max.filippov@cogentembedded.com> >> [Sergei: changed the guard macro name, fixed comment, extended copyright.] >> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> >> --- >> arch/arm/mach-shmobile/include/mach/dma.h | 25 ++++++++++++++++++++++++- >> 1 file changed, 24 insertions(+), 1 deletion(-) >> Index: renesas/arch/arm/mach-shmobile/include/mach/dma.h >> =================================>> --- renesas.orig/arch/arm/mach-shmobile/include/mach/dma.h >> +++ renesas/arch/arm/mach-shmobile/include/mach/dma.h >> @@ -1 +1,24 @@ >> -/* empty */ >> +/* >> + * Copyright (C) 2011-2013 Renesas Electronics Corporation >> + * Copyright (C) 2013 Cogent Embedded, Inc. >> + * >> + * This program is free software; you can redistribute it and/or modify >> + * it under the terms of the GNU General Public License version 2 >> + * as published by the Free Software Foundation. >> + */ >> + >> +#ifndef __MACH_DMA_H >> +#define __MACH_DMA_H >> + >> +/* HPB-DMA slave IDs */ >> +enum { >> + HPBDMA_SLAVE_DUMMY, >> + HPBDMA_SLAVE_SDHI0_TX, >> + HPBDMA_SLAVE_SDHI0_RX, >> + HPBDMA_SLAVE_SSI0_TX_ST, >> + HPBDMA_SLAVE_SSI0_TX_MN, >> + HPBDMA_SLAVE_SSI1_RX_ST, >> + HPBDMA_SLAVE_SSI1_RX_MN, >> +}; > it should be in r8a7778.h It will be shared with R8A7779. WBR, Sergei ^ permalink raw reply [flat|nested] 30+ messages in thread
* [PATCH 1/3] ARM: shmobile: use common platform header for HPB-DMAC @ 2013-07-01 12:11 ` Sergei Shtylyov 0 siblings, 0 replies; 30+ messages in thread From: Sergei Shtylyov @ 2013-07-01 12:11 UTC (permalink / raw) To: linux-arm-kernel Hello. On 01-07-2013 8:11, Kuninori Morimoto wrote: >> From: Max Filippov <max.filippov@cogentembedded.com> >> Use previously empty <mach/dma.h> to declare HPB-DMA slave IDs. >> Signed-off-by: Max Filippov <max.filippov@cogentembedded.com> >> [Sergei: changed the guard macro name, fixed comment, extended copyright.] >> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> >> --- >> arch/arm/mach-shmobile/include/mach/dma.h | 25 ++++++++++++++++++++++++- >> 1 file changed, 24 insertions(+), 1 deletion(-) >> Index: renesas/arch/arm/mach-shmobile/include/mach/dma.h >> =================================================================== >> --- renesas.orig/arch/arm/mach-shmobile/include/mach/dma.h >> +++ renesas/arch/arm/mach-shmobile/include/mach/dma.h >> @@ -1 +1,24 @@ >> -/* empty */ >> +/* >> + * Copyright (C) 2011-2013 Renesas Electronics Corporation >> + * Copyright (C) 2013 Cogent Embedded, Inc. >> + * >> + * This program is free software; you can redistribute it and/or modify >> + * it under the terms of the GNU General Public License version 2 >> + * as published by the Free Software Foundation. >> + */ >> + >> +#ifndef __MACH_DMA_H >> +#define __MACH_DMA_H >> + >> +/* HPB-DMA slave IDs */ >> +enum { >> + HPBDMA_SLAVE_DUMMY, >> + HPBDMA_SLAVE_SDHI0_TX, >> + HPBDMA_SLAVE_SDHI0_RX, >> + HPBDMA_SLAVE_SSI0_TX_ST, >> + HPBDMA_SLAVE_SSI0_TX_MN, >> + HPBDMA_SLAVE_SSI1_RX_ST, >> + HPBDMA_SLAVE_SSI1_RX_MN, >> +}; > it should be in r8a7778.h It will be shared with R8A7779. WBR, Sergei ^ permalink raw reply [flat|nested] 30+ messages in thread
* Re: [PATCH 1/3] ARM: shmobile: use common platform header for HPB-DMAC 2013-07-01 12:11 ` Sergei Shtylyov @ 2013-07-18 21:56 ` Sergei Shtylyov -1 siblings, 0 replies; 30+ messages in thread From: Sergei Shtylyov @ 2013-07-18 21:56 UTC (permalink / raw) To: linux-arm-kernel Hello. On 07/01/2013 04:11 PM, Sergei Shtylyov wrote: >>> From: Max Filippov <max.filippov@cogentembedded.com> >>> Use previously empty <mach/dma.h> to declare HPB-DMA slave IDs. >>> Signed-off-by: Max Filippov <max.filippov@cogentembedded.com> >>> [Sergei: changed the guard macro name, fixed comment, extended copyright.] >>> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> >>> --- >>> arch/arm/mach-shmobile/include/mach/dma.h | 25 ++++++++++++++++++++++++- >>> 1 file changed, 24 insertions(+), 1 deletion(-) >>> Index: renesas/arch/arm/mach-shmobile/include/mach/dma.h >>> =================================>>> --- renesas.orig/arch/arm/mach-shmobile/include/mach/dma.h >>> +++ renesas/arch/arm/mach-shmobile/include/mach/dma.h >>> @@ -1 +1,24 @@ >>> -/* empty */ >>> +/* >>> + * Copyright (C) 2011-2013 Renesas Electronics Corporation >>> + * Copyright (C) 2013 Cogent Embedded, Inc. >>> + * >>> + * This program is free software; you can redistribute it and/or modify >>> + * it under the terms of the GNU General Public License version 2 >>> + * as published by the Free Software Foundation. >>> + */ >>> + >>> +#ifndef __MACH_DMA_H >>> +#define __MACH_DMA_H >>> + >>> +/* HPB-DMA slave IDs */ >>> +enum { >>> + HPBDMA_SLAVE_DUMMY, >>> + HPBDMA_SLAVE_SDHI0_TX, >>> + HPBDMA_SLAVE_SDHI0_RX, >>> + HPBDMA_SLAVE_SSI0_TX_ST, >>> + HPBDMA_SLAVE_SSI0_TX_MN, >>> + HPBDMA_SLAVE_SSI1_RX_ST, >>> + HPBDMA_SLAVE_SSI1_RX_MN, >>> +}; >> it should be in r8a7778.h > It will be shared with R8A7779. Apparently not anymore as this file is gone now. WBR, Sergei ^ permalink raw reply [flat|nested] 30+ messages in thread
* [PATCH 1/3] ARM: shmobile: use common platform header for HPB-DMAC @ 2013-07-18 21:56 ` Sergei Shtylyov 0 siblings, 0 replies; 30+ messages in thread From: Sergei Shtylyov @ 2013-07-18 21:56 UTC (permalink / raw) To: linux-arm-kernel Hello. On 07/01/2013 04:11 PM, Sergei Shtylyov wrote: >>> From: Max Filippov <max.filippov@cogentembedded.com> >>> Use previously empty <mach/dma.h> to declare HPB-DMA slave IDs. >>> Signed-off-by: Max Filippov <max.filippov@cogentembedded.com> >>> [Sergei: changed the guard macro name, fixed comment, extended copyright.] >>> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> >>> --- >>> arch/arm/mach-shmobile/include/mach/dma.h | 25 ++++++++++++++++++++++++- >>> 1 file changed, 24 insertions(+), 1 deletion(-) >>> Index: renesas/arch/arm/mach-shmobile/include/mach/dma.h >>> =================================================================== >>> --- renesas.orig/arch/arm/mach-shmobile/include/mach/dma.h >>> +++ renesas/arch/arm/mach-shmobile/include/mach/dma.h >>> @@ -1 +1,24 @@ >>> -/* empty */ >>> +/* >>> + * Copyright (C) 2011-2013 Renesas Electronics Corporation >>> + * Copyright (C) 2013 Cogent Embedded, Inc. >>> + * >>> + * This program is free software; you can redistribute it and/or modify >>> + * it under the terms of the GNU General Public License version 2 >>> + * as published by the Free Software Foundation. >>> + */ >>> + >>> +#ifndef __MACH_DMA_H >>> +#define __MACH_DMA_H >>> + >>> +/* HPB-DMA slave IDs */ >>> +enum { >>> + HPBDMA_SLAVE_DUMMY, >>> + HPBDMA_SLAVE_SDHI0_TX, >>> + HPBDMA_SLAVE_SDHI0_RX, >>> + HPBDMA_SLAVE_SSI0_TX_ST, >>> + HPBDMA_SLAVE_SSI0_TX_MN, >>> + HPBDMA_SLAVE_SSI1_RX_ST, >>> + HPBDMA_SLAVE_SSI1_RX_MN, >>> +}; >> it should be in r8a7778.h > It will be shared with R8A7779. Apparently not anymore as this file is gone now. WBR, Sergei ^ permalink raw reply [flat|nested] 30+ messages in thread
* Re: [PATCH 1/3] ARM: shmobile: use common platform header for HPB-DMAC 2013-06-29 22:45 ` Sergei Shtylyov @ 2013-07-02 4:59 ` Magnus Damm -1 siblings, 0 replies; 30+ messages in thread From: Magnus Damm @ 2013-07-02 4:59 UTC (permalink / raw) To: linux-arm-kernel Hi Sergei, On Sun, Jun 30, 2013 at 7:45 AM, Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> wrote: > From: Max Filippov <max.filippov@cogentembedded.com> > > Use previously empty <mach/dma.h> to declare HPB-DMA slave IDs. > > Signed-off-by: Max Filippov <max.filippov@cogentembedded.com> > [Sergei: changed the guard macro name, fixed comment, extended copyright.] > Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> > > --- > arch/arm/mach-shmobile/include/mach/dma.h | 25 ++++++++++++++++++++++++- > 1 file changed, 24 insertions(+), 1 deletion(-) Thanks for your patch. Can you please explain the reason why you put this shared information under mach/? From my point of view we should really have as few special things as possible under mach/. Starting to share DMA stuff here looks special. I would like mach-shmobile to be as standard as possible (basically only ARM code in the future), and in the case we have I/O devices or other IOMMUs or DMA controllers then we should use include/linux or that recently added platform data directory if needed. We need to make sure that our headers stay compatible with multiplatform support. I realize you may want to share this header between multiple SoCs, but I really want to avoid creating local common code under arch/arm/ that has nothing to do with ARM. Also, this is IMO a total non-issue, instead of merging code and making back porting more difficult for any particular SoC, then how about spending effort on the ling term instead, like DMA DT? So my recommendation is to keep this information together with each SoC. And also work on DT. In between have a coffee or work on DT, but please do not spend time on merging short term per-SoC information and making back porting more difficult. Thanks, / magnus ^ permalink raw reply [flat|nested] 30+ messages in thread
* [PATCH 1/3] ARM: shmobile: use common platform header for HPB-DMAC @ 2013-07-02 4:59 ` Magnus Damm 0 siblings, 0 replies; 30+ messages in thread From: Magnus Damm @ 2013-07-02 4:59 UTC (permalink / raw) To: linux-arm-kernel Hi Sergei, On Sun, Jun 30, 2013 at 7:45 AM, Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> wrote: > From: Max Filippov <max.filippov@cogentembedded.com> > > Use previously empty <mach/dma.h> to declare HPB-DMA slave IDs. > > Signed-off-by: Max Filippov <max.filippov@cogentembedded.com> > [Sergei: changed the guard macro name, fixed comment, extended copyright.] > Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> > > --- > arch/arm/mach-shmobile/include/mach/dma.h | 25 ++++++++++++++++++++++++- > 1 file changed, 24 insertions(+), 1 deletion(-) Thanks for your patch. Can you please explain the reason why you put this shared information under mach/? >From my point of view we should really have as few special things as possible under mach/. Starting to share DMA stuff here looks special. I would like mach-shmobile to be as standard as possible (basically only ARM code in the future), and in the case we have I/O devices or other IOMMUs or DMA controllers then we should use include/linux or that recently added platform data directory if needed. We need to make sure that our headers stay compatible with multiplatform support. I realize you may want to share this header between multiple SoCs, but I really want to avoid creating local common code under arch/arm/ that has nothing to do with ARM. Also, this is IMO a total non-issue, instead of merging code and making back porting more difficult for any particular SoC, then how about spending effort on the ling term instead, like DMA DT? So my recommendation is to keep this information together with each SoC. And also work on DT. In between have a coffee or work on DT, but please do not spend time on merging short term per-SoC information and making back porting more difficult. Thanks, / magnus ^ permalink raw reply [flat|nested] 30+ messages in thread
* Re: [PATCH 1/3] ARM: shmobile: use common platform header for HPB-DMAC 2013-07-02 4:59 ` Magnus Damm @ 2013-07-18 22:02 ` Sergei Shtylyov -1 siblings, 0 replies; 30+ messages in thread From: Sergei Shtylyov @ 2013-07-18 22:02 UTC (permalink / raw) To: linux-arm-kernel Hello. On 07/02/2013 08:59 AM, Magnus Damm wrote: >> From: Max Filippov <max.filippov@cogentembedded.com> >> Use previously empty <mach/dma.h> to declare HPB-DMA slave IDs. >> Signed-off-by: Max Filippov <max.filippov@cogentembedded.com> >> [Sergei: changed the guard macro name, fixed comment, extended copyright.] >> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> >> --- >> arch/arm/mach-shmobile/include/mach/dma.h | 25 ++++++++++++++++++++++++- >> 1 file changed, 24 insertions(+), 1 deletion(-) > Thanks for your patch. Can you please explain the reason why you put > this shared information under mach/? I'm afraid that's more a question to the original author, Phil Edworthy. He put this stuff into <mach/hpb-dmae.h>. > From my point of view we should really have as few special things as > possible under mach/. Starting to share DMA stuff here looks special. > I would like mach-shmobile to be as standard as possible (basically > only ARM code in the future), and in the case we have I/O devices or > other IOMMUs or DMA controllers then we should use include/linux or > that recently added platform data directory if needed. We need to make > sure that our headers stay compatible with multiplatform support. > I realize you may want to share this header between multiple SoCs, but > I really want to avoid creating local common code under arch/arm/ that > has nothing to do with ARM. Also, this is IMO a total non-issue, > instead of merging code and making back porting more difficult for any > particular SoC, then how about spending effort on the ling term > instead, like DMA DT? > So my recommendation is to keep this information together with each > SoC. And also work on DT. In between have a coffee or work on DT, but > please do not spend time on merging short term per-SoC information and > making back porting more difficult. OK. DT would be a next step I guess. > Thanks, > / magnus WBR, Sergei ^ permalink raw reply [flat|nested] 30+ messages in thread
* [PATCH 1/3] ARM: shmobile: use common platform header for HPB-DMAC @ 2013-07-18 22:02 ` Sergei Shtylyov 0 siblings, 0 replies; 30+ messages in thread From: Sergei Shtylyov @ 2013-07-18 22:02 UTC (permalink / raw) To: linux-arm-kernel Hello. On 07/02/2013 08:59 AM, Magnus Damm wrote: >> From: Max Filippov <max.filippov@cogentembedded.com> >> Use previously empty <mach/dma.h> to declare HPB-DMA slave IDs. >> Signed-off-by: Max Filippov <max.filippov@cogentembedded.com> >> [Sergei: changed the guard macro name, fixed comment, extended copyright.] >> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> >> --- >> arch/arm/mach-shmobile/include/mach/dma.h | 25 ++++++++++++++++++++++++- >> 1 file changed, 24 insertions(+), 1 deletion(-) > Thanks for your patch. Can you please explain the reason why you put > this shared information under mach/? I'm afraid that's more a question to the original author, Phil Edworthy. He put this stuff into <mach/hpb-dmae.h>. > From my point of view we should really have as few special things as > possible under mach/. Starting to share DMA stuff here looks special. > I would like mach-shmobile to be as standard as possible (basically > only ARM code in the future), and in the case we have I/O devices or > other IOMMUs or DMA controllers then we should use include/linux or > that recently added platform data directory if needed. We need to make > sure that our headers stay compatible with multiplatform support. > I realize you may want to share this header between multiple SoCs, but > I really want to avoid creating local common code under arch/arm/ that > has nothing to do with ARM. Also, this is IMO a total non-issue, > instead of merging code and making back porting more difficult for any > particular SoC, then how about spending effort on the ling term > instead, like DMA DT? > So my recommendation is to keep this information together with each > SoC. And also work on DT. In between have a coffee or work on DT, but > please do not spend time on merging short term per-SoC information and > making back porting more difficult. OK. DT would be a next step I guess. > Thanks, > / magnus WBR, Sergei ^ permalink raw reply [flat|nested] 30+ messages in thread
* Re: [PATCH 1/3] ARM: shmobile: use common platform header for HPB-DMAC 2013-07-18 22:02 ` Sergei Shtylyov @ 2013-07-19 0:16 ` Kuninori Morimoto -1 siblings, 0 replies; 30+ messages in thread From: Kuninori Morimoto @ 2013-07-19 0:16 UTC (permalink / raw) To: linux-arm-kernel Hi > >> From: Max Filippov <max.filippov@cogentembedded.com> > > >> Use previously empty <mach/dma.h> to declare HPB-DMA slave IDs. > > >> Signed-off-by: Max Filippov <max.filippov@cogentembedded.com> > >> [Sergei: changed the guard macro name, fixed comment, extended copyright.] > >> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> > > >> --- > >> arch/arm/mach-shmobile/include/mach/dma.h | 25 ++++++++++++++++++++++++- > >> 1 file changed, 24 insertions(+), 1 deletion(-) > > > Thanks for your patch. Can you please explain the reason why you put > > this shared information under mach/? > > I'm afraid that's more a question to the original author, Phil Edworthy. > He put this stuff into <mach/hpb-dmae.h>. Then, why you could send this patch instead of original author, even though you can't answer any question about it ? Best regards --- Kuninori Morimoto ^ permalink raw reply [flat|nested] 30+ messages in thread
* [PATCH 1/3] ARM: shmobile: use common platform header for HPB-DMAC @ 2013-07-19 0:16 ` Kuninori Morimoto 0 siblings, 0 replies; 30+ messages in thread From: Kuninori Morimoto @ 2013-07-19 0:16 UTC (permalink / raw) To: linux-arm-kernel Hi > >> From: Max Filippov <max.filippov@cogentembedded.com> > > >> Use previously empty <mach/dma.h> to declare HPB-DMA slave IDs. > > >> Signed-off-by: Max Filippov <max.filippov@cogentembedded.com> > >> [Sergei: changed the guard macro name, fixed comment, extended copyright.] > >> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> > > >> --- > >> arch/arm/mach-shmobile/include/mach/dma.h | 25 ++++++++++++++++++++++++- > >> 1 file changed, 24 insertions(+), 1 deletion(-) > > > Thanks for your patch. Can you please explain the reason why you put > > this shared information under mach/? > > I'm afraid that's more a question to the original author, Phil Edworthy. > He put this stuff into <mach/hpb-dmae.h>. Then, why you could send this patch instead of original author, even though you can't answer any question about it ? Best regards --- Kuninori Morimoto ^ permalink raw reply [flat|nested] 30+ messages in thread
* Re: [PATCH 1/3] ARM: shmobile: use common platform header for HPB-DMAC 2013-07-19 0:16 ` Kuninori Morimoto @ 2013-07-19 11:17 ` Sergei Shtylyov -1 siblings, 0 replies; 30+ messages in thread From: Sergei Shtylyov @ 2013-07-19 11:17 UTC (permalink / raw) To: linux-arm-kernel Hello. On 19-07-2013 4:16, Kuninori Morimoto wrote: >>>> From: Max Filippov <max.filippov@cogentembedded.com> >>>> Use previously empty <mach/dma.h> to declare HPB-DMA slave IDs. >>>> Signed-off-by: Max Filippov <max.filippov@cogentembedded.com> >>>> [Sergei: changed the guard macro name, fixed comment, extended copyright.] >>>> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> >>>> --- >>>> arch/arm/mach-shmobile/include/mach/dma.h | 25 ++++++++++++++++++++++++- >>>> 1 file changed, 24 insertions(+), 1 deletion(-) >>> Thanks for your patch. Can you please explain the reason why you put >>> this shared information under mach/? >> I'm afraid that's more a question to the original author, Phil Edworthy. >> He put this stuff into <mach/hpb-dmae.h>. > Then, why you could send this patch instead of original author, Ask Phil why didn't he upstream his stuff. We, unlike him, do have obligation to upstream it. > even though you can't answer any question about it ? I'm not the author of the patch, let me remind you, Max Filippov is. He was on vacations. > Best regards > --- > Kuninori Morimoto WBR, Sergei ^ permalink raw reply [flat|nested] 30+ messages in thread
* [PATCH 1/3] ARM: shmobile: use common platform header for HPB-DMAC @ 2013-07-19 11:17 ` Sergei Shtylyov 0 siblings, 0 replies; 30+ messages in thread From: Sergei Shtylyov @ 2013-07-19 11:17 UTC (permalink / raw) To: linux-arm-kernel Hello. On 19-07-2013 4:16, Kuninori Morimoto wrote: >>>> From: Max Filippov <max.filippov@cogentembedded.com> >>>> Use previously empty <mach/dma.h> to declare HPB-DMA slave IDs. >>>> Signed-off-by: Max Filippov <max.filippov@cogentembedded.com> >>>> [Sergei: changed the guard macro name, fixed comment, extended copyright.] >>>> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> >>>> --- >>>> arch/arm/mach-shmobile/include/mach/dma.h | 25 ++++++++++++++++++++++++- >>>> 1 file changed, 24 insertions(+), 1 deletion(-) >>> Thanks for your patch. Can you please explain the reason why you put >>> this shared information under mach/? >> I'm afraid that's more a question to the original author, Phil Edworthy. >> He put this stuff into <mach/hpb-dmae.h>. > Then, why you could send this patch instead of original author, Ask Phil why didn't he upstream his stuff. We, unlike him, do have obligation to upstream it. > even though you can't answer any question about it ? I'm not the author of the patch, let me remind you, Max Filippov is. He was on vacations. > Best regards > --- > Kuninori Morimoto WBR, Sergei ^ permalink raw reply [flat|nested] 30+ messages in thread
* Re: [PATCH 1/3] ARM: shmobile: use common platform header for HPB-DMAC 2013-07-19 11:17 ` Sergei Shtylyov @ 2013-07-19 11:32 ` Sergei Shtylyov -1 siblings, 0 replies; 30+ messages in thread From: Sergei Shtylyov @ 2013-07-19 11:32 UTC (permalink / raw) To: linux-arm-kernel Hello. On 19-07-2013 15:17, Sergei Shtylyov wrote: >>>>> From: Max Filippov <max.filippov@cogentembedded.com> >>>>> Use previously empty <mach/dma.h> to declare HPB-DMA slave IDs. >>>>> Signed-off-by: Max Filippov <max.filippov@cogentembedded.com> >>>>> [Sergei: changed the guard macro name, fixed comment, extended copyright.] >>>>> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> >>>>> --- >>>>> arch/arm/mach-shmobile/include/mach/dma.h | 25 >>>>> ++++++++++++++++++++++++- >>>>> 1 file changed, 24 insertions(+), 1 deletion(-) >>>> Thanks for your patch. Can you please explain the reason why you put >>>> this shared information under mach/? IIUC, because the only 2 file using it were under arch/arm/mach-shmobile/. >>> I'm afraid that's more a question to the original author, Phil Edworthy. >>> He put this stuff into <mach/hpb-dmae.h>. >> Then, why you could send this patch instead of original author, > Ask Phil why didn't he upstream his stuff. We, unlike him, do have > obligation to upstream it. >> even though you can't answer any question about it ? > I'm not the author of the patch, let me remind you, Max Filippov is. > He was on vacations. Phil's patches were only for R8A7778, IIRC (yet, the platfrom DMA code was placed in a separate file, not r8a7778.c). Max decided to use the <mach/hpb-dmae.h> header for sharing the DMA channel IDs between R8A777[89], I agreed but suggested to use the empty <mach/dma.h> instead. >> Best regards >> --- >> Kuninori Morimoto WBR, Sergei ^ permalink raw reply [flat|nested] 30+ messages in thread
* [PATCH 1/3] ARM: shmobile: use common platform header for HPB-DMAC @ 2013-07-19 11:32 ` Sergei Shtylyov 0 siblings, 0 replies; 30+ messages in thread From: Sergei Shtylyov @ 2013-07-19 11:32 UTC (permalink / raw) To: linux-arm-kernel Hello. On 19-07-2013 15:17, Sergei Shtylyov wrote: >>>>> From: Max Filippov <max.filippov@cogentembedded.com> >>>>> Use previously empty <mach/dma.h> to declare HPB-DMA slave IDs. >>>>> Signed-off-by: Max Filippov <max.filippov@cogentembedded.com> >>>>> [Sergei: changed the guard macro name, fixed comment, extended copyright.] >>>>> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> >>>>> --- >>>>> arch/arm/mach-shmobile/include/mach/dma.h | 25 >>>>> ++++++++++++++++++++++++- >>>>> 1 file changed, 24 insertions(+), 1 deletion(-) >>>> Thanks for your patch. Can you please explain the reason why you put >>>> this shared information under mach/? IIUC, because the only 2 file using it were under arch/arm/mach-shmobile/. >>> I'm afraid that's more a question to the original author, Phil Edworthy. >>> He put this stuff into <mach/hpb-dmae.h>. >> Then, why you could send this patch instead of original author, > Ask Phil why didn't he upstream his stuff. We, unlike him, do have > obligation to upstream it. >> even though you can't answer any question about it ? > I'm not the author of the patch, let me remind you, Max Filippov is. > He was on vacations. Phil's patches were only for R8A7778, IIRC (yet, the platfrom DMA code was placed in a separate file, not r8a7778.c). Max decided to use the <mach/hpb-dmae.h> header for sharing the DMA channel IDs between R8A777[89], I agreed but suggested to use the empty <mach/dma.h> instead. >> Best regards >> --- >> Kuninori Morimoto WBR, Sergei ^ permalink raw reply [flat|nested] 30+ messages in thread
* [PATCH 2/3] ARM: shmobile: r8a7778: add HPB-DMAC support 2013-06-29 22:43 ` Sergei Shtylyov @ 2013-06-29 22:48 ` Sergei Shtylyov -1 siblings, 0 replies; 30+ messages in thread From: Sergei Shtylyov @ 2013-06-29 22:48 UTC (permalink / raw) To: linux-arm-kernel From: Max Filippov <max.filippov@cogentembedded.com> Add HPB-DMAC platform device on R8A7778 SoC along with its slave and channel configurations. Signed-off-by: Max Filippov <max.filippov@cogentembedded.com> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> --- arch/arm/mach-shmobile/setup-r8a7778.c | 115 +++++++++++++++++++++++++++++++++ 1 file changed, 115 insertions(+) Index: renesas/arch/arm/mach-shmobile/setup-r8a7778.c =================================--- renesas.orig/arch/arm/mach-shmobile/setup-r8a7778.c +++ renesas/arch/arm/mach-shmobile/setup-r8a7778.c @@ -24,6 +24,7 @@ #include <linux/irqchip/arm-gic.h> #include <linux/of.h> #include <linux/of_platform.h> +#include <linux/platform_data/dma-rcar-hpbdma.h> #include <linux/platform_data/gpio-rcar.h> #include <linux/platform_data/irq-renesas-intc-irqpin.h> #include <linux/platform_device.h> @@ -36,6 +37,7 @@ #include <linux/usb/ehci_pdriver.h> #include <linux/usb/ohci_pdriver.h> #include <linux/dma-mapping.h> +#include <mach/dma.h> #include <mach/irqs.h> #include <mach/r8a7778.h> #include <mach/common.h> @@ -333,6 +335,118 @@ void __init r8a7778_add_mmc_device(struc info, sizeof(*info)); } +/* HPB-DMA */ + +/* Asynchronous mode register (ASYNCMDR) bits */ +#define ASYNCMDR_ASMD22_MASK BIT(2) /* SDHI0 */ +#define ASYNCMDR_ASMD22_SINGLE BIT(2) /* SDHI0 */ +#define ASYNCMDR_ASMD22_MULTI 0 /* SDHI0 */ +#define ASYNCMDR_ASMD21_MASK BIT(1) /* SDHI0 */ +#define ASYNCMDR_ASMD21_SINGLE BIT(1) /* SDHI0 */ +#define ASYNCMDR_ASMD21_MULTI 0 /* SDHI0 */ + +static const struct hpb_dmae_slave_config hpb_dmae_slaves[] = { + { + .id = HPBDMA_SLAVE_SDHI0_TX, + .addr = 0xffe4c000 + 0x30, + .dcr = DCR_SPDS_16BIT | DCR_DMDL | DCR_DPDS_16BIT, + .rstr = ASYNCRSTR_ASRST21 | ASYNCRSTR_ASRST22 | + ASYNCRSTR_ASRST23, + .mdr = ASYNCMDR_ASMD21_SINGLE, + .mdm = ASYNCMDR_ASMD21_MASK, + .port = 0x0D0C, + .flags = HPB_DMAE_SET_ASYNC_RESET | HPB_DMAE_SET_ASYNC_MODE, + .dma_ch = 21, + }, { + .id = HPBDMA_SLAVE_SDHI0_RX, + .addr = 0xffe4c000 + 0x30, + .dcr = DCR_SMDL | DCR_SPDS_16BIT | DCR_DPDS_16BIT, + .rstr = ASYNCRSTR_ASRST21 | ASYNCRSTR_ASRST22 | + ASYNCRSTR_ASRST23, + .mdr = ASYNCMDR_ASMD22_SINGLE, + .mdm = ASYNCMDR_ASMD22_MASK, + .port = 0x0D0C, + .flags = HPB_DMAE_SET_ASYNC_RESET | HPB_DMAE_SET_ASYNC_MODE, + .dma_ch = 22, + }, { + .id = HPBDMA_SLAVE_SSI0_TX_ST, + .addr = 0xffd90000 + 0x1008, + .dcr = DCR_CT | DCR_DIP | DCR_SPDS_32BIT | DCR_DMDL | + DCR_DPDS_32BIT, + .port = 0x0000, + .dma_ch = 28, + }, { + .id = HPBDMA_SLAVE_SSI0_TX_MN, + .addr = 0xffd90000 + 0x1008, + .dcr = DCR_CT | DCR_DIP | DCR_SPDS_16BIT | DCR_DMDL | + DCR_DPDS_16BIT, + .port = 0x0000, + .dma_ch = 28, + }, { + .id = HPBDMA_SLAVE_SSI1_RX_ST, + .addr = 0xffd90000 + 0x104c, + .dcr = DCR_CT | DCR_DIP | DCR_SMDL | DCR_SPDS_32BIT | + DCR_DPDAM | DCR_DPDS_32BIT, + .port = 0x0101, + .dma_ch = 29, + }, { + .id = HPBDMA_SLAVE_SSI1_RX_MN, + .addr = 0xffd90000 + 0x104c, + .dcr = DCR_CT | DCR_DIP | DCR_SMDL | DCR_SPDS_16BIT | + DCR_DPDAM | DCR_DPDS_16BIT, + .port = 0x0101, + .dma_ch = 29, + }, +}; + +static const struct hpb_dmae_channel hpb_dmae_channels[] = { + /* ch.21 SD0 */ + HPB_DMAE_CHANNEL(0x7e, HPBDMA_SLAVE_SDHI0_TX), + /* ch.22 SD0 */ + HPB_DMAE_CHANNEL(0x7e, HPBDMA_SLAVE_SDHI0_RX), + /* ch.28 SSI0 */ + HPB_DMAE_CHANNEL(0x7f, HPBDMA_SLAVE_SSI0_TX_ST), + HPB_DMAE_CHANNEL(0x7f, HPBDMA_SLAVE_SSI0_TX_MN), + /* ch.29 SSI1 */ + HPB_DMAE_CHANNEL(0x7f, HPBDMA_SLAVE_SSI1_RX_ST), + HPB_DMAE_CHANNEL(0x7f, HPBDMA_SLAVE_SSI1_RX_MN), +}; + +static struct hpb_dmae_pdata dma_platform_data __initdata = { + .slaves = hpb_dmae_slaves, + .num_slaves = ARRAY_SIZE(hpb_dmae_slaves), + .channels = hpb_dmae_channels, + .num_channels = ARRAY_SIZE(hpb_dmae_channels), + .ts_shift = { + [XMIT_SZ_8BIT] = 0, + [XMIT_SZ_16BIT] = 1, + [XMIT_SZ_32BIT] = 2, + }, + .num_hw_channels = 39, +}; + +static struct resource hpb_dmae_resources[] __initdata = { + /* Channel registers */ + DEFINE_RES_MEM(0xffc08000, 0x1000), + /* Common registers */ + DEFINE_RES_MEM(0xffc09000, 0x170), + /* Asynchronous reset registers */ + DEFINE_RES_MEM(0xffc00300, 4), + /* Asynchronous mode registers */ + DEFINE_RES_MEM(0xffc00400, 4), + /* IRQ for DMA channels */ + DEFINE_RES_NAMED(gic_iid(0x7b), 5, NULL, IORESOURCE_IRQ), +}; + +static void __init r8a7778_register_hpb_dmae(void) +{ + platform_device_register_resndata(&platform_bus, "hpb-dma-engine", -1, + hpb_dmae_resources, + ARRAY_SIZE(hpb_dmae_resources), + &dma_platform_data, + sizeof(dma_platform_data)); +} + void __init r8a7778_add_standard_devices(void) { int i; @@ -355,6 +469,7 @@ void __init r8a7778_add_standard_devices r8a7778_register_tmu(0); r8a7778_register_tmu(1); + r8a7778_register_hpb_dmae(); } void __init r8a7778_init_late(void) ^ permalink raw reply [flat|nested] 30+ messages in thread
* [PATCH 2/3] ARM: shmobile: r8a7778: add HPB-DMAC support @ 2013-06-29 22:48 ` Sergei Shtylyov 0 siblings, 0 replies; 30+ messages in thread From: Sergei Shtylyov @ 2013-06-29 22:48 UTC (permalink / raw) To: linux-arm-kernel From: Max Filippov <max.filippov@cogentembedded.com> Add HPB-DMAC platform device on R8A7778 SoC along with its slave and channel configurations. Signed-off-by: Max Filippov <max.filippov@cogentembedded.com> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> --- arch/arm/mach-shmobile/setup-r8a7778.c | 115 +++++++++++++++++++++++++++++++++ 1 file changed, 115 insertions(+) Index: renesas/arch/arm/mach-shmobile/setup-r8a7778.c =================================================================== --- renesas.orig/arch/arm/mach-shmobile/setup-r8a7778.c +++ renesas/arch/arm/mach-shmobile/setup-r8a7778.c @@ -24,6 +24,7 @@ #include <linux/irqchip/arm-gic.h> #include <linux/of.h> #include <linux/of_platform.h> +#include <linux/platform_data/dma-rcar-hpbdma.h> #include <linux/platform_data/gpio-rcar.h> #include <linux/platform_data/irq-renesas-intc-irqpin.h> #include <linux/platform_device.h> @@ -36,6 +37,7 @@ #include <linux/usb/ehci_pdriver.h> #include <linux/usb/ohci_pdriver.h> #include <linux/dma-mapping.h> +#include <mach/dma.h> #include <mach/irqs.h> #include <mach/r8a7778.h> #include <mach/common.h> @@ -333,6 +335,118 @@ void __init r8a7778_add_mmc_device(struc info, sizeof(*info)); } +/* HPB-DMA */ + +/* Asynchronous mode register (ASYNCMDR) bits */ +#define ASYNCMDR_ASMD22_MASK BIT(2) /* SDHI0 */ +#define ASYNCMDR_ASMD22_SINGLE BIT(2) /* SDHI0 */ +#define ASYNCMDR_ASMD22_MULTI 0 /* SDHI0 */ +#define ASYNCMDR_ASMD21_MASK BIT(1) /* SDHI0 */ +#define ASYNCMDR_ASMD21_SINGLE BIT(1) /* SDHI0 */ +#define ASYNCMDR_ASMD21_MULTI 0 /* SDHI0 */ + +static const struct hpb_dmae_slave_config hpb_dmae_slaves[] = { + { + .id = HPBDMA_SLAVE_SDHI0_TX, + .addr = 0xffe4c000 + 0x30, + .dcr = DCR_SPDS_16BIT | DCR_DMDL | DCR_DPDS_16BIT, + .rstr = ASYNCRSTR_ASRST21 | ASYNCRSTR_ASRST22 | + ASYNCRSTR_ASRST23, + .mdr = ASYNCMDR_ASMD21_SINGLE, + .mdm = ASYNCMDR_ASMD21_MASK, + .port = 0x0D0C, + .flags = HPB_DMAE_SET_ASYNC_RESET | HPB_DMAE_SET_ASYNC_MODE, + .dma_ch = 21, + }, { + .id = HPBDMA_SLAVE_SDHI0_RX, + .addr = 0xffe4c000 + 0x30, + .dcr = DCR_SMDL | DCR_SPDS_16BIT | DCR_DPDS_16BIT, + .rstr = ASYNCRSTR_ASRST21 | ASYNCRSTR_ASRST22 | + ASYNCRSTR_ASRST23, + .mdr = ASYNCMDR_ASMD22_SINGLE, + .mdm = ASYNCMDR_ASMD22_MASK, + .port = 0x0D0C, + .flags = HPB_DMAE_SET_ASYNC_RESET | HPB_DMAE_SET_ASYNC_MODE, + .dma_ch = 22, + }, { + .id = HPBDMA_SLAVE_SSI0_TX_ST, + .addr = 0xffd90000 + 0x1008, + .dcr = DCR_CT | DCR_DIP | DCR_SPDS_32BIT | DCR_DMDL | + DCR_DPDS_32BIT, + .port = 0x0000, + .dma_ch = 28, + }, { + .id = HPBDMA_SLAVE_SSI0_TX_MN, + .addr = 0xffd90000 + 0x1008, + .dcr = DCR_CT | DCR_DIP | DCR_SPDS_16BIT | DCR_DMDL | + DCR_DPDS_16BIT, + .port = 0x0000, + .dma_ch = 28, + }, { + .id = HPBDMA_SLAVE_SSI1_RX_ST, + .addr = 0xffd90000 + 0x104c, + .dcr = DCR_CT | DCR_DIP | DCR_SMDL | DCR_SPDS_32BIT | + DCR_DPDAM | DCR_DPDS_32BIT, + .port = 0x0101, + .dma_ch = 29, + }, { + .id = HPBDMA_SLAVE_SSI1_RX_MN, + .addr = 0xffd90000 + 0x104c, + .dcr = DCR_CT | DCR_DIP | DCR_SMDL | DCR_SPDS_16BIT | + DCR_DPDAM | DCR_DPDS_16BIT, + .port = 0x0101, + .dma_ch = 29, + }, +}; + +static const struct hpb_dmae_channel hpb_dmae_channels[] = { + /* ch.21 SD0 */ + HPB_DMAE_CHANNEL(0x7e, HPBDMA_SLAVE_SDHI0_TX), + /* ch.22 SD0 */ + HPB_DMAE_CHANNEL(0x7e, HPBDMA_SLAVE_SDHI0_RX), + /* ch.28 SSI0 */ + HPB_DMAE_CHANNEL(0x7f, HPBDMA_SLAVE_SSI0_TX_ST), + HPB_DMAE_CHANNEL(0x7f, HPBDMA_SLAVE_SSI0_TX_MN), + /* ch.29 SSI1 */ + HPB_DMAE_CHANNEL(0x7f, HPBDMA_SLAVE_SSI1_RX_ST), + HPB_DMAE_CHANNEL(0x7f, HPBDMA_SLAVE_SSI1_RX_MN), +}; + +static struct hpb_dmae_pdata dma_platform_data __initdata = { + .slaves = hpb_dmae_slaves, + .num_slaves = ARRAY_SIZE(hpb_dmae_slaves), + .channels = hpb_dmae_channels, + .num_channels = ARRAY_SIZE(hpb_dmae_channels), + .ts_shift = { + [XMIT_SZ_8BIT] = 0, + [XMIT_SZ_16BIT] = 1, + [XMIT_SZ_32BIT] = 2, + }, + .num_hw_channels = 39, +}; + +static struct resource hpb_dmae_resources[] __initdata = { + /* Channel registers */ + DEFINE_RES_MEM(0xffc08000, 0x1000), + /* Common registers */ + DEFINE_RES_MEM(0xffc09000, 0x170), + /* Asynchronous reset registers */ + DEFINE_RES_MEM(0xffc00300, 4), + /* Asynchronous mode registers */ + DEFINE_RES_MEM(0xffc00400, 4), + /* IRQ for DMA channels */ + DEFINE_RES_NAMED(gic_iid(0x7b), 5, NULL, IORESOURCE_IRQ), +}; + +static void __init r8a7778_register_hpb_dmae(void) +{ + platform_device_register_resndata(&platform_bus, "hpb-dma-engine", -1, + hpb_dmae_resources, + ARRAY_SIZE(hpb_dmae_resources), + &dma_platform_data, + sizeof(dma_platform_data)); +} + void __init r8a7778_add_standard_devices(void) { int i; @@ -355,6 +469,7 @@ void __init r8a7778_add_standard_devices r8a7778_register_tmu(0); r8a7778_register_tmu(1); + r8a7778_register_hpb_dmae(); } void __init r8a7778_init_late(void) ^ permalink raw reply [flat|nested] 30+ messages in thread
* Re: [PATCH 2/3] ARM: shmobile: r8a7778: add HPB-DMAC support 2013-06-29 22:48 ` Sergei Shtylyov @ 2013-07-01 4:12 ` Kuninori Morimoto -1 siblings, 0 replies; 30+ messages in thread From: Kuninori Morimoto @ 2013-07-01 4:12 UTC (permalink / raw) To: linux-arm-kernel Hi > From: Max Filippov <max.filippov@cogentembedded.com> > > Add HPB-DMAC platform device on R8A7778 SoC along with its slave and channel > configurations. > > Signed-off-by: Max Filippov <max.filippov@cogentembedded.com> > Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> > > --- (snip) > +static const struct hpb_dmae_slave_config hpb_dmae_slaves[] = { > + { > + .id = HPBDMA_SLAVE_SDHI0_TX, > + .addr = 0xffe4c000 + 0x30, > + .dcr = DCR_SPDS_16BIT | DCR_DMDL | DCR_DPDS_16BIT, > + .rstr = ASYNCRSTR_ASRST21 | ASYNCRSTR_ASRST22 | > + ASYNCRSTR_ASRST23, > + .mdr = ASYNCMDR_ASMD21_SINGLE, > + .mdm = ASYNCMDR_ASMD21_MASK, > + .port = 0x0D0C, > + .flags = HPB_DMAE_SET_ASYNC_RESET | HPB_DMAE_SET_ASYNC_MODE, > + .dma_ch = 21, > + }, { > + .id = HPBDMA_SLAVE_SDHI0_RX, > + .addr = 0xffe4c000 + 0x30, > + .dcr = DCR_SMDL | DCR_SPDS_16BIT | DCR_DPDS_16BIT, > + .rstr = ASYNCRSTR_ASRST21 | ASYNCRSTR_ASRST22 | > + ASYNCRSTR_ASRST23, > + .mdr = ASYNCMDR_ASMD22_SINGLE, > + .mdm = ASYNCMDR_ASMD22_MASK, > + .port = 0x0D0C, > + .flags = HPB_DMAE_SET_ASYNC_RESET | HPB_DMAE_SET_ASYNC_MODE, > + .dma_ch = 22, > + }, { > + .id = HPBDMA_SLAVE_SSI0_TX_ST, > + .addr = 0xffd90000 + 0x1008, > + .dcr = DCR_CT | DCR_DIP | DCR_SPDS_32BIT | DCR_DMDL | > + DCR_DPDS_32BIT, > + .port = 0x0000, > + .dma_ch = 28, > + }, { > + .id = HPBDMA_SLAVE_SSI0_TX_MN, > + .addr = 0xffd90000 + 0x1008, > + .dcr = DCR_CT | DCR_DIP | DCR_SPDS_16BIT | DCR_DMDL | > + DCR_DPDS_16BIT, > + .port = 0x0000, > + .dma_ch = 28, > + }, { > + .id = HPBDMA_SLAVE_SSI1_RX_ST, > + .addr = 0xffd90000 + 0x104c, > + .dcr = DCR_CT | DCR_DIP | DCR_SMDL | DCR_SPDS_32BIT | > + DCR_DPDAM | DCR_DPDS_32BIT, > + .port = 0x0101, > + .dma_ch = 29, > + }, { > + .id = HPBDMA_SLAVE_SSI1_RX_MN, > + .addr = 0xffd90000 + 0x104c, > + .dcr = DCR_CT | DCR_DIP | DCR_SMDL | DCR_SPDS_16BIT | > + DCR_DPDAM | DCR_DPDS_16BIT, > + .port = 0x0101, > + .dma_ch = 29, > + }, > +}; How did you test SSI ? remove these Best regards --- Kuninori Morimoto ^ permalink raw reply [flat|nested] 30+ messages in thread
* [PATCH 2/3] ARM: shmobile: r8a7778: add HPB-DMAC support @ 2013-07-01 4:12 ` Kuninori Morimoto 0 siblings, 0 replies; 30+ messages in thread From: Kuninori Morimoto @ 2013-07-01 4:12 UTC (permalink / raw) To: linux-arm-kernel Hi > From: Max Filippov <max.filippov@cogentembedded.com> > > Add HPB-DMAC platform device on R8A7778 SoC along with its slave and channel > configurations. > > Signed-off-by: Max Filippov <max.filippov@cogentembedded.com> > Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> > > --- (snip) > +static const struct hpb_dmae_slave_config hpb_dmae_slaves[] = { > + { > + .id = HPBDMA_SLAVE_SDHI0_TX, > + .addr = 0xffe4c000 + 0x30, > + .dcr = DCR_SPDS_16BIT | DCR_DMDL | DCR_DPDS_16BIT, > + .rstr = ASYNCRSTR_ASRST21 | ASYNCRSTR_ASRST22 | > + ASYNCRSTR_ASRST23, > + .mdr = ASYNCMDR_ASMD21_SINGLE, > + .mdm = ASYNCMDR_ASMD21_MASK, > + .port = 0x0D0C, > + .flags = HPB_DMAE_SET_ASYNC_RESET | HPB_DMAE_SET_ASYNC_MODE, > + .dma_ch = 21, > + }, { > + .id = HPBDMA_SLAVE_SDHI0_RX, > + .addr = 0xffe4c000 + 0x30, > + .dcr = DCR_SMDL | DCR_SPDS_16BIT | DCR_DPDS_16BIT, > + .rstr = ASYNCRSTR_ASRST21 | ASYNCRSTR_ASRST22 | > + ASYNCRSTR_ASRST23, > + .mdr = ASYNCMDR_ASMD22_SINGLE, > + .mdm = ASYNCMDR_ASMD22_MASK, > + .port = 0x0D0C, > + .flags = HPB_DMAE_SET_ASYNC_RESET | HPB_DMAE_SET_ASYNC_MODE, > + .dma_ch = 22, > + }, { > + .id = HPBDMA_SLAVE_SSI0_TX_ST, > + .addr = 0xffd90000 + 0x1008, > + .dcr = DCR_CT | DCR_DIP | DCR_SPDS_32BIT | DCR_DMDL | > + DCR_DPDS_32BIT, > + .port = 0x0000, > + .dma_ch = 28, > + }, { > + .id = HPBDMA_SLAVE_SSI0_TX_MN, > + .addr = 0xffd90000 + 0x1008, > + .dcr = DCR_CT | DCR_DIP | DCR_SPDS_16BIT | DCR_DMDL | > + DCR_DPDS_16BIT, > + .port = 0x0000, > + .dma_ch = 28, > + }, { > + .id = HPBDMA_SLAVE_SSI1_RX_ST, > + .addr = 0xffd90000 + 0x104c, > + .dcr = DCR_CT | DCR_DIP | DCR_SMDL | DCR_SPDS_32BIT | > + DCR_DPDAM | DCR_DPDS_32BIT, > + .port = 0x0101, > + .dma_ch = 29, > + }, { > + .id = HPBDMA_SLAVE_SSI1_RX_MN, > + .addr = 0xffd90000 + 0x104c, > + .dcr = DCR_CT | DCR_DIP | DCR_SMDL | DCR_SPDS_16BIT | > + DCR_DPDAM | DCR_DPDS_16BIT, > + .port = 0x0101, > + .dma_ch = 29, > + }, > +}; How did you test SSI ? remove these Best regards --- Kuninori Morimoto ^ permalink raw reply [flat|nested] 30+ messages in thread
* Re: [PATCH 2/3] ARM: shmobile: r8a7778: add HPB-DMAC support 2013-07-01 4:12 ` Kuninori Morimoto @ 2013-07-18 22:04 ` Sergei Shtylyov -1 siblings, 0 replies; 30+ messages in thread From: Sergei Shtylyov @ 2013-07-18 22:04 UTC (permalink / raw) To: linux-arm-kernel Hello. On 07/01/2013 08:12 AM, Kuninori Morimoto wrote: >> From: Max Filippov <max.filippov@cogentembedded.com> >> Add HPB-DMAC platform device on R8A7778 SoC along with its slave and channel >> configurations. >> Signed-off-by: Max Filippov <max.filippov@cogentembedded.com> >> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> >> --- > (snip) >> +static const struct hpb_dmae_slave_config hpb_dmae_slaves[] = { >> + { >> + .id = HPBDMA_SLAVE_SDHI0_TX, >> + .addr = 0xffe4c000 + 0x30, >> + .dcr = DCR_SPDS_16BIT | DCR_DMDL | DCR_DPDS_16BIT, >> + .rstr = ASYNCRSTR_ASRST21 | ASYNCRSTR_ASRST22 | >> + ASYNCRSTR_ASRST23, >> + .mdr = ASYNCMDR_ASMD21_SINGLE, >> + .mdm = ASYNCMDR_ASMD21_MASK, >> + .port = 0x0D0C, >> + .flags = HPB_DMAE_SET_ASYNC_RESET | HPB_DMAE_SET_ASYNC_MODE, >> + .dma_ch = 21, >> + }, { >> + .id = HPBDMA_SLAVE_SDHI0_RX, >> + .addr = 0xffe4c000 + 0x30, >> + .dcr = DCR_SMDL | DCR_SPDS_16BIT | DCR_DPDS_16BIT, >> + .rstr = ASYNCRSTR_ASRST21 | ASYNCRSTR_ASRST22 | >> + ASYNCRSTR_ASRST23, >> + .mdr = ASYNCMDR_ASMD22_SINGLE, >> + .mdm = ASYNCMDR_ASMD22_MASK, >> + .port = 0x0D0C, >> + .flags = HPB_DMAE_SET_ASYNC_RESET | HPB_DMAE_SET_ASYNC_MODE, >> + .dma_ch = 22, >> + }, { >> + .id = HPBDMA_SLAVE_SSI0_TX_ST, >> + .addr = 0xffd90000 + 0x1008, >> + .dcr = DCR_CT | DCR_DIP | DCR_SPDS_32BIT | DCR_DMDL | >> + DCR_DPDS_32BIT, >> + .port = 0x0000, >> + .dma_ch = 28, >> + }, { >> + .id = HPBDMA_SLAVE_SSI0_TX_MN, >> + .addr = 0xffd90000 + 0x1008, >> + .dcr = DCR_CT | DCR_DIP | DCR_SPDS_16BIT | DCR_DMDL | >> + DCR_DPDS_16BIT, >> + .port = 0x0000, >> + .dma_ch = 28, >> + }, { >> + .id = HPBDMA_SLAVE_SSI1_RX_ST, >> + .addr = 0xffd90000 + 0x104c, >> + .dcr = DCR_CT | DCR_DIP | DCR_SMDL | DCR_SPDS_32BIT | >> + DCR_DPDAM | DCR_DPDS_32BIT, >> + .port = 0x0101, >> + .dma_ch = 29, >> + }, { >> + .id = HPBDMA_SLAVE_SSI1_RX_MN, >> + .addr = 0xffd90000 + 0x104c, >> + .dcr = DCR_CT | DCR_DIP | DCR_SMDL | DCR_SPDS_16BIT | >> + DCR_DPDAM | DCR_DPDS_16BIT, >> + .port = 0x0101, >> + .dma_ch = 29, >> + }, >> +}; > How did you test SSI ? > remove these OK. The SSI stuff came from the original patch. > Best regards > --- > Kuninori Morimoto WBR, Sergei ^ permalink raw reply [flat|nested] 30+ messages in thread
* [PATCH 2/3] ARM: shmobile: r8a7778: add HPB-DMAC support @ 2013-07-18 22:04 ` Sergei Shtylyov 0 siblings, 0 replies; 30+ messages in thread From: Sergei Shtylyov @ 2013-07-18 22:04 UTC (permalink / raw) To: linux-arm-kernel Hello. On 07/01/2013 08:12 AM, Kuninori Morimoto wrote: >> From: Max Filippov <max.filippov@cogentembedded.com> >> Add HPB-DMAC platform device on R8A7778 SoC along with its slave and channel >> configurations. >> Signed-off-by: Max Filippov <max.filippov@cogentembedded.com> >> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> >> --- > (snip) >> +static const struct hpb_dmae_slave_config hpb_dmae_slaves[] = { >> + { >> + .id = HPBDMA_SLAVE_SDHI0_TX, >> + .addr = 0xffe4c000 + 0x30, >> + .dcr = DCR_SPDS_16BIT | DCR_DMDL | DCR_DPDS_16BIT, >> + .rstr = ASYNCRSTR_ASRST21 | ASYNCRSTR_ASRST22 | >> + ASYNCRSTR_ASRST23, >> + .mdr = ASYNCMDR_ASMD21_SINGLE, >> + .mdm = ASYNCMDR_ASMD21_MASK, >> + .port = 0x0D0C, >> + .flags = HPB_DMAE_SET_ASYNC_RESET | HPB_DMAE_SET_ASYNC_MODE, >> + .dma_ch = 21, >> + }, { >> + .id = HPBDMA_SLAVE_SDHI0_RX, >> + .addr = 0xffe4c000 + 0x30, >> + .dcr = DCR_SMDL | DCR_SPDS_16BIT | DCR_DPDS_16BIT, >> + .rstr = ASYNCRSTR_ASRST21 | ASYNCRSTR_ASRST22 | >> + ASYNCRSTR_ASRST23, >> + .mdr = ASYNCMDR_ASMD22_SINGLE, >> + .mdm = ASYNCMDR_ASMD22_MASK, >> + .port = 0x0D0C, >> + .flags = HPB_DMAE_SET_ASYNC_RESET | HPB_DMAE_SET_ASYNC_MODE, >> + .dma_ch = 22, >> + }, { >> + .id = HPBDMA_SLAVE_SSI0_TX_ST, >> + .addr = 0xffd90000 + 0x1008, >> + .dcr = DCR_CT | DCR_DIP | DCR_SPDS_32BIT | DCR_DMDL | >> + DCR_DPDS_32BIT, >> + .port = 0x0000, >> + .dma_ch = 28, >> + }, { >> + .id = HPBDMA_SLAVE_SSI0_TX_MN, >> + .addr = 0xffd90000 + 0x1008, >> + .dcr = DCR_CT | DCR_DIP | DCR_SPDS_16BIT | DCR_DMDL | >> + DCR_DPDS_16BIT, >> + .port = 0x0000, >> + .dma_ch = 28, >> + }, { >> + .id = HPBDMA_SLAVE_SSI1_RX_ST, >> + .addr = 0xffd90000 + 0x104c, >> + .dcr = DCR_CT | DCR_DIP | DCR_SMDL | DCR_SPDS_32BIT | >> + DCR_DPDAM | DCR_DPDS_32BIT, >> + .port = 0x0101, >> + .dma_ch = 29, >> + }, { >> + .id = HPBDMA_SLAVE_SSI1_RX_MN, >> + .addr = 0xffd90000 + 0x104c, >> + .dcr = DCR_CT | DCR_DIP | DCR_SMDL | DCR_SPDS_16BIT | >> + DCR_DPDAM | DCR_DPDS_16BIT, >> + .port = 0x0101, >> + .dma_ch = 29, >> + }, >> +}; > How did you test SSI ? > remove these OK. The SSI stuff came from the original patch. > Best regards > --- > Kuninori Morimoto WBR, Sergei ^ permalink raw reply [flat|nested] 30+ messages in thread
* [PATCH 3/3] ARM: shmobile: BOCK-W: enable DMA for SDHI0 2013-06-29 22:43 ` Sergei Shtylyov @ 2013-06-29 22:50 ` Sergei Shtylyov -1 siblings, 0 replies; 30+ messages in thread From: Sergei Shtylyov @ 2013-06-29 22:50 UTC (permalink / raw) To: linux-arm-kernel From: Max Filippov <max.filippov@cogentembedded.com> Pass HPB-DMA slave IDs in the SDHI0 platform data to enable DMA in the SDHI driver. Signed-off-by: Max Filippov <max.filippov@cogentembedded.com> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> --- arch/arm/mach-shmobile/board-bockw.c | 3 +++ 1 file changed, 3 insertions(+) Index: renesas/arch/arm/mach-shmobile/board-bockw.c =================================--- renesas.orig/arch/arm/mach-shmobile/board-bockw.c +++ renesas/arch/arm/mach-shmobile/board-bockw.c @@ -29,6 +29,7 @@ #include <linux/spi/spi.h> #include <linux/spi/flash.h> #include <mach/common.h> +#include <mach/dma.h> #include <mach/irqs.h> #include <mach/r8a7778.h> #include <asm/mach/arch.h> @@ -78,6 +79,8 @@ static struct resource smsc911x_resource /* SDHI */ static struct sh_mobile_sdhi_info sdhi0_info = { + .dma_slave_tx = HPBDMA_SLAVE_SDHI0_TX, + .dma_slave_rx = HPBDMA_SLAVE_SDHI0_RX, .tmio_caps = MMC_CAP_SD_HIGHSPEED, .tmio_ocr_mask = MMC_VDD_165_195 | MMC_VDD_32_33 | MMC_VDD_33_34, .tmio_flags = TMIO_MMC_HAS_IDLE_WAIT, ^ permalink raw reply [flat|nested] 30+ messages in thread
* [PATCH 3/3] ARM: shmobile: BOCK-W: enable DMA for SDHI0 @ 2013-06-29 22:50 ` Sergei Shtylyov 0 siblings, 0 replies; 30+ messages in thread From: Sergei Shtylyov @ 2013-06-29 22:50 UTC (permalink / raw) To: linux-arm-kernel From: Max Filippov <max.filippov@cogentembedded.com> Pass HPB-DMA slave IDs in the SDHI0 platform data to enable DMA in the SDHI driver. Signed-off-by: Max Filippov <max.filippov@cogentembedded.com> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> --- arch/arm/mach-shmobile/board-bockw.c | 3 +++ 1 file changed, 3 insertions(+) Index: renesas/arch/arm/mach-shmobile/board-bockw.c =================================================================== --- renesas.orig/arch/arm/mach-shmobile/board-bockw.c +++ renesas/arch/arm/mach-shmobile/board-bockw.c @@ -29,6 +29,7 @@ #include <linux/spi/spi.h> #include <linux/spi/flash.h> #include <mach/common.h> +#include <mach/dma.h> #include <mach/irqs.h> #include <mach/r8a7778.h> #include <asm/mach/arch.h> @@ -78,6 +79,8 @@ static struct resource smsc911x_resource /* SDHI */ static struct sh_mobile_sdhi_info sdhi0_info = { + .dma_slave_tx = HPBDMA_SLAVE_SDHI0_TX, + .dma_slave_rx = HPBDMA_SLAVE_SDHI0_RX, .tmio_caps = MMC_CAP_SD_HIGHSPEED, .tmio_ocr_mask = MMC_VDD_165_195 | MMC_VDD_32_33 | MMC_VDD_33_34, .tmio_flags = TMIO_MMC_HAS_IDLE_WAIT, ^ permalink raw reply [flat|nested] 30+ messages in thread
* Re: [PATCH 0/3] R8A7778/BOCK-W: HPB-DMAC support for SDHI0 2013-06-29 22:43 ` Sergei Shtylyov @ 2013-07-01 1:30 ` Simon Horman -1 siblings, 0 replies; 30+ messages in thread From: Simon Horman @ 2013-07-01 1:30 UTC (permalink / raw) To: linux-arm-kernel On Sun, Jun 30, 2013 at 02:43:56AM +0400, Sergei Shtylyov wrote: > Hello. > > Here's the set of 3 patches against the Simon Horman's 'renesas.git' repo, > 'renesas-next-20130628' tag. Here we add the HPB-DMAC platform code working on > the R8A7778 and enable DMA for SDHI0 on BOCK-W. The patchset depends on the > R-Car HPB-DMAC driver patch posted earlier. > > [1/3] ARM: shmobile: use common platform header for HPB-DMAC > [2/3] ARM: shmobile: r8a7778: add HPB-DMAC support > [3/3] ARM: shmobile: BOCK-W: enable DMA for SDHI0 Hi Magnus, Hi Morimoto-san, could you please review is series? ^ permalink raw reply [flat|nested] 30+ messages in thread
* [PATCH 0/3] R8A7778/BOCK-W: HPB-DMAC support for SDHI0 @ 2013-07-01 1:30 ` Simon Horman 0 siblings, 0 replies; 30+ messages in thread From: Simon Horman @ 2013-07-01 1:30 UTC (permalink / raw) To: linux-arm-kernel On Sun, Jun 30, 2013 at 02:43:56AM +0400, Sergei Shtylyov wrote: > Hello. > > Here's the set of 3 patches against the Simon Horman's 'renesas.git' repo, > 'renesas-next-20130628' tag. Here we add the HPB-DMAC platform code working on > the R8A7778 and enable DMA for SDHI0 on BOCK-W. The patchset depends on the > R-Car HPB-DMAC driver patch posted earlier. > > [1/3] ARM: shmobile: use common platform header for HPB-DMAC > [2/3] ARM: shmobile: r8a7778: add HPB-DMAC support > [3/3] ARM: shmobile: BOCK-W: enable DMA for SDHI0 Hi Magnus, Hi Morimoto-san, could you please review is series? ^ permalink raw reply [flat|nested] 30+ messages in thread
end of thread, other threads:[~2013-07-19 11:32 UTC | newest] Thread overview: 30+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2013-06-29 22:43 [PATCH 0/3] R8A7778/BOCK-W: HPB-DMAC support for SDHI0 Sergei Shtylyov 2013-06-29 22:43 ` Sergei Shtylyov 2013-06-29 22:45 ` [PATCH 1/3] ARM: shmobile: use common platform header for HPB-DMAC Sergei Shtylyov 2013-06-29 22:45 ` Sergei Shtylyov 2013-07-01 4:11 ` Kuninori Morimoto 2013-07-01 4:11 ` Kuninori Morimoto 2013-07-01 12:11 ` Sergei Shtylyov 2013-07-01 12:11 ` Sergei Shtylyov 2013-07-18 21:56 ` Sergei Shtylyov 2013-07-18 21:56 ` Sergei Shtylyov 2013-07-02 4:59 ` Magnus Damm 2013-07-02 4:59 ` Magnus Damm 2013-07-18 22:02 ` Sergei Shtylyov 2013-07-18 22:02 ` Sergei Shtylyov 2013-07-19 0:16 ` Kuninori Morimoto 2013-07-19 0:16 ` Kuninori Morimoto 2013-07-19 11:17 ` Sergei Shtylyov 2013-07-19 11:17 ` Sergei Shtylyov 2013-07-19 11:32 ` Sergei Shtylyov 2013-07-19 11:32 ` Sergei Shtylyov 2013-06-29 22:48 ` [PATCH 2/3] ARM: shmobile: r8a7778: add HPB-DMAC support Sergei Shtylyov 2013-06-29 22:48 ` Sergei Shtylyov 2013-07-01 4:12 ` Kuninori Morimoto 2013-07-01 4:12 ` Kuninori Morimoto 2013-07-18 22:04 ` Sergei Shtylyov 2013-07-18 22:04 ` Sergei Shtylyov 2013-06-29 22:50 ` [PATCH 3/3] ARM: shmobile: BOCK-W: enable DMA for SDHI0 Sergei Shtylyov 2013-06-29 22:50 ` Sergei Shtylyov 2013-07-01 1:30 ` [PATCH 0/3] R8A7778/BOCK-W: HPB-DMAC support " Simon Horman 2013-07-01 1:30 ` Simon Horman
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.