From mboxrd@z Thu Jan 1 00:00:00 1970 From: Frank Wunderlich Date: Fri, 24 Jan 2020 19:15:48 +0100 Subject: [BUG] binman: Add a library to access binman entries In-Reply-To: References: <20191207044315.51770-1-sjg@chromium.org> <20191206213936.v6.1.I8c03ae7027508705f6f535ba5137b5e6c5dea840@changeid> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi, a bit more info about this... this line [1] (in my case) breaks the init-chain: return log_msg_ret("binman node", -EINVAL); the binman_init [2] is added to init_sequence_r[] which is executed by initcall_run_list ./common/board_r.c:897: if (initcall_run_list(init_sequence_r)) exiting the binman-function [3] with error-code (return <> 0) exits the full chain (./include/initcall.h) [4] with message initcall sequence %p failed at call %p how to deal with this? - do not select binman as default=y in Kconfig - adding the binman-node [1] to all dts - do not exit with error-code (only print/log message) - do not exit the init-sequence on binman-error [3] - more ideas? in our case we disabled option CONFIG_BINMAN_FDT [5] regards Frank [1] https://gitlab.denx.de/u-boot/u-boot/blob/master/lib/binman.c#L45 [2] https://gitlab.denx.de/u-boot/u-boot/blob/master/common/board_r.c#L722 [3] https://gitlab.denx.de/u-boot/u-boot/blob/master/common/board_r.c#L369 [4] https://gitlab.denx.de/u-boot/u-boot/blob/master/include/initcall.h#L42 [5] http://forum.banana-pi.org/t/bpi-r64-current-u-boot-support/10077/69