On Sat, 2 Oct 2010 11:51:54 -0700 Greg KH <gregkh@xxxxxxx> wrote: > On Fri, Oct 01, 2010 at 07:24:54PM -0700, Andres Salomon wrote: > > On Fri, 1 Oct 2010 15:45:31 -0700 > > Randy Dunlap <randy.dunlap@xxxxxxxxxx> wrote: > > > > > On Fri, 1 Oct 2010 15:37:56 +1000 Stephen Rothwell wrote: > > > > > > > Hi all, > > > > > > > > Changes since 20100930: > > > > > > > > > drivers/staging/olpc_dcon/olpc_dcon_xo_1.c:57: error: implicit > > > declaration of function 'geode_gpio_event_irq' > > > drivers/staging/olpc_dcon/olpc_dcon_xo_1.c:57: error: implicit > > > declaration of function 'geode_gpio' > > > > Yep, it's pending a cs5535-gpio patch > > (https://patchwork.kernel.org/patch/209482/). > > Wait, as I didn't apply the later patches, what I did apply should > have still built properly (remember our rule, no breakage at any > point in a patch series.) So, should I revert the last olpc patch > that caused this problem? > Sorry, I thought that was made clear by the "this patch is necessary for building the driver" comments in each patch description. The original driver that was submitted doesn't build unless all patches are applied (the only which touches external APIs is the pending cs5535-gpio one). _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/devel