From: Stuart Yoder <stuart.yoder@nxp.com>
To: Greg KH <gregkh@linuxfoundation.org>
CC: Ruxandra Ioana Radulescu <ruxandra.radulescu@nxp.com>,
        "devel@driverdev.osuosl.org" <devel@driverdev.osuosl.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "agraf@suse.de" <agraf@suse.de>, "arnd@arndb.de" <arnd@arndb.de>,
        "Alexandru Marginean" <alexandru.marginean@nxp.com>,
        Bogdan Hamciuc <bogdan.hamciuc@nxp.com>,
        Roy Pledge <roy.pledge@nxp.com>,
        Laurentiu Tudor <laurentiu.tudor@nxp.com>
Subject: RE: [PATCH 7/8] staging: fsl-dpaa2/eth: Add TODO file
Thread-Topic: [PATCH 7/8] staging: fsl-dpaa2/eth: Add TODO file
Thread-Index: AQHST6P8gLAcyQr7sk6cziYpfvtUZaD6rywAgAACWoCAAAPcAIAAJqoQgABYp4CAAANakA==
Date: Tue, 6 Dec 2016 18:10:38 +0000
Message-ID: <VI1PR0401MB2638EFD5D0419CD72FA702E18D820@VI1PR0401MB2638.eurprd04.prod.outlook.com>
References: <20161206093442.11362-1-ruxandra.radulescu@nxp.com>
 <20161206093442.11362-8-ruxandra.radulescu@nxp.com>
 <20161206095800.GA4247@kroah.com>
 <VI1PR0402MB284774B6E78A67F19888A63894820@VI1PR0402MB2847.eurprd04.prod.outlook.com>
 <20161206102014.GA7899@kroah.com>
 <VI1PR0401MB26389BEBF08D585B0C99C0AF8D820@VI1PR0401MB2638.eurprd04.prod.outlook.com>
 <20161206175555.GB15607@kroah.com>
In-Reply-To: <20161206175555.GB15607@kroah.com>
Accept-Language: en-US
Content-Language: en-US
spamdiagnosticoutput: 1:99
spamdiagnosticmetadata: NSPM
Content-Type: text/plain; charset="us-ascii"
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-originalarrivaltime: 06 Dec 2016 18:10:38.2071
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR0401MB2655
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
Content-Transfer-Encoding: 8bit
X-MIME-Autoconverted: from quoted-printable to 8bit by mail.home.local id uB6MkcJq020396
Content-Length: 5101
Lines: 106


