dmaengine Archive on lore.kernel.org
 help / color / Atom feed
* [V2] dmaengine: fsl-edma: Add eDMA support for QorIQ LS1028A platform
@ 2019-10-21  2:21 Peng Ma
  2019-10-21 21:54 ` Leo Li
  0 siblings, 1 reply; 3+ messages in thread
From: Peng Ma @ 2019-10-21  2:21 UTC (permalink / raw)
  To: vkoul
  Cc: dan.j.williams, leoyang.li, k.kozlowski.k, fabio.estevam,
	dmaengine, linux-kernel, Peng Ma

Our platforms(such as LS1021A, LS1012A, LS1043A, LS1046A, LS1028A) with below
registers(CHCFG0 - CHCFG15) of eDMA as follows:
*-----------------------------------------------------------*
|     Offset   |	OTHERS      |		LS1028A	    |
|--------------|--------------------|-----------------------|
|     0x0      |        CHCFG0      |           CHCFG3      |
|--------------|--------------------|-----------------------|
|     0x1      |        CHCFG1      |           CHCFG2      |
|--------------|--------------------|-----------------------|
|     0x2      |        CHCFG2      |           CHCFG1      |
|--------------|--------------------|-----------------------|
|     0x3      |        CHCFG3      |           CHCFG0      |
|--------------|--------------------|-----------------------|
|     ...      |        ......      |           ......      |
|--------------|--------------------|-----------------------|
|     0xC      |        CHCFG12     |           CHCFG15     |
|--------------|--------------------|-----------------------|
|     0xD      |        CHCFG13     |           CHCFG14     |
|--------------|--------------------|-----------------------|
|     0xE      |        CHCFG14     |           CHCFG13     |
|--------------|--------------------|-----------------------|
|     0xF      |        CHCFG15     |           CHCFG12     |
*-----------------------------------------------------------*

This patch is to improve edma driver to fit LS1028A platform.

Signed-off-by: Peng Ma <peng.ma@nxp.com>
---
Changed for V2:
	- Explaining what's the "Our platforms"

 drivers/dma/fsl-edma-common.c | 12 ++++++++++++
 1 file changed, 12 insertions(+)

diff --git a/drivers/dma/fsl-edma-common.c b/drivers/dma/fsl-edma-common.c
index b1a7ca9..611186b 100644
--- a/drivers/dma/fsl-edma-common.c
+++ b/drivers/dma/fsl-edma-common.c
@@ -7,6 +7,7 @@
 #include <linux/module.h>
 #include <linux/slab.h>
 #include <linux/dma-mapping.h>
+#include <linux/sys_soc.h>
 
 #include "fsl-edma-common.h"
 
@@ -42,6 +43,11 @@
 
 #define EDMA_TCD		0x1000
 
+static struct soc_device_attribute soc_fixup_tuning[] = {
+	{ .family = "QorIQ LS1028A"},
+	{ },
+};
+
 static void fsl_edma_enable_request(struct fsl_edma_chan *fsl_chan)
 {
 	struct edma_regs *regs = &fsl_chan->edma->regs;
@@ -109,10 +115,16 @@ void fsl_edma_chan_mux(struct fsl_edma_chan *fsl_chan,
 	u32 ch = fsl_chan->vchan.chan.chan_id;
 	void __iomem *muxaddr;
 	unsigned int chans_per_mux, ch_off;
+	int endian_diff[4] = {3, 1, -1, -3};
 	u32 dmamux_nr = fsl_chan->edma->drvdata->dmamuxs;
 
 	chans_per_mux = fsl_chan->edma->n_chans / dmamux_nr;
 	ch_off = fsl_chan->vchan.chan.chan_id % chans_per_mux;
+
+	if (!fsl_chan->edma->big_endian &&
+	    soc_device_match(soc_fixup_tuning))
+		ch_off += endian_diff[ch_off % 4];
+
 	muxaddr = fsl_chan->edma->muxbase[ch / chans_per_mux];
 	slot = EDMAMUX_CHCFG_SOURCE(slot);
 
-- 
2.9.5


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

* RE: [V2] dmaengine: fsl-edma: Add eDMA support for QorIQ LS1028A platform
  2019-10-21  2:21 [V2] dmaengine: fsl-edma: Add eDMA support for QorIQ LS1028A platform Peng Ma
@ 2019-10-21 21:54 ` Leo Li
  2019-10-22  0:25   ` Li Yang
  0 siblings, 1 reply; 3+ messages in thread
From: Leo Li @ 2019-10-21 21:54 UTC (permalink / raw)
  To: Peng Ma, vkoul, Robin Gong
  Cc: dan.j.williams, k.kozlowski.k, Fabio Estevam, dmaengine,
	linux-kernel, Peng Ma



> -----Original Message-----
> From: Peng Ma <peng.ma@nxp.com>
> Sent: Sunday, October 20, 2019 9:22 PM
> To: vkoul@kernel.org
> Cc: dan.j.williams@intel.com; Leo Li <leoyang.li@nxp.com>;
> k.kozlowski.k@gmail.com; Fabio Estevam <fabio.estevam@nxp.com>;
> dmaengine@vger.kernel.org; linux-kernel@vger.kernel.org; Peng Ma
> <peng.ma@nxp.com>
> Subject: [V2] dmaengine: fsl-edma: Add eDMA support for QorIQ LS1028A
> platform
> 
> Our platforms(such as LS1021A, LS1012A, LS1043A, LS1046A, LS1028A) with
> below

You only covered QorIQ SoCs, how about the situation for IMX SoCs?

> registers(CHCFG0 - CHCFG15) of eDMA as follows:
> *-----------------------------------------------------------*
> |     Offset   |	OTHERS      |		LS1028A	    |
> |--------------|--------------------|-----------------------|
> |     0x0      |        CHCFG0      |           CHCFG3      |
> |--------------|--------------------|-----------------------|
> |     0x1      |        CHCFG1      |           CHCFG2      |
> |--------------|--------------------|-----------------------|
> |     0x2      |        CHCFG2      |           CHCFG1      |
> |--------------|--------------------|-----------------------|
> |     0x3      |        CHCFG3      |           CHCFG0      |
> |--------------|--------------------|-----------------------|
> |     ...      |        ......      |           ......      |
> |--------------|--------------------|-----------------------|
> |     0xC      |        CHCFG12     |           CHCFG15     |
> |--------------|--------------------|-----------------------|
> |     0xD      |        CHCFG13     |           CHCFG14     |
> |--------------|--------------------|-----------------------|
> |     0xE      |        CHCFG14     |           CHCFG13     |
> |--------------|--------------------|-----------------------|
> |     0xF      |        CHCFG15     |           CHCFG12     |
> *-----------------------------------------------------------*
> 
> This patch is to improve edma driver to fit LS1028A platform.
> 
> Signed-off-by: Peng Ma <peng.ma@nxp.com>
> ---
> Changed for V2:
> 	- Explaining what's the "Our platforms"
> 
>  drivers/dma/fsl-edma-common.c | 12 ++++++++++++
>  1 file changed, 12 insertions(+)
> 
> diff --git a/drivers/dma/fsl-edma-common.c b/drivers/dma/fsl-edma-
> common.c index b1a7ca9..611186b 100644
> --- a/drivers/dma/fsl-edma-common.c
> +++ b/drivers/dma/fsl-edma-common.c
> @@ -7,6 +7,7 @@
>  #include <linux/module.h>
>  #include <linux/slab.h>
>  #include <linux/dma-mapping.h>
> +#include <linux/sys_soc.h>
> 
>  #include "fsl-edma-common.h"
> 
> @@ -42,6 +43,11 @@
> 
>  #define EDMA_TCD		0x1000
> 
> +static struct soc_device_attribute soc_fixup_tuning[] = {
> +	{ .family = "QorIQ LS1028A"},
> +	{ },
> +};
> +
>  static void fsl_edma_enable_request(struct fsl_edma_chan *fsl_chan)  {
>  	struct edma_regs *regs = &fsl_chan->edma->regs; @@ -109,10
> +115,16 @@ void fsl_edma_chan_mux(struct fsl_edma_chan *fsl_chan,
>  	u32 ch = fsl_chan->vchan.chan.chan_id;
>  	void __iomem *muxaddr;
>  	unsigned int chans_per_mux, ch_off;
> +	int endian_diff[4] = {3, 1, -1, -3};
>  	u32 dmamux_nr = fsl_chan->edma->drvdata->dmamuxs;
> 
>  	chans_per_mux = fsl_chan->edma->n_chans / dmamux_nr;
>  	ch_off = fsl_chan->vchan.chan.chan_id % chans_per_mux;
> +
> +	if (!fsl_chan->edma->big_endian &&
> +	    soc_device_match(soc_fixup_tuning))
> +		ch_off += endian_diff[ch_off % 4];
> +

This probably is not the best fix now.  There is a new mux_configure32() API added but it doesn't consider endianness.  How about making it properly taken care of endianness?  And use it to set these registers?

>  	muxaddr = fsl_chan->edma->muxbase[ch / chans_per_mux];
>  	slot = EDMAMUX_CHCFG_SOURCE(slot);
> 
> --
> 2.9.5


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

* Re: [V2] dmaengine: fsl-edma: Add eDMA support for QorIQ LS1028A platform
  2019-10-21 21:54 ` Leo Li
