[PATCH 8/8] firmware: mark some structs as packed

Alexander Huemer alexander.huemer at xx.vu
Thu Nov 24 11:36:47 CET 2011


On Thu, Nov 24, 2011 at 10:49:43AM +0100, Holger Hans Peter Freyther wrote:
> On 11/24/2011 10:36 AM, Alexander Huemer wrote:
> 
> > Steve and Holger raised some reasonable concerns on some of the other
> > patches, it seems I was quite unconcentrated and they are mostly crap.
> 
> crap? no, not at all. I agree that we should get the number of warnings down,
> sometimes a warning just needs something else than the straight forward fix.
> 
Compiler warnings are of course controversial.
I personally think that osmocom-bb and friends are fine pieces of code
and deserve a kind of "polished" look when compiling.

Some warnings are also quite strong indications of bugs like "... makes
pointer from integer without a cast" and things like that.

I myself don't know how to handle e.g. "discards qualifiers from pointer
target type" when the constness of lvalue and rvalue don't match.

Maybe it doesn't actually matter so much and I am just pedantic.



More information about the baseband-devel mailing list