RE: [PATCH v2 8/9] staging: fsl-dpaa2/eth: Add TODO file

From: Ruxandra Ioana Radulescu
Date: Thu Apr 20 2017 - 04:31:22 EST


> -----Original Message-----
> From: Stuart Yoder [mailto:stuyoder@xxxxxxxxx]
> Sent: Wednesday, April 19, 2017 6:44 AM
> To: Ruxandra Ioana Radulescu <ruxandra.radulescu@xxxxxxx>
> Cc: gregkh@xxxxxxxxxxxxxxxxxxx; devel@xxxxxxxxxxxxxxxxxxxx; linux-
> kernel@xxxxxxxxxxxxxxx; Alexander Graf <agraf@xxxxxxx>; Arnd Bergmann
> <arnd@xxxxxxxx>; linux-arm-kernel@xxxxxxxxxxxxxxxxxxx; Alexandru
> Marginean <alexandru.marginean@xxxxxxx>; Bogdan Hamciuc
> <bogdan.hamciuc@xxxxxxx>; Laurentiu Tudor <laurentiu.tudor@xxxxxxx>;
> Roy Pledge <roy.pledge@xxxxxxx>; Haiying Wang
> <haiying.wang@xxxxxxx>
> Subject: Re: [PATCH v2 8/9] staging: fsl-dpaa2/eth: Add TODO file
>
> On Wed, Apr 12, 2017 at 11:25 AM, Ioana Radulescu
> <ruxandra.radulescu@xxxxxxx> wrote:
> > Add a list of TODO items for the Ethernet driver
> >
> > Signed-off-by: Ioana Radulescu <ruxandra.radulescu@xxxxxxx>
> > ---
> > v2: Add note
> >
> > drivers/staging/fsl-dpaa2/ethernet/TODO | 14 ++++++++++++++
> > 1 file changed, 14 insertions(+)
> > 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 000000000000..110e66d44b42
> > --- /dev/null
> > +++ b/drivers/staging/fsl-dpaa2/ethernet/TODO
> > @@ -0,0 +1,14 @@
> > +* 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
> > +
> > +NOTE: None of the above is must-have before getting the DPAA2
> Ethernet driver
> > +out of staging. The main requirement for that is to have the drivers it
> > +depends on, fsl-mc bus and DPIO driver, moved to drivers/bus and
> drivers/soc
> > +respectively.
>
> The TODO file should have contact info (I think)...look at other
> drivers/staging TODO
> for examples.

Thanks Stuart, I missed that. If I send a v3 of this patchset I'll include
this update, otherwise I'll send a separate patch for it.

Ioana