Commit ea11a89c authored by Jacobe Zang's avatar Jacobe Zang Committed by Kalle Valo
Browse files

wifi: brcmfmac: add flag for random seed during firmware download 



Providing the random seed to firmware was tied to the fact that the
device has a valid OTP, which worked for some Apple chips. However,
it turns out the BCM43752 device also needs the random seed in order
to get firmware running. Suspect it is simply tied to the firmware
branch used for the device. Introducing a mechanism to allow setting
it for a device through the device table.

Co-developed-by: default avatarOndrej Jirman <megi@xff.cz>
Signed-off-by: default avatarOndrej Jirman <megi@xff.cz>
Co-developed-by: default avatarArend van Spriel <arend.vanspriel@broadcom.com>
Signed-off-by: default avatarArend van Spriel <arend.vanspriel@broadcom.com>
Signed-off-by: default avatarJacobe Zang <jacobe.zang@wesion.com>
Reviewed-by: default avatarSebastian Reichel <sebastian.reichel@collabora.com>
Tested-by: Sebastian Reichel <sebastian.reichel@collabora.com> # On RK3588 EVB1
Acked-by: default avatarArend van Spriel <arend.vanspriel@broadcom.com>
Signed-off-by: default avatarKalle Valo <kvalo@kernel.org>
Link: https://patch.msgid.link/20240910-wireless-mainline-v14-4-9d80fea5326d@wesion.com
parent 0ff08433

drivers/net/wireless/broadcom/brcm80211/brcmfmac/pcie.c

+44 −8
Original line number Diff line number Diff line
@@ -66,6 +66,7 @@ BRCMF_FW_DEF(4365C, "brcmfmac4365c-pcie"); 
BRCMF_FW_DEF(4366B, "brcmfmac4366b-pcie");

BRCMF_FW_DEF(4366C, "brcmfmac4366c-pcie");

BRCMF_FW_DEF(4371, "brcmfmac4371-pcie");

BRCMF_FW_CLM_DEF(43752, "brcmfmac43752-pcie");

BRCMF_FW_CLM_DEF(4377B3, "brcmfmac4377b3-pcie");

BRCMF_FW_CLM_DEF(4378B1, "brcmfmac4378b1-pcie");

