[v2, 1/5] net: dpaa2: move DPAA2 PTP driver out of staging/

Y.b. Lu yangbo.lu at nxp.com
Mon Oct 8 07:50:15 UTC 2018


Let me just add an additional patch in v3.
I had sent the new v3 patch-set out. Please help to review.

Thanks a lot.

Best regards,
Yangbo Lu

> -----Original Message-----
> From: Y.b. Lu
> Sent: Monday, October 8, 2018 11:11 AM
> To: 'Andrew Lunn' <andrew at lunn.ch>; Ioana Ciocoi Radulescu
> <ruxandra.radulescu at nxp.com>
> Cc: linux-kernel at vger.kernel.org; devel at driverdev.osuosl.org;
> netdev at vger.kernel.org; Richard Cochran <richardcochran at gmail.com>;
> David S . Miller <davem at davemloft.net>; Ioana Ciocoi Radulescu
> <ruxandra.radulescu at nxp.com>; Greg Kroah-Hartman
> <gregkh at linuxfoundation.org>
> Subject: RE: [v2, 1/5] net: dpaa2: move DPAA2 PTP driver out of staging/
> 
> Hi Andrew,
> 
> Sorry for late. Just come back from holiday.
> 
> Hi Ioana,
> 
> Could you generate an additional patch for dpaa2_eth to address Andrew's
> comments?
> There was also a warning when checked patch. FSL_DPAA2_ETH needed more
> description.
> 
> WARNING: please write a paragraph that describes the config symbol fully
> #66: FILE: drivers/net/ethernet/freescale/dpaa2/Kconfig:1:
> +config FSL_DPAA2_ETH
> 
> Thanks a lot.
> 
> Best regards,
> Yangbo Lu
> 
> > -----Original Message-----
> > From: Andrew Lunn <andrew at lunn.ch>
> > Sent: Sunday, September 30, 2018 3:46 AM
> > To: Y.b. Lu <yangbo.lu at nxp.com>
> > Cc: linux-kernel at vger.kernel.org; devel at driverdev.osuosl.org;
> > netdev at vger.kernel.org; Richard Cochran <richardcochran at gmail.com>;
> > David S . Miller <davem at davemloft.net>; Ioana Ciocoi Radulescu
> > <ruxandra.radulescu at nxp.com>; Greg Kroah-Hartman
> > <gregkh at linuxfoundation.org>
> > Subject: Re: [v2, 1/5] net: dpaa2: move DPAA2 PTP driver out of
> > staging/
> >
> > > +++ b/drivers/net/ethernet/freescale/dpaa2/Kconfig
> > > @@ -0,0 +1,15 @@
> > > +config FSL_DPAA2_ETH
> > > +	tristate "Freescale DPAA2 Ethernet"
> > > +	depends on FSL_MC_BUS && FSL_MC_DPIO
> >
> > Could you add in here COMPILE_TEST?
> >
> > > +	depends on NETDEVICES && ETHERNET
> >
> > With the move out of staging, i don't think these two are required.
> >
> >      Andrew


More information about the devel mailing list