Re: [PATCH v2 2/2] Makefile: Only build dtc if needed

2020-04-30 Thread Simon Glass
Hi Tom, On Thu, 30 Apr 2020 at 09:05, Tom Rini wrote: > On Tue, Apr 28, 2020 at 04:40:47PM -0600, Simon Glass wrote: > > Hi Tom, > > > > On Tue, 28 Apr 2020 at 09:52, Tom Rini wrote: > > > > > > On Tue, Apr 28, 2020 at 09:41:14AM -0600, Simon Glass wrote: > > > > Hi Tom. > > > > > > > > On

Re: [PATCH v2 2/2] Makefile: Only build dtc if needed

2020-04-30 Thread Tom Rini
On Tue, Apr 28, 2020 at 04:40:47PM -0600, Simon Glass wrote: > Hi Tom, > > On Tue, 28 Apr 2020 at 09:52, Tom Rini wrote: > > > > On Tue, Apr 28, 2020 at 09:41:14AM -0600, Simon Glass wrote: > > > Hi Tom. > > > > > > On Tue, 28 Apr 2020 at 08:19, Tom Rini wrote: > > > > > > > > On Mon, Apr 27,

Re: [PATCH v2 2/2] Makefile: Only build dtc if needed

2020-04-28 Thread Simon Glass
Hi Tom, On Tue, 28 Apr 2020 at 09:52, Tom Rini wrote: > > On Tue, Apr 28, 2020 at 09:41:14AM -0600, Simon Glass wrote: > > Hi Tom. > > > > On Tue, 28 Apr 2020 at 08:19, Tom Rini wrote: > > > > > > On Mon, Apr 27, 2020 at 04:10:06PM -0700, Vagrant Cascadian wrote: > > > > On 2020-04-27, Simon

Re: [PATCH v2 2/2] Makefile: Only build dtc if needed

2020-04-28 Thread Tom Rini
On Tue, Apr 28, 2020 at 09:41:14AM -0600, Simon Glass wrote: > Hi Tom. > > On Tue, 28 Apr 2020 at 08:19, Tom Rini wrote: > > > > On Mon, Apr 27, 2020 at 04:10:06PM -0700, Vagrant Cascadian wrote: > > > On 2020-04-27, Simon Glass wrote: > > > > On Sun, 26 Apr 2020 at 18:58, Heinrich Schuchardt

Re: [PATCH v2 2/2] Makefile: Only build dtc if needed

2020-04-28 Thread Simon Glass
Hi Tom. On Tue, 28 Apr 2020 at 08:19, Tom Rini wrote: > > On Mon, Apr 27, 2020 at 04:10:06PM -0700, Vagrant Cascadian wrote: > > On 2020-04-27, Simon Glass wrote: > > > On Sun, 26 Apr 2020 at 18:58, Heinrich Schuchardt > > > wrote: > > >> > > >> Am April 27, 2020 12:29:29 AM UTC schrieb Simon

Re: [PATCH v2 2/2] Makefile: Only build dtc if needed

2020-04-28 Thread Tom Rini
On Mon, Apr 27, 2020 at 04:10:06PM -0700, Vagrant Cascadian wrote: > On 2020-04-27, Simon Glass wrote: > > On Sun, 26 Apr 2020 at 18:58, Heinrich Schuchardt > > wrote: > >> > >> Am April 27, 2020 12:29:29 AM UTC schrieb Simon Glass : > >> >At present U-Boot always builds dtc if CONFIG_OF_CONTROL

Re: [PATCH v2 2/2] Makefile: Only build dtc if needed

2020-04-27 Thread Vagrant Cascadian
On 2020-04-27, Simon Glass wrote: > On Sun, 26 Apr 2020 at 18:58, Heinrich Schuchardt wrote: >> >> Am April 27, 2020 12:29:29 AM UTC schrieb Simon Glass : >> >At present U-Boot always builds dtc if CONFIG_OF_CONTROL is defined. >> >This >> >is wasteful when the system already has a suitable

Re: [PATCH v2 2/2] Makefile: Only build dtc if needed

2020-04-27 Thread Simon Glass
Hi Heinrich, On Sun, 26 Apr 2020 at 18:58, Heinrich Schuchardt wrote: > > Am April 27, 2020 12:29:29 AM UTC schrieb Simon Glass : > >At present U-Boot always builds dtc if CONFIG_OF_CONTROL is defined. > >This > >is wasteful when the system already has a suitable version available. > > > >Update

Re: [PATCH v2 2/2] Makefile: Only build dtc if needed

2020-04-26 Thread Heinrich Schuchardt
Am April 27, 2020 12:29:29 AM UTC schrieb Simon Glass : >At present U-Boot always builds dtc if CONFIG_OF_CONTROL is defined. >This >is wasteful when the system already has a suitable version available. > >Update the Makefile logic to build dtc only if the version available is >too old. > >This

[PATCH v2 2/2] Makefile: Only build dtc if needed

2020-04-26 Thread Simon Glass
At present U-Boot always builds dtc if CONFIG_OF_CONTROL is defined. This is wasteful when the system already has a suitable version available. Update the Makefile logic to build dtc only if the version available is too old. This saves about 2.5 seconds of elapsed time on a clean build for me.