Since already Helmut complains about coding style, I'll add my 2 cents too: On Wed, Feb 08, 2012 at 09:00:02PM +0800, John Li wrote: > - if (!rt2x00_rt(rt2x00dev, RT5390)) { > + if (!rt2x00_rt(rt2x00dev, RT5390) || > + !rt2x00_rt(rt2x00dev, RT5392)) { Please make alignments like in example below (few spaces instead of last tab). if (!rt2x00_rt(rt2x00dev, RT5390) || !rt2x00_rt(rt2x00dev, RT5392)) { Thanks Stanislaw -- 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