Hi John, Please see in-line. On Tue, Apr 12, 2011 at 01:28:32PM -0700, John W. Linville wrote: > On Sat, Apr 09, 2011 at 12:25:37AM +0530, Yogesh Ashok Powar wrote: > > Fix checkpatch errors and warnings comprising of indent errors, spaces and > > __packed warnings. Also fix 'make C = 2' warnings. > > > > Signed-off-by: Yogesh Ashok Powar <yogeshp@xxxxxxxxxxx> > > --- > > drivers/net/wireless/mwl8k.c | 27 +++++++++++++-------------- > > 1 files changed, 13 insertions(+), 14 deletions(-) > > > > diff --git a/drivers/net/wireless/mwl8k.c b/drivers/net/wireless/mwl8k.c > > index 8913180..fb472f4 100644 > > --- a/drivers/net/wireless/mwl8k.c > > +++ b/drivers/net/wireless/mwl8k.c > > @@ -701,7 +701,7 @@ static int mwl8k_load_firmware(struct ieee80211_hw *hw) > > "helper image\n", pci_name(priv->pdev)); > > return rc; > > } > > - msleep(5); > > + msleep(20); > > > > rc = mwl8k_feed_fw_image(priv, fw->data, fw->size); > > } else { > > Is this an indent error, a space, or a __packed warning?? This was checkpatch warning. The exact warning is as below >WARNING: msleep < 20ms can sleep for up to 20ms; see Documentation/timers/timers-howto.txt >+ msleep(5); Thanks Yogesh Powar -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html