Re: [PATCH v2 2/3] lib/vsprintf: Add support for generic FOURCCs by extending %p4cc

From: Aditya Garg
Date: Sun Feb 23 2025 - 10:17:03 EST



>
> Looking at the header files, it looks like doing cpu_to_le32 on that variable and doing le32_to_cpu will actually reverse the order twice, on big endian systems, thus technically all way would not swap the order at all.
>
> I'm not really sure how to manage the sparse warnings here.



Not sure whether the maintainers would like it, but we can do something like this:

case 'l’:
#ifdef __LITTLE_ENDIAN
val = orig;
#else
orig = swab32(orig);
val = orig;
#endif
break;

case 'b’:
#ifdef __LITTLE_ENDIAN
orig = swab32(orig);
val = orig;
#else
val = orig;
#endif
break;