Hi On Mon, Jul 23, 2012 at 03:01:08PM +0800, Matt Chen wrote: > >> + > >> + rt2800_register_write(rt2x00dev, 0x58, 0x018); > >> + udelay(REGISTER_BUSY_DELAY); > >> + rt2800_register_write(rt2x00dev, 0x58, 0x418); > >> + udelay(REGISTER_BUSY_DELAY); > >> + rt2800_register_write(rt2x00dev, 0x58, 0x618); > > We really like to know what those black magic numbers mean. > I have no idea, Ralink gave it to me for the fixing. So perhaps patch should be posted by Ralink developer, who can fix that accordingly? Also it's clear now that you are not the author of the patch. Would be good to indicate true author of the patch by adding "From:" and "Signed-off-by:" lines. 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