75 lines
2.0 KiB
Diff
75 lines
2.0 KiB
Diff
From: Arend van Spriel <arend@broadcom.com>
|
|
Date: Wed, 11 Mar 2015 16:11:28 +0100
|
|
Subject: [PATCH] brcmfmac: always perform cores checks
|
|
|
|
Instead of checking the cores in the chip only if CONFIG_BRCMDBG
|
|
is selected perform the check always and extend it with more sanity
|
|
checking.
|
|
|
|
Reviewed-by: Hante Meuleman <meuleman@broadcom.com>
|
|
Reviewed-by: Pieter-Paul Giesberts <pieterpg@broadcom.com>
|
|
Signed-off-by: Arend van Spriel <arend@broadcom.com>
|
|
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
|
|
---
|
|
|
|
--- a/drivers/net/wireless/brcm80211/brcmfmac/chip.c
|
|
+++ b/drivers/net/wireless/brcm80211/brcmfmac/chip.c
|
|
@@ -419,13 +419,13 @@ static struct brcmf_core *brcmf_chip_add
|
|
return &core->pub;
|
|
}
|
|
|
|
-#ifdef DEBUG
|
|
/* safety check for chipinfo */
|
|
static int brcmf_chip_cores_check(struct brcmf_chip_priv *ci)
|
|
{
|
|
struct brcmf_core_priv *core;
|
|
bool need_socram = false;
|
|
bool has_socram = false;
|
|
+ bool cpu_found = false;
|
|
int idx = 1;
|
|
|
|
list_for_each_entry(core, &ci->cores, list) {
|
|
@@ -435,12 +435,14 @@ static int brcmf_chip_cores_check(struct
|
|
|
|
switch (core->pub.id) {
|
|
case BCMA_CORE_ARM_CM3:
|
|
+ cpu_found = true;
|
|
need_socram = true;
|
|
break;
|
|
case BCMA_CORE_INTERNAL_MEM:
|
|
has_socram = true;
|
|
break;
|
|
case BCMA_CORE_ARM_CR4:
|
|
+ cpu_found = true;
|
|
if (ci->pub.rambase == 0) {
|
|
brcmf_err("RAM base not provided with ARM CR4 core\n");
|
|
return -ENOMEM;
|
|
@@ -451,19 +453,21 @@ static int brcmf_chip_cores_check(struct
|
|
}
|
|
}
|
|
|
|
+ if (!cpu_found) {
|
|
+ brcmf_err("CPU core not detected\n");
|
|
+ return -ENXIO;
|
|
+ }
|
|
/* check RAM core presence for ARM CM3 core */
|
|
if (need_socram && !has_socram) {
|
|
brcmf_err("RAM core not provided with ARM CM3 core\n");
|
|
return -ENODEV;
|
|
}
|
|
+ if (!ci->pub.ramsize) {
|
|
+ brcmf_err("RAM size is undetermined\n");
|
|
+ return -ENOMEM;
|
|
+ }
|
|
return 0;
|
|
}
|
|
-#else /* DEBUG */
|
|
-static inline int brcmf_chip_cores_check(struct brcmf_chip_priv *ci)
|
|
-{
|
|
- return 0;
|
|
-}
|
|
-#endif
|
|
|
|
static void brcmf_chip_get_raminfo(struct brcmf_chip_priv *ci)
|
|
{
|