Thread (6 messages) 6 messages, 3 authors, 2004-11-06

Re: [patch 1/9] arcnet fixes

From: Andrew Morton <hidden>
Date: 2004-10-29 20:39:23

Esben Nielsen [off-list ref] wrote:
quoted
One problem with the patch though.
 > 
 > > diff -puN include/linux/if_ether.h~arcnet-fixes include/linux/if_ether.h
 > > --- 25/include/linux/if_ether.h~arcnet-fixes	2004-10-28 00:17:07.230704752 -0700
 > > +++ 25-akpm/include/linux/if_ether.h	2004-10-28 00:17:07.247702168 -0700
 > > @@ -91,6 +91,7 @@
 > >  #define ETH_P_IRDA	0x0017		/* Linux-IrDA			*/
 > >  #define ETH_P_ECONET	0x0018		/* Acorn Econet			*/
 > >  #define ETH_P_HDLC	0x0019		/* HDLC frames			*/
 > > +#define ETH_P_ARCNET	0x0020		/* ArcNet			*/
 > 
 > You should use 0x001a or similar, so you don't have
 > to do this next part of your patch, bumping NPROTO
 > in net.h
 > 

 Doh! I can't count in hex it seems. This would probably also mean I won't
 have to recompile the whole kernel but just the modules and I don't have 
 to reboot to make it work on a new machine - but just until somebody else
 picks the same protocol number ofcourse.

 Do I have to fix the problem and resubmit?
Yes please.  A patch relative to the previous one would be nice.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help