> -----Original Message-----
> From: Greg KH [mailto:gregkh@linuxfoundation.org]
> Sent: Tuesday, December 06, 2016 11:56 AM
> To: Stuart Yoder <stuart.yoder@nxp.com>
> Cc: Ruxandra Ioana Radulescu <ruxandra.radulescu@nxp.com>; devel@driverdev.osuosl.org; linux-
> kernel@vger.kernel.org; agraf@suse.de; arnd@arndb.de; Alexandru Marginean <alexandru.marginean@nxp.com>;
> Bogdan Hamciuc <bogdan.hamciuc@nxp.com>; Roy Pledge <roy.pledge@nxp.com>; Laurentiu Tudor
> <laurentiu.tudor@nxp.com>
> Subject: Re: [PATCH 7/8] staging: fsl-dpaa2/eth: Add TODO file
> 
> On Tue, Dec 06, 2016 at 12:59:59PM +0000, Stuart Yoder wrote:
> >
> >
> > > -----Original Message-----
> > > From: Greg KH [mailto:gregkh@linuxfoundation.org]
> > > Sent: Tuesday, December 06, 2016 4:20 AM
> > > To: Ruxandra Ioana Radulescu <ruxandra.radulescu@nxp.com>
> > > Cc: devel@driverdev.osuosl.org; linux-kernel@vger.kernel.org; agraf@suse.de; arnd@arndb.de;
> Alexandru
> > > Marginean <alexandru.marginean@nxp.com>; Bogdan Hamciuc <bogdan.hamciuc@nxp.com>; Stuart Yoder
> > > <stuart.yoder@nxp.com>; Roy Pledge <roy.pledge@nxp.com>; Laurentiu Tudor <laurentiu.tudor@nxp.com>
> > > Subject: Re: [PATCH 7/8] staging: fsl-dpaa2/eth: Add TODO file
> > >
> > > On Tue, Dec 06, 2016 at 10:06:25AM +0000, Ruxandra Ioana Radulescu wrote:
> > > > > -----Original Message-----
> > > > > From: Greg KH [mailto:gregkh@linuxfoundation.org]
> > > > > Sent: Tuesday, December 06, 2016 11:58 AM
> > > > > To: Ruxandra Ioana Radulescu <ruxandra.radulescu@nxp.com>
> > > > > Cc: devel@driverdev.osuosl.org; linux-kernel@vger.kernel.org;
> > > > > agraf@suse.de; arnd@arndb.de; Alexandru Marginean
> > > > > <alexandru.marginean@nxp.com>; Bogdan Hamciuc
> > > > > <bogdan.hamciuc@nxp.com>; Stuart Yoder <stuart.yoder@nxp.com>; Roy
> > > > > Pledge <roy.pledge@nxp.com>; Laurentiu Tudor
> > > > > <laurentiu.tudor@nxp.com>
> > > > > Subject: Re: [PATCH 7/8] staging: fsl-dpaa2/eth: Add TODO file
> > > > >
> > > > > On Tue, Dec 06, 2016 at 03:34:41AM -0600, Ioana Radulescu wrote:
> > > > > > Add a list of TODO items for the Ethernet driver
> > > > > >
> > > > > > Signed-off-by: Ioana Radulescu <ruxandra.radulescu@nxp.com>
> > > > > > ---
> > > > > >  drivers/staging/fsl-dpaa2/ethernet/TODO |    9 +++++++++
> > > > > >  1 files changed, 9 insertions(+), 0 deletions(-)
> > > > > >  create mode 100644 drivers/staging/fsl-dpaa2/ethernet/TODO
> > > > > >
> > > > > > diff --git a/drivers/staging/fsl-dpaa2/ethernet/TODO b/drivers/staging/fsl-
> > > > > dpaa2/ethernet/TODO
> > > > > > new file mode 100644
> > > > > > index 0000000..833265b
> > > > > > --- /dev/null
> > > > > > +++ b/drivers/staging/fsl-dpaa2/ethernet/TODO
> > > > > > @@ -0,0 +1,9 @@
> > > > > > +* Add a DPAA2 MAC kernel driver in order to allow PHY management;
> > > > > currently
> > > > > > +  the DPMAC objects and their link to DPNIs are handled by MC internally
> > > > > > +  and all PHYs are seen as fixed-link
> > > > > > +* add more debug support: decide how to expose detailed debug
> > > > > statistics,
> > > > > > +  add ingress error queue support
> > > > > > +* MC firmware uprev; the DPAA2 objects used by the Ethernet driver
> > > > > need to
> > > > > > +  be kept in sync with binary interface changes in MC
> > > > > > +* refine README file
> > > > > > +* cleanup
> > > > >
> > > > > These seem like very minor things, why not just spend a week and do this
> > > > > work and get it merged to the "correct" portion of the kernel tree?  Why
> > > > > does this have to go into staging?
> > > >
> > > > Actually the first bullet is not minor at all and requires some design
> > > > choices that we aren't yet completely clear with, and which in turn may
> > > > affect parts of the Ethernet driver. We figured it would be best to try
> > > > adding this in staging first (and also provide this way an example of using
> > > > the fsl-mc bus and dpio driver) than wait until all MAC development
> > > > questions are ironed-out.
> > >
> > > Ok, that makes sense.
> > >
> > > > I can remove the other bullets from the TODO list if you think they're
> > > > not worth mentioning.
> > >
> > > No, they should be mentioned, I just didn't think they are all that much
> > > work, and if you didn't have major things needed to get done, you could
> > > just knock it all out in a week of local development.
> > >
> > > I'll look into taking this into the tree later today...
> >
> > Note, as mentioned in the cover letter, in it's current form this patch
> > series is based on the series:
> > [PATCH v3 0/9] staging: fsl-mc: move bus driver out of staging, add dpio
> >
> > ...which means that it won't build or run without that series being
> > applied first, due to header file dependencies.  It also functionally
> > depends on the DPIO driver.  So we need the dpio driver merged first.
> >
> > Is moving the fsl-mc bus driver out of staging a possibility now?
> 
> I'm ok with it, but I really haven't looked at the patches in a while, I
> keep seeing others have problems with it.  Want me to review it now?

Yes, would like that very much.

Thanks,
Stuart