Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
From:   Ioana Ciocoi Radulescu <ruxandra.radulescu@nxp.com>
To:     Roy Pledge <roy.pledge@nxp.com>,
        Horia Geanta <horia.geanta@nxp.com>,
        "devel@driverdev.osuosl.org" <devel@driverdev.osuosl.org>,
        "linux-arm-kernel@lists.infradead.org" 
        <linux-arm-kernel@lists.infradead.org>,
        "gregkh@linuxfoundation.org" <gregkh@linuxfoundation.org>,
        Leo Li <leoyang.li@nxp.com>
CC:     Laurentiu Tudor <laurentiu.tudor@nxp.com>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "arnd@arndb.de" <arnd@arndb.de>,
        "catalin.marinas@arm.com" <catalin.marinas@arm.com>,
        "robin.murphy@arm.com" <robin.murphy@arm.com>
Subject: RE: [PATCH 0/2] staging/fsl-mc/bus: Move DPIO from staging to
 drivers/soc/fsl
Thread-Topic: [PATCH 0/2] staging/fsl-mc/bus: Move DPIO from staging to
 drivers/soc/fsl
Thread-Index: AQHUFJgtMRuXJ8mvPEGdOgjBQpkT8qSCjKbg
Date:   Fri, 6 Jul 2018 18:57:19 +0000
Message-ID: <AM3PR04MB338125F44EC72982F0F128694470@AM3PR04MB338.eurprd04.prod.outlook.com>
References: <20180705194113.16256-1-roy.pledge@nxp.com>
 <VI1PR0402MB34853F03375D3BCA4791432698470@VI1PR0402MB3485.eurprd04.prod.outlook.com>
 <VI1PR04MB4494A72D6734ECAA57BA93C586470@VI1PR04MB4494.eurprd04.prod.outlook.com>
In-Reply-To: <VI1PR04MB4494A72D6734ECAA57BA93C586470@VI1PR04MB4494.eurprd04.prod.outlook.com>
Accept-Language: en-US
Content-Language: en-US
received-spf: None (protection.outlook.com: nxp.com does not designate
 permitted sender hosts)
spamdiagnosticoutput: 1:99
spamdiagnosticmetadata: NSPM
Content-Type: text/plain; charset="iso-8859-2"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: 86ba4a01-23f3-4aac-2001-08d5e3724cee
X-MS-Exchange-CrossTenant-originalarrivaltime: 06 Jul 2018 18:57:19.8463
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM3PR04MB1361
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

> -----Original Message-----
> From: Roy Pledge
> Sent: Friday, July 6, 2018 9:52 PM
> To: Horia Geanta <horia.geanta@nxp.com>; devel@driverdev.osuosl.org;
> linux-arm-kernel@lists.infradead.org; gregkh@linuxfoundation.org; Leo Li
> <leoyang.li@nxp.com>; Ioana Ciocoi Radulescu
> <ruxandra.radulescu@nxp.com>
> Cc: Laurentiu Tudor <laurentiu.tudor@nxp.com>; linux-
> kernel@vger.kernel.org; arnd@arndb.de; catalin.marinas@arm.com;
> robin.murphy@arm.com
> Subject: Re: [PATCH 0/2] staging/fsl-mc/bus: Move DPIO from staging to
> drivers/soc/fsl
>=20
> On 7/6/2018 8:25 AM, Horia Geanta wrote:
> > On 7/5/2018 10:41 PM, Roy Pledge wrote:
> >> Move the NXP DPIO (Datapath I/O driver) from the staging/fsl-
> mc/bus/dpio
> >> directory to the drivers/soc/fsl directory.
> >>
> >> The DPIO driver enables access to the Queue and Buffer Managemer
> (QBMAN)
> >> hardware of NXP DPAA2 devices. This is a prerequiste for moving the
> DPAA2
> >> Ethernet device driver from the staging directory.
> >>
> > Roy, Ioana,
> >
> > Would it be ok to add the following patch on top of the series?
> > It's a dependency for dpseci object.
> > If not added now I fear it won't be accepted until dpaa2-ethernet
> > moves out of staging, thus gating dpseci upstreaming.
> >
> > Thanks,
> > Horia
> I can add this to the series - I will respin and send a v2.

Roy, when you send the v2 you can also add for this patch:

Acked-by: Ioana Radulescu <ruxandra.radulescu@nxp.com>

