Re: [GIT PULL] MIPS changes

From: Linus Torvalds
Date: Mon Sep 23 2019 - 14:18:41 EST


On Mon, Sep 23, 2019 at 11:07 AM Paul Burton <paul.burton@xxxxxxxx> wrote:
>
> Another issue is that there are currently 'expected' warnings dotted
> through the tree for various defconfigs

This is why I refuse to have _any_ warnings at all in my tree during
the merge window.

If you have expected warnings, you will ignore the new and valid ones.
So the only acceptable situation is "no warnings".

In honesty, I actually do have one warning in my tree:

samples/vfs/test-statx.c:24:15: warning: âstruct fooâ declared
inside parameter list will not be visible outside of this definition
or declaration
24 | #define statx foo
| ^~~

but because it's in the sample code, it pretty much never gets rebuilt
for me unless I basically do a "git clean" to get rid of everything,
so I don't normally see it for any normal pull.

So I've ignored that one warning, although I've actually been tempted
to just remove the sample because of it.

Adding David and Al to the cc just in case they have some simple fixup
for it that is likely to work across different user headers.

I considered just adding a

struct foo;

declaration, but the whole thing is incredibly ugly.

Linus