All of lore.kernel.org
 help / color / mirror / Atom feed
* [PULL] ath11k firmware 20211223
@ 2021-12-23 14:03 ` Kalle Valo
  0 siblings, 0 replies; 27+ messages in thread
From: Kalle Valo @ 2021-12-23 14:03 UTC (permalink / raw)
  To: linux-firmware; +Cc: linux-wireless, ath11k

Hi linux-firmware maintainers,

here's a pull request to update ath11k firmware images. We have new hardware support WCN6855 and QCN9074, but also updates to existing hardware.

Please let me know if there are any problems.

Kalle

The following changes since commit f682ecb015df087613f91872712eb6c0f6c998a8:

  Merge https://github.com/pkshih/linux-firmware into main (2021-12-16 09:42:51 -0500)

are available in the Git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/kvalo/linux-firmware.git ath10k-20211223

for you to fetch changes up to 0f161f385f3854086b4fdae2dae786ab6bd3efa5:

  ath11k: WCN6855 hw2.0: add to WLAN.HSP.1.1-03003-QCAHSPSWPL_V1_V2_SILICONZ_LITE-2 (2021-12-23 12:40:55 +0200)

----------------------------------------------------------------
Kalle Valo (5):
      ath11k: IPQ6018 hw1.0: update to WLAN.HK.2.5.0.1-01100-QCAHKSWPL_SILICONZ-1
      ath11k: IPQ8074 hw2.0: update to WLAN.HK.2.5.0.1-01100-QCAHKSWPL_SILICONZ-1
      ath11k: QCN9074 hw1.0: add to WLAN.HK.2.5.0.1-01100-QCAHKSWPL_SILICONZ-1
      ath11k: WCN6855 hw2.0: add board-2.bin and regdb.bin
      ath11k: WCN6855 hw2.0: add to WLAN.HSP.1.1-03003-QCAHSPSWPL_V1_V2_SILICONZ_LITE-2

 WHENCE                           |   14 +-
 ath11k/IPQ6018/hw1.0/Notice.txt  |   48 +-
 ath11k/IPQ6018/hw1.0/m3_fw.b01   |  Bin 6712 -> 6712 bytes
 ath11k/IPQ6018/hw1.0/m3_fw.b02   |  Bin 294912 -> 294912 bytes
 ath11k/IPQ6018/hw1.0/m3_fw.mdt   |  Bin 6860 -> 6860 bytes
 ath11k/IPQ6018/hw1.0/q6_fw.b00   |  Bin 340 -> 340 bytes
 ath11k/IPQ6018/hw1.0/q6_fw.b01   |  Bin 7000 -> 7000 bytes
 ath11k/IPQ6018/hw1.0/q6_fw.b02   |  Bin 4696 -> 4696 bytes
 ath11k/IPQ6018/hw1.0/q6_fw.b03   |  Bin 2357712 -> 2557456 bytes
 ath11k/IPQ6018/hw1.0/q6_fw.b04   |  Bin 370560 -> 412192 bytes
 ath11k/IPQ6018/hw1.0/q6_fw.b05   |  Bin 175588 -> 197988 bytes
 ath11k/IPQ6018/hw1.0/q6_fw.b07   |  Bin 8936 -> 9408 bytes
 ath11k/IPQ6018/hw1.0/q6_fw.b08   |  Bin 409914 -> 452780 bytes
 ath11k/IPQ6018/hw1.0/q6_fw.mdt   |  Bin 7340 -> 7340 bytes
 ath11k/IPQ8074/hw2.0/Notice.txt  |   48 +-
 ath11k/IPQ8074/hw2.0/m3_fw.b01   |  Bin 136 -> 136 bytes
 ath11k/IPQ8074/hw2.0/m3_fw.b02   |  Bin 327680 -> 327680 bytes
 ath11k/IPQ8074/hw2.0/m3_fw.mdt   |  Bin 284 -> 284 bytes
 ath11k/IPQ8074/hw2.0/q6_fw.b00   |  Bin 340 -> 340 bytes
 ath11k/IPQ8074/hw2.0/q6_fw.b01   |  Bin 328 -> 328 bytes
 ath11k/IPQ8074/hw2.0/q6_fw.b02   |  Bin 4696 -> 4696 bytes
 ath11k/IPQ8074/hw2.0/q6_fw.b03   |  Bin 2589264 -> 2934416 bytes
 ath11k/IPQ8074/hw2.0/q6_fw.b04   |  Bin 985504 -> 1009760 bytes
 ath11k/IPQ8074/hw2.0/q6_fw.b05   |  Bin 387076 -> 243588 bytes
 ath11k/IPQ8074/hw2.0/q6_fw.b07   |  Bin 7048 -> 9816 bytes
 ath11k/IPQ8074/hw2.0/q6_fw.b08   |  Bin 424524 -> 485768 bytes
 ath11k/IPQ8074/hw2.0/q6_fw.mdt   |  Bin 668 -> 668 bytes
 ath11k/QCN9074/hw1.0/Notice.txt  |  802 ++++++++++++++++++++
 ath11k/QCN9074/hw1.0/amss.bin    |  Bin 0 -> 11897676 bytes
 ath11k/QCN9074/hw1.0/m3.bin      |  Bin 0 -> 340108 bytes
 ath11k/WCN6855/hw2.0/Notice.txt  | 1504 ++++++++++++++++++++++++++++++++++++++
 ath11k/WCN6855/hw2.0/amss.bin    |  Bin 0 -> 4984832 bytes
 ath11k/WCN6855/hw2.0/board-2.bin |  Bin 0 -> 721328 bytes
 ath11k/WCN6855/hw2.0/m3.bin      |  Bin 0 -> 266684 bytes
 ath11k/WCN6855/hw2.0/regdb.bin   |  Bin 0 -> 24310 bytes
 35 files changed, 2382 insertions(+), 34 deletions(-)
 create mode 100644 ath11k/QCN9074/hw1.0/Notice.txt
 create mode 100644 ath11k/QCN9074/hw1.0/amss.bin
 create mode 100644 ath11k/QCN9074/hw1.0/m3.bin
 create mode 100644 ath11k/WCN6855/hw2.0/Notice.txt
 create mode 100644 ath11k/WCN6855/hw2.0/amss.bin
 create mode 100644 ath11k/WCN6855/hw2.0/board-2.bin
 create mode 100644 ath11k/WCN6855/hw2.0/m3.bin
 create mode 100644 ath11k/WCN6855/hw2.0/regdb.bin

^ permalink raw reply	[flat|nested] 27+ messages in thread

* [PULL] ath11k firmware 20211223
@ 2021-12-23 14:03 ` Kalle Valo
  0 siblings, 0 replies; 27+ messages in thread
From: Kalle Valo @ 2021-12-23 14:03 UTC (permalink / raw)
  To: linux-firmware; +Cc: linux-wireless, ath11k

Hi linux-firmware maintainers,

here's a pull request to update ath11k firmware images. We have new hardware support WCN6855 and QCN9074, but also updates to existing hardware.

Please let me know if there are any problems.

Kalle

The following changes since commit f682ecb015df087613f91872712eb6c0f6c998a8:

  Merge https://github.com/pkshih/linux-firmware into main (2021-12-16 09:42:51 -0500)

are available in the Git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/kvalo/linux-firmware.git ath10k-20211223

for you to fetch changes up to 0f161f385f3854086b4fdae2dae786ab6bd3efa5:

  ath11k: WCN6855 hw2.0: add to WLAN.HSP.1.1-03003-QCAHSPSWPL_V1_V2_SILICONZ_LITE-2 (2021-12-23 12:40:55 +0200)

----------------------------------------------------------------
Kalle Valo (5):
      ath11k: IPQ6018 hw1.0: update to WLAN.HK.2.5.0.1-01100-QCAHKSWPL_SILICONZ-1
      ath11k: IPQ8074 hw2.0: update to WLAN.HK.2.5.0.1-01100-QCAHKSWPL_SILICONZ-1
      ath11k: QCN9074 hw1.0: add to WLAN.HK.2.5.0.1-01100-QCAHKSWPL_SILICONZ-1
      ath11k: WCN6855 hw2.0: add board-2.bin and regdb.bin
      ath11k: WCN6855 hw2.0: add to WLAN.HSP.1.1-03003-QCAHSPSWPL_V1_V2_SILICONZ_LITE-2

 WHENCE                           |   14 +-
 ath11k/IPQ6018/hw1.0/Notice.txt  |   48 +-
 ath11k/IPQ6018/hw1.0/m3_fw.b01   |  Bin 6712 -> 6712 bytes
 ath11k/IPQ6018/hw1.0/m3_fw.b02   |  Bin 294912 -> 294912 bytes
 ath11k/IPQ6018/hw1.0/m3_fw.mdt   |  Bin 6860 -> 6860 bytes
 ath11k/IPQ6018/hw1.0/q6_fw.b00   |  Bin 340 -> 340 bytes
 ath11k/IPQ6018/hw1.0/q6_fw.b01   |  Bin 7000 -> 7000 bytes
 ath11k/IPQ6018/hw1.0/q6_fw.b02   |  Bin 4696 -> 4696 bytes
 ath11k/IPQ6018/hw1.0/q6_fw.b03   |  Bin 2357712 -> 2557456 bytes
 ath11k/IPQ6018/hw1.0/q6_fw.b04   |  Bin 370560 -> 412192 bytes
 ath11k/IPQ6018/hw1.0/q6_fw.b05   |  Bin 175588 -> 197988 bytes
 ath11k/IPQ6018/hw1.0/q6_fw.b07   |  Bin 8936 -> 9408 bytes
 ath11k/IPQ6018/hw1.0/q6_fw.b08   |  Bin 409914 -> 452780 bytes
 ath11k/IPQ6018/hw1.0/q6_fw.mdt   |  Bin 7340 -> 7340 bytes
 ath11k/IPQ8074/hw2.0/Notice.txt  |   48 +-
 ath11k/IPQ8074/hw2.0/m3_fw.b01   |  Bin 136 -> 136 bytes
 ath11k/IPQ8074/hw2.0/m3_fw.b02   |  Bin 327680 -> 327680 bytes
 ath11k/IPQ8074/hw2.0/m3_fw.mdt   |  Bin 284 -> 284 bytes
 ath11k/IPQ8074/hw2.0/q6_fw.b00   |  Bin 340 -> 340 bytes
 ath11k/IPQ8074/hw2.0/q6_fw.b01   |  Bin 328 -> 328 bytes
 ath11k/IPQ8074/hw2.0/q6_fw.b02   |  Bin 4696 -> 4696 bytes
 ath11k/IPQ8074/hw2.0/q6_fw.b03   |  Bin 2589264 -> 2934416 bytes
 ath11k/IPQ8074/hw2.0/q6_fw.b04   |  Bin 985504 -> 1009760 bytes
 ath11k/IPQ8074/hw2.0/q6_fw.b05   |  Bin 387076 -> 243588 bytes
 ath11k/IPQ8074/hw2.0/q6_fw.b07   |  Bin 7048 -> 9816 bytes
 ath11k/IPQ8074/hw2.0/q6_fw.b08   |  Bin 424524 -> 485768 bytes
 ath11k/IPQ8074/hw2.0/q6_fw.mdt   |  Bin 668 -> 668 bytes
 ath11k/QCN9074/hw1.0/Notice.txt  |  802 ++++++++++++++++++++
 ath11k/QCN9074/hw1.0/amss.bin    |  Bin 0 -> 11897676 bytes
 ath11k/QCN9074/hw1.0/m3.bin      |  Bin 0 -> 340108 bytes
 ath11k/WCN6855/hw2.0/Notice.txt  | 1504 ++++++++++++++++++++++++++++++++++++++
 ath11k/WCN6855/hw2.0/amss.bin    |  Bin 0 -> 4984832 bytes
 ath11k/WCN6855/hw2.0/board-2.bin |  Bin 0 -> 721328 bytes
 ath11k/WCN6855/hw2.0/m3.bin      |  Bin 0 -> 266684 bytes
 ath11k/WCN6855/hw2.0/regdb.bin   |  Bin 0 -> 24310 bytes
 35 files changed, 2382 insertions(+), 34 deletions(-)
 create mode 100644 ath11k/QCN9074/hw1.0/Notice.txt
 create mode 100644 ath11k/QCN9074/hw1.0/amss.bin
 create mode 100644 ath11k/QCN9074/hw1.0/m3.bin
 create mode 100644 ath11k/WCN6855/hw2.0/Notice.txt
 create mode 100644 ath11k/WCN6855/hw2.0/amss.bin
 create mode 100644 ath11k/WCN6855/hw2.0/board-2.bin
 create mode 100644 ath11k/WCN6855/hw2.0/m3.bin
 create mode 100644 ath11k/WCN6855/hw2.0/regdb.bin

-- 
ath11k mailing list
ath11k@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/ath11k

^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [PULL] ath11k firmware 20211223
  2021-12-23 14:03 ` Kalle Valo
  (?)
@ 2021-12-23 17:24 ` Mark Herbert
  -1 siblings, 0 replies; 27+ messages in thread
