Hi David, Sorry for the noise. i will send the fixed patch.
Thanks. -Varka Bhadram On Thu, Jun 12, 2014 at 11:35 PM, David Miller <da...@davemloft.net> wrote: > From: David Miller <da...@davemloft.net> > Date: Thu, 12 Jun 2014 11:03:24 -0700 (PDT) > >> From: Varka Bhadram <varkabhad...@gmail.com> >> Date: Thu, 12 Jun 2014 14:40:48 +0530 >> >>> separate the mrf24j40 hardware initialisation from probe() >>> and adds the sanity checkings. >>> >>> These checkings are required if somebody hasn't a right spi configuration >>> the probe function should fail. So we have to return from there. >>> >>> Signed-off-by: Varka Bhadram <var...@cdac.in> >> >> Applied, thanks. > > Reverted, did you even compile test this change? > > drivers/net/ieee802154/mrf24j40.c: In function ‘mrf24j40_probe’: > drivers/net/ieee802154/mrf24j40.c:741:10: error: ‘err_hw_init’ undeclared > (first use in this function) > drivers/net/ieee802154/mrf24j40.c:741:10: note: each undeclared identifier is > reported only once for each function it appears in > drivers/net/ieee802154/mrf24j40.c:759:1: warning: label ‘err_hw_init’ defined > but not used [-Wunused-label] > drivers/net/ieee802154/mrf24j40.c:702:5: warning: unused variable ‘val’ > [-Wunused-variable] > make[1]: *** [drivers/net/ieee802154/mrf24j40.o] Error 1 > make: *** [drivers/net/ieee802154/mrf24j40.o] Error 2 -- 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/
