I don't usually weigh in on makefile issues, but if Stephen says it's needed, let's get it in.
-----Original Message----- From: Tom Rini <tr...@konsulko.com> Sent: Thursday, July 26, 2018 12:40 PM To: Stephen Warren <swar...@wwwdotorg.org> Cc: Simon Glass <s...@chromium.org>; U-Boot Mailing List <u-boot@lists.denx.de>; Stephen Warren <swar...@nvidia.com>; Tom Warren <twar...@nvidia.com> Subject: Re: [PATCH] tegra: Indicate that binman makes all three output files On Thu, Jul 26, 2018 at 01:37:31PM -0600, Stephen Warren wrote: > On 07/23/2018 09:27 AM, Stephen Warren wrote: > >On 07/20/2018 07:49 PM, Simon Glass wrote: > >>Use GNU make pattern rules to indicate that a single run of binman > >>produces all three Tegra output files. The avoids make running > >>binman three times (perhaps in parallel) and those instances > >>inteferring with each other. > >> > >>See http://patchwork.ozlabs.org/patch/944611/ for the bug report. > > > >Tested-by: Stephen Warren <swar...@nvidia.com> > > > >(This make feature is very odd; specifying n targets without a % > >means they're separate rules, but specifying n targets with a % means > >it's a single rule that generates n targets. Nice inconsistency on > >make's part!) > > Any chance we can apply this ASAP to the main u-boot.git master branch? > Pretty much all my Jenkins builds are failing some random subset of > boards because of this issue. Thanks. If Tom W is fine, I'm happy to pick it up directly. -- Tom ----------------------------------------------------------------------------------- This email message is for the sole use of the intended recipient(s) and may contain confidential information. Any unauthorized review, use, disclosure or distribution is prohibited. If you are not the intended recipient, please contact the sender by reply email and destroy all copies of the original message. ----------------------------------------------------------------------------------- _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
