Re: [PATCH 1/3] arm: imx6: move imx6_init_lowlevel to single initcall

2014-06-30 Thread Sascha Hauer
On Tue, Jun 24, 2014 at 09:17:49AM +0200, Sascha Hauer wrote: > On Tue, Jun 24, 2014 at 08:33:14AM +0200, Lucas Stach wrote: > > Instead of repeating the same lowlevel init for every board > > move it to it's own initcall. > > > > Avoids code bloat and shaves off almost 1.5kB of uncompressed > > b

Re: [PATCH 1/3] arm: imx6: move imx6_init_lowlevel to single initcall

2014-06-27 Thread Holger Schurig
Good idea. In addition to my board, the Sabre-Lite board also produces this warning, even twice. ___ barebox mailing list barebox@lists.infradead.org http://lists.infradead.org/mailman/listinfo/barebox

Re: [PATCH 1/3] arm: imx6: move imx6_init_lowlevel to single initcall

2014-06-26 Thread Sascha Hauer
On Thu, Jun 26, 2014 at 02:02:07PM +0200, Holger Schurig wrote: > Hmm, I have removed the imx6_init_lowlevel() from my (out-of-tree) > board.c file. Still I get this warning: > > __request_region: 0x1000:0x4fff conflicts with 0x1000:0x4fff > > Could this be related to your patchse

Re: [PATCH 1/3] arm: imx6: move imx6_init_lowlevel to single initcall

2014-06-26 Thread Holger Schurig
Hmm, I have removed the imx6_init_lowlevel() from my (out-of-tree) board.c file. Still I get this warning: __request_region: 0x1000:0x4fff conflicts with 0x1000:0x4fff Could this be related to your patchset? In the log below, the text " ahs2_postcore_init, after barebox_set_hostn

Re: [PATCH 1/3] arm: imx6: move imx6_init_lowlevel to single initcall

2014-06-24 Thread Sascha Hauer
On Tue, Jun 24, 2014 at 08:33:14AM +0200, Lucas Stach wrote: > Instead of repeating the same lowlevel init for every board > move it to it's own initcall. > > Avoids code bloat and shaves off almost 1.5kB of uncompressed > barebox size for a default imx_v7_defconfig build. > > For boards wherethe

[PATCH 1/3] arm: imx6: move imx6_init_lowlevel to single initcall

2014-06-23 Thread Lucas Stach
Instead of repeating the same lowlevel init for every board move it to it's own initcall. Avoids code bloat and shaves off almost 1.5kB of uncompressed barebox size for a default imx_v7_defconfig build. For boards wherethe hostname setup was done in the postcore initcall we move this to a device