Manikanta Pubbisetty <quic_mpubbise@xxxxxxxxxxx> writes:
> Since WPPS Q6 does the PCIe enumeration of WCN6750, device
> information like BAR and BAR size is not known to the APPS
> processor (Application Processor SubSystem). In order to
> fetch these details, a QMI message called device info request
> will be sent to the target. Therefore, add logic to fetch
> BAR details from the target.
>
> Tested-on: WCN6750 hw1.0 AHB WLAN.MSL.1.0.1-00887-QCAMSLSWPLZ-1
> Tested-on: WCN6855 hw2.0 PCI WLAN.HSP.1.1-01720.1-QCAHSPSWPL_V1_V2_SILICONZ_LITE-1
> Tested-on: QCN9074 hw1.0 PCI WLAN.HK.2.5.0.1-01100-QCAHKSWPL_SILICONZ-1
> Tested-on: IPQ8074 hw2.0 AHB WLAN.HK.2.4.0.1-00192-QCAHKSWPL_SILICONZ-1
>
> Signed-off-by: Manikanta Pubbisetty <quic_mpubbise@xxxxxxxxxxx>
[...]
> +static int ath11k_qmi_request_device_info(struct ath11k_base *ab)
> +{
> + struct qmi_wlanfw_device_info_req_msg_v01 req = {};
> + struct qmi_wlanfw_device_info_resp_msg_v01 resp = {};
> + struct qmi_txn txn;
> + void __iomem *bar_addr_va;
> + int ret;
> +
> + /* device info message req is only sent for hybrid bus devices */
> + if (!ab->hw_params.hybrid_bus_type)
> + return 0;
> +
> + ret = qmi_txn_init(&ab->qmi.handle, &txn,
> + qmi_wlfw_device_info_resp_msg_v01_ei, &resp);
> + if (ret < 0)
> + goto out;
> +
> + ret = qmi_send_request(&ab->qmi.handle, NULL, &txn,
> + QMI_WLANFW_DEVICE_INFO_REQ_V01,
> + QMI_WLANFW_DEVICE_INFO_REQ_MSG_V01_MAX_LEN,
> + qmi_wlanfw_device_info_req_msg_v01_ei, &req);
> + if (ret < 0) {
> + qmi_txn_cancel(&txn);
> + ath11k_warn(ab, "qmi failed to send target device info request, err = %d\n",
> + ret);
> + goto out;
> + }
> +
> + ret = qmi_txn_wait(&txn, msecs_to_jiffies(ATH11K_QMI_WLANFW_TIMEOUT_MS));
> + if (ret < 0) {
> + ath11k_warn(ab, "qmi failed target device info request %d\n", ret);
> + goto out;
> + }
> +
> + if (resp.resp.result != QMI_RESULT_SUCCESS_V01) {
> + ath11k_warn(ab, "qmi device info req failed, result: %d, err: %d\n",
> + resp.resp.result, resp.resp.error);
> + ret = -EINVAL;
> + goto out;
> + }
> +
> + if (!resp.bar_addr_valid || !resp.bar_size_valid) {
> + ath11k_warn(ab, "qmi device info response invalid, result: %d, err: %d\n",
> + resp.resp.result, resp.resp.error);
> + ret = -EINVAL;
> + goto out;
> + }
> +
> + if (!resp.bar_addr ||
> + resp.bar_size != ATH11K_QMI_DEVICE_BAR_SIZE) {
> + ath11k_warn(ab, "qmi device info invalid addr and size, result: %d, err: %d\n",
> + resp.resp.result, resp.resp.error);
> + ret = -EINVAL;
> + goto out;
> + }
> +
> + bar_addr_va = devm_ioremap(ab->dev, resp.bar_addr, resp.bar_size);
> +
> + if (!bar_addr_va) {
> + ath11k_warn(ab, "qmi device info ioremap failed\n");
> + ab->mem_len = 0;
> + ret = -EIO;
> + goto out;
> + }
> +
> + ab->mem = bar_addr_va;
> + ab->mem_len = resp.bar_size;
> +
> + return 0;
> +out:
> + return ret;
> +}
In the pending branch I changed the warning messages to follow the style
used in ath11k.
> +
> static int ath11k_qmi_request_target_cap(struct ath11k_base *ab)
> {
> struct qmi_wlanfw_cap_req_msg_v01 req;
> @@ -2749,6 +2886,12 @@ static int ath11k_qmi_event_load_bdf(struct ath11k_qmi *qmi)
> return ret;
> }
>
> + ret = ath11k_qmi_request_device_info(ab);
> + if (ret < 0) {
> + ath11k_warn(ab, "failed to request qmi device info %d\n", ret);
> + return ret;
> + }
Here too.
--
https://patchwork.kernel.org/project/linux-wireless/list/
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
