| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <198521ec-cb41-701c-06d7-0432564dd0d2@cypress.com>
Date: Tue, 10 Dec 2019 03:36:59 +0000
From: Chi-Hsien Lin <Chi-Hsien.Lin@...ress.com>
To: Soeren Moch <smoch@....de>, Kalle Valo <kvalo@...eaurora.org>
CC: Wright Feng <Wright.Feng@...ress.com>,
Arend van Spriel <arend.vanspriel@...adcom.com>,
Franky Lin <franky.lin@...adcom.com>,
Hante Meuleman <hante.meuleman@...adcom.com>,
"linux-wireless@...r.kernel.org" <linux-wireless@...r.kernel.org>,
"brcm80211-dev-list.pdl@...adcom.com"
<brcm80211-dev-list.pdl@...adcom.com>,
brcm80211-dev-list <brcm80211-dev-list@...ress.com>,
"netdev@...r.kernel.org" <netdev@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 1/8] brcmfmac: reset two D11 cores if chip has two D11
cores
On 12/10/2019 6:38, Soeren Moch wrote:
> From: Wright Feng <wright.feng@...ress.com>
>
> There are two D11 cores in RSDB chips like 4359. We have to reset two
> D11 cores simutaneously before firmware download, or the firmware may
> not be initialized correctly and cause "fw initialized failed" error.
>
> Signed-off-by: Wright Feng <wright.feng@...ress.com>
Reviewed-by: Chi-Hsien Lin <chi-hsien.lin@...ress.com>
> ---
> Cc: Kalle Valo <kvalo@...eaurora.org>
> Cc: Arend van Spriel <arend.vanspriel@...adcom.com>
> Cc: Franky Lin <franky.lin@...adcom.com>
> Cc: Hante Meuleman <hante.meuleman@...adcom.com>
> Cc: Chi-Hsien Lin <chi-hsien.lin@...ress.com>
> Cc: Wright Feng <wright.feng@...ress.com>
> Cc: linux-wireless@...r.kernel.org
> Cc: brcm80211-dev-list.pdl@...adcom.com
> Cc: brcm80211-dev-list@...ress.com
> Cc: netdev@...r.kernel.org
> Cc: linux-kernel@...r.kernel.org
> ---
> .../broadcom/brcm80211/brcmfmac/chip.c | 50 +++++++++++++++++++
> .../broadcom/brcm80211/brcmfmac/chip.h | 1 +
> .../broadcom/brcm80211/brcmfmac/pcie.c | 2 +-
> 3 files changed, 52 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/chip.c b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/chip.c
> index a795d781b4c5..0b5fbe5d8270 100644
> --- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/chip.c
> +++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/chip.c
> @@ -433,11 +433,25 @@ static void brcmf_chip_ai_resetcore(struct brcmf_core_priv *core, u32 prereset,
> {
> struct brcmf_chip_priv *ci;
> int count;
> + struct brcmf_core *d11core2 = NULL;
> + struct brcmf_core_priv *d11priv2 = NULL;
>
> ci = core->chip;
>
> + /* special handle two D11 cores reset */
> + if (core->pub.id == BCMA_CORE_80211) {
> + d11core2 = brcmf_chip_get_d11core(&ci->pub, 1);
> + if (d11core2) {
> + brcmf_dbg(INFO, "found two d11 cores, reset both\n");
> + d11priv2 = container_of(d11core2,
> + struct brcmf_core_priv, pub);
> + }
> + }
> +
> /* must disable first to work for arbitrary current core state */
> brcmf_chip_ai_coredisable(core, prereset, reset);
> + if (d11priv2)
> + brcmf_chip_ai_coredisable(d11priv2, prereset, reset);
>
> count = 0;
> while (ci->ops->read32(ci->ctx, core->wrapbase + BCMA_RESET_CTL) &
> @@ -449,9 +463,30 @@ static void brcmf_chip_ai_resetcore(struct brcmf_core_priv *core, u32 prereset,
> usleep_range(40, 60);
> }
>
> + if (d11priv2) {
> + count = 0;
> + while (ci->ops->read32(ci->ctx,
> + d11priv2->wrapbase + BCMA_RESET_CTL) &
> + BCMA_RESET_CTL_RESET) {
> + ci->ops->write32(ci->ctx,
> + d11priv2->wrapbase + BCMA_RESET_CTL,
> + 0);
> + count++;
> + if (count > 50)
> + break;
> + usleep_range(40, 60);
> + }
> + }
> +
> ci->ops->write32(ci->ctx, core->wrapbase + BCMA_IOCTL,
> postreset | BCMA_IOCTL_CLK);
> ci->ops->read32(ci->ctx, core->wrapbase + BCMA_IOCTL);
> +
> + if (d11priv2) {
> + ci->ops->write32(ci->ctx, d11priv2->wrapbase + BCMA_IOCTL,
> + postreset | BCMA_IOCTL_CLK);
> + ci->ops->read32(ci->ctx, d11priv2->wrapbase + BCMA_IOCTL);
> + }
> }
>
> char *brcmf_chip_name(u32 id, u32 rev, char *buf, uint len)
> @@ -1109,6 +1144,21 @@ void brcmf_chip_detach(struct brcmf_chip *pub)
> kfree(chip);
> }
>
> +struct brcmf_core *brcmf_chip_get_d11core(struct brcmf_chip *pub, u8 unit)
> +{
> + struct brcmf_chip_priv *chip;
> + struct brcmf_core_priv *core;
> +
> + chip = container_of(pub, struct brcmf_chip_priv, pub);
> + list_for_each_entry(core, &chip->cores, list) {
> + if (core->pub.id == BCMA_CORE_80211) {
> + if (unit-- == 0)
> + return &core->pub;
> + }
> + }
> + return NULL;
> +}
> +
> struct brcmf_core *brcmf_chip_get_core(struct brcmf_chip *pub, u16 coreid)
> {
> struct brcmf_chip_priv *chip;
> diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/chip.h b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/chip.h
> index 7b00f6a59e89..8fa38658e727 100644
> --- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/chip.h
> +++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/chip.h
> @@ -74,6 +74,7 @@ struct brcmf_chip *brcmf_chip_attach(void *ctx,
> const struct brcmf_buscore_ops *ops);
> void brcmf_chip_detach(struct brcmf_chip *chip);
> struct brcmf_core *brcmf_chip_get_core(struct brcmf_chip *chip, u16 coreid);
> +struct brcmf_core *brcmf_chip_get_d11core(struct brcmf_chip *pub, u8 unit);
> struct brcmf_core *brcmf_chip_get_chipcommon(struct brcmf_chip *chip);
> struct brcmf_core *brcmf_chip_get_pmu(struct brcmf_chip *pub);
> bool brcmf_chip_iscoreup(struct brcmf_core *core);
> diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/pcie.c b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/pcie.c
> index f64ce5074a55..7ac72804e285 100644
> --- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/pcie.c
> +++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/pcie.c
> @@ -78,7 +78,7 @@ static const struct brcmf_firmware_mapping brcmf_pcie_fwnames[] = {
> BRCMF_FW_ENTRY(BRCM_CC_4371_CHIP_ID, 0xFFFFFFFF, 4371),
> };
>
> -#define BRCMF_PCIE_FW_UP_TIMEOUT 2000 /* msec */
> +#define BRCMF_PCIE_FW_UP_TIMEOUT 5000 /* msec */
>
> #define BRCMF_PCIE_REG_MAP_SIZE (32 * 1024)
>
> --
> 2.17.1
>
> .
>
Powered by blists - more mailing lists