* 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 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
* 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
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