2010/2/9 Michael Buesch <mb@xxxxxxxxx>: > On Tuesday 09 February 2010 21:04:40 Rafał Miłecki wrote: >> Signed-off-by: Rafał Miłecki <zajec5@xxxxxxxxx> >> --- >> 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ł -- 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
