[U-Boot] [BUG] booting on the Tinker Board (tinker-rk3288_defconfig) is broken

[Tom Rini <trini@konsulko.com>]

On Wed, Feb 13, 2019 at 11:13:19AM +0100, Philipp Tomsich wrote:
> 
> 
> > On 13.02.2019, at 11:10, David Wu <david.wu@rock-chips.com> wrote:
> > 
> > Hi Philipp,
> > 
> > 在 2019/2/12 下午9:54, Philipp Tomsich 写道:
> >>> On 12.02.2019, at 13:38, David Wu <david.wu@rock-chips.com> wrote:
> >>> 
> >>> Hi Philipp,
> >>> 
> >>> 在 2019/2/10 上午2:24, Philipp Tomsich 写道:
> >>>> That said, I have been fighting issues from this patchset when trying to read
> >>>> from devices on the i2c0 bus on a RK3399—which had me add a “simple”
> >>>> pinctrl device again, as pulling all the dependencies into the SPL is a pain.
> >>> 
> >>> I think you can also add the simple ops if necessary. Full pinctrl
> >>> should only be used at the U-boot stage, or the SPL stage where the TPL is available.
> >>> 
> >>> For most boards about 3288, there is no need to enable pinctrl at SPL
> >>> stage, such as Tinker board.
> >> Could you check whether TPL_MAX_SIZE and SPL_MAX_SIZE are correct for
> >> the RK3288? If not, please submit a patch.
> > 
> > It is correct, 32KB.
> 
> Interesting...
> I wonder why the build did not fail then, if SPL became too big.

This is probably the "we need BOARD_SIZE_CHECK used more widely too"
issue that's come up in other threads.  The linker-only check does not
catch the CONFIG_OF_SEPARATE dtb exceeds limit problem as it's
post-link.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20190213/1e604583/attachment.sig>