Hi Alex,
On Sat, 20 Mar 2021 at 03:53, Alex G. mr.nuke.me@gmail.com wrote:
On 3/18/21 1:44 AM, Simon Glass wrote:
Hi Alexandru,
On Thu, 11 Mar 2021 at 07:04, Alexandru Gagniuc mr.nuke.me@gmail.com wrote:
Commit 4afc4f37c70e ("doc: FIT image: Clarify format and simplify syntax") requires that FPGA images be referenced through the "loadables" in the config node. This means that "fpga" properties in config nodes are deprecated.
Given that there are likely FIT images which use "fpga", let's not break those right away. Print a warning message that such use is deprecated, and give users a couple of releases to update their
Signed-off-by: Alexandru Gagniuc mr.nuke.me@gmail.com
common/spl/spl_fit.c | 9 +++++++++ 1 file changed, 9 insertions(+)
diff --git a/common/spl/spl_fit.c b/common/spl/spl_fit.c index 55fca9f399..68f29c0026 100644 --- a/common/spl/spl_fit.c +++ b/common/spl/spl_fit.c @@ -526,6 +526,13 @@ __weak bool spl_load_simple_fit_skip_processing(void) return false; }
+static void warn_deprecated(const char *msg) +{
printf("DEPRECATED: %s\n", msg);printf("\tThis will stop working in a future u-boot release\n");printf("\tSee doc/uImage.FIT/source_file_format.txt\n");That is a lot of text to add...can it be shorter?
Sure. The idea was that we'd remove this message in a couple of releases anyway, and we want it very loud until then. If I remove the middle printf(), will thgat work ?
Sounds good, people can always look at the code for a message.
Regards, Simon