Attention is currently required from: laforge, neels, pespin.
2 comments:
Commit Message:
this
Done
Patchset:
I'm not sure this is entirely fixing the issue. […]
As far as I understand the line struct {} __attribute__ ((aligned)); will force data[0] into an aligned position, relative to the beginning of the address where struct osmo_ecu_state is stored. This will be an aligned address since the compiler has knowledge about what is stored, but it can not know what we store in the memory beginning from data[0]. However since data[0] is now at an aligned position this is not a problem anymore. To make it look less ugly we can also just define a an uint8_t *data pointer followed by an uint8_t _data[0] array. The pointer member will automatically be in an aligned position and so _data[0] will also be aligned. (see also @laforge@gnumonks.org comments in the referenced ticket.)
To view, visit change 35212. To unsubscribe, or for help writing mail filters, visit settings.