diff mbox series

linux-firmware: upgrade 20250109 -> 20250211

Message ID 20250211185151.995828-1-quic_vpuar@quicinc.com
State Accepted, archived
Commit 6148a282509106879388138fe682721d07968452
Headers show
Series linux-firmware: upgrade 20250109 -> 20250211 | expand

Commit Message

Vivek Puar Feb. 11, 2025, 6:51 p.m. UTC
License-Update: additional files

Signed-off-by: Vivek Puar <quic_vpuar@quicinc.com>
Cc: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> 
---
 ...{linux-firmware_20250109.bb => linux-firmware_20250211.bb} | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
 rename meta/recipes-kernel/linux-firmware/{linux-firmware_20250109.bb => linux-firmware_20250211.bb} (99%)

Comments

Dmitry Baryshkov Feb. 11, 2025, 9:23 p.m. UTC | #1
On Tue, 11 Feb 2025 at 20:52, Vivek Puar <quic_vpuar@quicinc.com> wrote:

> License-Update: additional files

Not only "additional files", there is a new Qualcomm licence, which is
not caught by this patch, NAK

While taking care of the new Qualcomm licence, please also note new
ath*, qca and qcom/ firmware. Please extend corresponding split
packages.

>
> Signed-off-by: Vivek Puar <quic_vpuar@quicinc.com>
> Cc: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> ---
>  ...{linux-firmware_20250109.bb => linux-firmware_20250211.bb} | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>  rename meta/recipes-kernel/linux-firmware/{linux-firmware_20250109.bb => linux-firmware_20250211.bb} (99%)
>
> diff --git a/meta/recipes-kernel/linux-firmware/linux-firmware_20250109.bb b/meta/recipes-kernel/linux-firmware/linux-firmware_20250211.bb
> similarity index 99%
> rename from meta/recipes-kernel/linux-firmware/linux-firmware_20250109.bb
> rename to meta/recipes-kernel/linux-firmware/linux-firmware_20250211.bb
> index b932300742..6cadf99d47 100644
> --- a/meta/recipes-kernel/linux-firmware/linux-firmware_20250109.bb
> +++ b/meta/recipes-kernel/linux-firmware/linux-firmware_20250211.bb
> @@ -158,7 +158,7 @@ LIC_FILES_CHKSUM = "file://LICENCE.Abilis;md5=b5ee3f410780e56711ad48eadc22b8bc \
>                      "
>  # WHENCE checksum is defined separately to ease overriding it if
>  # class-devupstream is selected.
> -WHENCE_CHKSUM  = "c90a25ccbd027eb8d61b2943ab68447c"
> +WHENCE_CHKSUM  = "65c2919e6511a0d348a009ac419375b7"
>
>  # These are not common licenses, set NO_GENERIC_LICENSE for them
>  # so that the license files will be copied from fetched source
> @@ -247,7 +247,7 @@ SRC_URI:class-devupstream = "git://git.kernel.org/pub/scm/linux/kernel/git/firmw
>  # Pin this to the 20220509 release, override this in local.conf
>  SRCREV:class-devupstream ?= "b19cbdca78ab2adfd210c91be15a22568e8b8cae"
>
> -SRC_URI[sha256sum] = "a12cb38f4793394bf56c6702be29b6e4f09deca0714943bba98a492030651570"
> +SRC_URI[sha256sum] = "2de1345897bf839d532c5de0fdb348770ca2a5f4edfb21971582597abb45297d"
>
>  inherit allarch
Vivek Puar Feb. 12, 2025, 4:13 a.m. UTC | #2
Sure will cover all and send the patch.
I was going to upload a new patch covering them.


