[IDEA RFC][PATCH] b43: G-PHY: drop unused PHY rev 1 code
Larry Finger
Larry.Finger at lwfinger.net
Sun Jul 22 16:42:38 EDT 2012
On 07/22/2012 08:00 AM, Rafał Miłecki wrote:
> b43 does not support devices with G-PHY rev 1. They are handled by
> b43legacy.
I agree with this change.
Larry
> ---
> This change matches http://bcm-v4.sipsolutions.net/802.11/PHY/Init/G
> routine. Since b43 and b43legacy were splitted, I believe PHY support
> can be splitted too.
>
> I think similar change (just "reversed" can be applied to the b43legacy)
> ---
> drivers/net/wireless/b43/main.c | 2 +-
> drivers/net/wireless/b43/phy_g.c | 103 +-------------------------------------
> 2 files changed, 2 insertions(+), 103 deletions(-)
>
> diff --git a/drivers/net/wireless/b43/main.c b/drivers/net/wireless/b43/main.c
> index b80352b..8e5f4e8 100644
> --- a/drivers/net/wireless/b43/main.c
> +++ b/drivers/net/wireless/b43/main.c
> @@ -4306,7 +4306,7 @@ static int b43_phy_versioning(struct b43_wldev *dev)
> unsupported = 1;
> break;
> case B43_PHYTYPE_G:
> - if (phy_rev > 9)
> + if (phy_rev < 2 || phy_rev > 9)
> unsupported = 1;
> break;
> #ifdef CONFIG_B43_PHY_N
> diff --git a/drivers/net/wireless/b43/phy_g.c b/drivers/net/wireless/b43/phy_g.c
> index 12f467b..a67f278 100644
> --- a/drivers/net/wireless/b43/phy_g.c
> +++ b/drivers/net/wireless/b43/phy_g.c
> @@ -1489,104 +1489,6 @@ static u16 b43_radio_init2050(struct b43_wldev *dev)
> return ret;
> }
>
> -static void b43_phy_initb5(struct b43_wldev *dev)
> -{
> - struct b43_phy *phy = &dev->phy;
> - struct b43_phy_g *gphy = phy->g;
> - u16 offset, value;
> - u8 old_channel;
> -
> - if (phy->analog == 1) {
> - b43_radio_set(dev, 0x007A, 0x0050);
> - }
> - if ((dev->dev->board_vendor != SSB_BOARDVENDOR_BCM) &&
> - (dev->dev->board_type != SSB_BOARD_BU4306)) {
> - value = 0x2120;
> - for (offset = 0x00A8; offset < 0x00C7; offset++) {
> - b43_phy_write(dev, offset, value);
> - value += 0x202;
> - }
> - }
> - b43_phy_maskset(dev, 0x0035, 0xF0FF, 0x0700);
> - if (phy->radio_ver == 0x2050)
> - b43_phy_write(dev, 0x0038, 0x0667);
> -
> - if (phy->gmode || phy->rev >= 2) {
> - if (phy->radio_ver == 0x2050) {
> - b43_radio_set(dev, 0x007A, 0x0020);
> - b43_radio_set(dev, 0x0051, 0x0004);
> - }
> - b43_write16(dev, B43_MMIO_PHY_RADIO, 0x0000);
> -
> - b43_phy_set(dev, 0x0802, 0x0100);
> - b43_phy_set(dev, 0x042B, 0x2000);
> -
> - b43_phy_write(dev, 0x001C, 0x186A);
> -
> - b43_phy_maskset(dev, 0x0013, 0x00FF, 0x1900);
> - b43_phy_maskset(dev, 0x0035, 0xFFC0, 0x0064);
> - b43_phy_maskset(dev, 0x005D, 0xFF80, 0x000A);
> - }
> -
> - if (dev->bad_frames_preempt) {
> - b43_phy_set(dev, B43_PHY_RADIO_BITFIELD, (1 << 11));
> - }
> -
> - if (phy->analog == 1) {
> - b43_phy_write(dev, 0x0026, 0xCE00);
> - b43_phy_write(dev, 0x0021, 0x3763);
> - b43_phy_write(dev, 0x0022, 0x1BC3);
> - b43_phy_write(dev, 0x0023, 0x06F9);
> - b43_phy_write(dev, 0x0024, 0x037E);
> - } else
> - b43_phy_write(dev, 0x0026, 0xCC00);
> - b43_phy_write(dev, 0x0030, 0x00C6);
> - b43_write16(dev, 0x03EC, 0x3F22);
> -
> - if (phy->analog == 1)
> - b43_phy_write(dev, 0x0020, 0x3E1C);
> - else
> - b43_phy_write(dev, 0x0020, 0x301C);
> -
> - if (phy->analog == 0)
> - b43_write16(dev, 0x03E4, 0x3000);
> -
> - old_channel = phy->channel;
> - /* Force to channel 7, even if not supported. */
> - b43_gphy_channel_switch(dev, 7, 0);
> -
> - if (phy->radio_ver != 0x2050) {
> - b43_radio_write16(dev, 0x0075, 0x0080);
> - b43_radio_write16(dev, 0x0079, 0x0081);
> - }
> -
> - b43_radio_write16(dev, 0x0050, 0x0020);
> - b43_radio_write16(dev, 0x0050, 0x0023);
> -
> - if (phy->radio_ver == 0x2050) {
> - b43_radio_write16(dev, 0x0050, 0x0020);
> - b43_radio_write16(dev, 0x005A, 0x0070);
> - }
> -
> - b43_radio_write16(dev, 0x005B, 0x007B);
> - b43_radio_write16(dev, 0x005C, 0x00B0);
> -
> - b43_radio_set(dev, 0x007A, 0x0007);
> -
> - b43_gphy_channel_switch(dev, old_channel, 0);
> -
> - b43_phy_write(dev, 0x0014, 0x0080);
> - b43_phy_write(dev, 0x0032, 0x00CA);
> - b43_phy_write(dev, 0x002A, 0x88A3);
> -
> - b43_set_txpower_g(dev, &gphy->bbatt, &gphy->rfatt, gphy->tx_control);
> -
> - if (phy->radio_ver == 0x2050)
> - b43_radio_write16(dev, 0x005D, 0x000D);
> -
> - b43_write16(dev, 0x03E4, (b43_read16(dev, 0x03E4) & 0xFFC0) | 0x0004);
> -}
> -
> static void b43_phy_initb6(struct b43_wldev *dev)
> {
> struct b43_phy *phy = &dev->phy;
> @@ -1993,10 +1895,7 @@ static void b43_phy_initg(struct b43_wldev *dev)
> struct b43_phy_g *gphy = phy->g;
> u16 tmp;
>
> - if (phy->rev == 1)
> - b43_phy_initb5(dev);
> - else
> - b43_phy_initb6(dev);
> + b43_phy_initb6(dev);
>
> if (phy->rev >= 2 || phy->gmode)
> b43_phy_inita(dev);
>
More information about the b43-dev
mailing list