diff mbox series

[SRU,Xenial] ath10k: update the IRAM bank number for QCA9377

Message ID 20180521081307.3910-2-acelan.kao@canonical.com
State New
Headers show
Series [SRU,Xenial] ath10k: update the IRAM bank number for QCA9377 | expand

Commit Message

AceLan Kao May 21, 2018, 8:13 a.m. UTC
From: Ryan Hsu <ryanhsu@codeaurora.org>

BugLink: https://bugs.launchpad.net/bugs/1748345

Preparation for a new QCA9377 firmware release. The new firmware release
requires more IRAM banks, hence update that on ath10k.

The IRAM banks promotion won't break any backwards compatibility, as those IRAM
banks were not getting used in previous firmware releases.

Signed-off-by: Ryan Hsu <ryanhsu@codeaurora.org>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
(backported from commit fc8b92635f79cfc4dd3015384bebafada0b08f19)
Signed-off-by: AceLan Kao <acelan.kao@canonical.com>
---
 drivers/net/wireless/ath/ath10k/pci.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Colin Ian King May 21, 2018, 8:16 a.m. UTC | #1
On 21/05/18 09:13, AceLan Kao wrote:
> From: Ryan Hsu <ryanhsu@codeaurora.org>
> 
> BugLink: https://bugs.launchpad.net/bugs/1748345
> 
> Preparation for a new QCA9377 firmware release. The new firmware release
> requires more IRAM banks, hence update that on ath10k.
> 
> The IRAM banks promotion won't break any backwards compatibility, as those IRAM
> banks were not getting used in previous firmware releases.
> 
> Signed-off-by: Ryan Hsu <ryanhsu@codeaurora.org>
> Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
> (backported from commit fc8b92635f79cfc4dd3015384bebafada0b08f19)
> Signed-off-by: AceLan Kao <acelan.kao@canonical.com>
> ---
>  drivers/net/wireless/ath/ath10k/pci.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/net/wireless/ath/ath10k/pci.c b/drivers/net/wireless/ath/ath10k/pci.c
> index 907fd60c4241..7cc780f4044b 100644
> --- a/drivers/net/wireless/ath/ath10k/pci.c
> +++ b/drivers/net/wireless/ath/ath10k/pci.c
> @@ -1886,7 +1886,7 @@ static int ath10k_pci_get_num_banks(struct ath10k *ar)
>  		}
>  		break;
>  	case QCA9377_1_0_DEVICE_ID:
> -		return 2;
> +		return 9;
>  	}
>  
>  	ath10k_warn(ar, "unknown number of banks, assuming 1\n");
> 
Clean backport, makes sense and has minimal regression potential.

Acked-by: Colin Ian King <colin.king@canonical.com>
Anthony Wong May 21, 2018, 8:22 a.m. UTC | #2
On Mon, May 21, 2018 at 04:13:07PM +0800, AceLan Kao wrote:
> From: Ryan Hsu <ryanhsu@codeaurora.org>
> 
> BugLink: https://bugs.launchpad.net/bugs/1748345
> 
> Preparation for a new QCA9377 firmware release. The new firmware release
> requires more IRAM banks, hence update that on ath10k.
> 
> The IRAM banks promotion won't break any backwards compatibility, as those IRAM
> banks were not getting used in previous firmware releases.
> 
> Signed-off-by: Ryan Hsu <ryanhsu@codeaurora.org>
> Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
> (backported from commit fc8b92635f79cfc4dd3015384bebafada0b08f19)
> Signed-off-by: AceLan Kao <acelan.kao@canonical.com>
> ---
>  drivers/net/wireless/ath/ath10k/pci.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/net/wireless/ath/ath10k/pci.c b/drivers/net/wireless/ath/ath10k/pci.c
> index 907fd60c4241..7cc780f4044b 100644
> --- a/drivers/net/wireless/ath/ath10k/pci.c
> +++ b/drivers/net/wireless/ath/ath10k/pci.c
> @@ -1886,7 +1886,7 @@ static int ath10k_pci_get_num_banks(struct ath10k *ar)
>  		}
>  		break;
>  	case QCA9377_1_0_DEVICE_ID:
> -		return 2;
> +		return 9;
>  	}
>  
>  	ath10k_warn(ar, "unknown number of banks, assuming 1\n");
> -- 
> 2.17.0

Looks good, clean cherry-pick.

Acked-by: Anthony Wong <anthony.wong@canonical.com>
Wen-chien Jesse Sung May 21, 2018, 8:27 a.m. UTC | #3
Acked-By: Wen-chien Jesse Sung <jesse.sung@canonical.com>
diff mbox series

Patch

diff --git a/drivers/net/wireless/ath/ath10k/pci.c b/drivers/net/wireless/ath/ath10k/pci.c
index 907fd60c4241..7cc780f4044b 100644
--- a/drivers/net/wireless/ath/ath10k/pci.c
+++ b/drivers/net/wireless/ath/ath10k/pci.c
@@ -1886,7 +1886,7 @@  static int ath10k_pci_get_num_banks(struct ath10k *ar)
 		}
 		break;
 	case QCA9377_1_0_DEVICE_ID:
-		return 2;
+		return 9;
 	}
 
 	ath10k_warn(ar, "unknown number of banks, assuming 1\n");