public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* staging: where to put system-wide includes
@ 2009-10-08 11:28 Pavel Machek
  2009-10-08 15:00 ` Greg KH
  2009-10-08 18:40 ` Christoph Hellwig
  0 siblings, 2 replies; 5+ messages in thread
From: Pavel Machek @ 2009-10-08 11:28 UTC (permalink / raw)
  To: kernel list, Greg KH

Hi!

In staging, I need some includes that will become "system-wide" at the
end.

Is it acceptable to create drivers/staging/include/linux (and
friends), and then use -I magic to include those headers?
									Pavel
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: staging: where to put system-wide includes
  2009-10-08 11:28 staging: where to put system-wide includes Pavel Machek
@ 2009-10-08 15:00 ` Greg KH
  2009-10-17 18:26   ` Pavel Machek
  2009-10-08 18:40 ` Christoph Hellwig
  1 sibling, 1 reply; 5+ messages in thread
From: Greg KH @ 2009-10-08 15:00 UTC (permalink / raw)
  To: Pavel Machek; +Cc: kernel list

On Thu, Oct 08, 2009 at 01:28:18PM +0200, Pavel Machek wrote:
> Hi!
> 
> In staging, I need some includes that will become "system-wide" at the
> end.
> 
> Is it acceptable to create drivers/staging/include/linux (and
> friends), and then use -I magic to include those headers?

No, keep them in the subdir for the drivers you are adding please and
reference them by the location directly, no -I magic.

thanks,

greg k-h

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: staging: where to put system-wide includes
  2009-10-08 11:28 staging: where to put system-wide includes Pavel Machek
  2009-10-08 15:00 ` Greg KH
@ 2009-10-08 18:40 ` Christoph Hellwig
  2009-10-09 20:38   ` Pavel Machek
  1 sibling, 1 reply; 5+ messages in thread
From: Christoph Hellwig @ 2009-10-08 18:40 UTC (permalink / raw)
  To: Pavel Machek; +Cc: kernel list, Greg KH

On Thu, Oct 08, 2009 at 01:28:18PM +0200, Pavel Machek wrote:
> Hi!
> 
> In staging, I need some includes that will become "system-wide" at the
> end.
> 
> Is it acceptable to create drivers/staging/include/linux (and
> friends), and then use -I magic to include those headers?

If you need system-wide inclusions it's pretty good sign that what
you add is not a simple driver and thus should not just go into the
staging tree.  Given that you're a long-timer kernel hackers what's the
problem with getting your code fully up to kernel standards anyway?


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: staging: where to put system-wide includes
  2009-10-08 18:40 ` Christoph Hellwig
@ 2009-10-09 20:38   ` Pavel Machek
  0 siblings, 0 replies; 5+ messages in thread
From: Pavel Machek @ 2009-10-09 20:38 UTC (permalink / raw)
  To: Christoph Hellwig; +Cc: kernel list, Greg KH

On Thu 2009-10-08 14:40:27, Christoph Hellwig wrote:
> On Thu, Oct 08, 2009 at 01:28:18PM +0200, Pavel Machek wrote:
> > Hi!
> > 
> > In staging, I need some includes that will become "system-wide" at the
> > end.
> > 
> > Is it acceptable to create drivers/staging/include/linux (and
> > friends), and then use -I magic to include those headers?
> 
> If you need system-wide inclusions it's pretty good sign that what
> you add is not a simple driver and thus should not just go into the
> staging tree.  Given that you're a long-timer kernel hackers what's the
> problem with getting your code fully up to kernel standards anyway?

Well, it is not exactly my code -- I'm trying to merge HTC Dream
support. Problem with "simply cleaning it up" is that its 122Kloc.

									Pavel

-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: staging: where to put system-wide includes
  2009-10-08 15:00 ` Greg KH
@ 2009-10-17 18:26   ` Pavel Machek
  0 siblings, 0 replies; 5+ messages in thread
From: Pavel Machek @ 2009-10-17 18:26 UTC (permalink / raw)
  To: Greg KH; +Cc: kernel list

On Thu 2009-10-08 08:00:10, Greg KH wrote:
> On Thu, Oct 08, 2009 at 01:28:18PM +0200, Pavel Machek wrote:
> > Hi!
> > 
> > In staging, I need some includes that will become "system-wide" at the
> > end.
> > 
> > Is it acceptable to create drivers/staging/include/linux (and
> > friends), and then use -I magic to include those headers?
> 
> No, keep them in the subdir for the drivers you are adding please and
> reference them by the location directly, no -I magic.

That will make moving drivers to final place "interesting", but I'll
do as you wish.
									Pavel
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2009-10-17 18:26 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-10-08 11:28 staging: where to put system-wide includes Pavel Machek
2009-10-08 15:00 ` Greg KH
2009-10-17 18:26   ` Pavel Machek
2009-10-08 18:40 ` Christoph Hellwig
2009-10-09 20:38   ` Pavel Machek

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox