On Saturday 27 February 2010 12:09:30 Rafał Miłecki wrote: > 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.
That doesn't hurt. It probably won't help much either, as the whole driver depends on implicit write posting. But I don't care whether you add it or not. -- Greetings, Michael. _______________________________________________ Bcm43xx-dev mailing list Bcm43xx-dev@lists.berlios.de https://lists.berlios.de/mailman/listinfo/bcm43xx-dev