BRCMF_FW_CLM_DEF(4378B3, "brcmfmac4378b3-pcie");
@@ -104,6 +105,7 @@ static const struct brcmf_firmware_mapping brcmf_pcie_fwnames[] = { 
	BRCMF_FW_ENTRY(BRCM_CC_43664_CHIP_ID, 0xFFFFFFF0, 4366C),

	BRCMF_FW_ENTRY(BRCM_CC_43666_CHIP_ID, 0xFFFFFFF0, 4366C),

	BRCMF_FW_ENTRY(BRCM_CC_4371_CHIP_ID, 0xFFFFFFFF, 4371),

	BRCMF_FW_ENTRY(BRCM_CC_43752_CHIP_ID, 0xFFFFFFFF, 43752),

	BRCMF_FW_ENTRY(BRCM_CC_4377_CHIP_ID, 0xFFFFFFFF, 4377B3), /* revision ID 4 */

	BRCMF_FW_ENTRY(BRCM_CC_4378_CHIP_ID, 0x0000000F, 4378B1), /* revision ID 3 */

	BRCMF_FW_ENTRY(BRCM_CC_4378_CHIP_ID, 0xFFFFFFE0, 4378B3), /* revision ID 5 */
@@ -353,6 +355,7 @@ struct brcmf_pciedev_info { 
			  u16 value);

	struct brcmf_mp_device *settings;

	struct brcmf_otp_params otp;

	bool fwseed;

#ifdef DEBUG

	u32 console_interval;

	bool console_active;
@@ -1715,14 +1718,14 @@ static int brcmf_pcie_download_fw_nvram(struct brcmf_pciedev_info *devinfo, 
		memcpy_toio(devinfo->tcm + address, nvram, nvram_len);

		brcmf_fw_nvram_free(nvram);



		if (devinfo->otp.valid) {

		if (devinfo->fwseed) {

			size_t rand_len = BRCMF_RANDOM_SEED_LENGTH;

			struct brcmf_random_seed_footer footer = {

				.length = cpu_to_le32(rand_len),

				.magic = cpu_to_le32(BRCMF_RANDOM_SEED_MAGIC),

			};



			/* Some Apple chips/firmwares expect a buffer of random

			/* Some chips/firmwares expect a buffer of random

			 * data to be present before NVRAM

			 */

			brcmf_dbg(PCIE, "Download random seed\n");
@@ -2394,6 +2397,37 @@ static void brcmf_pcie_debugfs_create(struct device *dev) 
}

#endif



struct brcmf_pcie_drvdata {

	enum brcmf_fwvendor vendor;

	bool fw_seed;

};



enum {

	BRCMF_DRVDATA_CYW,

	BRCMF_DRVDATA_BCA,

	BRCMF_DRVDATA_WCC,

	BRCMF_DRVDATA_WCC_SEED,

};



static const struct brcmf_pcie_drvdata drvdata[] = {

	[BRCMF_DRVDATA_CYW] = {

		.vendor = BRCMF_FWVENDOR_CYW,

		.fw_seed = false,

	},

	[BRCMF_DRVDATA_BCA] = {

		.vendor = BRCMF_FWVENDOR_BCA,

		.fw_seed = false,

	},

	[BRCMF_DRVDATA_WCC] = {

		.vendor = BRCMF_FWVENDOR_WCC,

		.fw_seed = false,

	},

	[BRCMF_DRVDATA_WCC_SEED] = {

		.vendor = BRCMF_FWVENDOR_WCC,

		.fw_seed = true,

	},

};



/* Forward declaration for pci_match_id() call */

static const struct pci_device_id brcmf_pcie_devid_table[];
@@ -2475,9 +2509,10 @@ brcmf_pcie_probe(struct pci_dev *pdev, const struct pci_device_id *id) 
	bus->bus_priv.pcie = pcie_bus_dev;

	bus->ops = &brcmf_pcie_bus_ops;

	bus->proto_type = BRCMF_PROTO_MSGBUF;

	bus->fwvid = id->driver_data;

	bus->chip = devinfo->coreid;

	bus->wowl_supported = pci_pme_capable(pdev, PCI_D3hot);

	bus->fwvid = drvdata[id->driver_data].vendor;

	devinfo->fwseed = drvdata[id->driver_data].fw_seed;

	dev_set_drvdata(&pdev->dev, bus);



	ret = brcmf_alloc(&devinfo->pdev->dev, devinfo->settings);
@@ -2663,14 +2698,14 @@ static const struct dev_pm_ops brcmf_pciedrvr_pm = { 
		BRCM_PCIE_VENDOR_ID_BROADCOM, (dev_id), \

		PCI_ANY_ID, PCI_ANY_ID, \

		PCI_CLASS_NETWORK_OTHER << 8, 0xffff00, \

		BRCMF_FWVENDOR_ ## fw_vend \

		BRCMF_DRVDATA_ ## fw_vend \

	}

#define BRCMF_PCIE_DEVICE_SUB(dev_id, subvend, subdev, fw_vend) \

	{ \

		BRCM_PCIE_VENDOR_ID_BROADCOM, (dev_id), \

		(subvend), (subdev), \

		PCI_CLASS_NETWORK_OTHER << 8, 0xffff00, \

		BRCMF_FWVENDOR_ ## fw_vend \

		BRCMF_DRVDATA_ ## fw_vend \

	}



static const struct pci_device_id brcmf_pcie_devid_table[] = {
@@ -2698,9 +2733,10 @@ static const struct pci_device_id brcmf_pcie_devid_table[] = { 
	BRCMF_PCIE_DEVICE(BRCM_PCIE_4366_5G_DEVICE_ID, BCA),

	BRCMF_PCIE_DEVICE(BRCM_PCIE_4371_DEVICE_ID, WCC),

	BRCMF_PCIE_DEVICE(BRCM_PCIE_43596_DEVICE_ID, CYW),

	BRCMF_PCIE_DEVICE(BRCM_PCIE_4377_DEVICE_ID, WCC),

	BRCMF_PCIE_DEVICE(BRCM_PCIE_4378_DEVICE_ID, WCC),

	BRCMF_PCIE_DEVICE(BRCM_PCIE_4387_DEVICE_ID, WCC),

	BRCMF_PCIE_DEVICE(BRCM_PCIE_4377_DEVICE_ID, WCC_SEED),

	BRCMF_PCIE_DEVICE(BRCM_PCIE_4378_DEVICE_ID, WCC_SEED),

	BRCMF_PCIE_DEVICE(BRCM_PCIE_4387_DEVICE_ID, WCC_SEED),

	BRCMF_PCIE_DEVICE(BRCM_PCIE_43752_DEVICE_ID, WCC_SEED),



	{ /* end: all zeroes */ }

};

drivers/net/wireless/broadcom/brcm80211/include/brcm_hw_ids.h

+2 −0
Original line number Diff line number Diff line
@@ -52,6 +52,7 @@ 
#define BRCM_CC_43664_CHIP_ID		43664

#define BRCM_CC_43666_CHIP_ID		43666

#define BRCM_CC_4371_CHIP_ID		0x4371

#define BRCM_CC_43752_CHIP_ID		43752

#define BRCM_CC_4377_CHIP_ID		0x4377

#define BRCM_CC_4378_CHIP_ID		0x4378

#define BRCM_CC_4387_CHIP_ID		0x4387
@@ -94,6 +95,7 @@ 
#define BRCM_PCIE_4366_5G_DEVICE_ID	0x43c5

#define BRCM_PCIE_4371_DEVICE_ID	0x440d

#define BRCM_PCIE_43596_DEVICE_ID	0x4415

#define BRCM_PCIE_43752_DEVICE_ID	0x449d

#define BRCM_PCIE_4377_DEVICE_ID	0x4488

#define BRCM_PCIE_4378_DEVICE_ID	0x4425

#define BRCM_PCIE_4387_DEVICE_ID	0x4433