[PATCH 8/8] firmware: mark some structs as packed
Sylvain Munaut
246tnt at gmail.com
Thu Nov 24 07:58:33 CET 2011
Hi,
First, thanks for this series of patch, I'll be sure to review and test them.
> this eliminates the occurrance of gcc warning
> warning: cast increases required alignment of target type
> ---
> src/target/firmware/include/comm/timer.h | 4 +++-
> src/target/firmware/include/layer1/sched_gsmtime.h | 5 ++++-
> 2 files changed, 7 insertions(+), 2 deletions(-)
> -};
> +} __packed;
I'm not conviced by this one : Why should we use packed for structures
not used as 'communication packets' ?
Cheers,
Sylvain
More information about the baseband-devel
mailing list