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

[Heinrich Schuchardt]

On 2/13/19 1:25 PM, Tom Rini wrote:
> On Wed, Feb 13, 2019 at 11:13:19AM +0100, Philipp Tomsich wrote:
>>
>>
>>> On 13.02.2019, at 11:10, David Wu <david.wu at rock-chips.com> wrote:
>>>
>>> Hi Philipp,
>>>
>>> 在 2019/2/12 下午9:54, Philipp Tomsich 写道:
>>>>> On 12.02.2019, at 13:38, David Wu <david.wu at 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.
> 

This is the error message I currently get for U-Boot 2019.04 when
building for the Tinker Board:

mkdir -p foo/usr/lib/u-boot/tinker/
denx/tools/mkimage -n rk3288 -T rksd -d denx/spl/u-boot-spl-dtb.bin \
  foo/usr/lib/u-boot/tinker/u-boot.img
Warning: SPL image is too large (size 0x8800) and will not boot
Error: image verification failed
cat denx/u-boot-dtb.bin >> foo/usr/lib/u-boot/tinker/u-boot.img
cp sd_fusing.sh foo/usr/lib/u-boot/tinker/

@Tom
Shouldn't the mkimage step be included in Travis CI testing?

Best regards

Heinrich