From: Mark Herbert @ 2021-12-23 17:24 UTC (permalink / raw)
  To: Kalle Valo; +Cc: linux-firmware, linux-wireless, ath11k

[-- Attachment #1: Type: text/plain, Size: 4469 bytes --]

Will there be a new firmware fo 6390? It is the oldest one in the
repository and buggy as hell. Hibernation is impossible uless the newer
firmware hacked from Windows drivers , and firmware is being updated for
all ath11k chips except this one. 14 month - still same buggy blob...


чт, 23 дек. 2021 г., 17:03 Kalle Valo <kvalo@qca.qualcomm.com>:

> Hi linux-firmware maintainers,
>
> here's a pull request to update ath11k firmware images. We have new
> hardware support WCN6855 and QCN9074, but also updates to existing hardware.
>
> Please let me know if there are any problems.
>
> Kalle
>
> The following changes since commit
> f682ecb015df087613f91872712eb6c0f6c998a8:
>
>   Merge https://github.com/pkshih/linux-firmware into main (2021-12-16
> 09:42:51 -0500)
>
> are available in the Git repository at:
>
>   git://git.kernel.org/pub/scm/linux/kernel/git/kvalo/linux-firmware.git
> ath10k-20211223
>
> for you to fetch changes up to 0f161f385f3854086b4fdae2dae786ab6bd3efa5:
>
>   ath11k: WCN6855 hw2.0: add to
> WLAN.HSP.1.1-03003-QCAHSPSWPL_V1_V2_SILICONZ_LITE-2 (2021-12-23 12:40:55
> +0200)
>
> ----------------------------------------------------------------
> Kalle Valo (5):
>       ath11k: IPQ6018 hw1.0: update to
> WLAN.HK.2.5.0.1-01100-QCAHKSWPL_SILICONZ-1
>       ath11k: IPQ8074 hw2.0: update to
> WLAN.HK.2.5.0.1-01100-QCAHKSWPL_SILICONZ-1
>       ath11k: QCN9074 hw1.0: add to
> WLAN.HK.2.5.0.1-01100-QCAHKSWPL_SILICONZ-1
>       ath11k: WCN6855 hw2.0: add board-2.bin and regdb.bin
>       ath11k: WCN6855 hw2.0: add to
> WLAN.HSP.1.1-03003-QCAHSPSWPL_V1_V2_SILICONZ_LITE-2
>
>  WHENCE                           |   14 +-
>  ath11k/IPQ6018/hw1.0/Notice.txt  |   48 +-
>  ath11k/IPQ6018/hw1.0/m3_fw.b01   |  Bin 6712 -> 6712 bytes
>  ath11k/IPQ6018/hw1.0/m3_fw.b02   |  Bin 294912 -> 294912 bytes
>  ath11k/IPQ6018/hw1.0/m3_fw.mdt   |  Bin 6860 -> 6860 bytes
>  ath11k/IPQ6018/hw1.0/q6_fw.b00   |  Bin 340 -> 340 bytes
>  ath11k/IPQ6018/hw1.0/q6_fw.b01   |  Bin 7000 -> 7000 bytes
>  ath11k/IPQ6018/hw1.0/q6_fw.b02   |  Bin 4696 -> 4696 bytes
>  ath11k/IPQ6018/hw1.0/q6_fw.b03   |  Bin 2357712 -> 2557456 bytes
>  ath11k/IPQ6018/hw1.0/q6_fw.b04   |  Bin 370560 -> 412192 bytes
>  ath11k/IPQ6018/hw1.0/q6_fw.b05   |  Bin 175588 -> 197988 bytes
>  ath11k/IPQ6018/hw1.0/q6_fw.b07   |  Bin 8936 -> 9408 bytes
>  ath11k/IPQ6018/hw1.0/q6_fw.b08   |  Bin 409914 -> 452780 bytes
>  ath11k/IPQ6018/hw1.0/q6_fw.mdt   |  Bin 7340 -> 7340 bytes
>  ath11k/IPQ8074/hw2.0/Notice.txt  |   48 +-
>  ath11k/IPQ8074/hw2.0/m3_fw.b01   |  Bin 136 -> 136 bytes
>  ath11k/IPQ8074/hw2.0/m3_fw.b02   |  Bin 327680 -> 327680 bytes
>  ath11k/IPQ8074/hw2.0/m3_fw.mdt   |  Bin 284 -> 284 bytes
>  ath11k/IPQ8074/hw2.0/q6_fw.b00   |  Bin 340 -> 340 bytes
>  ath11k/IPQ8074/hw2.0/q6_fw.b01   |  Bin 328 -> 328 bytes
>  ath11k/IPQ8074/hw2.0/q6_fw.b02   |  Bin 4696 -> 4696 bytes
>  ath11k/IPQ8074/hw2.0/q6_fw.b03   |  Bin 2589264 -> 2934416 bytes
>  ath11k/IPQ8074/hw2.0/q6_fw.b04   |  Bin 985504 -> 1009760 bytes
>  ath11k/IPQ8074/hw2.0/q6_fw.b05   |  Bin 387076 -> 243588 bytes
>  ath11k/IPQ8074/hw2.0/q6_fw.b07   |  Bin 7048 -> 9816 bytes
>  ath11k/IPQ8074/hw2.0/q6_fw.b08   |  Bin 424524 -> 485768 bytes
>  ath11k/IPQ8074/hw2.0/q6_fw.mdt   |  Bin 668 -> 668 bytes
>  ath11k/QCN9074/hw1.0/Notice.txt  |  802 ++++++++++++++++++++
>  ath11k/QCN9074/hw1.0/amss.bin    |  Bin 0 -> 11897676 bytes
>  ath11k/QCN9074/hw1.0/m3.bin      |  Bin 0 -> 340108 bytes
>  ath11k/WCN6855/hw2.0/Notice.txt  | 1504
> ++++++++++++++++++++++++++++++++++++++
>  ath11k/WCN6855/hw2.0/amss.bin    |  Bin 0 -> 4984832 bytes
>  ath11k/WCN6855/hw2.0/board-2.bin |  Bin 0 -> 721328 bytes
>  ath11k/WCN6855/hw2.0/m3.bin      |  Bin 0 -> 266684 bytes
>  ath11k/WCN6855/hw2.0/regdb.bin   |  Bin 0 -> 24310 bytes
>  35 files changed, 2382 insertions(+), 34 deletions(-)
>  create mode 100644 ath11k/QCN9074/hw1.0/Notice.txt
>  create mode 100644 ath11k/QCN9074/hw1.0/amss.bin
>  create mode 100644 ath11k/QCN9074/hw1.0/m3.bin
>  create mode 100644 ath11k/WCN6855/hw2.0/Notice.txt
>  create mode 100644 ath11k/WCN6855/hw2.0/amss.bin
>  create mode 100644 ath11k/WCN6855/hw2.0/board-2.bin
>  create mode 100644 ath11k/WCN6855/hw2.0/m3.bin
>  create mode 100644 ath11k/WCN6855/hw2.0/regdb.bin
>
> --
> ath11k mailing list
> ath11k@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/ath11k
>

