Hello. On Tue, 2014-04-22 at 20:55, Alexander Aring wrote: > This patch removes a unnecessary state read. The status value is never > evaluate after reading the state. > > Signed-off-by: Alexander Aring <alex.ar...@gmail.com> > --- > drivers/net/ieee802154/at86rf230.c | 4 ---- > 1 file changed, 4 deletions(-) > > diff --git a/drivers/net/ieee802154/at86rf230.c > b/drivers/net/ieee802154/at86rf230.c > index 5079036..6e6e742 100644 > --- a/drivers/net/ieee802154/at86rf230.c > +++ b/drivers/net/ieee802154/at86rf230.c > @@ -967,10 +967,6 @@ static int at86rf230_hw_init(struct at86rf230_local *lp) > u8 status; > u8 csma_seed[2]; > > - rc = at86rf230_read_subreg(lp, SR_TRX_STATUS, &status); > - if (rc) > - return rc; > -
I don't have the full code around right now but if status is never used you could also remove the variable all together or is it used again later? regards Stefan Schmidt ------------------------------------------------------------------------------ Start Your Social Network Today - Download eXo Platform Build your Enterprise Intranet with eXo Platform Software Java Based Open Source Intranet - Social, Extensible, Cloud Ready Get Started Now And Turn Your Intranet Into A Collaboration Platform http://p.sf.net/sfu/ExoPlatform _______________________________________________ Linux-zigbee-devel mailing list Linux-zigbee-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/linux-zigbee-devel
