On Thu, 10 Apr 2008, Chas Williams (CONTRACTOR) wrote:

quoted text
> In message <Pine.LNX.4.64.0804092259460.21022@ask.diku.dk>,Jesper Dangaard Brou
> er writes:
>> +#define ATM_CELL_SIZE               53  /* ATM cell size incl. header */
>> +#define ATM_CELL_PAYLOAD            48  /* ATM payload size */
>
> these constants are already in linux/atm.h

Yes, I remember.  My original patch also contained some #ifndef's to 
handle that case.  I cannot remember, but there was some reason why I 
needed those, something about the ATM part of TC not always being 
included.

+#ifndef ATM_CELL_SIZE
+#define ATM_CELL_SIZE               53  /* ATM cell size incl. header */
+#endif
+#ifndef ATM_CELL_PAYLOAD
+#define ATM_CELL_PAYLOAD            48  /* ATM payload size */
+#endif

Hilsen
   Jesper Brouer

--
-------------------------------------------------------------------
MSc. Master of Computer Science
Dept. of Computer Science, University of Copenhagen
Author of http://www.adsl-optimizer.dk
-------------------------------------------------------------------
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html