From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from hera.kernel.org ([140.211.167.34]) by bombadil.infradead.org with esmtps (Exim 4.69 #1 (Red Hat Linux)) id 1LPR8D-0003Mx-G2 for linux-mtd@lists.infradead.org; Wed, 21 Jan 2009 00:39:45 +0000 Subject: Re: [GIT PULL -tip] fix 33 make headers_check warnings From: Jaswinder Singh Rajput To: "H. Peter Anvin" In-Reply-To: <49725B1E.604@kernel.org> References: <1232223441.3966.22.camel@localhost.localdomain> <1232228261.27094.0.camel@macbook.infradead.org> <20090117221932.GA4956@uranus.ravnborg.org> <49725B1E.604@kernel.org> Content-Type: text/plain Date: Wed, 21 Jan 2009 06:08:39 +0530 Message-Id: <1232498319.3123.34.camel@localhost.localdomain> Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Cc: urs.thuermann@volkswagen.de, neilb@suse.de, greg@kroah.com, bfields@fieldses.org, linux-mtd@lists.infradead.org, Sam Ravnborg , dbrownell@users.sourceforge.net, x86 maintainers , tiwai@suse.de, Ingo Molnar , linux-media@vger.kernel.org, mchehab@infradead.org, linux-raid@vger.kernel.org, netfilter@vger.kernel.org, linux-nfs@vger.kernel.org, netdev@vger.kernel.org, linux-usb@vger.kernel.org, LKML , netfilter-devel@vger.kernel.org, oliver.hartkopp@volkswagen.de, spi-devel-general@lists.sourceforge.net, Andrew Morton , David Woodhouse List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Sat, 2009-01-17 at 14:26 -0800, H. Peter Anvin wrote: > Sam Ravnborg wrote: > >>> > >> That patch looks wrong, and unnecessary. It was fine before. > > Nope - include/linux/dvb/audio.h failed to include linux/types.h > > despite the fact that is uses __u32 etc. > > > > But why the _kernel_ should include a userspace header is > > much more questionable. > > > > is one of a handful of headers provided by gcc itself. > Should I reintroduce my patch to solve this warning of 'make headers_check': usr/include/linux/dvb/audio.h:133: found __[us]{8,16,32,64} type without #include diff --git a/include/linux/dvb/audio.h b/include/linux/dvb/audio.h index 89412e1..758a48c 100644 --- a/include/linux/dvb/audio.h +++ b/include/linux/dvb/audio.h @@ -24,9 +24,8 @@ #ifndef _DVBAUDIO_H_ #define _DVBAUDIO_H_ -#ifdef __KERNEL__ #include -#else +#ifndef __KERNEL__ #include #endif Or you guys have some better plan. Thanks, -- JSR