On Fri, 12 Mar 2021 at 10:32, Alexandru Gagniuc <mr.nuke...@gmail.com> wrote: > > OP-TEE images are normally packaged with > type = "tee; > os = "tee"; > > However, fit_image_load() thinks that is somehow invalid. however if > they were declared as type = "kernel", os = "linux", fit_image_load() > would happily accept them and allow the boot to continue. There is no > technical limitation to excluding "tee". > > Allowing "tee" images is useful in a boot flow where OP-TEE is > executed before linux. > > In fact, I think it's unintuitive for a "load"ing function to also do > parsing and contain a bunch ad-hoc heuristics that only its caller > might know. But I don't make the rules, I just write fixes. In more > polite terms: refactoring the fit_image API is beyond the scope of > this change. > > Signed-off-by: Alexandru Gagniuc <mr.nuke...@gmail.com> > --- > common/image-fit.c | 2 ++ > 1 file changed, 2 insertions(+)
Reviewed-by: Simon Glass <s...@chromium.org>
