From: "Linus Walleij" <linus.walleij@xxxxxxxxxx>
Date: Wed, 25 Apr 2012 17:30:08 +0200
> As a first step to consolidate the RNDIS implementations, break out
> a common file with all the #defines and move it to <linux/rndis.h>.
> The compilation screams about double-defines all over the place
> (as could be expected, this is why we're consolidating) but it
> compiles cleanly still.
>
> Signed-off-by: Linus Walleij <linus.walleij@xxxxxxxxxx>
After this patch the build emits tons of warnings because you've
left a situation where there will be duplicate definitions:
In file included from drivers/net/hyperv/hyperv_net.h:30:0,
from drivers/net/hyperv/netvsc_drv.c:41:
include/linux/rndis.h:93:0: warning: "OID_GEN_CURRENT_PACKET_FILTER" redefined [enabled by default]
include/linux/rndis.h:51:0: note: this is the location of the previous definition
include/linux/rndis.h:239:0: warning: "RNDIS_STATUS_SUCCESS" redefined [enabled by default]
...
This is unacceptable, the tree must build cleanly and work properly at
each step in the patch series, in order to ensure bisectability. You
can't "fix it up" in a patch later in the series and just assume that
the whole series is applied.
Also, in the new file you created you added a spurious empty line at
the end of the file. Please remove that as GIT notices this when
applying your patch and thus warns about it.
In fact, please go through this entire patch set (like I was) making
sure the build is clean and no bugs or bisectability issues are
introduced each step of the way.
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
