On Tue, 25 Mar 2025 17:47:44 +0000, Andre Przywara wrote:

> In console_setfile(), there is some #ifdef'ed code, updating monitor
> functions for a U-Boot proper build. This is called inside a switch/case
> statement, but the closing "break;" is inside the #ifdef section.
> This doesn't look right: we should not fall through to the error case
> for an SPL/TPL build.
> 
> Move the "break" to be always effective, solving a compiler warning about
> an untagged implicit fallthrough.
> 
> [...]

Applied to u-boot/master, thanks!

[1/1] common: console: move break; statement
      commit: 82f5ce11e669d86ac08e6ba800d4a97ad734665b
-- 
Tom


Reply via email to