On 2/12/2025 2:53 AM, Dmitry Baryshkov via lists.openembedded.org wrote:
> On Tue, 11 Feb 2025 at 20:52, Vivek Puar <quic_vpuar@quicinc.com> wrote:
>
>> License-Update: additional files
> Not only "additional files", there is a new Qualcomm licence, which is
> not caught by this patch, NAK
>
> While taking care of the new Qualcomm licence, please also note new
> ath*, qca and qcom/ firmware. Please extend corresponding split
> packages.
>
>> Signed-off-by: Vivek Puar <quic_vpuar@quicinc.com>
>> Cc: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
>> ---
>>   ...{linux-firmware_20250109.bb => linux-firmware_20250211.bb} | 4 ++--
>>   1 file changed, 2 insertions(+), 2 deletions(-)
>>   rename meta/recipes-kernel/linux-firmware/{linux-firmware_20250109.bb => linux-firmware_20250211.bb} (99%)
>>
>> diff --git a/meta/recipes-kernel/linux-firmware/linux-firmware_20250109.bb b/meta/recipes-kernel/linux-firmware/linux-firmware_20250211.bb
>> similarity index 99%
>> rename from meta/recipes-kernel/linux-firmware/linux-firmware_20250109.bb
>> rename to meta/recipes-kernel/linux-firmware/linux-firmware_20250211.bb
>> index b932300742..6cadf99d47 100644
>> --- a/meta/recipes-kernel/linux-firmware/linux-firmware_20250109.bb
>> +++ b/meta/recipes-kernel/linux-firmware/linux-firmware_20250211.bb
>> @@ -158,7 +158,7 @@ LIC_FILES_CHKSUM = "file://LICENCE.Abilis;md5=b5ee3f410780e56711ad48eadc22b8bc \
>>                       "
>>   # WHENCE checksum is defined separately to ease overriding it if
>>   # class-devupstream is selected.
>> -WHENCE_CHKSUM  = "c90a25ccbd027eb8d61b2943ab68447c"
>> +WHENCE_CHKSUM  = "65c2919e6511a0d348a009ac419375b7"
>>
>>   # These are not common licenses, set NO_GENERIC_LICENSE for them
>>   # so that the license files will be copied from fetched source
>> @@ -247,7 +247,7 @@ SRC_URI:class-devupstream = "git://git.kernel.org/pub/scm/linux/kernel/git/firmw
>>   # Pin this to the 20220509 release, override this in local.conf
>>   SRCREV:class-devupstream ?= "b19cbdca78ab2adfd210c91be15a22568e8b8cae"
>>
>> -SRC_URI[sha256sum] = "a12cb38f4793394bf56c6702be29b6e4f09deca0714943bba98a492030651570"
>> +SRC_URI[sha256sum] = "2de1345897bf839d532c5de0fdb348770ca2a5f4edfb21971582597abb45297d"
>>
>>   inherit allarch
Regards,
Vivek
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#211186): https://lists.openembedded.org/g/openembedded-core/message/211186
> Mute This Topic: https://lists.openembedded.org/mt/111128390/8903276
> Group Owner: openembedded-core+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [quic_vpuar@quicinc.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
Vivek Puar Feb. 12, 2025, 7:10 a.m. UTC | #3
On 2/12/2025 2:53 AM, Dmitry Baryshkov via lists.openembedded.org wrote:
> On Tue, 11 Feb 2025 at 20:52, Vivek Puar<quic_vpuar@quicinc.com> wrote:
>
>> License-Update: additional files
> Not only "additional files", there is a new Qualcomm licence, which is
> not caught by this patch, NAK
>
> While taking care of the new Qualcomm licence, please also note new
> ath*, qca and qcom/ firmware. Please extend corresponding split
> packages.

1) Now as qca firmware for WCN3950 chips comes from the RB1 firmware 
release through Linaro,

     a) so does it mean we will be adding "Firmware-qcom" to 
"*LICENSE:${PN}-qca*"?
     b) or do we need to split  ${PN}-qca into multiple packages?
     c) or the firmware needs to be added as separate package, for 
example: ${PN}-qca-wcn3950, but in this case how to exclude the firmware 
from ${PN}-qca?

2) ath11k depends on "*ath10k-license*", so is there any split or change 
required for the new QCA6698AQ firmware?

