All of lore.kernel.org
 help / color / mirror / Atom feed
From: Takashi Iwai <tiwai@suse.de>
To: Fred Gleason <fredg@salemradiolabs.com>
Cc: Eliot Blennerhassett <bigblen@icqmail.com>,
	alsa-devel@lists.sourceforge.net
Subject: Re: AudioScience driver
Date: Wed, 02 Jul 2003 12:42:18 +0200	[thread overview]
Message-ID: <s5h7k71ky5x.wl@alsa2.suse.de> (raw)
In-Reply-To: <200307011652.48863.fredg@salemradiolabs.com>

At Tue, 1 Jul 2003 16:52:48 -0400,
Fred Gleason wrote:
> 
> On Monday 30 June 2003 09:26, Takashi Iwai wrote:
> 
> > > Good idea.
> > > how about /usr/include/asihpi/hpi.h
> >
> > well, assuming the certain header location is not recommended.
> >
> > basically, the kernel sources must not refer to the standard header
> > files for user-space.  if you need a header file for the kernel,
> > include it in the kernel tree.  or, more easily, specify the header
> > path explicitly with -I option in Makefile (which may be given by the
> > configure script).
> 
> Then perhaps '/usr/include/linux/hpi.h'.  

the problem is that you cannot assume that the driver source accepts
the "standard" header-file locations, namely, /usr/include, etc.
these locations should not be referred by the kernel sources. 
all header files for the kernel codes should be private.

in the case of alsa-driver package, this is not strictly prohibited.
but it's better to avoid assume the fixed location.

so, as i suggested, it's better to have an option, such as

	% ./configure --with-hpi=/usr/local/src/hpi

so that the configure script can check the location of hpi.h and
enable the compilation of asihpi driver only when it's found.

i have no objection to /usr/include/asihpi/hpi.h, btw.  it's just the
matter of alsa-driver side.


Takashi


-------------------------------------------------------
This SF.Net email sponsored by: Free pre-built ASP.NET sites including
Data Reports, E-commerce, Portals, and Forums are available now.
Download today and enter to win an XBOX or Visual Studio .NET.
http://aspnet.click-url.com/go/psa00100006ave/direct;at.asp_061203_01/01

  reply	other threads:[~2003-07-02 10:42 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-06-25 21:58 AudioScience driver Eliot Blennerhassett
2003-06-30 13:26 ` Takashi Iwai
2003-07-01 20:52   ` Fred Gleason
2003-07-02 10:42     ` Takashi Iwai [this message]
  -- strict thread matches above, loose matches on Subject: below --
2003-06-24  5:14 Eliot Blennerhassett
2003-06-24  9:40 ` Takashi Iwai
2003-06-25  3:23 ` Fred Gleason

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=s5h7k71ky5x.wl@alsa2.suse.de \
    --to=tiwai@suse.de \
    --cc=alsa-devel@lists.sourceforge.net \
    --cc=bigblen@icqmail.com \
    --cc=fredg@salemradiolabs.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.