[PATCH] fs: jffs2: Remove always true test

Quentin Schulz quentin.schulz at cherry.de
Mon Oct 6 16:07:39 CEST 2025


Hi Andrew,

On 10/2/25 11:29 AM, Andrew Goodbody wrote:
> Testing an unsigned variable to be >= 0 will always be true so remove
> this redundant test.
> 

I checked if it wouldn't rather be an issue of the wrong type being 
used, but i_version is:
- initialized to 0,
- set to b2->version with b2 being pL->frag list element which defines 
version as a u32 as well. The version member is set from jffs2 raw inode 
u32 version member, so we're good as well on this side.

Thus,

Reviewed-by: Quentin Schulz <quentin.schulz at cherry.de>

Thanks!
Quentin


More information about the U-Boot mailing list