Re: [PATCH v5 26/26] fdt: Don't call board_fdt_blob_setup() without OF_BOARD

2021-10-27 Thread Tom Rini
On Wed, Oct 27, 2021 at 11:33:53PM +0200, François Ozog wrote: > Hi Tom > > Le mer. 27 oct. 2021 à 21:12, Tom Rini a écrit : > > > On Wed, Oct 27, 2021 at 10:17:05AM +0300, Ilias Apalodimas wrote: > > > Hi Simon, > > > > > > On Tue, 26 Oct 2021 at 18:27, Simon Glass wrote: > > > > > > > > Hi

Re: [PATCH v5 26/26] fdt: Don't call board_fdt_blob_setup() without OF_BOARD

2021-10-27 Thread François Ozog
Hi Tom Le mer. 27 oct. 2021 à 21:12, Tom Rini a écrit : > On Wed, Oct 27, 2021 at 10:17:05AM +0300, Ilias Apalodimas wrote: > > Hi Simon, > > > > On Tue, 26 Oct 2021 at 18:27, Simon Glass wrote: > > > > > > Hi Ilias, > > > > > > On Tue, 26 Oct 2021 at 07:56, Ilias Apalodimas > > > wrote: > >

Re: [PATCH v5 26/26] fdt: Don't call board_fdt_blob_setup() without OF_BOARD

2021-10-27 Thread Tom Rini
On Wed, Oct 27, 2021 at 10:17:05AM +0300, Ilias Apalodimas wrote: > Hi Simon, > > On Tue, 26 Oct 2021 at 18:27, Simon Glass wrote: > > > > Hi Ilias, > > > > On Tue, 26 Oct 2021 at 07:56, Ilias Apalodimas > > wrote: > > > > > > Hi Simon, > > > > > > As I said here [1], this is moving on an

Re: [PATCH v5 26/26] fdt: Don't call board_fdt_blob_setup() without OF_BOARD

2021-10-27 Thread Ilias Apalodimas
Hi Simon, On Tue, 26 Oct 2021 at 18:27, Simon Glass wrote: > > Hi Ilias, > > On Tue, 26 Oct 2021 at 07:56, Ilias Apalodimas > wrote: > > > > Hi Simon, > > > > As I said here [1], this is moving on an entirely different direction I had > > in mind. I'd much prefer starting the discussions for a

Re: [PATCH v5 26/26] fdt: Don't call board_fdt_blob_setup() without OF_BOARD

2021-10-26 Thread Simon Glass
Hi Ilias, On Tue, 26 Oct 2021 at 07:56, Ilias Apalodimas wrote: > > Hi Simon, > > As I said here [1], this is moving on an entirely different direction I had > in mind. I'd much prefer starting the discussions for a solution that > allows us to scale. I am missing the point here. Is there

Re: [PATCH v5 26/26] fdt: Don't call board_fdt_blob_setup() without OF_BOARD

2021-10-26 Thread Ilias Apalodimas
Hi Simon, As I said here [1], this is moving on an entirely different direction I had in mind. I'd much prefer starting the discussions for a solution that allows us to scale. FWIW I think the current code is still not clean for my taste. Commit 3b595da441cf ("fdtdec: allow board to provide fdt

[PATCH v5 26/26] fdt: Don't call board_fdt_blob_setup() without OF_BOARD

2021-10-25 Thread Simon Glass
At present this override function is called even when OF_BOARD Is not enabled. This makes it impossible to disable this feature and in fact makes the OF_BOARD option useless. Reinstate its intended purpose, so that it is possible to switch between the appended devicetree and one provided by the