[ewg] [PATCH 14/14] nes: kernel build infrastructure

Roland Dreier rdreier at cisco.com
Thu Aug 9 10:34:22 PDT 2007


 > +ifdef CONFIG_INFINIBAND_NES_DEBUG
 > +EXTRA_CFLAGS += -DNES_DEBUG
 > +endif

I would just use the CONFIG_INFINIBAND_NES_DEBUG symbol directly in
your code.

 > +EXTRA_CFLAGS += -DNES_MINICM

If you're defining this unconditionally, can you just get rid of all
the tests of this symbol?

 - R.



More information about the ewg mailing list