* [RFC PULL] ath11k: move WCN6750 firmware to the device-specific subdir
@ 2024-10-11 6:16 Dmitry Baryshkov
2024-10-18 22:16 ` Jeff Johnson
0 siblings, 1 reply; 5+ messages in thread
From: Dmitry Baryshkov @ 2024-10-11 6:16 UTC (permalink / raw)
To: Kalle Valo, Jeff Johnson, Bjorn Andersson, Konrad Dybcio,
linux-firmware
Cc: ath11k, linux-arm-msm
The firmware under ath11k/WCN6750 can not be utilized on other devices using
WCN6750 to provide WiFi (e.g. on the Qualcomm RB3 gen2 board). This is an
attempt to propose a solution by moving the file into device-specifig subdir.
Other platforms can place firmware files in other subdirs, depending on the
SoC.
Note, while performing the move I also propose squashing the firmware file.
This is the practice recommended by the msm platform maintainers for the DSP
firmware (not to mention that it also saves us from the extra symlinks). If
Kalle, ath11k maintainer, objects, I will keep the firmware in the split form.
For now this is an RFC in order to gather opinion form respective platform and
driver maintainers.
----------------------------------------------------------------
The following changes since commit d55315295cccf8ce5dc288a74d75cf09b8063cfc:
Merge branch 'superm1/qci-patches' into 'main' (2024-10-10 19:11:13 +0000)
are available in the Git repository at:
https://gitlab.com/lumag/linux-firmware wcn6750-device-specific
for you to fetch changes up to abe8c109a2b4c422b627335d124473734c8790d8:
ath11k: move WCN6750 firmware to the device-specific subdir (2024-10-10 22:26:20 +0300)
----------------------------------------------------------------
Dmitry Baryshkov (1):
ath11k: move WCN6750 firmware to the device-specific subdir
WHENCE | 12 ++----------
ath11k/WCN6750/hw1.0/{wpss.b04 => sc7280/wpss.mbn} | Bin 5819999 -> 7463728 bytes
ath11k/WCN6750/hw1.0/wpss.b00 | Bin 340 -> 0 bytes
ath11k/WCN6750/hw1.0/wpss.b01 | Bin 6848 -> 0 bytes
ath11k/WCN6750/hw1.0/wpss.b02 | Bin 10300 -> 0 bytes
ath11k/WCN6750/hw1.0/wpss.b03 | Bin 4096 -> 0 bytes
ath11k/WCN6750/hw1.0/wpss.b05 | Bin 164332 -> 0 bytes
ath11k/WCN6750/hw1.0/wpss.b06 | Bin 266684 -> 0 bytes
ath11k/WCN6750/hw1.0/wpss.b07 | Bin 1176368 -> 0 bytes
ath11k/WCN6750/hw1.0/wpss.b08 | 0
ath11k/WCN6750/hw1.0/wpss.mdt | Bin 7188 -> 0 bytes
11 files changed, 2 insertions(+), 10 deletions(-)
rename ath11k/WCN6750/hw1.0/{wpss.b04 => sc7280/wpss.mbn} (77%)
delete mode 100644 ath11k/WCN6750/hw1.0/wpss.b00
delete mode 100644 ath11k/WCN6750/hw1.0/wpss.b01
delete mode 100644 ath11k/WCN6750/hw1.0/wpss.b02
delete mode 100644 ath11k/WCN6750/hw1.0/wpss.b03
delete mode 100644 ath11k/WCN6750/hw1.0/wpss.b05
delete mode 100644 ath11k/WCN6750/hw1.0/wpss.b06
delete mode 100644 ath11k/WCN6750/hw1.0/wpss.b07
delete mode 100644 ath11k/WCN6750/hw1.0/wpss.b08
delete mode 100644 ath11k/WCN6750/hw1.0/wpss.mdt
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [RFC PULL] ath11k: move WCN6750 firmware to the device-specific subdir
2024-10-11 6:16 [RFC PULL] ath11k: move WCN6750 firmware to the device-specific subdir Dmitry Baryshkov
@ 2024-10-18 22:16 ` Jeff Johnson
2024-10-21 18:22 ` Kalle Valo
0 siblings, 1 reply; 5+ messages in thread
From: Jeff Johnson @ 2024-10-18 22:16 UTC (permalink / raw)
To: Dmitry Baryshkov, Kalle Valo, Jeff Johnson, Bjorn Andersson,
Konrad Dybcio, linux-firmware
Cc: ath11k, linux-arm-msm
On 10/10/2024 11:16 PM, Dmitry Baryshkov wrote:
> The firmware under ath11k/WCN6750 can not be utilized on other devices using
> WCN6750 to provide WiFi (e.g. on the Qualcomm RB3 gen2 board). This is an
> attempt to propose a solution by moving the file into device-specifig subdir.
> Other platforms can place firmware files in other subdirs, depending on the
> SoC.
>
> Note, while performing the move I also propose squashing the firmware file.
> This is the practice recommended by the msm platform maintainers for the DSP
> firmware (not to mention that it also saves us from the extra symlinks). If
> Kalle, ath11k maintainer, objects, I will keep the firmware in the split form.
>
> For now this is an RFC in order to gather opinion form respective platform and
> driver maintainers.
>
> ----------------------------------------------------------------
>
> The following changes since commit d55315295cccf8ce5dc288a74d75cf09b8063cfc:
>
> Merge branch 'superm1/qci-patches' into 'main' (2024-10-10 19:11:13 +0000)
>
> are available in the Git repository at:
>
> https://gitlab.com/lumag/linux-firmware wcn6750-device-specific
>
> for you to fetch changes up to abe8c109a2b4c422b627335d124473734c8790d8:
>
> ath11k: move WCN6750 firmware to the device-specific subdir (2024-10-10 22:26:20 +0300)
>
> ----------------------------------------------------------------
> Dmitry Baryshkov (1):
> ath11k: move WCN6750 firmware to the device-specific subdir
>
> WHENCE | 12 ++----------
> ath11k/WCN6750/hw1.0/{wpss.b04 => sc7280/wpss.mbn} | Bin 5819999 -> 7463728 bytes
> ath11k/WCN6750/hw1.0/wpss.b00 | Bin 340 -> 0 bytes
> ath11k/WCN6750/hw1.0/wpss.b01 | Bin 6848 -> 0 bytes
> ath11k/WCN6750/hw1.0/wpss.b02 | Bin 10300 -> 0 bytes
> ath11k/WCN6750/hw1.0/wpss.b03 | Bin 4096 -> 0 bytes
> ath11k/WCN6750/hw1.0/wpss.b05 | Bin 164332 -> 0 bytes
> ath11k/WCN6750/hw1.0/wpss.b06 | Bin 266684 -> 0 bytes
> ath11k/WCN6750/hw1.0/wpss.b07 | Bin 1176368 -> 0 bytes
> ath11k/WCN6750/hw1.0/wpss.b08 | 0
> ath11k/WCN6750/hw1.0/wpss.mdt | Bin 7188 -> 0 bytes
> 11 files changed, 2 insertions(+), 10 deletions(-)
> rename ath11k/WCN6750/hw1.0/{wpss.b04 => sc7280/wpss.mbn} (77%)
> delete mode 100644 ath11k/WCN6750/hw1.0/wpss.b00
> delete mode 100644 ath11k/WCN6750/hw1.0/wpss.b01
> delete mode 100644 ath11k/WCN6750/hw1.0/wpss.b02
> delete mode 100644 ath11k/WCN6750/hw1.0/wpss.b03
> delete mode 100644 ath11k/WCN6750/hw1.0/wpss.b05
> delete mode 100644 ath11k/WCN6750/hw1.0/wpss.b06
> delete mode 100644 ath11k/WCN6750/hw1.0/wpss.b07
> delete mode 100644 ath11k/WCN6750/hw1.0/wpss.b08
> delete mode 100644 ath11k/WCN6750/hw1.0/wpss.mdt
I'm waiting for Kalle to respond, but at a minimum I suspect our internal
firmware tooling would be impacted by this change.
Kalle, do you have concerns or suggestions on how to handle this issue?
/jeff
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [RFC PULL] ath11k: move WCN6750 firmware to the device-specific subdir
2024-10-18 22:16 ` Jeff Johnson
@ 2024-10-21 18:22 ` Kalle Valo
2024-10-21 18:34 ` Dmitry Baryshkov
0 siblings, 1 reply; 5+ messages in thread
From: Kalle Valo @ 2024-10-21 18:22 UTC (permalink / raw)
To: Jeff Johnson
Cc: Dmitry Baryshkov, Jeff Johnson, Bjorn Andersson, Konrad Dybcio,
linux-firmware, ath11k, linux-arm-msm
Jeff Johnson <quic_jjohnson@quicinc.com> writes:
> On 10/10/2024 11:16 PM, Dmitry Baryshkov wrote:
>
>> The firmware under ath11k/WCN6750 can not be utilized on other devices using
>> WCN6750 to provide WiFi (e.g. on the Qualcomm RB3 gen2 board). This is an
>> attempt to propose a solution by moving the file into device-specifig subdir.
>> Other platforms can place firmware files in other subdirs, depending on the
>> SoC.
>>
>> Note, while performing the move I also propose squashing the firmware
>> file.
I assume you mean squashing the files to wpss.mbn?
>> This is the practice recommended by the msm platform maintainers for the DSP
>> firmware (not to mention that it also saves us from the extra symlinks). If
>> Kalle, ath11k maintainer, objects, I will keep the firmware in the split form.
>>
>> For now this is an RFC in order to gather opinion form respective platform and
>> driver maintainers.
>>
>> ----------------------------------------------------------------
>>
>> The following changes since commit d55315295cccf8ce5dc288a74d75cf09b8063cfc:
>>
>> Merge branch 'superm1/qci-patches' into 'main' (2024-10-10 19:11:13 +0000)
>>
>> are available in the Git repository at:
>>
>> https://gitlab.com/lumag/linux-firmware wcn6750-device-specific
>>
>> for you to fetch changes up to abe8c109a2b4c422b627335d124473734c8790d8:
>>
>> ath11k: move WCN6750 firmware to the device-specific subdir (2024-10-10 22:26:20 +0300)
>>
>> ----------------------------------------------------------------
>> Dmitry Baryshkov (1):
>> ath11k: move WCN6750 firmware to the device-specific subdir
>>
>> WHENCE | 12 ++----------
>> ath11k/WCN6750/hw1.0/{wpss.b04 => sc7280/wpss.mbn} | Bin 5819999 -> 7463728 bytes
>> ath11k/WCN6750/hw1.0/wpss.b00 | Bin 340 -> 0 bytes
>> ath11k/WCN6750/hw1.0/wpss.b01 | Bin 6848 -> 0 bytes
>> ath11k/WCN6750/hw1.0/wpss.b02 | Bin 10300 -> 0 bytes
>> ath11k/WCN6750/hw1.0/wpss.b03 | Bin 4096 -> 0 bytes
>> ath11k/WCN6750/hw1.0/wpss.b05 | Bin 164332 -> 0 bytes
>> ath11k/WCN6750/hw1.0/wpss.b06 | Bin 266684 -> 0 bytes
>> ath11k/WCN6750/hw1.0/wpss.b07 | Bin 1176368 -> 0 bytes
>> ath11k/WCN6750/hw1.0/wpss.b08 | 0
>> ath11k/WCN6750/hw1.0/wpss.mdt | Bin 7188 -> 0 bytes
>> 11 files changed, 2 insertions(+), 10 deletions(-)
>> rename ath11k/WCN6750/hw1.0/{wpss.b04 => sc7280/wpss.mbn} (77%)
So this follow basically the same directory hierarchy as you Dmitry
implemented for ath10k in commit 5abf259772df ("wifi: ath10k: support
board-specific firmware overrides"), right? Just making sure we are on
the same page. I think ath10k/ath11k/ath12k drivers should follow the
same design as much as possible.
>> delete mode 100644 ath11k/WCN6750/hw1.0/wpss.b00
>> delete mode 100644 ath11k/WCN6750/hw1.0/wpss.b01
>> delete mode 100644 ath11k/WCN6750/hw1.0/wpss.b02
>> delete mode 100644 ath11k/WCN6750/hw1.0/wpss.b03
>> delete mode 100644 ath11k/WCN6750/hw1.0/wpss.b05
>> delete mode 100644 ath11k/WCN6750/hw1.0/wpss.b06
>> delete mode 100644 ath11k/WCN6750/hw1.0/wpss.b07
>> delete mode 100644 ath11k/WCN6750/hw1.0/wpss.b08
>> delete mode 100644 ath11k/WCN6750/hw1.0/wpss.mdt
>
> I'm waiting for Kalle to respond, but at a minimum I suspect our internal
> firmware tooling would be impacted by this change.
Yeah, most likely we need to update our tools. But we would have to do
it anyway so I'm not concerned about that.
> Kalle, do you have concerns or suggestions on how to handle this issue?
Looks good to me.
Dmitry, if you have the time please take a look at this:
https://patchwork.kernel.org/project/linux-wireless/cover/20241001033053.2084360-1-quic_miaoqing@quicinc.com/
Based on a quick look it seems to do things differently and not really a
fan of that.
--
https://patchwork.kernel.org/project/linux-wireless/list/
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [RFC PULL] ath11k: move WCN6750 firmware to the device-specific subdir
2024-10-21 18:22 ` Kalle Valo
@ 2024-10-21 18:34 ` Dmitry Baryshkov
2024-10-28 17:37 ` Kalle Valo
0 siblings, 1 reply; 5+ messages in thread
From: Dmitry Baryshkov @ 2024-10-21 18:34 UTC (permalink / raw)
To: Kalle Valo
Cc: Jeff Johnson, Jeff Johnson, Bjorn Andersson, Konrad Dybcio,
linux-firmware, ath11k, linux-arm-msm
On Mon, 21 Oct 2024 at 21:22, Kalle Valo <kvalo@kernel.org> wrote:
>
> Jeff Johnson <quic_jjohnson@quicinc.com> writes:
>
> > On 10/10/2024 11:16 PM, Dmitry Baryshkov wrote:
> >
> >> The firmware under ath11k/WCN6750 can not be utilized on other devices using
> >> WCN6750 to provide WiFi (e.g. on the Qualcomm RB3 gen2 board). This is an
> >> attempt to propose a solution by moving the file into device-specifig subdir.
> >> Other platforms can place firmware files in other subdirs, depending on the
> >> SoC.
> >>
> >> Note, while performing the move I also propose squashing the firmware
> >> file.
>
> I assume you mean squashing the files to wpss.mbn?
>
> >> This is the practice recommended by the msm platform maintainers for the DSP
> >> firmware (not to mention that it also saves us from the extra symlinks). If
> >> Kalle, ath11k maintainer, objects, I will keep the firmware in the split form.
> >>
> >> For now this is an RFC in order to gather opinion form respective platform and
> >> driver maintainers.
> >>
> >> ----------------------------------------------------------------
> >>
> >> The following changes since commit d55315295cccf8ce5dc288a74d75cf09b8063cfc:
> >>
> >> Merge branch 'superm1/qci-patches' into 'main' (2024-10-10 19:11:13 +0000)
> >>
> >> are available in the Git repository at:
> >>
> >> https://gitlab.com/lumag/linux-firmware wcn6750-device-specific
> >>
> >> for you to fetch changes up to abe8c109a2b4c422b627335d124473734c8790d8:
> >>
> >> ath11k: move WCN6750 firmware to the device-specific subdir (2024-10-10 22:26:20 +0300)
> >>
> >> ----------------------------------------------------------------
> >> Dmitry Baryshkov (1):
> >> ath11k: move WCN6750 firmware to the device-specific subdir
> >>
> >> WHENCE | 12 ++----------
> >> ath11k/WCN6750/hw1.0/{wpss.b04 => sc7280/wpss.mbn} | Bin 5819999 -> 7463728 bytes
> >> ath11k/WCN6750/hw1.0/wpss.b00 | Bin 340 -> 0 bytes
> >> ath11k/WCN6750/hw1.0/wpss.b01 | Bin 6848 -> 0 bytes
> >> ath11k/WCN6750/hw1.0/wpss.b02 | Bin 10300 -> 0 bytes
> >> ath11k/WCN6750/hw1.0/wpss.b03 | Bin 4096 -> 0 bytes
> >> ath11k/WCN6750/hw1.0/wpss.b05 | Bin 164332 -> 0 bytes
> >> ath11k/WCN6750/hw1.0/wpss.b06 | Bin 266684 -> 0 bytes
> >> ath11k/WCN6750/hw1.0/wpss.b07 | Bin 1176368 -> 0 bytes
> >> ath11k/WCN6750/hw1.0/wpss.b08 | 0
> >> ath11k/WCN6750/hw1.0/wpss.mdt | Bin 7188 -> 0 bytes
> >> 11 files changed, 2 insertions(+), 10 deletions(-)
> >> rename ath11k/WCN6750/hw1.0/{wpss.b04 => sc7280/wpss.mbn} (77%)
>
> So this follow basically the same directory hierarchy as you Dmitry
> implemented for ath10k in commit 5abf259772df ("wifi: ath10k: support
> board-specific firmware overrides"), right? Just making sure we are on
> the same page. I think ath10k/ath11k/ath12k drivers should follow the
> same design as much as possible.
Yes, that was my goal. Having subdirs under the main dir.
>
> >> delete mode 100644 ath11k/WCN6750/hw1.0/wpss.b00
> >> delete mode 100644 ath11k/WCN6750/hw1.0/wpss.b01
> >> delete mode 100644 ath11k/WCN6750/hw1.0/wpss.b02
> >> delete mode 100644 ath11k/WCN6750/hw1.0/wpss.b03
> >> delete mode 100644 ath11k/WCN6750/hw1.0/wpss.b05
> >> delete mode 100644 ath11k/WCN6750/hw1.0/wpss.b06
> >> delete mode 100644 ath11k/WCN6750/hw1.0/wpss.b07
> >> delete mode 100644 ath11k/WCN6750/hw1.0/wpss.b08
> >> delete mode 100644 ath11k/WCN6750/hw1.0/wpss.mdt
> >
> > I'm waiting for Kalle to respond, but at a minimum I suspect our internal
> > firmware tooling would be impacted by this change.
>
> Yeah, most likely we need to update our tools. But we would have to do
> it anyway so I'm not concerned about that.
Ok. If there are no concerns from your side, could you possibly
respond with Acked-by? Then we can unblock and merge this.
> > Kalle, do you have concerns or suggestions on how to handle this issue?
>
> Looks good to me.
>
> Dmitry, if you have the time please take a look at this:
>
> https://patchwork.kernel.org/project/linux-wireless/cover/20241001033053.2084360-1-quic_miaoqing@quicinc.com/
>
> Based on a quick look it seems to do things differently and not really a
> fan of that.
Yes, I'd prefer a subdir rather than this implementation. I'll respond
either today or tomorrow.
--
With best wishes
Dmitry
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [RFC PULL] ath11k: move WCN6750 firmware to the device-specific subdir
2024-10-21 18:34 ` Dmitry Baryshkov
@ 2024-10-28 17:37 ` Kalle Valo
0 siblings, 0 replies; 5+ messages in thread
From: Kalle Valo @ 2024-10-28 17:37 UTC (permalink / raw)
To: Dmitry Baryshkov
Cc: Jeff Johnson, Jeff Johnson, Bjorn Andersson, Konrad Dybcio,
linux-firmware, ath11k, linux-arm-msm
Dmitry Baryshkov <dmitry.baryshkov@linaro.org> writes:
> On Mon, 21 Oct 2024 at 21:22, Kalle Valo <kvalo@kernel.org> wrote:
>
>>
>> Jeff Johnson <quic_jjohnson@quicinc.com> writes:
>>
>> >> Dmitry Baryshkov (1):
>> >> ath11k: move WCN6750 firmware to the device-specific subdir
>> >>
>> >> WHENCE | 12 ++----------
>> >> ath11k/WCN6750/hw1.0/{wpss.b04 => sc7280/wpss.mbn} | Bin 5819999 -> 7463728 bytes
>> >> ath11k/WCN6750/hw1.0/wpss.b00 | Bin 340 -> 0 bytes
>> >> ath11k/WCN6750/hw1.0/wpss.b01 | Bin 6848 -> 0 bytes
>> >> ath11k/WCN6750/hw1.0/wpss.b02 | Bin 10300 -> 0 bytes
>> >> ath11k/WCN6750/hw1.0/wpss.b03 | Bin 4096 -> 0 bytes
>> >> ath11k/WCN6750/hw1.0/wpss.b05 | Bin 164332 -> 0 bytes
>> >> ath11k/WCN6750/hw1.0/wpss.b06 | Bin 266684 -> 0 bytes
>> >> ath11k/WCN6750/hw1.0/wpss.b07 | Bin 1176368 -> 0 bytes
>> >> ath11k/WCN6750/hw1.0/wpss.b08 | 0
>> >> ath11k/WCN6750/hw1.0/wpss.mdt | Bin 7188 -> 0 bytes
>> >> 11 files changed, 2 insertions(+), 10 deletions(-)
>> >> rename ath11k/WCN6750/hw1.0/{wpss.b04 => sc7280/wpss.mbn} (77%)
>>
>> So this follow basically the same directory hierarchy as you Dmitry
>> implemented for ath10k in commit 5abf259772df ("wifi: ath10k: support
>> board-specific firmware overrides"), right? Just making sure we are on
>> the same page. I think ath10k/ath11k/ath12k drivers should follow the
>> same design as much as possible.
>
> Yes, that was my goal. Having subdirs under the main dir.
Very good.
>> >> delete mode 100644 ath11k/WCN6750/hw1.0/wpss.b00
>> >> delete mode 100644 ath11k/WCN6750/hw1.0/wpss.b01
>> >> delete mode 100644 ath11k/WCN6750/hw1.0/wpss.b02
>> >> delete mode 100644 ath11k/WCN6750/hw1.0/wpss.b03
>> >> delete mode 100644 ath11k/WCN6750/hw1.0/wpss.b05
>> >> delete mode 100644 ath11k/WCN6750/hw1.0/wpss.b06
>> >> delete mode 100644 ath11k/WCN6750/hw1.0/wpss.b07
>> >> delete mode 100644 ath11k/WCN6750/hw1.0/wpss.b08
>> >> delete mode 100644 ath11k/WCN6750/hw1.0/wpss.mdt
>> >
>> > I'm waiting for Kalle to respond, but at a minimum I suspect our internal
>> > firmware tooling would be impacted by this change.
>>
>> Yeah, most likely we need to update our tools. But we would have to do
>> it anyway so I'm not concerned about that.
>
> Ok. If there are no concerns from your side, could you possibly
> respond with Acked-by? Then we can unblock and merge this.
Sure:
Acked-by: Kalle Valo <kvalo@kernel.org>
>> > Kalle, do you have concerns or suggestions on how to handle this issue?
>>
>> Looks good to me.
>>
>> Dmitry, if you have the time please take a look at this:
>>
>> https://patchwork.kernel.org/project/linux-wireless/cover/20241001033053.2084360-1-quic_miaoqing@quicinc.com/
>>
>> Based on a quick look it seems to do things differently and not really a
>> fan of that.
>
> Yes, I'd prefer a subdir rather than this implementation. I'll respond
> either today or tomorrow.
Thanks a lot, I appreciate you helping us.
--
https://patchwork.kernel.org/project/linux-wireless/list/
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2024-10-28 17:37 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-10-11 6:16 [RFC PULL] ath11k: move WCN6750 firmware to the device-specific subdir Dmitry Baryshkov
2024-10-18 22:16 ` Jeff Johnson
2024-10-21 18:22 ` Kalle Valo
2024-10-21 18:34 ` Dmitry Baryshkov
2024-10-28 17:37 ` Kalle Valo
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox