On Saturday 27 February 2010 12:12:23 Rafał Miłecki wrote: > 2010/2/10 Michael Buesch <mb@xxxxxxxxx>: > > On Tuesday 09 February 2010 21:04:33 Rafał Miłecki wrote: > >> +#define B43_MMIO_PSM_PHY_HDR 0x492 /* programmable state machine */ > > > > The comment doesn't make a lot of sense. > > In case you don't know, the PSM is the part of the hardware > > that executes the firmware. > > Well, guess you're right. It was me hearing for the first time "PSM" > so decided to write it. Guess it's pretty obvious for every device > driver developer :) > Well, what I wanted to say is that it's more important to explain what PHY_HDR means. Something like /* PSM PHY header */ ? -- Greetings, Michael. -- 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