@ 2019-10-22  0:25   ` Li Yang
  0 siblings, 0 replies; 3+ messages in thread
From: Li Yang @ 2019-10-22  0:25 UTC (permalink / raw)
  To: Peng Ma, vkoul, Robin Gong
  Cc: dan.j.williams, k.kozlowski.k, Fabio Estevam, dmaengine, linux-kernel

On Mon, Oct 21, 2019 at 4:57 PM Leo Li <leoyang.li@nxp.com> wrote:
>
>
>
> > -----Original Message-----
> > From: Peng Ma <peng.ma@nxp.com>
> > Sent: Sunday, October 20, 2019 9:22 PM
> > To: vkoul@kernel.org
> > Cc: dan.j.williams@intel.com; Leo Li <leoyang.li@nxp.com>;
> > k.kozlowski.k@gmail.com; Fabio Estevam <fabio.estevam@nxp.com>;
> > dmaengine@vger.kernel.org; linux-kernel@vger.kernel.org; Peng Ma
> > <peng.ma@nxp.com>
> > Subject: [V2] dmaengine: fsl-edma: Add eDMA support for QorIQ LS1028A
> > platform
> >
> > Our platforms(such as LS1021A, LS1012A, LS1043A, LS1046A, LS1028A) with
> > below
>
> You only covered QorIQ SoCs, how about the situation for IMX SoCs?
>
> > registers(CHCFG0 - CHCFG15) of eDMA as follows:
> > *-----------------------------------------------------------*
> > |     Offset   |      OTHERS      |           LS1028A     |
> > |--------------|--------------------|-----------------------|
> > |     0x0      |        CHCFG0      |           CHCFG3      |
> > |--------------|--------------------|-----------------------|
> > |     0x1      |        CHCFG1      |           CHCFG2      |
> > |--------------|--------------------|-----------------------|
> > |     0x2      |        CHCFG2      |           CHCFG1      |
> > |--------------|--------------------|-----------------------|
> > |     0x3      |        CHCFG3      |           CHCFG0      |
> > |--------------|--------------------|-----------------------|
> > |     ...      |        ......      |           ......      |
> > |--------------|--------------------|-----------------------|
> > |     0xC      |        CHCFG12     |           CHCFG15     |
> > |--------------|--------------------|-----------------------|
> > |     0xD      |        CHCFG13     |           CHCFG14     |
> > |--------------|--------------------|-----------------------|
> > |     0xE      |        CHCFG14     |           CHCFG13     |
> > |--------------|--------------------|-----------------------|
> > |     0xF      |        CHCFG15     |           CHCFG12     |
> > *-----------------------------------------------------------*
> >
> > This patch is to improve edma driver to fit LS1028A platform.
> >
> > Signed-off-by: Peng Ma <peng.ma@nxp.com>
> > ---
> > Changed for V2:
> >       - Explaining what's the "Our platforms"
> >
> >  drivers/dma/fsl-edma-common.c | 12 ++++++++++++
> >  1 file changed, 12 insertions(+)
> >
> > diff --git a/drivers/dma/fsl-edma-common.c b/drivers/dma/fsl-edma-
> > common.c index b1a7ca9..611186b 100644
> > --- a/drivers/dma/fsl-edma-common.c
> > +++ b/drivers/dma/fsl-edma-common.c
> > @@ -7,6 +7,7 @@
> >  #include <linux/module.h>
> >  #include <linux/slab.h>
> >  #include <linux/dma-mapping.h>
> > +#include <linux/sys_soc.h>
> >
> >  #include "fsl-edma-common.h"
> >
> > @@ -42,6 +43,11 @@
> >
> >  #define EDMA_TCD             0x1000
> >
> > +static struct soc_device_attribute soc_fixup_tuning[] = {
> > +     { .family = "QorIQ LS1028A"},
> > +     { },
> > +};
> > +
> >  static void fsl_edma_enable_request(struct fsl_edma_chan *fsl_chan)  {
> >       struct edma_regs *regs = &fsl_chan->edma->regs; @@ -109,10
> > +115,16 @@ void fsl_edma_chan_mux(struct fsl_edma_chan *fsl_chan,
> >       u32 ch = fsl_chan->vchan.chan.chan_id;
> >       void __iomem *muxaddr;
> >       unsigned int chans_per_mux, ch_off;
> > +     int endian_diff[4] = {3, 1, -1, -3};
> >       u32 dmamux_nr = fsl_chan->edma->drvdata->dmamuxs;
> >
> >       chans_per_mux = fsl_chan->edma->n_chans / dmamux_nr;
> >       ch_off = fsl_chan->vchan.chan.chan_id % chans_per_mux;
> > +
> > +     if (!fsl_chan->edma->big_endian &&
> > +         soc_device_match(soc_fixup_tuning))
> > +             ch_off += endian_diff[ch_off % 4];
> > +
>
> This probably is not the best fix now.  There is a new mux_configure32() API added but it doesn't consider endianness.  How about making it properly taken care of endianness?  And use it to set these registers?

You can ignore this comment.  The mux_configure32() seems to deal with
a different register layout(32-bit register per CH).

Considering the register per channel is defined as 8-bit in the
reference manual, the hardware seems to be weird to require
byte-swapping as if the register is 32-bit.  Probably you can rename
soc_fixup_tuning something like mux_byte_swap_quirk.

>
> >       muxaddr = fsl_chan->edma->muxbase[ch / chans_per_mux];
> >       slot = EDMAMUX_CHCFG_SOURCE(slot);
> >
> > --
> > 2.9.5
>

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

end of thread, back to index

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-10-21  2:21 [V2] dmaengine: fsl-edma: Add eDMA support for QorIQ LS1028A platform Peng Ma
2019-10-21 21:54 ` Leo Li
2019-10-22  0:25   ` Li Yang

dmaengine Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/dmaengine/0 dmaengine/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 dmaengine dmaengine/ https://lore.kernel.org/dmaengine \
		dmaengine@vger.kernel.org
	public-inbox-index dmaengine

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.kernel.vger.dmaengine


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git