On Sat, May 22, 2010 at 12:45, Dan Carpenter <erro...@gmail.com> wrote: > The bitwise negate is intended here. With the logical negate the > condition is always false. > > Signed-off-by: Dan Carpenter <erro...@gmail.com> > > diff --git a/drivers/net/e1000e/netdev.c b/drivers/net/e1000e/netdev.c > index 24507f3..57a7e41 100644 > --- a/drivers/net/e1000e/netdev.c > +++ b/drivers/net/e1000e/netdev.c > @@ -2554,7 +2554,7 @@ static void e1000_init_manageability_pt(struct > e1000_adapter *adapter) > mdef = er32(MDEF(i)); > > /* Ignore filters with anything other than IPMI ports > */ > - if (mdef & !(E1000_MDEF_PORT_623 | > E1000_MDEF_PORT_664)) > + if (mdef & ~(E1000_MDEF_PORT_623 | > E1000_MDEF_PORT_664)) > continue; > > /* Enable this decision filter in MANC2H */ > --
Thanks Dan, I have added the patch to my queue. -- Cheers, Jeff ------------------------------------------------------------------------------ _______________________________________________ E1000-devel mailing list E1000-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/e1000-devel To learn more about Intel® Ethernet, visit http://communities.intel.com/community/wired
