El jue., 13 feb. 2020 a las 15:44, Ahmad Fatoum (<a.fat...@pengutronix.de>) escribió: > > On 2/13/20 3:35 PM, Guillermo Rodriguez Garcia wrote: > > El jue., 13 feb. 2020 a las 15:32, Ahmad Fatoum > > (<a.fat...@pengutronix.de>) escribió: > >> > >> On 2/13/20 3:28 PM, Ahmad Fatoum wrote: > >>> 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? > >> > >> Ah, you mean this is too specific that it shouldn't be in tf-a.make? > > > > Yes. > > > >> > >> Well, Some Zynqs do this as well. If you are embedding OPTEE in TF-A, you > >> will need > >> to save optee somewhere too. It's not a niche use case. The split between > >> install > > > > For cases like this (optee) you're going to need to amend tf-a.in > > anyway because you need to add dependencies (you mentioned this > > yourself in a previous email) > > > >> and target-install is just because so many vendors have their own idea of > >> where the TF-A > >> comes from. If most of these can be made to work by installing the file to > >> sysroot along > >> with the IMAGEDIR, so why not do so? > > > > If it really covers "most of these" use cases, then I guess it may make > > sense. > > if it's consumed by image rules, it's in IMAGEDIR. If it's consumed by > "normal" rules, > it's in the sysroot at a fixed place. Not sure if there is another use case I > am missing. > > > > > Still -- shouldn't it at least be optional? > > Why should the .install stage be behind an option, but the .targetinstall not? > Should there be options for enabling/disabling each one of them? > > And why? Just to save a few hundred kilobytes extra in the _build_ directory? > Doesn't sound like a good enough reason to me.
OK this makes sense. But, you should at least make sure $(PTXCONF_SYSROOT_TARGET)/usr/lib/firmware exists before trying to copy anything into it. Guillermo > > Cheers > Ahmad > > > > > Guillermo > > > >> > >> Cheers > >> Ahmad > >> > >>> > >>> > >>> 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 | > > > > > > > > > -- > 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 | -- Guillermo Rodriguez Garcia guille.rodrig...@gmail.com _______________________________________________ ptxdist mailing list ptxdist@pengutronix.de