Hi Ingo, Will do. Thanks for reviewing it. -Amit
On Tuesday 07 November 2006 22:19, Ingo Oeser wrote: > Hi Amit, > > one minor nitpick: > > You wrote: > > diff --git a/drivers/net/netxen/netxen_nic_main.c > > b/drivers/net/netxen/netxen_nic_main.c index b54ea16..4effb87 100644 > > --- a/drivers/net/netxen/netxen_nic_main.c > > +++ b/drivers/net/netxen/netxen_nic_main.c > > [...] > > > @@ -1040,7 +1041,7 @@ static int netxen_nic_poll(struct net_de > > netxen_nic_enable_int(adapter); > > } > > > > - return (done ? 0 : 1); > > + return (!done); > > return !done; > > Please lose the braces here (CodingStyle). > > Just respin or send this change along with later patchsets. > > Regards > > Ingo Oeser - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html