On Thu, May 23, 2019 at 04:04:17PM +0200, Marek Vasut wrote:
> On 5/23/19 3:59 PM, Tom Rini wrote:
> > On Thu, May 23, 2019 at 10:49:11AM -0300, Fabio Estevam wrote:
> >> Hi Peng,
> >>
> >> On Tue, May 21, 2019 at 10:38 PM Peng Fan <peng....@nxp.com> wrote:
> >>
> >>>> I will send a v2 removing CONFIG_SPL_DM.
> >>>
> >>> So what is the real direction moving to use SPL driver, non-dm is allowed 
> >>> in future?
> >>
> >> We are very limited in space when CONFIG_SPL_DM is used on imx6q-sabresd.
> >>
> >> With CONFIG_SPL_DM selected the size increase due to the three dtb's
> >> is about 8kB.
> >>
> >> Even with CONFIG_SPL_DM disabled we ony have 1kB of free space, so I
> >> still want to optimize the size further.
> > 
> > So with CONFIG_SPL_DM we're overflowing by ~7kB?
> > 
> >> Two options that I want also to remove are:
> >>
> >> CONFIG_SPL_FS_EXT4=y
> >> CONFIG_SPL_I2C_SUPPORT=y
> >>
> >> We are at -rc2 now, and I would really like to have mx6sabresd booting 
> >> again.
> >>
> >> Please check my v2 series and see if it looks good.
> > 
> > We certainly want to do something to fix the regression, agreed.
> > 
> > Looking at the config, can we just use a fake DTB for SPL that works on
> > all 3 boards, rather than 3 DTBs?
> 
> Mind you, all of that seems like a workaround. Removing functionality
> just to fit the monster with DM/DT in is a no-go. R-Car Gen2 doesn't use
> DM/DT in SPL because it just won't fit and I believe that absolutely
> makes sense ; why should we impose DM/DT in SPL on MX6 only to make it
> less usable?

Setting aside that as I said you don't have to use DM in SPL, maybe it's
also worth noting you don't have to provide "describes the whole board"
DTB files.  If as far as SPL is concerned there's no difference between
board A, B and C, just make a DTS that describes the relevant hardware.

Also, platdata can help here too.

-- 
Tom

Attachment: signature.asc
Description: PGP signature

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to