Le Mon, May 19, 2025 at 03:06:08PM -0600, Tom Rini a écrit :
> Good catch. This should also get a:
> Fixes: c2211ff65136 ("bootm: Add more fields to bootm_info")

Thank you for the review.
I am preparing a v2 of this patch.

> And after reading the code and grepping around more, I think we should
> be doing the assignment unconditionally in the function. It's what we
> practically do today as only colibri_vf disables CMD_BOOTM but then also
> enables any other OS boot command.
> -- 
> Tom

I did a bit more analysis, I put my discoveries into the commit message of the 
second version of this patch. And yes, you are right, the assignment should be 
done unconditionally (as I understand it).

Benjamin

Reply via email to