[PATCH][SRU][linux-oem] QCA9377 requires more IRAM banks for its new firmware

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
6 messages Options
Reply | Threaded
Open this post in threaded view
|

[PATCH][SRU][linux-oem] QCA9377 requires more IRAM banks for its new firmware

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

[Impact]
The new firmware for QCA9377 supports Qualcomm's RED(Radio Equipment Directive) compliance and requires more IRAM banks. The origin 4 banks doesn't fit the new BDF and firmware.

[Fix]
Change the bank number from 4 to 9.

[Regression Potential]
Low, increasing the number of IRAM banks should be no harm.

[Misc]
Currently, both driver patch and new firmware do not upstream yet. For the sake of avoiding dkms package, we can change the IRAM bank number before their patch is ready.

AceLan Kao (1):
  UBUNTU: SAUCE: ath10k: change QCA9377 IRAM back to 9

 drivers/net/wireless/ath/ath10k/pci.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

--
2.7.4


--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|

[PATCH] UBUNTU: SAUCE: ath10k: change QCA9377 IRAM back to 9

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

The patch is for downloading QCA9377 new firmware which is compliant
with RED 2.1.1 to QCA9377 IRAM bank 9. Because this new firmware is
bigger, IRAM bank 4 space is not enough, so we need to change to bank 9.

Signed-off-by: AceLan Kao <[hidden email]>
---
 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 7ebfc40..619c115 100644
--- a/drivers/net/wireless/ath/ath10k/pci.c
+++ b/drivers/net/wireless/ath/ath10k/pci.c
@@ -1991,7 +1991,7 @@ static int ath10k_pci_get_num_banks(struct ath10k *ar)
  }
  break;
  case QCA9377_1_0_DEVICE_ID:
- return 4;
+ return 9;
  }
 
  ath10k_warn(ar, "unknown number of banks, assuming 1\n");
--
2.7.4


--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|

ACK: [PATCH][SRU][linux-oem] QCA9377 requires more IRAM banks for its new firmware

Wen-chien Jesse Sung
In reply to this post by AceLan Kao
Acked-By: Wen-chien Jesse Sung <[hidden email]>


--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|

ACK: [PATCH][SRU][linux-oem] QCA9377 requires more IRAM banks for its new firmware

Kai-Heng Feng
In reply to this post by AceLan Kao

> On Feb 9, 2018, at 11:07 AM, AceLan Kao <[hidden email]> wrote:
>
> BugLink: https://bugs.launchpad.net/bugs/1748345
>
> [Impact]
> The new firmware for QCA9377 supports Qualcomm's RED(Radio Equipment  
> Directive) compliance and requires more IRAM banks. The origin 4 banks  
> doesn't fit the new BDF and firmware.
>
> [Fix]
> Change the bank number from 4 to 9.
>
> [Regression Potential]
> Low, increasing the number of IRAM banks should be no harm.
>
> [Misc]
> Currently, both driver patch and new firmware do not upstream yet. For  
> the sake of avoiding dkms package, we can change the IRAM bank number  
> before their patch is ready.
>
> AceLan Kao (1):
>   UBUNTU: SAUCE: ath10k: change QCA9377 IRAM back to 9
>
>  drivers/net/wireless/ath/ath10k/pci.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> —

Do note that this cover-letter isn’t in the same thread with the patch.

Acked-by: Kai-Heng Feng <[hidden email]>

> 2.7.4
>
>
> --
> kernel-team mailing list
> [hidden email]
> https://lists.ubuntu.com/mailman/listinfo/kernel-team

--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|

Re: [PATCH][SRU][linux-oem] QCA9377 requires more IRAM banks for its new firmware

Timo Aaltonen-6
In reply to this post by AceLan Kao
On 09.02.2018 05:07, AceLan Kao wrote:

> BugLink: https://bugs.launchpad.net/bugs/1748345
>
> [Impact]
> The new firmware for QCA9377 supports Qualcomm's RED(Radio Equipment Directive) compliance and requires more IRAM banks. The origin 4 banks doesn't fit the new BDF and firmware.
>
> [Fix]
> Change the bank number from 4 to 9.
>
> [Regression Potential]
> Low, increasing the number of IRAM banks should be no harm.
>
> [Misc]
> Currently, both driver patch and new firmware do not upstream yet. For the sake of avoiding dkms package, we can change the IRAM bank number before their patch is ready.
>
> AceLan Kao (1):
>   UBUNTU: SAUCE: ath10k: change QCA9377 IRAM back to 9
>
>  drivers/net/wireless/ath/ath10k/pci.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>

applied to linux-oem, thanks

--
t

--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|

APPLIED/OEM: Re: [PATCH][SRU][linux-oem] QCA9377 requires more IRAM banks for its new firmware

Timo Aaltonen-6
In reply to this post by AceLan Kao
On 09.02.2018 05:07, AceLan Kao wrote:

> BugLink: https://bugs.launchpad.net/bugs/1748345
>
> [Impact]
> The new firmware for QCA9377 supports Qualcomm's RED(Radio Equipment Directive) compliance and requires more IRAM banks. The origin 4 banks doesn't fit the new BDF and firmware.
>
> [Fix]
> Change the bank number from 4 to 9.
>
> [Regression Potential]
> Low, increasing the number of IRAM banks should be no harm.
>
> [Misc]
> Currently, both driver patch and new firmware do not upstream yet. For the sake of avoiding dkms package, we can change the IRAM bank number before their patch is ready.
>
> AceLan Kao (1):
>   UBUNTU: SAUCE: ath10k: change QCA9377 IRAM back to 9
>
>  drivers/net/wireless/ath/ath10k/pci.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>

now with a proper subject

--
t

--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team