> >
> > --->8---
> > Previous commits:
> > 6e2387e8f19ed ("staging: fsl-dpaa2/eth: Add Freescale DPAA2 Ethernet
> driver")
> > 39163c0ce0f48 ("staging: fsl-dpaa2/eth: Errors checking update")
> > added bits that are not specific to the WRIOP accelerator.
> >
> > Move these where they belong (in DPIO) such that other accelerators
> > can make use of them.
> >
> > Signed-off-by: Horia Geant=E3 <horia.geanta@nxp.com>
> > ---
> >  drivers/staging/fsl-dpaa2/ethernet/dpaa2-eth.c |  4 ++--
> >  drivers/staging/fsl-dpaa2/ethernet/dpaa2-eth.h | 18 +++++-------------
> >  drivers/staging/fsl-mc/include/dpaa2-fd.h      | 12 ++++++++++++
> >  3 files changed, 19 insertions(+), 15 deletions(-)
> >
> > diff --git a/drivers/staging/fsl-dpaa2/ethernet/dpaa2-eth.c
> b/drivers/staging/fsl-dpaa2/ethernet/dpaa2-eth.c
> > index 396371728aa1..d5f0ac5c2d1f 100644
> > --- a/drivers/staging/fsl-dpaa2/ethernet/dpaa2-eth.c
> > +++ b/drivers/staging/fsl-dpaa2/ethernet/dpaa2-eth.c
> > @@ -455,7 +455,7 @@ static int build_sg_fd(struct dpaa2_eth_priv *priv,
> >  	dpaa2_fd_set_format(fd, dpaa2_fd_sg);
> >  	dpaa2_fd_set_addr(fd, addr);
> >  	dpaa2_fd_set_len(fd, skb->len);
> > -	dpaa2_fd_set_ctrl(fd, DPAA2_FD_CTRL_PTA |
> DPAA2_FD_CTRL_PTV1);
> > +	dpaa2_fd_set_ctrl(fd, FD_CTRL_PTA | FD_CTRL_PTV1);
> >
> >  	if (priv->tx_tstamp && skb_shinfo(skb)->tx_flags &
> SKBTX_HW_TSTAMP)
> >  		enable_tx_tstamp(fd, sgt_buf);
> > @@ -508,7 +508,7 @@ static int build_single_fd(struct dpaa2_eth_priv
> *priv,
> >  	dpaa2_fd_set_offset(fd, (u16)(skb->data - buffer_start));
> >  	dpaa2_fd_set_len(fd, skb->len);
> >  	dpaa2_fd_set_format(fd, dpaa2_fd_single);
> > -	dpaa2_fd_set_ctrl(fd, DPAA2_FD_CTRL_PTA |
> DPAA2_FD_CTRL_PTV1);
> > +	dpaa2_fd_set_ctrl(fd, FD_CTRL_PTA | FD_CTRL_PTV1);
> >
> >  	if (priv->tx_tstamp && skb_shinfo(skb)->tx_flags &
> SKBTX_HW_TSTAMP)
> >  		enable_tx_tstamp(fd, buffer_start);
> > diff --git a/drivers/staging/fsl-dpaa2/ethernet/dpaa2-eth.h
> b/drivers/staging/fsl-dpaa2/ethernet/dpaa2-eth.h
> > index 905a4e6be8fa..9269cb05a84b 100644
> > --- a/drivers/staging/fsl-dpaa2/ethernet/dpaa2-eth.h
> > +++ b/drivers/staging/fsl-dpaa2/ethernet/dpaa2-eth.h
> > @@ -124,21 +124,13 @@ struct dpaa2_eth_swa {
> >  #define DPAA2_FD_FRC_FAICFDV		0x0400
> >
> >  /* Error bits in FD CTRL */
> > -#define DPAA2_FD_CTRL_UFD		0x00000004
> > -#define DPAA2_FD_CTRL_SBE		0x00000008
> > -#define DPAA2_FD_CTRL_FSE		0x00000020
> > -#define DPAA2_FD_CTRL_FAERR		0x00000040
> > -
> > -#define DPAA2_FD_RX_ERR_MASK		(DPAA2_FD_CTRL_SBE	| \
> > -					 DPAA2_FD_CTRL_FAERR)
> > -#define DPAA2_FD_TX_ERR_MASK		(DPAA2_FD_CTRL_UFD	| \
> > -					 DPAA2_FD_CTRL_SBE	| \
> > -					 DPAA2_FD_CTRL_FSE	| \
> > -					 DPAA2_FD_CTRL_FAERR)
> > +#define DPAA2_FD_RX_ERR_MASK		(FD_CTRL_SBE |
> FD_CTRL_FAERR)
> > +#define DPAA2_FD_TX_ERR_MASK		(FD_CTRL_UFD	| \
> > +					 FD_CTRL_SBE	| \
> > +					 FD_CTRL_FSE	| \
> > +					 FD_CTRL_FAERR)
> >
> >  /* Annotation bits in FD CTRL */
> > -#define DPAA2_FD_CTRL_PTA		0x00800000
> > -#define DPAA2_FD_CTRL_PTV1		0x00400000
> >  #define DPAA2_FD_CTRL_ASAL		0x00020000	/* ASAL =3D
> 128B */
> >
> >  /* Frame annotation status */
> > diff --git a/drivers/staging/fsl-mc/include/dpaa2-fd.h b/drivers/stagin=
g/fsl-
> mc/include/dpaa2-fd.h
> > index b55b89ba4eda..2576abaa7779 100644
> > --- a/drivers/staging/fsl-mc/include/dpaa2-fd.h
> > +++ b/drivers/staging/fsl-mc/include/dpaa2-fd.h
> > @@ -67,6 +67,18 @@ struct dpaa2_fd {
> >  #define SG_FINAL_FLAG_MASK	0x1
> >  #define SG_FINAL_FLAG_SHIFT	15
> >
> > +/* Error bits in FD CTRL */
> > +#define FD_CTRL_ERR_MASK	0x000000FF
> > +#define FD_CTRL_UFD		0x00000004
> > +#define FD_CTRL_SBE		0x00000008
> > +#define FD_CTRL_FLC		0x00000010
> > +#define FD_CTRL_FSE		0x00000020
> > +#define FD_CTRL_FAERR		0x00000040
> > +
> > +/* Annotation bits in FD CTRL */
> > +#define FD_CTRL_PTA		0x00800000
> > +#define FD_CTRL_PTV1		0x00400000
> > +
> >  enum dpaa2_fd_format {
> >  	dpaa2_fd_single =3D 0,
> >  	dpaa2_fd_list,
>=20