> 
> 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;

Reply via email to