Received-SPF: pass (google.com: domain of linux-wireless+bounces-3624-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
From: Kalle Valo <kvalo@kernel.org>
To: Sven Eckelmann <sven@narfation.org>
Cc: ath12k@lists.infradead.org,  Raj Kumar Bhagat
 <quic_rajkbhag@quicinc.com>,  linux-wireless@vger.kernel.org,  Ganesh Babu
 Jothiram <quic_gjothira@quicinc.com>
Subject: Re: [PATCH v3 12/13] wifi: ath12k: Read board id to support
 split-PHY QCN9274
References: <20240129065724.2310207-1-quic_rajkbhag@quicinc.com>
	<20240129065724.2310207-13-quic_rajkbhag@quicinc.com>
	<23399810.6Emhk5qWAg@ripper>
Date: Thu, 15 Feb 2024 13:32:41 +0200
In-Reply-To: <23399810.6Emhk5qWAg@ripper> (Sven Eckelmann's message of "Wed,
	14 Feb 2024 15:17:11 +0100")
Message-ID: <87bk8inesm.fsf@kernel.org>
User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/28.2 (gnu/linux)
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain

Sven Eckelmann <sven@narfation.org> writes:

> On Monday, 29 January 2024 07:57:23 CET Raj Kumar Bhagat wrote:
>> diff --git a/drivers/net/wireless/ath/ath12k/fw.c
>> b/drivers/net/wireless/ath/ath12k/fw.c
>> index fbcf40c97792..5be4b2d4a19d 100644
>> --- a/drivers/net/wireless/ath/ath12k/fw.c
>> +++ b/drivers/net/wireless/ath/ath12k/fw.c
>> @@ -119,6 +119,13 @@ static int
>> ath12k_fw_request_firmware_api_n(struct ath12k_base *ab,
> [...]
>> +		case ATH12K_FW_IE_AMSS_DUALMAC_IMAGE:
>> +			ath12k_dbg(ab, ATH12K_DBG_BOOT,
>> +				   "found dualmac fw image ie (%zd B)\n",
>> +				   ie_len);
>> +			ab->fw.amss_dualmac_data = data;
>> +			ab->fw.amss_dualmac_len = ie_len;
>> +			break;
>>  		default:
>>  			ath12k_warn(ab, "Unknown FW IE: %u\n", ie_id);
>>  			break;
>
>> @@ -371,16 +375,43 @@ int ath12k_mhi_register(struct ath12k_pci *ab_pci)
> [...]
>> +	if (dualmac) {
>> +		if (ab->fw.amss_dualmac_data && ab->fw.amss_dualmac_len > 0) {
>> +			/* use MHI firmware file from firmware-N.bin */
>> +			mhi_ctrl->fw_data = ab->fw.amss_dualmac_data;
>> +			mhi_ctrl->fw_sz = ab->fw.amss_dualmac_len;
>> +		} else {
>> + ath12k_warn(ab, "dualmac firmware IE not present in
>> firmware-N.bin\n");
>> +			ret = -ENOENT;
>> +			goto free_controller;
>> +		}
>
> I saw this now already multiple times that the source code was referencing the 
> firmware-N.bin (like we had with ath10k). But they aren't published anywhere. 
> I can't find a single one at:
>
> *
> https://git.kernel.org/pub/scm/linux/kernel/git/firmware/linux-firmware.git/tree/ath11k
> *
> https://git.kernel.org/pub/scm/linux/kernel/git/firmware/linux-firmware.git/tree/ath12k
> * https://github.com/kvalo/ath11k-firmware
> * https://github.com/quic/upstream-wifi-fw
>
> So I find it odd that the new code now seems to depend on it without actually 
> providing these files anywhere.

For example, firmware-2.bin will be used for qrtr id to support multiple
devices on the same host.

> Might be good to have these soon in linux-firmware.git.

Yeah, we will but just slowly. We are currently moving
ath1?k-firmware.git repos to a new location, firmware-N.bin files should
start emerging after that.

-- 
https://patchwork.kernel.org/project/linux-wireless/list/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches