On 2/13/20 3:25 PM, Guillermo Rodriguez Garcia wrote: > El jue., 13 feb. 2020 a las 14:30, Ahmad Fatoum > (<a.fat...@pengutronix.de>) escribió:
>>>> +# >>>> ---------------------------------------------------------------------------- >>>> +# Install >>>> +# >>>> ---------------------------------------------------------------------------- >>>> + >>>> +$(STATEDIR)/tf-a.install: >>>> + @$(call targetinfo) >>>> +ifeq ($(TF_A_ARTIFACTS_SRC),) >>>> + $(warning TF_A_ARTIFACTS is empty. nothing to install.) >>>> +else >>>> + @install -m644 -D \ >>>> + >>>> --target-directory=$(PTXCONF_SYSROOT_TARGET)/usr/lib/firmware \ >>> >>> This doesn't look right. >>> Shouldn't the install stage install things to the package install >>> directory only? >>> And, in case you want to install something somewhere else, shouldn't >>> the actual target directory at least be configurable? >>> For example there is no /usr/lib/firmware in my platform. >> >> That's only in the sysroot for use by other rules (e.g. barebox embedding >> TF-A >> on some i.MX8). It's not installed in the target rootfs. > > Ok but that's specific to that particular configuration, and the > choice of directory also seems to be arbitrary. You got to standardize on something. If this is good, a $(PTXCONF_SYSROOT_FIRMWARE) might be a good thing to agree on. > Does this belong in a generic rules file ? How would that look like? Cheers Ahmad -- Pengutronix e.K. | | Steuerwalder Str. 21 | https://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ ptxdist mailing list ptxdist@pengutronix.de