Messages in this thread | | | From | Wei Ni <> | Date | Thu, 4 Mar 2010 19:09:29 +0800 | Subject | RE: [alsa-devel] [PATCH]Support MCP89 and GT21x hdmi audio |
| |
Hi, Takashi I test my patch, it build successful when CONFIG_SND_HDA_*=y. Could you give me your build error messages?
Thanks Wei. nvpublic
-----Original Message----- From: Takashi Iwai [mailto:tiwai@suse.de] Sent: Thursday, March 04, 2010 6:57 PM To: Wei Ni Cc: 'Pavel Hofman'; 'alsa-devel'; 'linux-kernel'; 'akpm' Subject: Re: [alsa-devel] [PATCH]Support MCP89 and GT21x hdmi audio
At Thu, 4 Mar 2010 18:46:59 +0800, Wei Ni wrote: > > Hi, Takashi > I think we can submit my patch first,
But your patch breaks the build when CONFIG_SND_HDA_*=y. So, it should be fixed in your patch.
thanks,
Takashi
> then generate another patch > to add EXPORT_SYMBOL() in hda_eld.c, and change Makefile to remove > hda_eld.o from snd-hda-codec-xxhdmi-objs. > > Thanks > Wei. > nvpublic > > -----Original Message----- > From: Takashi Iwai [mailto:tiwai@suse.de] > Sent: Thursday, March 04, 2010 5:47 PM > To: Wei Ni > Cc: 'Pavel Hofman'; 'alsa-devel'; 'linux-kernel'; 'akpm' > Subject: Re: [alsa-devel] [PATCH]Support MCP89 and GT21x hdmi audio > > At Thu, 4 Mar 2010 10:21:39 +0800, > Wei Ni wrote: > > > > Hi, Takashi > > 1. Yes, it can works for all Nvidia controller. > > OK. > > > 2. The hda_eld.o doesn't export any symbols. > > I tried to put hda_eld.o to snd-had-codec-*, and remove had_eldo.o > > from snd-had-codec-intelhdmi-objs, but it will build error. It need > > to modify hda_eld.c to export symbols, it will add many changes. > > Yes, this is unavoidable. Please add EXPORT_SYMBOL()'s > appropriately. > > We can reduce them again once after all HDMI stuff is merged into > one. > > > thanks, > > Takashi >
| |