Messages in this thread | | | From | hayeswang <> | Subject | RE: [PATCH net-next v3 5/5] r8169: support RTL8411 | Date | Thu, 5 Apr 2012 14:06:14 +0800 |
| |
Francois Romieu [mailto:romieu@fr.zoreil.com] > Sent: Friday, March 30, 2012 3:25 PM > To: Hayeswang > Cc: netdev@vger.kernel.org; linux-kernel@vger.kernel.org > Subject: Re: [PATCH net-next v3 5/5] r8169: support RTL8411 > > Hayes Wang <hayeswang@realtek.com> : > > Support the new chip RTL8411. > > > > Signed-off-by: Hayes Wang <hayeswang@realtek.com> > > I do not see any change related to rtl_init_jumbo_ops. If > NULL is ok for > jumbo_{enable / disable}, it deserves to be outlined in the commit > message. > > No reviewer can figure what's going on otherwise. >
There is nothing to do for rtl_init_jumbo_ops. It is all right for keeping NULL. Do I need to resend this patch?
Best Regards, Hayes
| |