Re: [PATCH] mac80211: check endianness/types in sparse runs

From: Johannes Berg
Date: Thu Feb 21 2008 - 15:09:52 EST



> > In any case, I would love to have __CHECK_ENDIAN__ enabled by default at
> > least on the wireless code (just caught another bug with it...)
>
> I should then add support for something like:
>
> checkflags-y := -D__CHECK_ENDIAN__
>
> to match the style of the rest.
>
> I do not like all the buried in assumption about the
> global variables.

That might be good. I retract my patch for now then.

> But I would prefer that someone would spend a few days looking into the
> warnings generated with sparse.

I did spend a bit of time running it on all the tree, but wireless,
where I do most work, is clean. I have some code in sound/ that
generates warnings and some drivers/macintosh/ code I worked on too, but
right now I don't want to pick up too many changes all over the tree
into my patches tree.

johannes

Attachment: signature.asc
Description: This is a digitally signed message part