From:   Kalle Valo <kvalo@kernel.org>
To:     carson.vandegriffe@candelatech.com
Cc:     linux-wireless@vger.kernel.org
Subject: Re: [RFC PATCH] wifi: mt76: mt7916: Support per-radio configuration 'firmware' file.
References: <20230123175500.3375542-1-carson.vandegriffe@candelatech.com>
Date:   Tue, 07 Feb 2023 17:13:43 +0200
In-Reply-To: <20230123175500.3375542-1-carson.vandegriffe@candelatech.com>
        (carson vandegriffe's message of "Mon, 23 Jan 2023 09:55:00 -0800")
Message-ID: <87lel97bko.fsf@kernel.org>
User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/26.1 (gnu/linux)
MIME-Version: 1.0
Content-Type: text/plain
Precedence: bulk

carson.vandegriffe@candelatech.com writes:

> From: Carson Vandegriffe <carson.vandegriffe@candelatech.com>
>
> This lets users specify the upper band that the 7916 radio should use.
> Upon reboot, the 7916 will be using that upper band.
>
> Example config file:
>
> myhost@: cat /usr/lib/firmware/mediatek/fwcfg-mmio-0000\:04\:00.0.txt
>
> high_band=6
>
> Signed-off-by: Carson Vandegriffe <carson.vandegriffe@candelatech.com>

So this is basically an .ini file with settings for the driver? It's a
long standing request feature request how wireless drivers should handle
thatbut there's still no resolution. Having an ASCII parser in the
driver does not sound a good idea.

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

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