>> Signed-off-by: Vivek Puar<quic_vpuar@quicinc.com>
>> Cc: Dmitry Baryshkov<dmitry.baryshkov@linaro.org>
>> ---
>>   ...{linux-firmware_20250109.bb => linux-firmware_20250211.bb} | 4 ++--
>>   1 file changed, 2 insertions(+), 2 deletions(-)
>>   rename meta/recipes-kernel/linux-firmware/{linux-firmware_20250109.bb => linux-firmware_20250211.bb} (99%)
>>
>> diff --git a/meta/recipes-kernel/linux-firmware/linux-firmware_20250109.bb b/meta/recipes-kernel/linux-firmware/linux-firmware_20250211.bb
>> similarity index 99%
>> rename from meta/recipes-kernel/linux-firmware/linux-firmware_20250109.bb
>> rename to meta/recipes-kernel/linux-firmware/linux-firmware_20250211.bb
>> index b932300742..6cadf99d47 100644
>> --- a/meta/recipes-kernel/linux-firmware/linux-firmware_20250109.bb
>> +++ b/meta/recipes-kernel/linux-firmware/linux-firmware_20250211.bb
>> @@ -158,7 +158,7 @@ LIC_FILES_CHKSUM ="file://LICENCE.Abilis;md5=b5ee3f410780e56711ad48eadc22b8bc \ "
>>   # WHENCE checksum is defined separately to ease overriding it if
>>   # class-devupstream is selected.
>> -WHENCE_CHKSUM  = "c90a25ccbd027eb8d61b2943ab68447c"
>> +WHENCE_CHKSUM  = "65c2919e6511a0d348a009ac419375b7"
>>
>>   # These are not common licenses, set NO_GENERIC_LICENSE for them
>>   # so that the license files will be copied from fetched source
>> @@ -247,7 +247,7 @@ SRC_URI:class-devupstream = "git://git.kernel.org/pub/scm/linux/kernel/git/firmw
>>   # Pin this to the 20220509 release, override this in local.conf
>>   SRCREV:class-devupstream ?= "b19cbdca78ab2adfd210c91be15a22568e8b8cae"
>>
>> -SRC_URI[sha256sum] = "a12cb38f4793394bf56c6702be29b6e4f09deca0714943bba98a492030651570"
>> +SRC_URI[sha256sum] = "2de1345897bf839d532c5de0fdb348770ca2a5f4edfb21971582597abb45297d"
>>
>>   inherit allarch
Regards,
Vivek
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#211186):https://lists.openembedded.org/g/openembedded-core/message/211186
> Mute This Topic:https://lists.openembedded.org/mt/111128390/8903276
> Group Owner:openembedded-core+owner@lists.openembedded.org
> Unsubscribe:https://lists.openembedded.org/g/openembedded-core/unsub [quic_vpuar@quicinc.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
Dmitry Baryshkov Feb. 12, 2025, 11:57 a.m. UTC | #4
On Wed, Feb 12, 2025 at 12:40:44PM +0530, Vivek Puar wrote:
> 
> On 2/12/2025 2:53 AM, Dmitry Baryshkov via lists.openembedded.org wrote:
> > On Tue, 11 Feb 2025 at 20:52, Vivek Puar<quic_vpuar@quicinc.com> wrote:
> > 
> > > License-Update: additional files
> > Not only "additional files", there is a new Qualcomm licence, which is
> > not caught by this patch, NAK
> > 
> > While taking care of the new Qualcomm licence, please also note new
> > ath*, qca and qcom/ firmware. Please extend corresponding split
> > packages.
> 
> 1) Now as qca firmware for WCN3950 chips comes from the RB1 firmware release
> through Linaro,
> 
> ��� a) so does it mean we will be adding "Firmware-qcom" to
> "*LICENSE:${PN}-qca*"?
> ��� b) or do we need to split� ${PN}-qca into multiple packages?
> ��� c) or the firmware needs to be added as separate package, for example:
> ${PN}-qca-wcn3950, but in this case how to exclude the firmware from
> ${PN}-qca?

If I haven't seen Git's index line in your email, I'd have questioned if
you are working on an older branch or just not up-to-date tip. However
your patch proves that you've seen the commit that was merged yesterday.
And as you've seen it, you could have noticed that Marc Ferland has
already split the qca package (and that change was comitted more than
two weeks ago).

> 2) ath11k depends on "*ath10k-license*", so is there any split or change
> required for the new QCA6698AQ firmware?

ath11k packages are already split. Please follow the established style.

