[PATCH] common: console: move break; statement
    Tom Rini 
    trini at konsulko.com
       
    Fri Apr 11 16:48:54 CEST 2025
    
    
  
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
    
    
More information about the U-Boot
mailing list