Re: [ptxdist] [PATCH v2] linux-firmware: Add new package. Version 20211216

[Denis OSTERLAND-HEIM <denis.osterland@diehl.com>]

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

> Hi,

> >> +
> >> +LINUX_FIRMWARE_SELECTED_LICENSES := $(call remove_quotes,
> >> +$(PTXCONF_LINUX_FIRMWARE_SELECTED_LICENSES))
> >> +ifneq ($(LINUX_FIRMWARE_SELECTED_LICENSES),)
> >> +LINUX_FIRMWARE_LICENSE_FILES := $(filter $(addsuffix %,$(addprefix 
> >> +file://,$(LINUX_FIRMWARE_SELECTED_LICENSES))),$(LINUX_FIRMWARE_LICEN
> >> +SE_
> >> +FILES))
> >> +endif
> > In my opinion LINUX_FIRMWARE_LICENSE_FILES should be 
> > LINUX_FIRMWARE_ALL_LICENSE_FILES and LINUX_FIRMWARE_SELECTED_LICENSES 
> > should be LINUX_FIRMWARE_LICENSE_FILES, so that the license report 
> > only picks up selected licenses.
> > 
>
> The point was that the filter should filter the selected ones from the
full list, to make sure that they do exist. Do you mean that it does not
work? Does the > license report pick licenses in some other way? When I do a
license check it seems to pick the ones I've selected.
>
Sorry, I got the code wrong. You are right.

Regards, Denis 

[-- Attachment #1.2: smime.p7s --]
[-- Type: application/x-pkcs7-signature, Size: 3908 bytes --]

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

_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de
To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de