Subject: Re: [PATCH v3 0/2] ath11k: add handler for board-2.bin without
 variant and read SMBIOS
To:     Wen Gong <quic_wgong@quicinc.com>, ath11k@lists.infradead.org
Cc:     linux-wireless@vger.kernel.org
References: <20211125084458.16352-1-quic_wgong@quicinc.com>
From:   Mark Herbert <mark.herbert42@gmail.com>
Message-ID: <e4edd060-8dcb-e1f9-27f4-3ba8c94404ea@gmail.com>
Date:   Thu, 25 Nov 2021 12:52:28 +0300
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101
 Thunderbird/78.14.0
MIME-Version: 1.0
In-Reply-To: <20211125084458.16352-1-quic_wgong@quicinc.com>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 8bit
Content-Language: en-US
Precedence: bulk

Tried on my Dell 9310. It works now.

Gives the warning that SMBIOS recommended firmware not found but 
succesfuly fallback to generic mode and starts.

[  118.422601] ath11k_pci 0000:72:00.0: chip_id 0x0 chip_family 0xb 
board_id 0xff soc_id 0xffffffff
[  118.422616] ath11k_pci 0000:72:00.0: fw_version 0x10110341 
fw_build_timestamp 2021-05-02 15:16 fw_build_id
[  118.422778] ath11k_pci 0000:72:00.0: failed to fetch board data for 
bus=pci,qmi-chip-id=0,qmi-board-id=255,variant=DE_1901 from 
ath11k/QCA6390/hw2.0/board-2.bin

On 25.11.2021 11:44, Wen Gong wrote:
> v3:
>     1. add patch "ath11k: add fallback board name without variant while searching board-2.bin"
>     2. reabsed to latest ath.git master ath-202111221436
>
> v2: rebased to latest ath.git master ath-202111170737
>
> When it does not have matched board name with variant in board-2.bin,
> ath11k will fallback to search without variant.
>
> Also read the variant from SMBIOS.
>
> Wen Gong (2):
>    ath11k: add fallback board name without variant while searching
>      board-2.bin
>    ath11k: add read variant from SMBIOS for download board data
>
>   drivers/net/wireless/ath/ath11k/core.c | 93 ++++++++++++++++++++++++--
>   drivers/net/wireless/ath/ath11k/core.h | 11 +++
>   drivers/net/wireless/ath/ath11k/qmi.c  |  4 ++
>   3 files changed, 104 insertions(+), 4 deletions(-)
>