Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1536825 > unrolled thread

Re: [PATCH 7/8] staging: fsl-dpaa2/eth: Add TODO file

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: [PATCH 7/8] staging: fsl-dpaa2/eth: Add TODO file Greg KH <gregkh@linuxfoundation.org> - 2016-12-06 11:20 +0100
    Re: [PATCH 7/8] staging: fsl-dpaa2/eth: Add TODO file Greg KH <gregkh@linuxfoundation.org> - 2016-12-06 11:30 +0100
      RE: [PATCH 7/8] staging: fsl-dpaa2/eth: Add TODO file Stuart Yoder <stuart.yoder@nxp.com> - 2016-12-06 14:40 +0100
        Re: [PATCH 7/8] staging: fsl-dpaa2/eth: Add TODO file Greg KH <gregkh@linuxfoundation.org> - 2016-12-06 19:10 +0100
          RE: [PATCH 7/8] staging: fsl-dpaa2/eth: Add TODO file Stuart Yoder <stuart.yoder@nxp.com> - 2016-12-06 23:50 +0100
    RE: [PATCH 7/8] staging: fsl-dpaa2/eth: Add TODO file Ruxandra Ioana Radulescu <ruxandra.radulescu@nxp.com> - 2016-12-06 11:50 +0100

#1536825 — 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?

thanks,

greg k-h

[toc] | [next] | [standalone]

#1536831

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...

thanks,

greg k-h

[toc] | [prev] | [next] | [standalone]

#1536981

> -----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?

Seems like there are several options--
A.  Keep bus driver in drivers/staging for now, create new series to
    add dpio driver into staging, add fsl-dpaa2/eth into staging and
    refactor it to assume include dependencies in staging directories.
B.  Apply the series:
    [PATCH v3 0/9] staging: fsl-mc: move bus driver out of staging, add dpio
    ...in something close to it's current form.  There is some minor 
    feedback on the dpio driver that will require a v4, which I can
    send in the next day or so.

The fsl-mc bus driver to do list is complete, with the exception
of demonstrating a functional driver on top of the bus driver,
and as described in the cover letter of the "move bus driver out
of staging" series the dpio driver demonstrates what an fsl-mc
device driver looks like.

Thanks,
Stuart
    

[toc] | [prev] | [next] | [standalone]

#1537175

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?

thanks,

greg k-h

[toc] | [prev] | [next] | [standalone]

#1537303

> -----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

[toc] | [prev] | [next] | [standalone]

#1536845

> -----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.
I can remove the other bullets from the TODO list if you think they're
not worth mentioning.

Thanks,
Ioana

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web