On Fri 2012-10-12 09:53:19, Joe Perches wrote: > On Fri, 2012-10-12 at 22:07 +0530, Adil Mujeeb wrote: > > Removed checkpatch.pl script reported ERRORs > [] > > diff --git a/linux-3.6-rc7/drivers/staging/winbond/phy_calibration.c > > b/linux-3.6-rc7/drivers/staging/winbond/phy_calibration.c > [] > > -#define AG_CONST 0.6072529350 > > +#define AG_CONST (0.6072529350) > > That one is unnecessary. > checkpatch is defective here.
Agreed. > I wonder why there is any floating point > in the code at all though. Some kind of signal strength computations. I'm not an wifi expert... :-). Pavel -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html -- 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/