On Sat, 2013-05-18 at 23:20 +0300, Petko Manolov wrote: > From: Petko Manolov <pet...@nucleusys.com> > > Moving constant and structure definitions out of rtl8150.c; > > Signed-off-by: Petko Manolov <pet...@nucleusys.com> > --- > drivers/net/usb/rtl8150.c | 121 +---------------------------------- > 1 file changed, 2 insertions(+), 119 deletions(-) > > diff --git a/drivers/net/usb/rtl8150.c b/drivers/net/usb/rtl8150.c > index a491d3a..7d1897b 100644 > --- a/drivers/net/usb/rtl8150.c > +++ b/drivers/net/usb/rtl8150.c > @@ -17,132 +17,15 @@ > #include <linux/usb.h> > #include <asm/uaccess.h> > > +#include "rtl8150.h" [...]
Now how is this driver going to compile, when rtl8150.h is only added by the next patch?! You need to squash these first 2 together, so that the driver can be compiled after each step. Ben. -- Ben Hutchings, Staff Engineer, Solarflare Not speaking for my employer; that's the marketing department's job. They asked us to note that Solarflare product names are trademarked. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/