Re: [PATCH wireless-next v2 06/31] wifi: mm81x: add core.h

[Johannes Berg]

Hi,

Sorry for the delay. You probably already decided, but FWIW

> since the kernel (AFAICT) only has a single host endian structure.
> However I am a bit cautious about redefining elf sections here since
> I cannot find much precedent in the tree. it does mean we don't need
> to use __force. Of course we could also do what you wrote above and
> convert in-place using __force to the host endian struct. Do you have
> any preferences here?

I don't really, though I'd probably try to capture this in as little
code as possible, perhaps a single conversion function.

> Seems most wireless drivers have much simpler
> TLV followed by blob firmware loaders.

Indeed, why bother with anything more complex if you get to define it,
but maybe you don't get to define it.

johannes