Re: linux-next: build warning after merge of the wireless-next tree

[Larry Finger]

On 04/03/2013 09:36 PM, Stephen Rothwell wrote:
> Hi Larry,
>
> On Wed, 03 Apr 2013 20:17:01 -0500 Larry Finger <Larry.Finger@xxxxxxxxxxxx> wrote:
> > On 04/03/2013 08:04 PM, Stephen Rothwell wrote:
> > > On Thu, 4 Apr 2013 12:00:43 +1100 Stephen Rothwell <sfr@xxxxxxxxxxxxxxxx> wrote:
> > > > After merging the wireless-next tree, today's linux-next build (x86_64
> > > > allmodconfig) produced this warning:
> > > >
> > > > x86_64-linux-gcc: warning: drivers/net/wireless/rtlwifi: linker input file unused because linking not done
> > > > x86_64-linux-gcc: warning: drivers/net/wireless/rtlwifi: linker input file unused because linking not done
> > > > x86_64-linux-gcc: warning: drivers/net/wireless/rtlwifi: linker input file unused because linking not done
> > > > x86_64-linux-gcc: warning: drivers/net/wireless/rtlwifi: linker input file unused because linking not done
> > > > x86_64-linux-gcc: warning: drivers/net/wireless/rtlwifi: linker input file unused because linking not done
> > > > x86_64-linux-gcc: warning: drivers/net/wireless/rtlwifi: linker input file unused because linking not done
> > > > x86_64-linux-gcc: warning: drivers/net/wireless/rtlwifi: linker input file unused because linking not done
> > > > x86_64-linux-gcc: warning: drivers/net/wireless/rtlwifi: linker input file unused because linking not done
> > > > x86_64-linux-gcc: warning: drivers/net/wireless/rtlwifi: linker input file unused because linking not done
> > > > x86_64-linux-gcc: warning: drivers/net/wireless/rtlwifi: linker input file unused because linking not done
> > > > x86_64-linux-gcc: warning: drivers/net/wireless/rtlwifi: linker input file unused because linking not done
> > > >
> > > > I have no idea what caused that.
> > >
> > > Though I suspect commit 8b138d4898a0 ("rtlwifi: rtl8188ee: Enable build
> > > of new driver") - I wonder if the gap between the "-I" and
> > > "drivers/net/wireless/rtlwifi" added to ccflags-y is the cause?
> > >
> > > I am using gcc 4.6.3 if that matters.
> >
> > I use 4.7.2.
> >
> > I'll send a patch with the space removed to see if that helps.
>
> I just tested without the space and it fixes the problem for my Debian
> 4.7.2 compiler.  Removing the space also fixes the problem with finding
> include files (after I removed the "../" bits again).

Stephan,

Ah, that was the real error. Thanks for testing.

John: Please take my patch of this evening entitles "[PATCH] rtlwifi: rtl8188ee: 
Fix linker warnings" and drop the one by Chen, Chien-Chia entitled "[PATCH] 
rtlwifi: rtl8188ee: Fix wrong header path". If we put the ../ into the header 
paths, I'll just have to take them out later. We might as well leave them out.

Larry


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/