Hi all, Sent out the v2. Please help to review. Thanks a lot. Hi Andrew,
> -----Original Message----- > From: Y.b. Lu > Sent: Friday, September 28, 2018 4:04 PM > To: 'Andrew Lunn' <and...@lunn.ch> > Cc: linux-ker...@vger.kernel.org; de...@driverdev.osuosl.org; > net...@vger.kernel.org; Richard Cochran <richardcoch...@gmail.com>; > David S . Miller <da...@davemloft.net>; Ioana Ciocoi Radulescu > <ruxandra.radule...@nxp.com>; Greg Kroah-Hartman > <gre...@linuxfoundation.org> > Subject: RE: [PATCH 1/2] net: dpaa2: move DPAA2 PTP driver out of staging/ > > Hi Andrew, > > Thanks a lot for your comments. > Please see my comments inline. > > Best regards, > Yangbo Lu > > > -----Original Message----- > > From: Andrew Lunn <and...@lunn.ch> > > Sent: Thursday, September 27, 2018 9:25 PM > > To: Y.b. Lu <yangbo...@nxp.com> > > Cc: linux-ker...@vger.kernel.org; de...@driverdev.osuosl.org; > > net...@vger.kernel.org; Richard Cochran <richardcoch...@gmail.com>; > > David S . Miller <da...@davemloft.net>; Ioana Ciocoi Radulescu > > <ruxandra.radule...@nxp.com>; Greg Kroah-Hartman > > <gre...@linuxfoundation.org> > > Subject: Re: [PATCH 1/2] net: dpaa2: move DPAA2 PTP driver out of > > staging/ > > > > On Thu, Sep 27, 2018 at 07:12:27PM +0800, Yangbo Lu wrote: > > > This patch is to move DPAA2 PTP driver out of staging/ since the > > > dpaa2-eth had been moved out. > > > > > > Signed-off-by: Yangbo Lu <yangbo...@nxp.com> > > > --- [...] > > > Can rtc_drv be made const? > > [Y.b. Lu] Will use const in next version. > [Y.b. Lu] Sorry, just found it couldn't be made const. Warnings showed when built with const. > > Is rtc.h used by anything other than rtc.c? It seems like it can be removed. > > [Y.b. Lu] Let me remove it in next version. [Y.b. Lu] Sorry, I still kept that header file since dpaa2_phc_index need to be declared. _______________________________________________ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel