Re: [alsa-devel] NULL pointer dereference in ALSA triggered through saa7134-alsa

2009-08-10 Thread Takashi Iwai
At Mon, 10 Aug 2009 19:06:29 +0300,
Ozan Çağlayan wrote:
> 
> Takashi Iwai wrote:
> > But /usr/include/sound isn't used for building kernel modules normally.
> > Unless any hack is added, these files have to be installed to the
> > kernel header directory.
> >   
> 
> Ah you're right, I totally missed that one. Thanks, will try to
> workaround that.

Well, the best would be to make alsa-driver backward compatible.
Basically we need to patch headers in alsa-kernel tree to be
binary-compatible with older versions, i.e. adjusting the fields
in struct snd_pcm or so, and supply the compatible function,
snd_card_new() in addition.

This could be done automatically in alsa-driver build process, but
I have little time to work on it...


Takashi


--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [alsa-devel] NULL pointer dereference in ALSA triggered through saa7134-alsa

2009-08-10 Thread Ozan Çağlayan
Takashi Iwai wrote:
> But /usr/include/sound isn't used for building kernel modules normally.
> Unless any hack is added, these files have to be installed to the
> kernel header directory.
>   

Ah you're right, I totally missed that one. Thanks, will try to
workaround that.
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [alsa-devel] NULL pointer dereference in ALSA triggered through saa7134-alsa

2009-08-10 Thread Takashi Iwai
At Mon, 10 Aug 2009 18:42:20 +0300,
Ozan Çağlayan wrote:
> 
> Takashi Iwai wrote:
> > At Mon, 10 Aug 2009 16:50:55 +0300,
> > Ozan Çağlayan wrote:
> >   
> >> Hi,
> >>
> >> I've finally succesfully compiled and linked saa7134-alsa driver using
> >> an external alsa-driver and its Module.symvers file. Everything seems
> >> okay, no undefined symbol or something else:
> >>
> >> - An installed 2.6.30.4 kernel which only builds and brings soundcore
> >> and sound_firmware,
> >> - Latest alsa-driver built externally and installed,
> >> - Latest saa7134-alsa, cx88-alsa, etc. code from linus-2.6 (seen that
> >> they don't affected by some API/ABI changes) patched on top of the
> >> alsa-driver tarball,
> >> 
> >
> > The external drivers using ALSA API have to be built with the
> > newer ALSA header files from alsa-driver tree.  It's not enough to
> > change snd_card_new() with snd_card_create().  The core structure
> > was changed, so the whole build has to be adjusted, too.
> >   
> Actually that was the 0th step that I forgot to mention. I'm installing
> the headers from the alsa-driver snapshot into /usr/include/sound and
> then I build alsa-driver. Then I use the symvers from the alsa-driver
> build alltogether with the headers that I've already installed into
> /usr/include/sound to build the V4L ones.

But /usr/include/sound isn't used for building kernel modules normally.
Unless any hack is added, these files have to be installed to the
kernel header directory.


Takashi
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [alsa-devel] NULL pointer dereference in ALSA triggered through saa7134-alsa

2009-08-10 Thread Ozan Çağlayan
Takashi Iwai wrote:
> At Mon, 10 Aug 2009 16:50:55 +0300,
> Ozan Çağlayan wrote:
>   
>> Hi,
>>
>> I've finally succesfully compiled and linked saa7134-alsa driver using
>> an external alsa-driver and its Module.symvers file. Everything seems
>> okay, no undefined symbol or something else:
>>
>> - An installed 2.6.30.4 kernel which only builds and brings soundcore
>> and sound_firmware,
>> - Latest alsa-driver built externally and installed,
>> - Latest saa7134-alsa, cx88-alsa, etc. code from linus-2.6 (seen that
>> they don't affected by some API/ABI changes) patched on top of the
>> alsa-driver tarball,
>> 
>
> The external drivers using ALSA API have to be built with the
> newer ALSA header files from alsa-driver tree.  It's not enough to
> change snd_card_new() with snd_card_create().  The core structure
> was changed, so the whole build has to be adjusted, too.
>   
Actually that was the 0th step that I forgot to mention. I'm installing
the headers from the alsa-driver snapshot into /usr/include/sound and
then I build alsa-driver. Then I use the symvers from the alsa-driver
build alltogether with the headers that I've already installed into
/usr/include/sound to build the V4L ones.

Sorry If I understand what you said wrongly.

Thanks,
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [alsa-devel] NULL pointer dereference in ALSA triggered through saa7134-alsa

2009-08-10 Thread Takashi Iwai
At Mon, 10 Aug 2009 16:50:55 +0300,
Ozan Çağlayan wrote:
> 
> Hi,
> 
> I've finally succesfully compiled and linked saa7134-alsa driver using
> an external alsa-driver and its Module.symvers file. Everything seems
> okay, no undefined symbol or something else:
> 
> - An installed 2.6.30.4 kernel which only builds and brings soundcore
> and sound_firmware,
> - Latest alsa-driver built externally and installed,
> - Latest saa7134-alsa, cx88-alsa, etc. code from linus-2.6 (seen that
> they don't affected by some API/ABI changes) patched on top of the
> alsa-driver tarball,

The external drivers using ALSA API have to be built with the
newer ALSA header files from alsa-driver tree.  It's not enough to
change snd_card_new() with snd_card_create().  The core structure
was changed, so the whole build has to be adjusted, too.


Takashi
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html