From 1425465accfb96b74df400131aeb55665e9d98d2 Mon Sep 17 00:00:00 2001 From: Andreas Dannenberg Date: Thu, 8 Aug 2019 12:54:49 -0500 Subject: [PATCH] README: Clarify use of BSS during SPL board_init_f() The earlier commit.... commit a5a5d997b41a ("spl: Allow performing BSS init early before board_init_f()") ...introduced the ability to use BSS from SPL's board_init_f() as it may be required in certain exceptional use cases so go ahead and update the README to reflect this change. Note that as highlighted with the changes the use of the associated CONFIG option is generally not recommended. Signed-off-by: Andreas Dannenberg Reviewed-by: Simon Goldschmidt --- README | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/README b/README index 2d91619a94..1389e8ff12 100644 --- a/README +++ b/README @@ -267,6 +267,13 @@ board_init_f(): - preloader_console_init() can be called here in extremis - should set up SDRAM, and anything needed to make the UART work - these is no need to clear BSS, it will be done by crt0.S + - for specific scenarios on certain architectures an early BSS *can* + be made available (via CONFIG_SPL_EARLY_BSS by moving the clearing + of BSS prior to entering board_init_f()) but doing so is discouraged. + Instead it is strongly recommended to architect any code changes + or additions such to not depend on the availability of BSS during + board_init_f() as indicated in other sections of this README to + maintain compatibility and consistency across the entire code base. - must return normally from this function (don't call board_init_r() directly)