2010/2/9 Michael Buesch <m...@bu3sch.de>: > On Tuesday 09 February 2010 21:04:40 Rafał Miłecki wrote: >> Signed-off-by: Rafał Miłecki <zaj...@gmail.com> >> --- >> drivers/net/wireless/b43/phy_common.c | 11 +++++++++++ >> drivers/net/wireless/b43/phy_common.h | 2 ++ >> 2 files changed, 13 insertions(+), 0 deletions(-) >> >> diff --git a/drivers/net/wireless/b43/phy_common.c >> b/drivers/net/wireless/b43/phy_common.c >> index 8f7d7ef..0b0f9df 100644 >> --- a/drivers/net/wireless/b43/phy_common.c >> +++ b/drivers/net/wireless/b43/phy_common.c >> @@ -466,3 +466,14 @@ struct b43_c32 b43_cordic(int theta) >> >> return ret; >> } >> + >> +/* http://bcm-v4.sipsolutions.net/802.11/PHY/BmacWriteShm */ >> +void b43_bmac_write_shm(struct b43_wldev *dev, u32 offset, u16 value) >> +{ >> + b43_write32(dev, B43_MMIO_SHM_CONTROL, 0x00010000 | (offset >> 2)); >> + b43_read32(dev, B43_MMIO_SHM_CONTROL); >> + if (offset & 2) >> + b43_write16(dev, 0x165, value); >> + else >> + b43_write16(dev, B43_MMIO_SHM_DATA, value); >> +} > > I'd like to put a biiiig questionmark on this. > We already have SHM access. Your function does exactly the same, except that > it > accesses the bullshit register h165.
Yeah, that 0x165 is really crappy. Let's hope it really should be 0x166 (B43_MMIO_SHM_DATA_UNALIGNED). Do you think we should add dummy-read to b43_shm_control_word as BmacWriteShm does? Currently we do not perform that. -- Rafał _______________________________________________ Bcm43xx-dev mailing list Bcm43xx-dev@lists.berlios.de https://lists.berlios.de/mailman/listinfo/bcm43xx-dev
