Daniel Golle <daniel@xxxxxxxxxxxxxx> writes: > From: Gabor Juhos <juhosg@xxxxxxxxxxx> > > On the RT3593 chipset, the beacon registers are located > in the high 8KB part of the shared memory. > > The high part of the shared memory is only accessible > if it is explicitly selected. Add a helper function > in order to be able to control the SHR_MSEL bit in > the PBF_SYS_CTRL register. Also add a few more helper > functions and use those to select the correct part of > the shared memory before and after accessing the beacon > registers. > > The base addresses of the beacon registers are also > different from the actually used values, so fix the > 'rt2800_hw_beacon_base' function to return the correct > values. > > Signed-off-by: Gabor Juhos <juhosg@xxxxxxxxxxx> Daniel, as you are submitting these patches you should add your Signed-off-by after the author's line. Documentation/SubmittingPatches contains more info about that and what Signed-off-by means. And also take a look at how Luca does it: http://www.spinics.net/lists/linux-wireless/msg158064.html -- Kalle Valo