[-- Attachment #2: Type: text/html, Size: 5571 bytes --]

^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [PULL] ath11k firmware 20211223
  2021-12-23 14:03 ` Kalle Valo
@ 2021-12-29 13:41   ` Josh Boyer
  -1 siblings, 0 replies; 27+ messages in thread
From: Josh Boyer @ 2021-12-29 13:41 UTC (permalink / raw)
  To: Kalle Valo; +Cc: linux-firmware, linux-wireless, ath11k

On Thu, Dec 23, 2021 at 9:03 AM Kalle Valo <kvalo@qca.qualcomm.com> wrote:
>
> Hi linux-firmware maintainers,
>
> here's a pull request to update ath11k firmware images. We have new hardware support WCN6855 and QCN9074, but also updates to existing hardware.
>
> Please let me know if there are any problems.
>
> Kalle
>
> The following changes since commit f682ecb015df087613f91872712eb6c0f6c998a8:
>
>   Merge https://github.com/pkshih/linux-firmware into main (2021-12-16 09:42:51 -0500)
>
> are available in the Git repository at:
>
>   git://git.kernel.org/pub/scm/linux/kernel/git/kvalo/linux-firmware.git ath10k-20211223
>
> for you to fetch changes up to 0f161f385f3854086b4fdae2dae786ab6bd3efa5:
>
>   ath11k: WCN6855 hw2.0: add to WLAN.HSP.1.1-03003-QCAHSPSWPL_V1_V2_SILICONZ_LITE-2 (2021-12-23 12:40:55 +0200)
>
> ----------------------------------------------------------------
> Kalle Valo (5):
>       ath11k: IPQ6018 hw1.0: update to WLAN.HK.2.5.0.1-01100-QCAHKSWPL_SILICONZ-1
>       ath11k: IPQ8074 hw2.0: update to WLAN.HK.2.5.0.1-01100-QCAHKSWPL_SILICONZ-1
>       ath11k: QCN9074 hw1.0: add to WLAN.HK.2.5.0.1-01100-QCAHKSWPL_SILICONZ-1
>       ath11k: WCN6855 hw2.0: add board-2.bin and regdb.bin
>       ath11k: WCN6855 hw2.0: add to WLAN.HSP.1.1-03003-QCAHSPSWPL_V1_V2_SILICONZ_LITE-2
>
>  WHENCE                           |   14 +-
>  ath11k/IPQ6018/hw1.0/Notice.txt  |   48 +-

These Notice files seem problematic.  They're clearly well intended,
but they have language that alludes to an agreement with Atheros and a
confidential nature to some of the files.  Specifically:

"...your use of these software
components together with the Qualcomm Atheros software (Qualcomm
Atheros software hereinafter referred to as “Software”) is
subject to the terms of your agreement from Qualcomm Atheros."

Nobody has an opportunity to agree to anything with Atheros when they
consume the files from linux-firmware.  There's also no explicit grant
of redistribution for any of these files.

Also:

"/*
 * Copyright (c) 2011 Qualcomm Technologies, Inc.
 * 2011 Qualcomm Atheros, Inc.
 * All Rights Reserved.
 * Confidential and Proprietary - Qualcomm Technologies, Inc.
 * Notifications and licenses are retained for attribution purposes only.
 */
"

Several of those are followed by "For this file....  has selected the
BSD license"  but it's unclear what file that's referring to.

I'm not comfortable pulling this with the licenses in an ambiguous
state.  Can you work with your team to clarify the text?

josh

>  ath11k/IPQ6018/hw1.0/m3_fw.b01   |  Bin 6712 -> 6712 bytes
>  ath11k/IPQ6018/hw1.0/m3_fw.b02   |  Bin 294912 -> 294912 bytes
>  ath11k/IPQ6018/hw1.0/m3_fw.mdt   |  Bin 6860 -> 6860 bytes
>  ath11k/IPQ6018/hw1.0/q6_fw.b00   |  Bin 340 -> 340 bytes
>  ath11k/IPQ6018/hw1.0/q6_fw.b01   |  Bin 7000 -> 7000 bytes
>  ath11k/IPQ6018/hw1.0/q6_fw.b02   |  Bin 4696 -> 4696 bytes
>  ath11k/IPQ6018/hw1.0/q6_fw.b03   |  Bin 2357712 -> 2557456 bytes
>  ath11k/IPQ6018/hw1.0/q6_fw.b04   |  Bin 370560 -> 412192 bytes
>  ath11k/IPQ6018/hw1.0/q6_fw.b05   |  Bin 175588 -> 197988 bytes
>  ath11k/IPQ6018/hw1.0/q6_fw.b07   |  Bin 8936 -> 9408 bytes
>  ath11k/IPQ6018/hw1.0/q6_fw.b08   |  Bin 409914 -> 452780 bytes
>  ath11k/IPQ6018/hw1.0/q6_fw.mdt   |  Bin 7340 -> 7340 bytes
>  ath11k/IPQ8074/hw2.0/Notice.txt  |   48 +-
>  ath11k/IPQ8074/hw2.0/m3_fw.b01   |  Bin 136 -> 136 bytes
>  ath11k/IPQ8074/hw2.0/m3_fw.b02   |  Bin 327680 -> 327680 bytes
>  ath11k/IPQ8074/hw2.0/m3_fw.mdt   |  Bin 284 -> 284 bytes
>  ath11k/IPQ8074/hw2.0/q6_fw.b00   |  Bin 340 -> 340 bytes
>  ath11k/IPQ8074/hw2.0/q6_fw.b01   |  Bin 328 -> 328 bytes
>  ath11k/IPQ8074/hw2.0/q6_fw.b02   |  Bin 4696 -> 4696 bytes
>  ath11k/IPQ8074/hw2.0/q6_fw.b03   |  Bin 2589264 -> 2934416 bytes
>  ath11k/IPQ8074/hw2.0/q6_fw.b04   |  Bin 985504 -> 1009760 bytes
>  ath11k/IPQ8074/hw2.0/q6_fw.b05   |  Bin 387076 -> 243588 bytes
>  ath11k/IPQ8074/hw2.0/q6_fw.b07   |  Bin 7048 -> 9816 bytes
>  ath11k/IPQ8074/hw2.0/q6_fw.b08   |  Bin 424524 -> 485768 bytes
>  ath11k/IPQ8074/hw2.0/q6_fw.mdt   |  Bin 668 -> 668 bytes
>  ath11k/QCN9074/hw1.0/Notice.txt  |  802 ++++++++++++++++++++
>  ath11k/QCN9074/hw1.0/amss.bin    |  Bin 0 -> 11897676 bytes
>  ath11k/QCN9074/hw1.0/m3.bin      |  Bin 0 -> 340108 bytes
>  ath11k/WCN6855/hw2.0/Notice.txt  | 1504 ++++++++++++++++++++++++++++++++++++++
>  ath11k/WCN6855/hw2.0/amss.bin    |  Bin 0 -> 4984832 bytes
>  ath11k/WCN6855/hw2.0/board-2.bin |  Bin 0 -> 721328 bytes
>  ath11k/WCN6855/hw2.0/m3.bin      |  Bin 0 -> 266684 bytes
>  ath11k/WCN6855/hw2.0/regdb.bin   |  Bin 0 -> 24310 bytes
>  35 files changed, 2382 insertions(+), 34 deletions(-)
>  create mode 100644 ath11k/QCN9074/hw1.0/Notice.txt
>  create mode 100644 ath11k/QCN9074/hw1.0/amss.bin
>  create mode 100644 ath11k/QCN9074/hw1.0/m3.bin
>  create mode 100644 ath11k/WCN6855/hw2.0/Notice.txt
>  create mode 100644 ath11k/WCN6855/hw2.0/amss.bin
>  create mode 100644 ath11k/WCN6855/hw2.0/board-2.bin
>  create mode 100644 ath11k/WCN6855/hw2.0/m3.bin
>  create mode 100644 ath11k/WCN6855/hw2.0/regdb.bin

^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [PULL] ath11k firmware 20211223
@ 2021-12-29 13:41   ` Josh Boyer
  0 siblings, 0 replies; 27+ messages in thread
From: Josh Boyer @ 2021-12-29 13:41 UTC (permalink / raw)
  To: Kalle Valo; +Cc: linux-firmware, linux-wireless, ath11k

On Thu, Dec 23, 2021 at 9:03 AM Kalle Valo <kvalo@qca.qualcomm.com> wrote:
>
> Hi linux-firmware maintainers,
>
> here's a pull request to update ath11k firmware images. We have new hardware support WCN6855 and QCN9074, but also updates to existing hardware.
>
> Please let me know if there are any problems.
>
> Kalle
>
> The following changes since commit f682ecb015df087613f91872712eb6c0f6c998a8:
>
>   Merge https://github.com/pkshih/linux-firmware into main (2021-12-16 09:42:51 -0500)
>
> are available in the Git repository at:
>
>   git://git.kernel.org/pub/scm/linux/kernel/git/kvalo/linux-firmware.git ath10k-20211223
>
> for you to fetch changes up to 0f161f385f3854086b4fdae2dae786ab6bd3efa5:
>
>   ath11k: WCN6855 hw2.0: add to WLAN.HSP.1.1-03003-QCAHSPSWPL_V1_V2_SILICONZ_LITE-2 (2021-12-23 12:40:55 +0200)
>
> ----------------------------------------------------------------
> Kalle Valo (5):
>       ath11k: IPQ6018 hw1.0: update to WLAN.HK.2.5.0.1-01100-QCAHKSWPL_SILICONZ-1
>       ath11k: IPQ8074 hw2.0: update to WLAN.HK.2.5.0.1-01100-QCAHKSWPL_SILICONZ-1
>       ath11k: QCN9074 hw1.0: add to WLAN.HK.2.5.0.1-01100-QCAHKSWPL_SILICONZ-1
>       ath11k: WCN6855 hw2.0: add board-2.bin and regdb.bin
>       ath11k: WCN6855 hw2.0: add to WLAN.HSP.1.1-03003-QCAHSPSWPL_V1_V2_SILICONZ_LITE-2
>
>  WHENCE                           |   14 +-
>  ath11k/IPQ6018/hw1.0/Notice.txt  |   48 +-

These Notice files seem problematic.  They're clearly well intended,
but they have language that alludes to an agreement with Atheros and a
confidential nature to some of the files.  Specifically:

"...your use of these software
components together with the Qualcomm Atheros software (Qualcomm
Atheros software hereinafter referred to as “Software”) is
subject to the terms of your agreement from Qualcomm Atheros."

Nobody has an opportunity to agree to anything with Atheros when they
consume the files from linux-firmware.  There's also no explicit grant
of redistribution for any of these files.

Also:

"/*
 * Copyright (c) 2011 Qualcomm Technologies, Inc.
 * 2011 Qualcomm Atheros, Inc.
 * All Rights Reserved.
 * Confidential and Proprietary - Qualcomm Technologies, Inc.
 * Notifications and licenses are retained for attribution purposes only.
 */
"

Several of those are followed by "For this file....  has selected the
BSD license"  but it's unclear what file that's referring to.

I'm not comfortable pulling this with the licenses in an ambiguous
state.  Can you work with your team to clarify the text?

josh

>  ath11k/IPQ6018/hw1.0/m3_fw.b01   |  Bin 6712 -> 6712 bytes
>  ath11k/IPQ6018/hw1.0/m3_fw.b02   |  Bin 294912 -> 294912 bytes
>  ath11k/IPQ6018/hw1.0/m3_fw.mdt   |  Bin 6860 -> 6860 bytes
>  ath11k/IPQ6018/hw1.0/q6_fw.b00   |  Bin 340 -> 340 bytes
>  ath11k/IPQ6018/hw1.0/q6_fw.b01   |  Bin 7000 -> 7000 bytes
>  ath11k/IPQ6018/hw1.0/q6_fw.b02   |  Bin 4696 -> 4696 bytes
>  ath11k/IPQ6018/hw1.0/q6_fw.b03   |  Bin 2357712 -> 2557456 bytes
>  ath11k/IPQ6018/hw1.0/q6_fw.b04   |  Bin 370560 -> 412192 bytes
>  ath11k/IPQ6018/hw1.0/q6_fw.b05   |  Bin 175588 -> 197988 bytes
>  ath11k/IPQ6018/hw1.0/q6_fw.b07   |  Bin 8936 -> 9408 bytes
>  ath11k/IPQ6018/hw1.0/q6_fw.b08   |  Bin 409914 -> 452780 bytes
>  ath11k/IPQ6018/hw1.0/q6_fw.mdt   |  Bin 7340 -> 7340 bytes
>  ath11k/IPQ8074/hw2.0/Notice.txt  |   48 +-
>  ath11k/IPQ8074/hw2.0/m3_fw.b01   |  Bin 136 -> 136 bytes
>  ath11k/IPQ8074/hw2.0/m3_fw.b02   |  Bin 327680 -> 327680 bytes
>  ath11k/IPQ8074/hw2.0/m3_fw.mdt   |  Bin 284 -> 284 bytes
>  ath11k/IPQ8074/hw2.0/q6_fw.b00   |  Bin 340 -> 340 bytes
>  ath11k/IPQ8074/hw2.0/q6_fw.b01   |  Bin 328 -> 328 bytes
>  ath11k/IPQ8074/hw2.0/q6_fw.b02   |  Bin 4696 -> 4696 bytes
>  ath11k/IPQ8074/hw2.0/q6_fw.b03   |  Bin 2589264 -> 2934416 bytes
>  ath11k/IPQ8074/hw2.0/q6_fw.b04   |  Bin 985504 -> 1009760 bytes
>  ath11k/IPQ8074/hw2.0/q6_fw.b05   |  Bin 387076 -> 243588 bytes
>  ath11k/IPQ8074/hw2.0/q6_fw.b07   |  Bin 7048 -> 9816 bytes
>  ath11k/IPQ8074/hw2.0/q6_fw.b08   |  Bin 424524 -> 485768 bytes
>  ath11k/IPQ8074/hw2.0/q6_fw.mdt   |  Bin 668 -> 668 bytes
>  ath11k/QCN9074/hw1.0/Notice.txt  |  802 ++++++++++++++++++++
>  ath11k/QCN9074/hw1.0/amss.bin    |  Bin 0 -> 11897676 bytes
>  ath11k/QCN9074/hw1.0/m3.bin      |  Bin 0 -> 340108 bytes
>  ath11k/WCN6855/hw2.0/Notice.txt  | 1504 ++++++++++++++++++++++++++++++++++++++
>  ath11k/WCN6855/hw2.0/amss.bin    |  Bin 0 -> 4984832 bytes
>  ath11k/WCN6855/hw2.0/board-2.bin |  Bin 0 -> 721328 bytes
>  ath11k/WCN6855/hw2.0/m3.bin      |  Bin 0 -> 266684 bytes
>  ath11k/WCN6855/hw2.0/regdb.bin   |  Bin 0 -> 24310 bytes
>  35 files changed, 2382 insertions(+), 34 deletions(-)
>  create mode 100644 ath11k/QCN9074/hw1.0/Notice.txt
>  create mode 100644 ath11k/QCN9074/hw1.0/amss.bin
>  create mode 100644 ath11k/QCN9074/hw1.0/m3.bin
>  create mode 100644 ath11k/WCN6855/hw2.0/Notice.txt
>  create mode 100644 ath11k/WCN6855/hw2.0/amss.bin
>  create mode 100644 ath11k/WCN6855/hw2.0/board-2.bin
>  create mode 100644 ath11k/WCN6855/hw2.0/m3.bin
>  create mode 100644 ath11k/WCN6855/hw2.0/regdb.bin

-- 
ath11k mailing list
ath11k@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/ath11k

^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [PULL] ath11k firmware 20211223
  2021-12-29 13:41   ` Josh Boyer
@ 2022-01-10 15:19     ` Kalle Valo
  -1 siblings, 0 replies; 27+ messages in thread
From: Kalle Valo @ 2022-01-10 15:19 UTC (permalink / raw)
  To: Josh Boyer; +Cc: linux-firmware, linux-wireless, ath11k

Josh Boyer <jwboyer@kernel.org> writes:

> On Thu, Dec 23, 2021 at 9:03 AM Kalle Valo <kvalo@qca.qualcomm.com> wrote:
>>
>> Hi linux-firmware maintainers,
>>
>> here's a pull request to update ath11k firmware images. We have new hardware support WCN6855 and QCN9074, but also updates to existing hardware.
>>
>> Please let me know if there are any problems.
>>
>> Kalle
>>
>> The following changes since commit f682ecb015df087613f91872712eb6c0f6c998a8:
>>
>>   Merge https://github.com/pkshih/linux-firmware into main (2021-12-16 09:42:51 -0500)
>>
>> are available in the Git repository at:
>>
>>   git://git.kernel.org/pub/scm/linux/kernel/git/kvalo/linux-firmware.git ath10k-20211223
>>
>> for you to fetch changes up to 0f161f385f3854086b4fdae2dae786ab6bd3efa5:
>>
>>   ath11k: WCN6855 hw2.0: add to WLAN.HSP.1.1-03003-QCAHSPSWPL_V1_V2_SILICONZ_LITE-2 (2021-12-23 12:40:55 +0200)
>>
>> ----------------------------------------------------------------
>> Kalle Valo (5):
>>       ath11k: IPQ6018 hw1.0: update to WLAN.HK.2.5.0.1-01100-QCAHKSWPL_SILICONZ-1
>>       ath11k: IPQ8074 hw2.0: update to WLAN.HK.2.5.0.1-01100-QCAHKSWPL_SILICONZ-1
>>       ath11k: QCN9074 hw1.0: add to WLAN.HK.2.5.0.1-01100-QCAHKSWPL_SILICONZ-1
>>       ath11k: WCN6855 hw2.0: add board-2.bin and regdb.bin
>>       ath11k: WCN6855 hw2.0: add to WLAN.HSP.1.1-03003-QCAHSPSWPL_V1_V2_SILICONZ_LITE-2
>>
>>  WHENCE                           |   14 +-
>>  ath11k/IPQ6018/hw1.0/Notice.txt  |   48 +-
>
> These Notice files seem problematic.  They're clearly well intended,
> but they have language that alludes to an agreement with Atheros and a
> confidential nature to some of the files.  Specifically:
>
> "...your use of these software
> components together with the Qualcomm Atheros software (Qualcomm
> Atheros software hereinafter referred to as “Software”) is
> subject to the terms of your agreement from Qualcomm Atheros."
>
> Nobody has an opportunity to agree to anything with Atheros when they
> consume the files from linux-firmware.  There's also no explicit grant
> of redistribution for any of these files.
>
> Also:
>
> "/*
>  * Copyright (c) 2011 Qualcomm Technologies, Inc.
>  * 2011 Qualcomm Atheros, Inc.
>  * All Rights Reserved.
>  * Confidential and Proprietary - Qualcomm Technologies, Inc.
>  * Notifications and licenses are retained for attribution purposes only.
>  */
> "
>
> Several of those are followed by "For this file....  has selected the
> BSD license"  but it's unclear what file that's referring to.
>
> I'm not comfortable pulling this with the licenses in an ambiguous
> state.  Can you work with your team to clarify the text?

Sorry for the delay, I was on vacation.

Thanks for the feedback, I'll report this and later send a new pull
request with better language. But this can take some time.

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

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

^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [PULL] ath11k firmware 20211223
@ 2022-01-10 15:19     ` Kalle Valo
  0 siblings, 0 replies; 27+ messages in thread
From: Kalle Valo @ 2022-01-10 15:19 UTC (permalink / raw)
  To: Josh Boyer; +Cc: linux-firmware, linux-wireless, ath11k

Josh Boyer <jwboyer@kernel.org> writes:

> On Thu, Dec 23, 2021 at 9:03 AM Kalle Valo <kvalo@qca.qualcomm.com> wrote:
>>
>> Hi linux-firmware maintainers,
>>
>> here's a pull request to update ath11k firmware images. We have new hardware support WCN6855 and QCN9074, but also updates to existing hardware.
>>
>> Please let me know if there are any problems.
>>
>> Kalle
>>
>> The following changes since commit f682ecb015df087613f91872712eb6c0f6c998a8:
>>
>>   Merge https://github.com/pkshih/linux-firmware into main (2021-12-16 09:42:51 -0500)
>>
>> are available in the Git repository at:
>>
>>   git://git.kernel.org/pub/scm/linux/kernel/git/kvalo/linux-firmware.git ath10k-20211223
>>
>> for you to fetch changes up to 0f161f385f3854086b4fdae2dae786ab6bd3efa5:
>>
>>   ath11k: WCN6855 hw2.0: add to WLAN.HSP.1.1-03003-QCAHSPSWPL_V1_V2_SILICONZ_LITE-2 (2021-12-23 12:40:55 +0200)
>>
>> ----------------------------------------------------------------
>> Kalle Valo (5):
>>       ath11k: IPQ6018 hw1.0: update to WLAN.HK.2.5.0.1-01100-QCAHKSWPL_SILICONZ-1
>>       ath11k: IPQ8074 hw2.0: update to WLAN.HK.2.5.0.1-01100-QCAHKSWPL_SILICONZ-1
>>       ath11k: QCN9074 hw1.0: add to WLAN.HK.2.5.0.1-01100-QCAHKSWPL_SILICONZ-1
>>       ath11k: WCN6855 hw2.0: add board-2.bin and regdb.bin
>>       ath11k: WCN6855 hw2.0: add to WLAN.HSP.1.1-03003-QCAHSPSWPL_V1_V2_SILICONZ_LITE-2
>>
>>  WHENCE                           |   14 +-
>>  ath11k/IPQ6018/hw1.0/Notice.txt  |   48 +-
>
> These Notice files seem problematic.  They're clearly well intended,
> but they have language that alludes to an agreement with Atheros and a
> confidential nature to some of the files.  Specifically:
>
> "...your use of these software
> components together with the Qualcomm Atheros software (Qualcomm
> Atheros software hereinafter referred to as “Software”) is
> subject to the terms of your agreement from Qualcomm Atheros."
>
> Nobody has an opportunity to agree to anything with Atheros when they
> consume the files from linux-firmware.  There's also no explicit grant
> of redistribution for any of these files.
>
> Also:
>
> "/*
>  * Copyright (c) 2011 Qualcomm Technologies, Inc.
>  * 2011 Qualcomm Atheros, Inc.
>  * All Rights Reserved.
>  * Confidential and Proprietary - Qualcomm Technologies, Inc.
>  * Notifications and licenses are retained for attribution purposes only.
>  */
> "
>
> Several of those are followed by "For this file....  has selected the
> BSD license"  but it's unclear what file that's referring to.
>
> I'm not comfortable pulling this with the licenses in an ambiguous
> state.  Can you work with your team to clarify the text?

Sorry for the delay, I was on vacation.

Thanks for the feedback, I'll report this and later send a new pull
request with better language. But this can take some time.

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

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

-- 
ath11k mailing list
ath11k@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/ath11k

^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [PULL] ath11k firmware 20211223
  2021-12-29 13:41   ` Josh Boyer
@ 2022-02-10 10:45     ` Kalle Valo
  -1 siblings, 0 replies; 27+ messages in thread
From: Kalle Valo @ 2022-02-10 10:45 UTC (permalink / raw)
  To: Josh Boyer; +Cc: linux-firmware, linux-wireless, ath11k

Hi Josh,

Josh Boyer <jwboyer@kernel.org> writes:

> On Thu, Dec 23, 2021 at 9:03 AM Kalle Valo <kvalo@qca.qualcomm.com> wrote:
>>
>>  WHENCE                           |   14 +-
>>  ath11k/IPQ6018/hw1.0/Notice.txt  |   48 +-
>
> These Notice files seem problematic.  They're clearly well intended,
> but they have language that alludes to an agreement with Atheros and a
> confidential nature to some of the files.  Specifically:
>
> "...your use of these software
> components together with the Qualcomm Atheros software (Qualcomm
> Atheros software hereinafter referred to as “Software”) is
> subject to the terms of your agreement from Qualcomm Atheros."
>
> Nobody has an opportunity to agree to anything with Atheros when they
> consume the files from linux-firmware.  There's also no explicit grant
> of redistribution for any of these files.

So this file we are discussing is the notice.txt file. In the WHENCE
file the actual license for ath11k is:

Licence: Redistributable. See LICENSE.QualcommAtheros_ath10k for details

Does that cover your concerns about redistribution?

I'm still working on your other comment about notice.txt, will get back
on that later.

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

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

^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [PULL] ath11k firmware 20211223
@ 2022-02-10 10:45     ` Kalle Valo
  0 siblings, 0 replies; 27+ messages in thread
From: Kalle Valo @ 2022-02-10 10:45 UTC (permalink / raw)
  To: Josh Boyer; +Cc: linux-firmware, linux-wireless, ath11k

Hi Josh,

Josh Boyer <jwboyer@kernel.org> writes:

> On Thu, Dec 23, 2021 at 9:03 AM Kalle Valo <kvalo@qca.qualcomm.com> wrote:
>>
>>  WHENCE                           |   14 +-
>>  ath11k/IPQ6018/hw1.0/Notice.txt  |   48 +-
>
> These Notice files seem problematic.  They're clearly well intended,
> but they have language that alludes to an agreement with Atheros and a
> confidential nature to some of the files.  Specifically:
>
> "...your use of these software
> components together with the Qualcomm Atheros software (Qualcomm
> Atheros software hereinafter referred to as “Software”) is
> subject to the terms of your agreement from Qualcomm Atheros."
>
> Nobody has an opportunity to agree to anything with Atheros when they
> consume the files from linux-firmware.  There's also no explicit grant
> of redistribution for any of these files.

So this file we are discussing is the notice.txt file. In the WHENCE
file the actual license for ath11k is:

Licence: Redistributable. See LICENSE.QualcommAtheros_ath10k for details

Does that cover your concerns about redistribution?

I'm still working on your other comment about notice.txt, will get back
on that later.

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

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

-- 
ath11k mailing list
ath11k@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/ath11k

^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [PULL] ath11k firmware 20211223
  2021-12-23 14:03 ` Kalle Valo
@ 2022-02-18 15:00   ` Sven Eckelmann
  -1 siblings, 0 replies; 27+ messages in thread
From: Sven Eckelmann @ 2022-02-18 15:00 UTC (permalink / raw)
  To: linux-firmware, ath11k, Kalle Valo; +Cc: linux-wireless

[-- Attachment #1: Type: text/plain, Size: 395 bytes --]

On Thursday, 23 December 2021 15:03:04 CET Kalle Valo wrote:
>  ath11k/QCN9074/hw1.0/Notice.txt  |  802 ++++++++++++++++++++
>  ath11k/QCN9074/hw1.0/amss.bin    |  Bin 0 -> 11897676 bytes
>  ath11k/QCN9074/hw1.0/m3.bin      |  Bin 0 -> 340108 bytes

This pull adds the firmware for QCN9074 but not the board-2.bin. As result, 
the initialization of the ath11k PHY will fail.

Kind regards,
	Sven

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [PULL] ath11k firmware 20211223
@ 2022-02-18 15:00   ` Sven Eckelmann
  0 siblings, 0 replies; 27+ messages in thread
From: Sven Eckelmann @ 2022-02-18 15:00 UTC (permalink / raw)
  To: linux-firmware, ath11k, Kalle Valo; +Cc: linux-wireless


[-- Attachment #1.1: Type: text/plain, Size: 395 bytes --]

On Thursday, 23 December 2021 15:03:04 CET Kalle Valo wrote:
>  ath11k/QCN9074/hw1.0/Notice.txt  |  802 ++++++++++++++++++++
>  ath11k/QCN9074/hw1.0/amss.bin    |  Bin 0 -> 11897676 bytes
>  ath11k/QCN9074/hw1.0/m3.bin      |  Bin 0 -> 340108 bytes

This pull adds the firmware for QCN9074 but not the board-2.bin. As result, 
the initialization of the ath11k PHY will fail.

Kind regards,
	Sven

[-- Attachment #1.2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

[-- Attachment #2: Type: text/plain, Size: 102 bytes --]

-- 
ath11k mailing list
ath11k@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/ath11k

^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [PULL] ath11k firmware 20211223
  2022-02-10 10:45     ` Kalle Valo
@ 2022-02-18 15:55       ` Josh Boyer
  -1 siblings, 0 replies; 27+ messages in thread
From: Josh Boyer @ 2022-02-18 15:55 UTC (permalink / raw)
  To: Kalle Valo; +Cc: linux-firmware, linux-wireless, ath11k

On Thu, Feb 10, 2022 at 5:45 AM Kalle Valo <kvalo@kernel.org> wrote:
>
> Hi Josh,
>
> Josh Boyer <jwboyer@kernel.org> writes:
>
> > On Thu, Dec 23, 2021 at 9:03 AM Kalle Valo <kvalo@qca.qualcomm.com> wrote:
> >>
> >>  WHENCE                           |   14 +-
> >>  ath11k/IPQ6018/hw1.0/Notice.txt  |   48 +-
> >
> > These Notice files seem problematic.  They're clearly well intended,
> > but they have language that alludes to an agreement with Atheros and a
> > confidential nature to some of the files.  Specifically:
> >
> > "...your use of these software
> > components together with the Qualcomm Atheros software (Qualcomm
> > Atheros software hereinafter referred to as “Software”) is
> > subject to the terms of your agreement from Qualcomm Atheros."
> >
> > Nobody has an opportunity to agree to anything with Atheros when they
> > consume the files from linux-firmware.  There's also no explicit grant
> > of redistribution for any of these files.
>
> So this file we are discussing is the notice.txt file. In the WHENCE
> file the actual license for ath11k is:
>
> Licence: Redistributable. See LICENSE.QualcommAtheros_ath10k for details

Then is the notice.txt file needed at all?  If a user or distribution
were to install this firmware, is the expectation that the notice.txt
file also be installed?

> Does that cover your concerns about redistribution?

Not really.  I understand the logic, but if the binding license for
the firmware is LICENSE.QualcommAtheros_ath10k but we still need
notice.txt for some reason, they conflict.  It's ambiguous at best.

josh

> I'm still working on your other comment about notice.txt, will get back
> on that later.
>
> --
> https://patchwork.kernel.org/project/linux-wireless/list/
>
> https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches

^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [PULL] ath11k firmware 20211223
@ 2022-02-18 15:55       ` Josh Boyer
  0 siblings, 0 replies; 27+ messages in thread
From: Josh Boyer @ 2022-02-18 15:55 UTC (permalink / raw)
  To: Kalle Valo; +Cc: linux-firmware, linux-wireless, ath11k

On Thu, Feb 10, 2022 at 5:45 AM Kalle Valo <kvalo@kernel.org> wrote:
>
> Hi Josh,
>
> Josh Boyer <jwboyer@kernel.org> writes:
>
> > On Thu, Dec 23, 2021 at 9:03 AM Kalle Valo <kvalo@qca.qualcomm.com> wrote:
> >>
> >>  WHENCE                           |   14 +-
> >>  ath11k/IPQ6018/hw1.0/Notice.txt  |   48 +-
> >
> > These Notice files seem problematic.  They're clearly well intended,
> > but they have language that alludes to an agreement with Atheros and a
> > confidential nature to some of the files.  Specifically:
> >
> > "...your use of these software
> > components together with the Qualcomm Atheros software (Qualcomm
> > Atheros software hereinafter referred to as “Software”) is
> > subject to the terms of your agreement from Qualcomm Atheros."
> >
> > Nobody has an opportunity to agree to anything with Atheros when they
> > consume the files from linux-firmware.  There's also no explicit grant
> > of redistribution for any of these files.
>
> So this file we are discussing is the notice.txt file. In the WHENCE
> file the actual license for ath11k is:
>
> Licence: Redistributable. See LICENSE.QualcommAtheros_ath10k for details

Then is the notice.txt file needed at all?  If a user or distribution
were to install this firmware, is the expectation that the notice.txt
file also be installed?

> Does that cover your concerns about redistribution?

Not really.  I understand the logic, but if the binding license for
the firmware is LICENSE.QualcommAtheros_ath10k but we still need
notice.txt for some reason, they conflict.  It's ambiguous at best.

josh

> I'm still working on your other comment about notice.txt, will get back
> on that later.
>
> --
> https://patchwork.kernel.org/project/linux-wireless/list/
>
> https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches

-- 
ath11k mailing list
ath11k@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/ath11k

^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [PULL] ath11k firmware 20211223
  2022-02-18 15:55       ` Josh Boyer
@ 2022-03-02 17:50         ` Jouni Malinen
  -1 siblings, 0 replies; 27+ messages in thread
From: Jouni Malinen @ 2022-03-02 17:50 UTC (permalink / raw)
  To: Josh Boyer; +Cc: Kalle Valo, linux-firmware, linux-wireless, ath11k

On Fri, Feb 18, 2022 at 10:55:37AM -0500, Josh Boyer wrote:
> On Thu, Feb 10, 2022 at 5:45 AM Kalle Valo <kvalo@kernel.org> wrote:
> > > On Thu, Dec 23, 2021 at 9:03 AM Kalle Valo <kvalo@qca.qualcomm.com> wrote:
> > >>
> > >>  WHENCE                           |   14 +-
> > >>  ath11k/IPQ6018/hw1.0/Notice.txt  |   48 +-

> > > These Notice files seem problematic.  They're clearly well intended,
> > > but they have language that alludes to an agreement with Atheros and a
> > > confidential nature to some of the files.  Specifically:
> > >
> > > "...your use of these software
> > > components together with the Qualcomm Atheros software (Qualcomm
> > > Atheros software hereinafter referred to as “Software”) is
> > > subject to the terms of your agreement from Qualcomm Atheros."
> > >
> > > Nobody has an opportunity to agree to anything with Atheros when they
> > > consume the files from linux-firmware.  There's also no explicit grant
> > > of redistribution for any of these files.

This is not the license; this is a description of the notice.txt file
containing the notices to cover the 3rd party license requirements for
attribution/notices included in redistribution of those parts.

> > So this file we are discussing is the notice.txt file. In the WHENCE
> > file the actual license for ath11k is:
> >
> > Licence: Redistributable. See LICENSE.QualcommAtheros_ath10k for details

And this is where the license is and this includes license to
redistribute.

> Then is the notice.txt file needed at all?  If a user or distribution
> were to install this firmware, is the expectation that the notice.txt
> file also be installed?

Those 3rd party license might require various notifications to be
included for notification/attribution purposes, so I would expect the
notice.txt file to be redistributed whenever the firmware image is
redistributed.

> > Does that cover your concerns about redistribution?
> 
> Not really.  I understand the logic, but if the binding license for
> the firmware is LICENSE.QualcommAtheros_ath10k but we still need
> notice.txt for some reason, they conflict.  It's ambiguous at best.

Could you please be more specific on where you see a conflict? Maybe
this would be clearer if the WHENCE file would not mark the notice.txt
files with the "License: <file>" lines since these are not the license
for the firmware binary and then there would be a single "License:" line
pointing out the exact license that applies?

Would the following in WHENCE work for you?

Driver: ath11k - Qualcomm Technologies 802.11ax chipset support

File: ath11k/IPQ6018/hw1.0/board-2.bin
...
Version: WLAN.HK.2.1.0.1-01238-QCAHKSWPL_SILICONZ-2
Notice: ath11k/IPQ6018/hw1.0/Notice.txt
...
File: ath11k/QCA6390/hw2.0/m3.bin
Version: WLAN.HST.1.0.1-01740-QCAHSTSWPLZ_V2_TO_X86-1
Notice: ath11k/QCA6390/hw2.0/Notice.txt

Licence: Redistributable. See LICENSE.QualcommAtheros_ath10k for details


In other words, there would be only a single "License:" line and one
"Notice:" line for each firmware version? The license itself (i.e.,
LICENSE.QualcommAtheros_ath10k) is same for all the versions while the
set of notices (i.e., those notice.txt files) can be different based on
what is included in the particular build.

> > I'm still working on your other comment about notice.txt, will get back
> > on that later.

This part about clearly identifying the files should be clear now, but
it would be good to resolve that part about the notice.txt files in
general before sending out an updated pull request.

-- 
Jouni Malinen                                            PGP id EFC895FA

^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [PULL] ath11k firmware 20211223
@ 2022-03-02 17:50         ` Jouni Malinen
  0 siblings, 0 replies; 27+ messages in thread
From: Jouni Malinen @ 2022-03-02 17:50 UTC (permalink / raw)
  To: Josh Boyer; +Cc: Kalle Valo, linux-firmware, linux-wireless, ath11k

On Fri, Feb 18, 2022 at 10:55:37AM -0500, Josh Boyer wrote:
> On Thu, Feb 10, 2022 at 5:45 AM Kalle Valo <kvalo@kernel.org> wrote:
> > > On Thu, Dec 23, 2021 at 9:03 AM Kalle Valo <kvalo@qca.qualcomm.com> wrote:
> > >>
> > >>  WHENCE                           |   14 +-
> > >>  ath11k/IPQ6018/hw1.0/Notice.txt  |   48 +-

> > > These Notice files seem problematic.  They're clearly well intended,
> > > but they have language that alludes to an agreement with Atheros and a
> > > confidential nature to some of the files.  Specifically:
> > >
> > > "...your use of these software
> > > components together with the Qualcomm Atheros software (Qualcomm
> > > Atheros software hereinafter referred to as “Software”) is
> > > subject to the terms of your agreement from Qualcomm Atheros."
> > >
> > > Nobody has an opportunity to agree to anything with Atheros when they
> > > consume the files from linux-firmware.  There's also no explicit grant
> > > of redistribution for any of these files.

This is not the license; this is a description of the notice.txt file
containing the notices to cover the 3rd party license requirements for
attribution/notices included in redistribution of those parts.

> > So this file we are discussing is the notice.txt file. In the WHENCE
> > file the actual license for ath11k is:
> >
> > Licence: Redistributable. See LICENSE.QualcommAtheros_ath10k for details

And this is where the license is and this includes license to
redistribute.

> Then is the notice.txt file needed at all?  If a user or distribution
> were to install this firmware, is the expectation that the notice.txt
> file also be installed?

Those 3rd party license might require various notifications to be
included for notification/attribution purposes, so I would expect the
notice.txt file to be redistributed whenever the firmware image is
redistributed.

> > Does that cover your concerns about redistribution?
> 
> Not really.  I understand the logic, but if the binding license for
> the firmware is LICENSE.QualcommAtheros_ath10k but we still need
> notice.txt for some reason, they conflict.  It's ambiguous at best.

Could you please be more specific on where you see a conflict? Maybe
this would be clearer if the WHENCE file would not mark the notice.txt
files with the "License: <file>" lines since these are not the license
for the firmware binary and then there would be a single "License:" line
pointing out the exact license that applies?

Would the following in WHENCE work for you?

Driver: ath11k - Qualcomm Technologies 802.11ax chipset support

File: ath11k/IPQ6018/hw1.0/board-2.bin
...
Version: WLAN.HK.2.1.0.1-01238-QCAHKSWPL_SILICONZ-2
Notice: ath11k/IPQ6018/hw1.0/Notice.txt
...
File: ath11k/QCA6390/hw2.0/m3.bin
Version: WLAN.HST.1.0.1-01740-QCAHSTSWPLZ_V2_TO_X86-1
Notice: ath11k/QCA6390/hw2.0/Notice.txt

Licence: Redistributable. See LICENSE.QualcommAtheros_ath10k for details


In other words, there would be only a single "License:" line and one
"Notice:" line for each firmware version? The license itself (i.e.,
LICENSE.QualcommAtheros_ath10k) is same for all the versions while the
set of notices (i.e., those notice.txt files) can be different based on
what is included in the particular build.

> > I'm still working on your other comment about notice.txt, will get back
> > on that later.

This part about clearly identifying the files should be clear now, but
it would be good to resolve that part about the notice.txt files in
general before sending out an updated pull request.

-- 
Jouni Malinen                                            PGP id EFC895FA

-- 
ath11k mailing list
ath11k@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/ath11k

^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [PULL] ath11k firmware 20211223
  2022-03-02 17:50         ` Jouni Malinen
@ 2022-03-04 13:04           ` Josh Boyer
  -1 siblings, 0 replies; 27+ messages in thread
From: Josh Boyer @ 2022-03-04 13:04 UTC (permalink / raw)
  To: Jouni Malinen; +Cc: Kalle Valo, linux-firmware, linux-wireless, ath11k

On Wed, Mar 2, 2022 at 12:50 PM Jouni Malinen <j@w1.fi> wrote:
>
> On Fri, Feb 18, 2022 at 10:55:37AM -0500, Josh Boyer wrote:
> > On Thu, Feb 10, 2022 at 5:45 AM Kalle Valo <kvalo@kernel.org> wrote:
> > > > On Thu, Dec 23, 2021 at 9:03 AM Kalle Valo <kvalo@qca.qualcomm.com> wrote:
> > > >>
> > > >>  WHENCE                           |   14 +-
> > > >>  ath11k/IPQ6018/hw1.0/Notice.txt  |   48 +-
>
> > > > These Notice files seem problematic.  They're clearly well intended,
> > > > but they have language that alludes to an agreement with Atheros and a
> > > > confidential nature to some of the files.  Specifically:
> > > >
> > > > "...your use of these software
> > > > components together with the Qualcomm Atheros software (Qualcomm
> > > > Atheros software hereinafter referred to as “Software”) is
> > > > subject to the terms of your agreement from Qualcomm Atheros."
> > > >
> > > > Nobody has an opportunity to agree to anything with Atheros when they
> > > > consume the files from linux-firmware.  There's also no explicit grant
> > > > of redistribution for any of these files.
>
> This is not the license; this is a description of the notice.txt file
> containing the notices to cover the 3rd party license requirements for
> attribution/notices included in redistribution of those parts.
>
> > > So this file we are discussing is the notice.txt file. In the WHENCE
> > > file the actual license for ath11k is:
> > >
> > > Licence: Redistributable. See LICENSE.QualcommAtheros_ath10k for details
>
> And this is where the license is and this includes license to
> redistribute.
>
> > Then is the notice.txt file needed at all?  If a user or distribution
> > were to install this firmware, is the expectation that the notice.txt
> > file also be installed?
>
> Those 3rd party license might require various notifications to be
> included for notification/attribution purposes, so I would expect the
> notice.txt file to be redistributed whenever the firmware image is
> redistributed.
>
> > > Does that cover your concerns about redistribution?
> >
> > Not really.  I understand the logic, but if the binding license for
> > the firmware is LICENSE.QualcommAtheros_ath10k but we still need
> > notice.txt for some reason, they conflict.  It's ambiguous at best.
>
> Could you please be more specific on where you see a conflict? Maybe
> this would be clearer if the WHENCE file would not mark the notice.txt
> files with the "License: <file>" lines since these are not the license
> for the firmware binary and then there would be a single "License:" line
> pointing out the exact license that applies?

That would help.  I spent some time digging into the existing ath
firmware in the tree and see they all have similar notices already.
It leaves me guessing how/why they were merged that way in the past,
but if we can disambiguate what the license actually is that would be
good.

> Would the following in WHENCE work for you?
>
> Driver: ath11k - Qualcomm Technologies 802.11ax chipset support
>
> File: ath11k/IPQ6018/hw1.0/board-2.bin
> ...
> Version: WLAN.HK.2.1.0.1-01238-QCAHKSWPL_SILICONZ-2
> Notice: ath11k/IPQ6018/hw1.0/Notice.txt

We'd have to teach copy-firmware.sh what to do with a "Notice:" key.
Is there a reason "File:" wouldn't work?  The intention is to install
the notices alongside the binaries, so that would accomplish it.

> ...
> File: ath11k/QCA6390/hw2.0/m3.bin
> Version: WLAN.HST.1.0.1-01740-QCAHSTSWPLZ_V2_TO_X86-1
> Notice: ath11k/QCA6390/hw2.0/Notice.txt
>
> Licence: Redistributable. See LICENSE.QualcommAtheros_ath10k for details
>
>
> In other words, there would be only a single "License:" line and one
> "Notice:" line for each firmware version? The license itself (i.e.,
> LICENSE.QualcommAtheros_ath10k) is same for all the versions while the
> set of notices (i.e., those notice.txt files) can be different based on
> what is included in the particular build.

Yes, that would help.  Would you be able to adjust the existing
entries for ath firmware in the same way?

> > > I'm still working on your other comment about notice.txt, will get back
> > > on that later.
>
> This part about clearly identifying the files should be clear now, but
> it would be good to resolve that part about the notice.txt files in
> general before sending out an updated pull request.

Given these were merged in the past, perhaps I'm being overly
pedantic.  If we can mark them as Files or Notices instead of
Licenses, I won't hold it up.  It leaves me slightly confused why
attribution files need to reference agreements with Qualcomm, splatter
Confidential and Proprietary throughout the file, and reference
COPYING and README in reference to GPLv2 when the BSD license was
clearly chosen.  Perhaps that could be cleaned up in the future.

josh

^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [PULL] ath11k firmware 20211223
@ 2022-03-04 13:04           ` Josh Boyer
  0 siblings, 0 replies; 27+ messages in thread
From: Josh Boyer @ 2022-03-04 13:04 UTC (permalink / raw)
  To: Jouni Malinen; +Cc: Kalle Valo, linux-firmware, linux-wireless, ath11k

On Wed, Mar 2, 2022 at 12:50 PM Jouni Malinen <j@w1.fi> wrote:
>
> On Fri, Feb 18, 2022 at 10:55:37AM -0500, Josh Boyer wrote:
> > On Thu, Feb 10, 2022 at 5:45 AM Kalle Valo <kvalo@kernel.org> wrote:
> > > > On Thu, Dec 23, 2021 at 9:03 AM Kalle Valo <kvalo@qca.qualcomm.com> wrote:
> > > >>
> > > >>  WHENCE                           |   14 +-
> > > >>  ath11k/IPQ6018/hw1.0/Notice.txt  |   48 +-
>
> > > > These Notice files seem problematic.  They're clearly well intended,
> > > > but they have language that alludes to an agreement with Atheros and a
> > > > confidential nature to some of the files.  Specifically:
> > > >
> > > > "...your use of these software
> > > > components together with the Qualcomm Atheros software (Qualcomm
> > > > Atheros software hereinafter referred to as “Software”) is
> > > > subject to the terms of your agreement from Qualcomm Atheros."
> > > >
> > > > Nobody has an opportunity to agree to anything with Atheros when they
> > > > consume the files from linux-firmware.  There's also no explicit grant
> > > > of redistribution for any of these files.
>
> This is not the license; this is a description of the notice.txt file
> containing the notices to cover the 3rd party license requirements for
> attribution/notices included in redistribution of those parts.
>
> > > So this file we are discussing is the notice.txt file. In the WHENCE
> > > file the actual license for ath11k is:
> > >
> > > Licence: Redistributable. See LICENSE.QualcommAtheros_ath10k for details
>
> And this is where the license is and this includes license to
> redistribute.
>
> > Then is the notice.txt file needed at all?  If a user or distribution
> > were to install this firmware, is the expectation that the notice.txt
> > file also be installed?
>
> Those 3rd party license might require various notifications to be
> included for notification/attribution purposes, so I would expect the
> notice.txt file to be redistributed whenever the firmware image is
> redistributed.
>
> > > Does that cover your concerns about redistribution?
> >
> > Not really.  I understand the logic, but if the binding license for
> > the firmware is LICENSE.QualcommAtheros_ath10k but we still need
> > notice.txt for some reason, they conflict.  It's ambiguous at best.
>
> Could you please be more specific on where you see a conflict? Maybe
> this would be clearer if the WHENCE file would not mark the notice.txt
> files with the "License: <file>" lines since these are not the license
> for the firmware binary and then there would be a single "License:" line
> pointing out the exact license that applies?

That would help.  I spent some time digging into the existing ath
firmware in the tree and see they all have similar notices already.
It leaves me guessing how/why they were merged that way in the past,
but if we can disambiguate what the license actually is that would be
good.

> Would the following in WHENCE work for you?
>
> Driver: ath11k - Qualcomm Technologies 802.11ax chipset support
>
> File: ath11k/IPQ6018/hw1.0/board-2.bin
> ...
> Version: WLAN.HK.2.1.0.1-01238-QCAHKSWPL_SILICONZ-2
> Notice: ath11k/IPQ6018/hw1.0/Notice.txt

We'd have to teach copy-firmware.sh what to do with a "Notice:" key.
Is there a reason "File:" wouldn't work?  The intention is to install
the notices alongside the binaries, so that would accomplish it.

> ...
> File: ath11k/QCA6390/hw2.0/m3.bin
> Version: WLAN.HST.1.0.1-01740-QCAHSTSWPLZ_V2_TO_X86-1
> Notice: ath11k/QCA6390/hw2.0/Notice.txt
>
> Licence: Redistributable. See LICENSE.QualcommAtheros_ath10k for details
>
>
> In other words, there would be only a single "License:" line and one
> "Notice:" line for each firmware version? The license itself (i.e.,
> LICENSE.QualcommAtheros_ath10k) is same for all the versions while the
> set of notices (i.e., those notice.txt files) can be different based on
> what is included in the particular build.

Yes, that would help.  Would you be able to adjust the existing
entries for ath firmware in the same way?

> > > I'm still working on your other comment about notice.txt, will get back
> > > on that later.
>
> This part about clearly identifying the files should be clear now, but
> it would be good to resolve that part about the notice.txt files in
> general before sending out an updated pull request.

Given these were merged in the past, perhaps I'm being overly
pedantic.  If we can mark them as Files or Notices instead of
Licenses, I won't hold it up.  It leaves me slightly confused why
attribution files need to reference agreements with Qualcomm, splatter
Confidential and Proprietary throughout the file, and reference
COPYING and README in reference to GPLv2 when the BSD license was
clearly chosen.  Perhaps that could be cleaned up in the future.

josh

-- 
ath11k mailing list
ath11k@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/ath11k

^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [PULL] ath11k firmware 20211223
  2022-03-04 13:04           ` Josh Boyer
@ 2022-03-04 16:14             ` Jouni Malinen
  -1 siblings, 0 replies; 27+ messages in thread
From: Jouni Malinen @ 2022-03-04 16:14 UTC (permalink / raw)
  To: Josh Boyer; +Cc: Kalle Valo, linux-firmware, linux-wireless, ath11k

On Fri, Mar 04, 2022 at 08:04:26AM -0500, Josh Boyer wrote:
> On Wed, Mar 2, 2022 at 12:50 PM Jouni Malinen <j@w1.fi> wrote:
> > Would the following in WHENCE work for you?
> >
> > Driver: ath11k - Qualcomm Technologies 802.11ax chipset support
> >
> > File: ath11k/IPQ6018/hw1.0/board-2.bin
> > ...
> > Version: WLAN.HK.2.1.0.1-01238-QCAHKSWPL_SILICONZ-2
> > Notice: ath11k/IPQ6018/hw1.0/Notice.txt
> 
> We'd have to teach copy-firmware.sh what to do with a "Notice:" key.
> Is there a reason "File:" wouldn't work?  The intention is to install
> the notices alongside the binaries, so that would accomplish it.

Yeah, that sounds reasonable to me. From copying/installing/distribution
view point, these can really be handled in the exact same way as the
actual firmware binaries in practice.

> > In other words, there would be only a single "License:" line and one
> > "Notice:" line for each firmware version? The license itself (i.e.,
> > LICENSE.QualcommAtheros_ath10k) is same for all the versions while the
> > set of notices (i.e., those notice.txt files) can be different based on
> > what is included in the particular build.
> 
> Yes, that would help.  Would you be able to adjust the existing
> entries for ath firmware in the same way?

Yes, I'll work with Kalle to update the existing ath* WLAN cases.

> > > > I'm still working on your other comment about notice.txt, will get back
> > > > on that later.
> >
> > This part about clearly identifying the files should be clear now, but
> > it would be good to resolve that part about the notice.txt files in
> > general before sending out an updated pull request.
> 
> Given these were merged in the past, perhaps I'm being overly
> pedantic.  If we can mark them as Files or Notices instead of
> Licenses, I won't hold it up.  It leaves me slightly confused why
> attribution files need to reference agreements with Qualcomm, splatter
> Confidential and Proprietary throughout the file, and reference
> COPYING and README in reference to GPLv2 when the BSD license was
> clearly chosen.  Perhaps that could be cleaned up in the future.

Thanks. We'll remove most of the unnecessary information from the new
notice.txt files and that should get rid of many of the potentially
confusing parts. If that cleanup leaves something confusing in place,
we are open to cleaning these up further in followup patches, but it
would be nice to be able to get the updated versions into
linux-firmware.git without much more additional delay and yes, this
would be with the File: instead of Licence: entries for the notice.txt
files.

-- 
Jouni Malinen                                            PGP id EFC895FA

^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [PULL] ath11k firmware 20211223
@ 2022-03-04 16:14             ` Jouni Malinen
  0 siblings, 0 replies; 27+ messages in thread
From: Jouni Malinen @ 2022-03-04 16:14 UTC (permalink / raw)
  To: Josh Boyer; +Cc: Kalle Valo, linux-firmware, linux-wireless, ath11k

On Fri, Mar 04, 2022 at 08:04:26AM -0500, Josh Boyer wrote:
> On Wed, Mar 2, 2022 at 12:50 PM Jouni Malinen <j@w1.fi> wrote:
> > Would the following in WHENCE work for you?
> >
> > Driver: ath11k - Qualcomm Technologies 802.11ax chipset support
> >
> > File: ath11k/IPQ6018/hw1.0/board-2.bin
> > ...
> > Version: WLAN.HK.2.1.0.1-01238-QCAHKSWPL_SILICONZ-2
> > Notice: ath11k/IPQ6018/hw1.0/Notice.txt
> 
> We'd have to teach copy-firmware.sh what to do with a "Notice:" key.
> Is there a reason "File:" wouldn't work?  The intention is to install
> the notices alongside the binaries, so that would accomplish it.

Yeah, that sounds reasonable to me. From copying/installing/distribution
view point, these can really be handled in the exact same way as the
actual firmware binaries in practice.

> > In other words, there would be only a single "License:" line and one
> > "Notice:" line for each firmware version? The license itself (i.e.,
> > LICENSE.QualcommAtheros_ath10k) is same for all the versions while the
> > set of notices (i.e., those notice.txt files) can be different based on
> > what is included in the particular build.
> 
> Yes, that would help.  Would you be able to adjust the existing
> entries for ath firmware in the same way?

Yes, I'll work with Kalle to update the existing ath* WLAN cases.

> > > > I'm still working on your other comment about notice.txt, will get back
> > > > on that later.
> >
> > This part about clearly identifying the files should be clear now, but
> > it would be good to resolve that part about the notice.txt files in
> > general before sending out an updated pull request.
> 
> Given these were merged in the past, perhaps I'm being overly
> pedantic.  If we can mark them as Files or Notices instead of
> Licenses, I won't hold it up.  It leaves me slightly confused why
> attribution files need to reference agreements with Qualcomm, splatter
> Confidential and Proprietary throughout the file, and reference
> COPYING and README in reference to GPLv2 when the BSD license was
> clearly chosen.  Perhaps that could be cleaned up in the future.

Thanks. We'll remove most of the unnecessary information from the new
notice.txt files and that should get rid of many of the potentially
confusing parts. If that cleanup leaves something confusing in place,
we are open to cleaning these up further in followup patches, but it
would be nice to be able to get the updated versions into
linux-firmware.git without much more additional delay and yes, this
would be with the File: instead of Licence: entries for the notice.txt
files.

-- 
Jouni Malinen                                            PGP id EFC895FA

-- 
ath11k mailing list
ath11k@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/ath11k

^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [PULL] ath11k firmware 20211223
  2022-03-04 16:14             ` Jouni Malinen
@ 2022-03-23  8:06               ` Takashi Iwai
  -1 siblings, 0 replies; 27+ messages in thread
From: Takashi Iwai @ 2022-03-23  8:06 UTC (permalink / raw)
  To: Jouni Malinen
  Cc: Josh Boyer, Kalle Valo, linux-firmware, linux-wireless, ath11k

On Fri, 04 Mar 2022 17:14:30 +0100,
Jouni Malinen wrote:
> 
> On Fri, Mar 04, 2022 at 08:04:26AM -0500, Josh Boyer wrote:
> > On Wed, Mar 2, 2022 at 12:50 PM Jouni Malinen <j@w1.fi> wrote:
> > > Would the following in WHENCE work for you?
> > >
> > > Driver: ath11k - Qualcomm Technologies 802.11ax chipset support
> > >
> > > File: ath11k/IPQ6018/hw1.0/board-2.bin
> > > ...
> > > Version: WLAN.HK.2.1.0.1-01238-QCAHKSWPL_SILICONZ-2
> > > Notice: ath11k/IPQ6018/hw1.0/Notice.txt
> > 
> > We'd have to teach copy-firmware.sh what to do with a "Notice:" key.
> > Is there a reason "File:" wouldn't work?  The intention is to install
> > the notices alongside the binaries, so that would accomplish it.
> 
> Yeah, that sounds reasonable to me. From copying/installing/distribution
> view point, these can really be handled in the exact same way as the
> actual firmware binaries in practice.
> 
> > > In other words, there would be only a single "License:" line and one
> > > "Notice:" line for each firmware version? The license itself (i.e.,
> > > LICENSE.QualcommAtheros_ath10k) is same for all the versions while the
> > > set of notices (i.e., those notice.txt files) can be different based on
> > > what is included in the particular build.
> > 
> > Yes, that would help.  Would you be able to adjust the existing
> > entries for ath firmware in the same way?
> 
> Yes, I'll work with Kalle to update the existing ath* WLAN cases.
> 
> > > > > I'm still working on your other comment about notice.txt, will get back
> > > > > on that later.
> > >
> > > This part about clearly identifying the files should be clear now, but
> > > it would be good to resolve that part about the notice.txt files in
> > > general before sending out an updated pull request.
> > 
> > Given these were merged in the past, perhaps I'm being overly
> > pedantic.  If we can mark them as Files or Notices instead of
> > Licenses, I won't hold it up.  It leaves me slightly confused why
> > attribution files need to reference agreements with Qualcomm, splatter
> > Confidential and Proprietary throughout the file, and reference
> > COPYING and README in reference to GPLv2 when the BSD license was
> > clearly chosen.  Perhaps that could be cleaned up in the future.
> 
> Thanks. We'll remove most of the unnecessary information from the new
> notice.txt files and that should get rid of many of the potentially
> confusing parts. If that cleanup leaves something confusing in place,
> we are open to cleaning these up further in followup patches, but it
> would be nice to be able to get the updated versions into
> linux-firmware.git without much more additional delay and yes, this
> would be with the File: instead of Licence: entries for the notice.txt
> files.

The problem of "File:" is that it's more or less intended to be
installed as the firmware files themselves, i.e. they are installed in
/lib/firmware/* that can be loaded to the kernel.  Putting such a
random (document) file there makes me a bit nervous.

We may introduce another tag to list up misc document files
(e.g. "Doc:" or whatever)?  Distros can pick up them and put to the
appropriate places in the package, too.


thanks,

Takashi

^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [PULL] ath11k firmware 20211223
@ 2022-03-23  8:06               ` Takashi Iwai
  0 siblings, 0 replies; 27+ messages in thread
From: Takashi Iwai @ 2022-03-23  8:06 UTC (permalink / raw)
  To: Jouni Malinen
  Cc: Josh Boyer, Kalle Valo, linux-firmware, linux-wireless, ath11k

On Fri, 04 Mar 2022 17:14:30 +0100,
Jouni Malinen wrote:
> 
> On Fri, Mar 04, 2022 at 08:04:26AM -0500, Josh Boyer wrote:
> > On Wed, Mar 2, 2022 at 12:50 PM Jouni Malinen <j@w1.fi> wrote:
> > > Would the following in WHENCE work for you?
> > >
> > > Driver: ath11k - Qualcomm Technologies 802.11ax chipset support
> > >
> > > File: ath11k/IPQ6018/hw1.0/board-2.bin
> > > ...
> > > Version: WLAN.HK.2.1.0.1-01238-QCAHKSWPL_SILICONZ-2
> > > Notice: ath11k/IPQ6018/hw1.0/Notice.txt
> > 
> > We'd have to teach copy-firmware.sh what to do with a "Notice:" key.
> > Is there a reason "File:" wouldn't work?  The intention is to install
> > the notices alongside the binaries, so that would accomplish it.
> 
> Yeah, that sounds reasonable to me. From copying/installing/distribution
> view point, these can really be handled in the exact same way as the
> actual firmware binaries in practice.
> 
> > > In other words, there would be only a single "License:" line and one
> > > "Notice:" line for each firmware version? The license itself (i.e.,
> > > LICENSE.QualcommAtheros_ath10k) is same for all the versions while the
> > > set of notices (i.e., those notice.txt files) can be different based on
> > > what is included in the particular build.
> > 
> > Yes, that would help.  Would you be able to adjust the existing
> > entries for ath firmware in the same way?
> 
> Yes, I'll work with Kalle to update the existing ath* WLAN cases.
> 
> > > > > I'm still working on your other comment about notice.txt, will get back
> > > > > on that later.
> > >
> > > This part about clearly identifying the files should be clear now, but
> > > it would be good to resolve that part about the notice.txt files in
> > > general before sending out an updated pull request.
> > 
> > Given these were merged in the past, perhaps I'm being overly
> > pedantic.  If we can mark them as Files or Notices instead of
> > Licenses, I won't hold it up.  It leaves me slightly confused why
> > attribution files need to reference agreements with Qualcomm, splatter
> > Confidential and Proprietary throughout the file, and reference
> > COPYING and README in reference to GPLv2 when the BSD license was
> > clearly chosen.  Perhaps that could be cleaned up in the future.
> 
> Thanks. We'll remove most of the unnecessary information from the new
> notice.txt files and that should get rid of many of the potentially
> confusing parts. If that cleanup leaves something confusing in place,
> we are open to cleaning these up further in followup patches, but it
> would be nice to be able to get the updated versions into
> linux-firmware.git without much more additional delay and yes, this
> would be with the File: instead of Licence: entries for the notice.txt
> files.

The problem of "File:" is that it's more or less intended to be
installed as the firmware files themselves, i.e. they are installed in
/lib/firmware/* that can be loaded to the kernel.  Putting such a
random (document) file there makes me a bit nervous.

We may introduce another tag to list up misc document files
(e.g. "Doc:" or whatever)?  Distros can pick up them and put to the
appropriate places in the package, too.


thanks,

Takashi

-- 
ath11k mailing list
ath11k@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/ath11k

^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [PULL] ath11k firmware 20211223
  2022-03-23  8:06               ` Takashi Iwai
@ 2022-04-13  8:26                 ` Kalle Valo
  -1 siblings, 0 replies; 27+ messages in thread
From: Kalle Valo @ 2022-04-13  8:26 UTC (permalink / raw)
  To: Takashi Iwai
  Cc: Jouni Malinen, Josh Boyer, linux-firmware, linux-wireless, ath11k

Takashi Iwai <tiwai@suse.de> writes:

> On Fri, 04 Mar 2022 17:14:30 +0100,
> Jouni Malinen wrote:
>> 
>> On Fri, Mar 04, 2022 at 08:04:26AM -0500, Josh Boyer wrote:
>> > On Wed, Mar 2, 2022 at 12:50 PM Jouni Malinen <j@w1.fi> wrote:
>> > > Would the following in WHENCE work for you?
>> > >
>> > > Driver: ath11k - Qualcomm Technologies 802.11ax chipset support
>> > >
>> > > File: ath11k/IPQ6018/hw1.0/board-2.bin
>> > > ...
>> > > Version: WLAN.HK.2.1.0.1-01238-QCAHKSWPL_SILICONZ-2
>> > > Notice: ath11k/IPQ6018/hw1.0/Notice.txt
>> > 
>> > We'd have to teach copy-firmware.sh what to do with a "Notice:" key.
>> > Is there a reason "File:" wouldn't work?  The intention is to install
>> > the notices alongside the binaries, so that would accomplish it.
>> 
>> Yeah, that sounds reasonable to me. From copying/installing/distribution
>> view point, these can really be handled in the exact same way as the
>> actual firmware binaries in practice.
>> 
>> > > In other words, there would be only a single "License:" line and one
>> > > "Notice:" line for each firmware version? The license itself (i.e.,
>> > > LICENSE.QualcommAtheros_ath10k) is same for all the versions while the
>> > > set of notices (i.e., those notice.txt files) can be different based on
>> > > what is included in the particular build.
>> > 
>> > Yes, that would help.  Would you be able to adjust the existing
>> > entries for ath firmware in the same way?
>> 
>> Yes, I'll work with Kalle to update the existing ath* WLAN cases.
>> 
>> > > > > I'm still working on your other comment about notice.txt, will get back
>> > > > > on that later.
>> > >
>> > > This part about clearly identifying the files should be clear now, but
>> > > it would be good to resolve that part about the notice.txt files in
>> > > general before sending out an updated pull request.
>> > 
>> > Given these were merged in the past, perhaps I'm being overly
>> > pedantic.  If we can mark them as Files or Notices instead of
>> > Licenses, I won't hold it up.  It leaves me slightly confused why
>> > attribution files need to reference agreements with Qualcomm, splatter
>> > Confidential and Proprietary throughout the file, and reference
>> > COPYING and README in reference to GPLv2 when the BSD license was
>> > clearly chosen.  Perhaps that could be cleaned up in the future.
>> 
>> Thanks. We'll remove most of the unnecessary information from the new
>> notice.txt files and that should get rid of many of the potentially
>> confusing parts. If that cleanup leaves something confusing in place,
>> we are open to cleaning these up further in followup patches, but it
>> would be nice to be able to get the updated versions into
>> linux-firmware.git without much more additional delay and yes, this
>> would be with the File: instead of Licence: entries for the notice.txt
>> files.
>
> The problem of "File:" is that it's more or less intended to be
> installed as the firmware files themselves, i.e. they are installed in
> /lib/firmware/* that can be loaded to the kernel.  Putting such a
> random (document) file there makes me a bit nervous.
>
> We may introduce another tag to list up misc document files
> (e.g. "Doc:" or whatever)?  Distros can pick up them and put to the
> appropriate places in the package, too.

What about "Notice:" which I proposed above? To me "Doc:" sounds like an
optional file, which notice.txt files are not. That's why I prefer
"Notice:".

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

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

^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [PULL] ath11k firmware 20211223
@ 2022-04-13  8:26                 ` Kalle Valo
  0 siblings, 0 replies; 27+ messages in thread
From: Kalle Valo @ 2022-04-13  8:26 UTC (permalink / raw)
  To: Takashi Iwai
  Cc: Jouni Malinen, Josh Boyer, linux-firmware, linux-wireless, ath11k

Takashi Iwai <tiwai@suse.de> writes:

> On Fri, 04 Mar 2022 17:14:30 +0100,
> Jouni Malinen wrote:
>> 
>> On Fri, Mar 04, 2022 at 08:04:26AM -0500, Josh Boyer wrote:
>> > On Wed, Mar 2, 2022 at 12:50 PM Jouni Malinen <j@w1.fi> wrote:
>> > > Would the following in WHENCE work for you?
>> > >
>> > > Driver: ath11k - Qualcomm Technologies 802.11ax chipset support
>> > >
>> > > File: ath11k/IPQ6018/hw1.0/board-2.bin
>> > > ...
>> > > Version: WLAN.HK.2.1.0.1-01238-QCAHKSWPL_SILICONZ-2
>> > > Notice: ath11k/IPQ6018/hw1.0/Notice.txt
>> > 
>> > We'd have to teach copy-firmware.sh what to do with a "Notice:" key.
>> > Is there a reason "File:" wouldn't work?  The intention is to install
>> > the notices alongside the binaries, so that would accomplish it.
>> 
>> Yeah, that sounds reasonable to me. From copying/installing/distribution
>> view point, these can really be handled in the exact same way as the
>> actual firmware binaries in practice.
>> 
>> > > In other words, there would be only a single "License:" line and one
>> > > "Notice:" line for each firmware version? The license itself (i.e.,
>> > > LICENSE.QualcommAtheros_ath10k) is same for all the versions while the
>> > > set of notices (i.e., those notice.txt files) can be different based on
>> > > what is included in the particular build.
>> > 
>> > Yes, that would help.  Would you be able to adjust the existing
>> > entries for ath firmware in the same way?
>> 
>> Yes, I'll work with Kalle to update the existing ath* WLAN cases.
>> 
>> > > > > I'm still working on your other comment about notice.txt, will get back
>> > > > > on that later.
>> > >
>> > > This part about clearly identifying the files should be clear now, but
>> > > it would be good to resolve that part about the notice.txt files in
>> > > general before sending out an updated pull request.
>> > 
>> > Given these were merged in the past, perhaps I'm being overly
>> > pedantic.  If we can mark them as Files or Notices instead of
>> > Licenses, I won't hold it up.  It leaves me slightly confused why
>> > attribution files need to reference agreements with Qualcomm, splatter
>> > Confidential and Proprietary throughout the file, and reference
>> > COPYING and README in reference to GPLv2 when the BSD license was
>> > clearly chosen.  Perhaps that could be cleaned up in the future.
>> 
>> Thanks. We'll remove most of the unnecessary information from the new
>> notice.txt files and that should get rid of many of the potentially
>> confusing parts. If that cleanup leaves something confusing in place,
>> we are open to cleaning these up further in followup patches, but it
>> would be nice to be able to get the updated versions into
>> linux-firmware.git without much more additional delay and yes, this
>> would be with the File: instead of Licence: entries for the notice.txt
>> files.
>
> The problem of "File:" is that it's more or less intended to be
> installed as the firmware files themselves, i.e. they are installed in
> /lib/firmware/* that can be loaded to the kernel.  Putting such a
> random (document) file there makes me a bit nervous.
>
> We may introduce another tag to list up misc document files
> (e.g. "Doc:" or whatever)?  Distros can pick up them and put to the
> appropriate places in the package, too.

What about "Notice:" which I proposed above? To me "Doc:" sounds like an
optional file, which notice.txt files are not. That's why I prefer
"Notice:".

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

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

-- 
ath11k mailing list
ath11k@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/ath11k

^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [PULL] ath11k firmware 20211223
  2022-02-18 15:00   ` Sven Eckelmann
@ 2022-04-13  8:28     ` Kalle Valo
  -1 siblings, 0 replies; 27+ messages in thread
From: Kalle Valo @ 2022-04-13  8:28 UTC (permalink / raw)
  To: Sven Eckelmann; +Cc: linux-firmware, ath11k, linux-wireless

Sven Eckelmann <sven@narfation.org> writes:

> On Thursday, 23 December 2021 15:03:04 CET Kalle Valo wrote:
>>  ath11k/QCN9074/hw1.0/Notice.txt  |  802 ++++++++++++++++++++
>>  ath11k/QCN9074/hw1.0/amss.bin    |  Bin 0 -> 11897676 bytes
>>  ath11k/QCN9074/hw1.0/m3.bin      |  Bin 0 -> 340108 bytes
>
> This pull adds the firmware for QCN9074 but not the board-2.bin. As result, 
> the initialization of the ath11k PHY will fail.

Good point, I missed that. My plan is to include both QCN9074 firmware
and board-2.bin in my next pull request.

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

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

^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [PULL] ath11k firmware 20211223
@ 2022-04-13  8:28     ` Kalle Valo
  0 siblings, 0 replies; 27+ messages in thread
From: Kalle Valo @ 2022-04-13  8:28 UTC (permalink / raw)
  To: Sven Eckelmann; +Cc: linux-firmware, ath11k, linux-wireless

Sven Eckelmann <sven@narfation.org> writes:

> On Thursday, 23 December 2021 15:03:04 CET Kalle Valo wrote:
>>  ath11k/QCN9074/hw1.0/Notice.txt  |  802 ++++++++++++++++++++
>>  ath11k/QCN9074/hw1.0/amss.bin    |  Bin 0 -> 11897676 bytes
>>  ath11k/QCN9074/hw1.0/m3.bin      |  Bin 0 -> 340108 bytes
>
> This pull adds the firmware for QCN9074 but not the board-2.bin. As result, 
> the initialization of the ath11k PHY will fail.

Good point, I missed that. My plan is to include both QCN9074 firmware
and board-2.bin in my next pull request.

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

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

-- 
ath11k mailing list
ath11k@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/ath11k

^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [PULL] ath11k firmware 20211223
  2022-04-13  8:26                 ` Kalle Valo
@ 2022-04-13  8:40                   ` Takashi Iwai
  -1 siblings, 0 replies; 27+ messages in thread
From: Takashi Iwai @ 2022-04-13  8:40 UTC (permalink / raw)
  To: Kalle Valo
  Cc: Takashi Iwai, Jouni Malinen, Josh Boyer, linux-firmware,
	linux-wireless, ath11k

On Wed, 13 Apr 2022 10:26:43 +0200,
Kalle Valo wrote:
> 
> Takashi Iwai <tiwai@suse.de> writes:
> 
> > On Fri, 04 Mar 2022 17:14:30 +0100,
> > Jouni Malinen wrote:
> >> 
> >> On Fri, Mar 04, 2022 at 08:04:26AM -0500, Josh Boyer wrote:
> >> > On Wed, Mar 2, 2022 at 12:50 PM Jouni Malinen <j@w1.fi> wrote:
> >> > > Would the following in WHENCE work for you?
> >> > >
> >> > > Driver: ath11k - Qualcomm Technologies 802.11ax chipset support
> >> > >
> >> > > File: ath11k/IPQ6018/hw1.0/board-2.bin
> >> > > ...
> >> > > Version: WLAN.HK.2.1.0.1-01238-QCAHKSWPL_SILICONZ-2
> >> > > Notice: ath11k/IPQ6018/hw1.0/Notice.txt
> >> > 
> >> > We'd have to teach copy-firmware.sh what to do with a "Notice:" key.
> >> > Is there a reason "File:" wouldn't work?  The intention is to install
> >> > the notices alongside the binaries, so that would accomplish it.
> >> 
> >> Yeah, that sounds reasonable to me. From copying/installing/distribution
> >> view point, these can really be handled in the exact same way as the
> >> actual firmware binaries in practice.
> >> 
> >> > > In other words, there would be only a single "License:" line and one
> >> > > "Notice:" line for each firmware version? The license itself (i.e.,
> >> > > LICENSE.QualcommAtheros_ath10k) is same for all the versions while the
> >> > > set of notices (i.e., those notice.txt files) can be different based on
> >> > > what is included in the particular build.
> >> > 
> >> > Yes, that would help.  Would you be able to adjust the existing
> >> > entries for ath firmware in the same way?
> >> 
> >> Yes, I'll work with Kalle to update the existing ath* WLAN cases.
> >> 
> >> > > > > I'm still working on your other comment about notice.txt, will get back
> >> > > > > on that later.
> >> > >
> >> > > This part about clearly identifying the files should be clear now, but
> >> > > it would be good to resolve that part about the notice.txt files in
> >> > > general before sending out an updated pull request.
> >> > 
> >> > Given these were merged in the past, perhaps I'm being overly
> >> > pedantic.  If we can mark them as Files or Notices instead of
> >> > Licenses, I won't hold it up.  It leaves me slightly confused why
> >> > attribution files need to reference agreements with Qualcomm, splatter
> >> > Confidential and Proprietary throughout the file, and reference
> >> > COPYING and README in reference to GPLv2 when the BSD license was
> >> > clearly chosen.  Perhaps that could be cleaned up in the future.
> >> 
> >> Thanks. We'll remove most of the unnecessary information from the new
> >> notice.txt files and that should get rid of many of the potentially
> >> confusing parts. If that cleanup leaves something confusing in place,
> >> we are open to cleaning these up further in followup patches, but it
> >> would be nice to be able to get the updated versions into
> >> linux-firmware.git without much more additional delay and yes, this
> >> would be with the File: instead of Licence: entries for the notice.txt
> >> files.
> >
> > The problem of "File:" is that it's more or less intended to be
> > installed as the firmware files themselves, i.e. they are installed in
> > /lib/firmware/* that can be loaded to the kernel.  Putting such a
> > random (document) file there makes me a bit nervous.
> >
> > We may introduce another tag to list up misc document files
> > (e.g. "Doc:" or whatever)?  Distros can pick up them and put to the
> > appropriate places in the package, too.
> 
> What about "Notice:" which I proposed above? To me "Doc:" sounds like an
> optional file, which notice.txt files are not. That's why I prefer
> "Notice:".

Sure, it'd work, too.  Only a patch is missing :)


thanks,

Takashi

^ permalink raw reply	[flat|nested] 27+ messages in thread

* Re: [PULL] ath11k firmware 20211223
@ 2022-04-13  8:40                   ` Takashi Iwai
  0 siblings, 0 replies; 27+ messages in thread
From: Takashi Iwai @ 2022-04-13  8:40 UTC (permalink / raw)
  To: Kalle Valo
  Cc: Takashi Iwai, Jouni Malinen, Josh Boyer, linux-firmware,
	linux-wireless, ath11k

On Wed, 13 Apr 2022 10:26:43 +0200,
Kalle Valo wrote:
> 
> Takashi Iwai <tiwai@suse.de> writes:
> 
> > On Fri, 04 Mar 2022 17:14:30 +0100,
> > Jouni Malinen wrote:
> >> 
> >> On Fri, Mar 04, 2022 at 08:04:26AM -0500, Josh Boyer wrote:
> >> > On Wed, Mar 2, 2022 at 12:50 PM Jouni Malinen <j@w1.fi> wrote:
> >> > > Would the following in WHENCE work for you?
> >> > >
> >> > > Driver: ath11k - Qualcomm Technologies 802.11ax chipset support
> >> > >
> >> > > File: ath11k/IPQ6018/hw1.0/board-2.bin
> >> > > ...
> >> > > Version: WLAN.HK.2.1.0.1-01238-QCAHKSWPL_SILICONZ-2
> >> > > Notice: ath11k/IPQ6018/hw1.0/Notice.txt
> >> > 
> >> > We'd have to teach copy-firmware.sh what to do with a "Notice:" key.
> >> > Is there a reason "File:" wouldn't work?  The intention is to install
> >> > the notices alongside the binaries, so that would accomplish it.
> >> 
> >> Yeah, that sounds reasonable to me. From copying/installing/distribution
> >> view point, these can really be handled in the exact same way as the
> >> actual firmware binaries in practice.
> >> 
> >> > > In other words, there would be only a single "License:" line and one
> >> > > "Notice:" line for each firmware version? The license itself (i.e.,
> >> > > LICENSE.QualcommAtheros_ath10k) is same for all the versions while the
> >> > > set of notices (i.e., those notice.txt files) can be different based on
> >> > > what is included in the particular build.
> >> > 
> >> > Yes, that would help.  Would you be able to adjust the existing
> >> > entries for ath firmware in the same way?
> >> 
> >> Yes, I'll work with Kalle to update the existing ath* WLAN cases.
> >> 
> >> > > > > I'm still working on your other comment about notice.txt, will get back
> >> > > > > on that later.
> >> > >
> >> > > This part about clearly identifying the files should be clear now, but
> >> > > it would be good to resolve that part about the notice.txt files in
> >> > > general before sending out an updated pull request.
> >> > 
> >> > Given these were merged in the past, perhaps I'm being overly
> >> > pedantic.  If we can mark them as Files or Notices instead of
> >> > Licenses, I won't hold it up.  It leaves me slightly confused why
> >> > attribution files need to reference agreements with Qualcomm, splatter
> >> > Confidential and Proprietary throughout the file, and reference
> >> > COPYING and README in reference to GPLv2 when the BSD license was
> >> > clearly chosen.  Perhaps that could be cleaned up in the future.
> >> 
> >> Thanks. We'll remove most of the unnecessary information from the new
> >> notice.txt files and that should get rid of many of the potentially
> >> confusing parts. If that cleanup leaves something confusing in place,
> >> we are open to cleaning these up further in followup patches, but it
> >> would be nice to be able to get the updated versions into
> >> linux-firmware.git without much more additional delay and yes, this
> >> would be with the File: instead of Licence: entries for the notice.txt
> >> files.
> >
> > The problem of "File:" is that it's more or less intended to be
> > installed as the firmware files themselves, i.e. they are installed in
> > /lib/firmware/* that can be loaded to the kernel.  Putting such a
> > random (document) file there makes me a bit nervous.
> >
> > We may introduce another tag to list up misc document files
> > (e.g. "Doc:" or whatever)?  Distros can pick up them and put to the
> > appropriate places in the package, too.
> 
> What about "Notice:" which I proposed above? To me "Doc:" sounds like an
> optional file, which notice.txt files are not. That's why I prefer
> "Notice:".

Sure, it'd work, too.  Only a patch is missing :)


thanks,

Takashi

-- 
ath11k mailing list
ath11k@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/ath11k

^ permalink raw reply	[flat|nested] 27+ messages in thread

end of thread, other threads:[~2022-04-13  8:41 UTC | newest]

Thread overview: 27+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-12-23 14:03 [PULL] ath11k firmware 20211223 Kalle Valo
2021-12-23 14:03 ` Kalle Valo
2021-12-23 17:24 ` Mark Herbert
2021-12-29 13:41 ` Josh Boyer
2021-12-29 13:41   ` Josh Boyer
2022-01-10 15:19   ` Kalle Valo
2022-01-10 15:19     ` Kalle Valo
2022-02-10 10:45   ` Kalle Valo
2022-02-10 10:45     ` Kalle Valo
2022-02-18 15:55     ` Josh Boyer
2022-02-18 15:55       ` Josh Boyer
2022-03-02 17:50       ` Jouni Malinen
2022-03-02 17:50         ` Jouni Malinen
2022-03-04 13:04         ` Josh Boyer
2022-03-04 13:04           ` Josh Boyer
2022-03-04 16:14           ` Jouni Malinen
2022-03-04 16:14             ` Jouni Malinen
2022-03-23  8:06             ` Takashi Iwai
2022-03-23  8:06               ` Takashi Iwai
2022-04-13  8:26               ` Kalle Valo
2022-04-13  8:26                 ` Kalle Valo
2022-04-13  8:40                 ` Takashi Iwai
2022-04-13  8:40                   ` Takashi Iwai
2022-02-18 15:00 ` Sven Eckelmann
2022-02-18 15:00   ` Sven Eckelmann
2022-04-13  8:28   ` Kalle Valo
2022-04-13  8:28     ` Kalle Valo

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.