> 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