| Message ID | 20240904095815.1572186-2-caleb.connolly@linaro.org |
|---|---|
| State | New |
| Headers | show |
| Series | ath11k: allow missing memory-regions | expand |
On 9/4/2024 3:27 PM, Caleb Connolly wrote: > On SC7280 platforms which are running with TrustZone, it is not > necessary to manually map the memory regions used by the wifi hardware. > However, ath11k will currently fail to load unless both memory regions > are specified. > > This breaks wifi on the rb3gen2 which only specifies the firmware memory > region and does not use the CE region. > > Adjust the order of operations in ath11k_ahb_fw_resources_init() to > check for the wifi-firmware subnode before attempting to parse the > memory regions. > > Signed-off-by: Caleb Connolly <caleb.connolly@linaro.org> Reviewed-by: Raj Kumar Bhagat <quic_rajkbhag@quicinc.com>
Caleb Connolly <caleb.connolly@linaro.org> wrote: > On SC7280 platforms which are running with TrustZone, it is not > necessary to manually map the memory regions used by the wifi hardware. > However, ath11k will currently fail to load unless both memory regions > are specified. > > This breaks wifi on the rb3gen2 which only specifies the firmware memory > region and does not use the CE region. > > Adjust the order of operations in ath11k_ahb_fw_resources_init() to > check for the wifi-firmware subnode before attempting to parse the > memory regions. > > Signed-off-by: Caleb Connolly <caleb.connolly@linaro.org> > Acked-by: Jeff Johnson <quic_jjohnson@quicinc.com> > Reviewed-by: Raj Kumar Bhagat <quic_rajkbhag@quicinc.com> > Signed-off-by: Kalle Valo <quic_kvalo@quicinc.com> Patch applied to ath-next branch of ath.git, thanks. 095cb947490c wifi: ath11k: allow missing memory-regions
diff --git a/drivers/net/wireless/ath/ath11k/ahb.c b/drivers/net/wireless/ath/ath11k/ahb.c index 634d385fd9ad..e24f8da565db 100644 --- a/drivers/net/wireless/ath/ath11k/ahb.c +++ b/drivers/net/wireless/ath/ath11k/ahb.c @@ -999,20 +999,20 @@ static int ath11k_ahb_fw_resources_init(struct ath11k_base *ab) */ if (!ab->hw_params.fixed_fw_mem) return 0; - ret = ath11k_ahb_setup_msa_resources(ab); - if (ret) { - ath11k_err(ab, "failed to setup msa resources\n"); - return ret; - } - node = of_get_child_by_name(host_dev->of_node, "wifi-firmware"); if (!node) { ab_ahb->fw.use_tz = true; return 0; } + ret = ath11k_ahb_setup_msa_resources(ab); + if (ret) { + ath11k_err(ab, "failed to setup msa resources\n"); + return ret; + } + info.fwnode = &node->fwnode; info.parent = host_dev; info.name = node->name; info.dma_mask = DMA_BIT_MASK(32);
On SC7280 platforms which are running with TrustZone, it is not necessary to manually map the memory regions used by the wifi hardware. However, ath11k will currently fail to load unless both memory regions are specified. This breaks wifi on the rb3gen2 which only specifies the firmware memory region and does not use the CE region. Adjust the order of operations in ath11k_ahb_fw_resources_init() to check for the wifi-firmware subnode before attempting to parse the memory regions. Signed-off-by: Caleb Connolly <caleb.connolly@linaro.org> --- Cc: linux-arm-msm@vger.kernel.org --- drivers/net/wireless/ath/ath11k/ahb.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-)