Commit 5a20ef3d authored by Hauke Mehrtens's avatar Hauke Mehrtens Committed by John W. Linville
Browse files

ssb: remove rev from boardinfo 



Previously the rev contained the revision read from the pci config
space and was used as board_rev in the wireless drivers. This is wrong
the board_rev is only fetched from the sprom accordingly to the open
source part of the Broadcom SDK and brcmsmac. This patch removes the
rev from the boardinfo structure and uses the board_rev attribute from
sprom instead. This attribute is filled by PCI, PCMCIA, SDIO and SoC
code.

Signed-off-by: default avatarHauke Mehrtens <hauke@hauke-m.de>
Tested-by: default avatarArend van Spriel <arend@broadcom.com>
Signed-off-by: default avatarJohn W. Linville <linville@tuxdriver.com>
parent 12d95683

arch/mips/bcm47xx/setup.c

+0 −2
Original line number Diff line number Diff line
@@ -115,8 +115,6 @@ static int bcm47xx_get_invariants(struct ssb_bus *bus, 
		iv->boardinfo.vendor = SSB_BOARDVENDOR_BCM;

	if (nvram_getenv("boardtype", buf, sizeof(buf)) >= 0)

		iv->boardinfo.type = (u16)simple_strtoul(buf, NULL, 0);

	if (nvram_getenv("boardrev", buf, sizeof(buf)) >= 0)

		iv->boardinfo.rev = (u16)simple_strtoul(buf, NULL, 0);



	bcm47xx_fill_sprom(&iv->sprom, NULL);

drivers/net/wireless/b43/bus.c

+1 −1
Original line number Diff line number Diff line
@@ -210,7 +210,7 @@ struct b43_bus_dev *b43_bus_dev_ssb_init(struct ssb_device *sdev) 


	dev->board_vendor = sdev->bus->boardinfo.vendor;

	dev->board_type = sdev->bus->boardinfo.type;

	dev->board_rev = sdev->bus->boardinfo.rev;

	dev->board_rev = sdev->bus->sprom.board_rev;



	dev->chip_id = sdev->bus->chip_id;

	dev->chip_rev = sdev->bus->chip_rev;

drivers/net/wireless/b43/main.c

+2 −2
Original line number Diff line number Diff line
@@ -5243,10 +5243,10 @@ static void b43_sprom_fixup(struct ssb_bus *bus) 


	/* boardflags workarounds */

	if (bus->boardinfo.vendor == SSB_BOARDVENDOR_DELL &&

	    bus->chip_id == 0x4301 && bus->boardinfo.rev == 0x74)

	    bus->chip_id == 0x4301 && bus->sprom.board_rev == 0x74)

		bus->sprom.boardflags_lo |= B43_BFL_BTCOEXIST;

	if (bus->boardinfo.vendor == PCI_VENDOR_ID_APPLE &&

	    bus->boardinfo.type == 0x4E && bus->boardinfo.rev > 0x40)

	    bus->boardinfo.type == 0x4E && bus->sprom.board_rev > 0x40)

		bus->sprom.boardflags_lo |= B43_BFL_PACTRL;

	if (bus->bustype == SSB_BUSTYPE_PCI) {

		pdev = bus->host_pci;

drivers/net/wireless/b43legacy/main.c

+1 −1
Original line number Diff line number Diff line
@@ -3779,7 +3779,7 @@ static void b43legacy_sprom_fixup(struct ssb_bus *bus) 
	/* boardflags workarounds */

	if (bus->boardinfo.vendor == PCI_VENDOR_ID_APPLE &&

	    bus->boardinfo.type == 0x4E &&

	    bus->boardinfo.rev > 0x40)

	    bus->sprom.board_rev > 0x40)

		bus->sprom.boardflags_lo |= B43legacy_BFL_PACTRL;

}

drivers/net/wireless/b43legacy/phy.c

+2 −2
Original line number Diff line number Diff line
@@ -408,7 +408,7 @@ static void b43legacy_phy_setupg(struct b43legacy_wldev *dev) 


		if (is_bcm_board_vendor(dev) &&

		    (dev->dev->bus->boardinfo.type == 0x0416) &&

		    (dev->dev->bus->boardinfo.rev == 0x0017))

		    (dev->dev->bus->sprom.board_rev == 0x0017))

			return;



		b43legacy_ilt_write(dev, 0x5001, 0x0002);
@@ -424,7 +424,7 @@ static void b43legacy_phy_setupg(struct b43legacy_wldev *dev) 


		if (is_bcm_board_vendor(dev) &&

		    (dev->dev->bus->boardinfo.type == 0x0416) &&

		    (dev->dev->bus->boardinfo.rev == 0x0017))

		    (dev->dev->bus->sprom.board_rev == 0x0017))

			return;



		b43legacy_ilt_write(dev, 0x0401, 0x0002);

CRA Git | Maintained and supported by SUSTech CRA and CCSE