> > > Signed-off-by: Vivek Puar<quic_vpuar@quicinc.com>
> > > Cc: Dmitry Baryshkov<dmitry.baryshkov@linaro.org>
> > > ---
> > >   ...{linux-firmware_20250109.bb => linux-firmware_20250211.bb} | 4 ++--
> > >   1 file changed, 2 insertions(+), 2 deletions(-)
> > >   rename meta/recipes-kernel/linux-firmware/{linux-firmware_20250109.bb => linux-firmware_20250211.bb} (99%)
> > > 
> > > diff --git a/meta/recipes-kernel/linux-firmware/linux-firmware_20250109.bb b/meta/recipes-kernel/linux-firmware/linux-firmware_20250211.bb
> > > similarity index 99%
> > > rename from meta/recipes-kernel/linux-firmware/linux-firmware_20250109.bb
> > > rename to meta/recipes-kernel/linux-firmware/linux-firmware_20250211.bb
> > > index b932300742..6cadf99d47 100644
> > > --- a/meta/recipes-kernel/linux-firmware/linux-firmware_20250109.bb
> > > +++ b/meta/recipes-kernel/linux-firmware/linux-firmware_20250211.bb
> > > @@ -158,7 +158,7 @@ LIC_FILES_CHKSUM ="file://LICENCE.Abilis;md5=b5ee3f410780e56711ad48eadc22b8bc \ "
> > >   # WHENCE checksum is defined separately to ease overriding it if
> > >   # class-devupstream is selected.
> > > -WHENCE_CHKSUM  = "c90a25ccbd027eb8d61b2943ab68447c"
> > > +WHENCE_CHKSUM  = "65c2919e6511a0d348a009ac419375b7"
> > > 
> > >   # These are not common licenses, set NO_GENERIC_LICENSE for them
> > >   # so that the license files will be copied from fetched source
> > > @@ -247,7 +247,7 @@ SRC_URI:class-devupstream = "git://git.kernel.org/pub/scm/linux/kernel/git/firmw
> > >   # Pin this to the 20220509 release, override this in local.conf
> > >   SRCREV:class-devupstream ?= "b19cbdca78ab2adfd210c91be15a22568e8b8cae"
> > > 
> > > -SRC_URI[sha256sum] = "a12cb38f4793394bf56c6702be29b6e4f09deca0714943bba98a492030651570"
> > > +SRC_URI[sha256sum] = "2de1345897bf839d532c5de0fdb348770ca2a5f4edfb21971582597abb45297d"
> > > 
> > >   inherit allarch
> Regards,
> Vivek
> >
diff mbox series

Patch

diff --git a/meta/recipes-kernel/linux-firmware/linux-firmware_20250109.bb b/meta/recipes-kernel/linux-firmware/linux-firmware_20250211.bb
similarity index 99%
rename from meta/recipes-kernel/linux-firmware/linux-firmware_20250109.bb
rename to meta/recipes-kernel/linux-firmware/linux-firmware_20250211.bb
index b932300742..6cadf99d47 100644
--- a/meta/recipes-kernel/linux-firmware/linux-firmware_20250109.bb
+++ b/meta/recipes-kernel/linux-firmware/linux-firmware_20250211.bb
@@ -158,7 +158,7 @@  LIC_FILES_CHKSUM = "file://LICENCE.Abilis;md5=b5ee3f410780e56711ad48eadc22b8bc \
                     "
 # WHENCE checksum is defined separately to ease overriding it if
 # class-devupstream is selected.
-WHENCE_CHKSUM  = "c90a25ccbd027eb8d61b2943ab68447c"
+WHENCE_CHKSUM  = "65c2919e6511a0d348a009ac419375b7"
 
 # These are not common licenses, set NO_GENERIC_LICENSE for them
 # so that the license files will be copied from fetched source
@@ -247,7 +247,7 @@  SRC_URI:class-devupstream = "git://git.kernel.org/pub/scm/linux/kernel/git/firmw
 # Pin this to the 20220509 release, override this in local.conf
 SRCREV:class-devupstream ?= "b19cbdca78ab2adfd210c91be15a22568e8b8cae"
 
-SRC_URI[sha256sum] = "a12cb38f4793394bf56c6702be29b6e4f09deca0714943bba98a492030651570"
+SRC_URI[sha256sum] = "2de1345897bf839d532c5de0fdb348770ca2a5f4edfb21971582597abb45297d"
 
 inherit allarch