From mboxrd@z Thu Jan 1 00:00:00 1970 From: Wu Fengguang Subject: Re: [PATCH]Support MCP89 and GT21x hdmi audio Date: Wed, 10 Mar 2010 14:01:07 +0800 Message-ID: <20100310060107.GA27334@localhost> References: <6B4D417B830BC44B8026029FD256F7F1C16B1AD556@HKMAIL01.nvidia.com> <6B4D417B830BC44B8026029FD256F7F1C16B1AD557@HKMAIL01.nvidia.com> <6B4D417B830BC44B8026029FD256F7F1C16B1AD558@HKMAIL01.nvidia.com> <6B4D417B830BC44B8026029FD256F7F1C16B1AD562@HKMAIL01.nvidia.com> <20100308044614.GA29026@localhost> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by alsa0.perex.cz (Postfix) with ESMTP id 7AF0E24368 for ; Wed, 10 Mar 2010 07:01:14 +0100 (CET) Content-Disposition: inline In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: alsa-devel-bounces@alsa-project.org Errors-To: alsa-devel-bounces@alsa-project.org To: VDR User Cc: Wei Ni , Takashi Iwai , alsa-devel , Jaroslav Kysela , linux-kernel , akpm , Pavel Hofman List-Id: alsa-devel@alsa-project.org T24gV2VkLCBNYXIgMTAsIDIwMTAgYXQgMDE6NDI6MDVQTSArMDgwMCwgVkRSIFVzZXIgd3JvdGU6 Cj4gT24gU3VuLCBNYXIgNywgMjAxMCBhdCA4OjQ2IFBNLCBXdSBGZW5nZ3VhbmcgPGZlbmdndWFu Zy53dUBpbnRlbC5jb20+IHdyb3RlOgo+ID4gQ0MgSmFyb3NsYXYsIG1heWJlIGhlIGhhcyBzb21l IGlkZWEgb24KPiA+Cj4gPiDCoCDCoCDCoCDCoHBjbV9ody5jOiBzbmRfcGNtX2h3X2RlbGF5KCkg U05EUlZfUENNX0lPQ1RMX0RFTEFZIGZhaWxlZC4KPiA+Cj4gPiBPbiBNb24sIE1hciAwOCwgMjAx MCBhdCAxMjozNTozN1BNICswODAwLCBXZWkgTmkgd3JvdGU6Cj4gPj4gSGksCj4gPj4gQmVjYXVz ZSBJIGRvbid0IGhhdmUgR1QyMjAgY2FyZCBvbiBoYW5kLCBJIGRpZG4ndCB0ZXN0IGl0IHlldC4K PiA+PiBTbyBJIGRpZG4ndCBhZGQgdGhpcyBjYXJkIHRvIHRoZSBwYXRjaCBmaWxlLiBJIHRoaW5r IHlvdSBjb3VsZCBhZGQgaXQgYWZ0ZXIKPiA+PiB5b3UgdGVzdCBpdC4KPiA+Pgo+ID4+IEFib3V0 IHRoZSB3YXJuaW5nIG1lc3NhZ2VzLCBJIGRvbid0IGtub3cgd2h5Lgo+ID4+IE1heSBiZSBzb21l IG90aGVycyBjYW4gYW5zd2VyLgo+ID4+Cj4gPj4gVGhhbmtzCj4gPj4gV2VpLgo+ID4+IG52cHVi bGljCj4gPj4KPiA+PiAtLS0tLU9yaWdpbmFsIE1lc3NhZ2UtLS0tLQo+ID4+IEZyb206IFZEUiBV c2VyIFttYWlsdG86dXNlci52ZHJAZ21haWwuY29tXQo+ID4+IFNlbnQ6IE1vbmRheSwgTWFyY2gg MDgsIDIwMTAgMzoyMiBBTQo+ID4+IFRvOiBXZWkgTmkKPiA+PiBDYzogVGFrYXNoaSBJd2FpOyBh a3BtOyBhbHNhLWRldmVsOyBXdSBGZW5nZ3Vhbmc7IGxpbnV4LWtlcm5lbDsgUGF2ZWwgSG9mbWFu Cj4gPj4gU3ViamVjdDogUmU6IFthbHNhLWRldmVsXSBbUEFUQ0hdU3VwcG9ydCBNQ1A4OSBhbmQg R1QyMXggaGRtaSBhdWRpbwo+ID4+Cj4gPj4gSSB0aGluayB0aGlzIG1heSBiZSByZWxldmFudCB0 byB0aGlzIHN1YmplY3QsIGlmIG5vdCB0aGVuIG15IGFwb2xvZ2llcy4KPiA+Pgo+ID4+IEkndmUg anVzdCBpbnN0YWxsZWQgYSBHVDIyMCBjYXJkIGFuZCB3YXMgYWJsZSB0byBnZXQgYXVkaW8tb3Zl ci1oZG1pCj4gPj4gd29ya2luZyBieSBjb21waWxpbmcgdGhlIHNuYXBzaG90IGRyaXZlcnMgZnJv bSAyMDEwMDMwNyB3aGljaCBjb250YWluCj4gPj4gV2VpJ3MgbmV3IGNvZGUuIMKgSSBmdXJ0aGVy IGhhZCB0byBwYXRjaCB0aGUgZHJpdmVyIHRvIHJlY29nbml6ZSBteQo+ID4+IGNhcmQgd2l0aDoK PiA+Pgo+ID4+IC0tLSBhbHNhLWRyaXZlci1vcmlnL2Fsc2Eta2VybmVsL3BjaS9oZGEvcGF0Y2hf bnZoZG1pLmMgMjAxMC0wMy0wNgo+ID4+IDE4OjAwOjEyLjAwMDAwMDAwMCAtMDgwMAo+ID4+ICsr KyBhbHNhLWRyaXZlci9hbHNhLWtlcm5lbC9wY2kvaGRhL3BhdGNoX252aGRtaS5jIMKgIMKgIMKg MjAxMC0wMy0wNgo+ID4+IDE4OjAyOjAwLjAwMDAwMDAwMCAtMDgwMAo+ID4+IEBAIC0xMzM1LDYg KzEzMzUsOCBAQCBzdGF0aWMgc3RydWN0IGhkYV9jb2RlY19wcmVzZXQgc25kX2hkYV9wCj4gPj4g wqAgwqAgwqAgwqAgwqAgLnBhdGNoID0gcGF0Y2hfbnZoZG1pXzhjaF84OSB9LAo+ID4+IMKgIMKg IMKgIMKgIHsgLmlkID0gMHgxMGRlMDAwYiwgLm5hbWUgPSAiR1QyMXggSERNSSIsCj4gPj4gwqAg wqAgwqAgwqAgwqAgLnBhdGNoID0gcGF0Y2hfbnZoZG1pXzhjaF84OSB9LAo+ID4+ICsgwqAgwqAg wqAgeyAuaWQgPSAweDEwZGUwMDBhLCAubmFtZSA9ICJHVDIyMCBIRE1JIiwKPiA+PiArIMKgIMKg IMKgIMKgIC5wYXRjaCA9IHBhdGNoX252aGRtaV84Y2hfODkgfSwKPiA+PiDCoCDCoCDCoCDCoCB7 IC5pZCA9IDB4MTBkZTAwMGQsIC5uYW1lID0gIkdUMjQwIEhETUkiLAo+ID4+IMKgIMKgIMKgIMKg IMKgIC5wYXRjaCA9IHBhdGNoX252aGRtaV84Y2hfODkgfSwKPiA+PiDCoCDCoCDCoCDCoCB7fSAv KiB0ZXJtaW5hdG9yICovCj4gPj4gQEAgLTEzNDcsNiArMTM0OSw3IEBAIE1PRFVMRV9BTElBUygi c25kLWhkYS1jb2RlYy1pZDoxMGRlMDAwNiIKPiA+PiDCoE1PRFVMRV9BTElBUygic25kLWhkYS1j b2RlYy1pZDoxMGRlMDAwNyIpOwo+ID4+IMKgTU9EVUxFX0FMSUFTKCJzbmQtaGRhLWNvZGVjLWlk OjEwZGUwMDY3Iik7Cj4gPj4gwqBNT0RVTEVfQUxJQVMoInNuZC1oZGEtY29kZWMtaWQ6MTBkZTgw MDEiKTsKPiA+PiArTU9EVUxFX0FMSUFTKCJzbmQtaGRhLWNvZGVjLWlkOjEwZGUwMDBhIik7Cj4g Pj4gwqBNT0RVTEVfQUxJQVMoInNuZC1oZGEtY29kZWMtaWQ6MTBkZTAwMGMiKTsKPiA+PiDCoE1P RFVMRV9BTElBUygic25kLWhkYS1jb2RlYy1pZDoxMGRlMDAwYiIpOwo+ID4+IMKgTU9EVUxFX0FM SUFTKCJzbmQtaGRhLWNvZGVjLWlkOjEwZGUwMDBkIik7Cj4gPj4KPiA+PiBUaGlzIHNlZW1zIHRv IHdvcmsgZmluZSB3aGVuIHJ1bm5pbmcgVkRSIGJ1dCBpZiBJIGRvIGFueXRoaW5nIGVsc2UKPiA+ PiBsaWtlIHVwZ3JhZGUgcGFja2FnZXMsIGNvbXBpbGUgc29tZXRoaW5nLCBvciBtYXliZSBjcmVh dGUgYSBiejIsIHRoZW4KPiA+PiBJIHN0YXJ0IHRvIHNlZSBfYSBsb3RfIG9mIHRoZSBmb2xsb3dp bmcgaW4gbXkgeGluZSBsb2c6Cj4gPj4KPiA+PiBwY21faHcuYzogc25kX3BjbV9od19kZWxheSgp IFNORFJWX1BDTV9JT0NUTF9ERUxBWSBmYWlsZWQuCj4gPj4KPiA+PiBXaXRoIHRoZSBvbGQgY29k ZSB0aGlzIGRpZG4ndCBoYXBwZW4uIMKgSG93ZXZlciwgbXkgc2V0dXAgd2FzIGFuIDg0MDBncwo+ ID4+IHVzaW5nIFNQRElGIE9VVCBmcm9tIG15IG1haW5ib2FyZCB0byBTUERJRiBJTiBvbiB0aGUg Y2FyZC4gwqBUaGVuIHVzaW5nCj4gPj4gYSBkdmktPmhkbWkgY2FibGUgaW50byB0aGUgdHYuCj4g Pj4KPiA+PiBBZ2FpbiwgSSBhcG9sb2dpemUgaWYgSSdtIHBvc3RpbmcgdGhpcyBpbiB0aGUgd3Jv bmcgcGxhY2UuCj4gPj4KPiA+PiBCZXN0IHJlZ2FyZHMuCj4gCj4gU28gbm9ib2R5IGhhcyBhbnkg aWRlYXMgYXQgYWxsPyAgVGhpcyBpcyB0dXJuaW5nIG91dCB0byBiZSBhIGJpZwo+IHByb2JsZW0g YW5kIEkndmUgZGlzY292ZXJlZCBpdCB3aWxsIG9jY3VyIGV2ZW4gd2l0aG91dCBhbnl0aGluZyBl bHNlCj4gaGFwcGVuaW5nIG9uIHRoZSBzeXN0ZW0uICAob3JpZ2luYWxseSBJIHRob3VnaHQgeW91 IG5lZWRlZCB0byBkbyBzb21lCj4gb3RoZXIgYWN0aXZpdHkuKSAgSSd2ZSBjaGVja2VkIHdpdGgg c29tZSBvdGhlciBkZXZzIEkga25vdyBhbmQKPiBldmVyeW9uZSBoYXMgc2FpZCB0aGUgcHJvYmxl bSBpcyB3aXRoIGFsc2Egc28gaXQgd291bGQgc2VlbSB0aGVyZSBpcyBhCj4gc2VyaW91cyBidWcg c29tZXdoZXJlIHNpbmNlIGV2ZW50dWFsbHkgdGhvc2UgZXJyb3JzIHdpbGwgaGFwcGVuIGFuZAo+ IHRoZSBhdWRpbyB3aWxsIHN0YXJ0IHNraXBwaW5nIGxpa2UgY3JhenkuCj4gCj4gUGxlYXNlIGxl dCBtZSBrbm93IGlmIHRoZXJlJ3MgYW55dGhpbmcgSSBjYW4gZG8gd2l0aGluIG15IGFiaWxpdHkg dG8gaGVscC4KClRoZSBmaXJzdCBzdGVwIHdvdWxkIGJlIHRvIHJ1biB2YW5pbGxhIDIuNi4zMyB3 aXRoIFdlaSBOaSdzIHBhdGNoLAppbiBvcmRlciB0byB0ZWxsIHdoZXRoZXIgdGhlIGJ1ZyBpcyBp bnRlcm5hbCB0byB0aGUgSERNSSBjb2RlLCBvciBpcwppbnRyb2R1Y2VkIGJ5IGJsZWVkaW5nIGVk Z2UgQUxTQSBkZXZlbG9wbWVudC4KClRoYW5rcywKRmVuZ2d1YW5nCl9fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCkFsc2EtZGV2ZWwgbWFpbGluZyBsaXN0CkFs c2EtZGV2ZWxAYWxzYS1wcm9qZWN0Lm9yZwpodHRwOi8vbWFpbG1hbi5hbHNhLXByb2plY3Qub3Jn L21haWxtYW4vbGlzdGluZm8vYWxzYS1kZXZlbAo= From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932089Ab0CJGBP (ORCPT ); Wed, 10 Mar 2010 01:01:15 -0500 Received: from mga01.intel.com ([192.55.52.88]:25270 "EHLO mga01.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754325Ab0CJGBM (ORCPT ); Wed, 10 Mar 2010 01:01:12 -0500 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.49,612,1262592000"; d="scan'208";a="547689212" Date: Wed, 10 Mar 2010 14:01:07 +0800 From: Wu Fengguang To: VDR User Cc: Wei Ni , Takashi Iwai , akpm , alsa-devel , linux-kernel , Pavel Hofman , Jaroslav Kysela Subject: Re: [alsa-devel] [PATCH]Support MCP89 and GT21x hdmi audio Message-ID: <20100310060107.GA27334@localhost> References: <6B4D417B830BC44B8026029FD256F7F1C16B1AD556@HKMAIL01.nvidia.com> <6B4D417B830BC44B8026029FD256F7F1C16B1AD557@HKMAIL01.nvidia.com> <6B4D417B830BC44B8026029FD256F7F1C16B1AD558@HKMAIL01.nvidia.com> <6B4D417B830BC44B8026029FD256F7F1C16B1AD562@HKMAIL01.nvidia.com> <20100308044614.GA29026@localhost> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: Mutt/1.5.18 (2008-05-17) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Mar 10, 2010 at 01:42:05PM +0800, VDR User wrote: > On Sun, Mar 7, 2010 at 8:46 PM, Wu Fengguang wrote: > > CC Jaroslav, maybe he has some idea on > > > >        pcm_hw.c: snd_pcm_hw_delay() SNDRV_PCM_IOCTL_DELAY failed. > > > > On Mon, Mar 08, 2010 at 12:35:37PM +0800, Wei Ni wrote: > >> Hi, > >> Because I don't have GT220 card on hand, I didn't test it yet. > >> So I didn't add this card to the patch file. I think you could add it after > >> you test it. > >> > >> About the warning messages, I don't know why. > >> May be some others can answer. > >> > >> Thanks > >> Wei. > >> nvpublic > >> > >> -----Original Message----- > >> From: VDR User [mailto:user.vdr@gmail.com] > >> Sent: Monday, March 08, 2010 3:22 AM > >> To: Wei Ni > >> Cc: Takashi Iwai; akpm; alsa-devel; Wu Fengguang; linux-kernel; Pavel Hofman > >> Subject: Re: [alsa-devel] [PATCH]Support MCP89 and GT21x hdmi audio > >> > >> I think this may be relevant to this subject, if not then my apologies. > >> > >> I've just installed a GT220 card and was able to get audio-over-hdmi > >> working by compiling the snapshot drivers from 20100307 which contain > >> Wei's new code.  I further had to patch the driver to recognize my > >> card with: > >> > >> --- alsa-driver-orig/alsa-kernel/pci/hda/patch_nvhdmi.c 2010-03-06 > >> 18:00:12.000000000 -0800 > >> +++ alsa-driver/alsa-kernel/pci/hda/patch_nvhdmi.c      2010-03-06 > >> 18:02:00.000000000 -0800 > >> @@ -1335,6 +1335,8 @@ static struct hda_codec_preset snd_hda_p > >>           .patch = patch_nvhdmi_8ch_89 }, > >>         { .id = 0x10de000b, .name = "GT21x HDMI", > >>           .patch = patch_nvhdmi_8ch_89 }, > >> +       { .id = 0x10de000a, .name = "GT220 HDMI", > >> +         .patch = patch_nvhdmi_8ch_89 }, > >>         { .id = 0x10de000d, .name = "GT240 HDMI", > >>           .patch = patch_nvhdmi_8ch_89 }, > >>         {} /* terminator */ > >> @@ -1347,6 +1349,7 @@ MODULE_ALIAS("snd-hda-codec-id:10de0006" > >>  MODULE_ALIAS("snd-hda-codec-id:10de0007"); > >>  MODULE_ALIAS("snd-hda-codec-id:10de0067"); > >>  MODULE_ALIAS("snd-hda-codec-id:10de8001"); > >> +MODULE_ALIAS("snd-hda-codec-id:10de000a"); > >>  MODULE_ALIAS("snd-hda-codec-id:10de000c"); > >>  MODULE_ALIAS("snd-hda-codec-id:10de000b"); > >>  MODULE_ALIAS("snd-hda-codec-id:10de000d"); > >> > >> This seems to work fine when running VDR but if I do anything else > >> like upgrade packages, compile something, or maybe create a bz2, then > >> I start to see _a lot_ of the following in my xine log: > >> > >> pcm_hw.c: snd_pcm_hw_delay() SNDRV_PCM_IOCTL_DELAY failed. > >> > >> With the old code this didn't happen.  However, my setup was an 8400gs > >> using SPDIF OUT from my mainboard to SPDIF IN on the card.  Then using > >> a dvi->hdmi cable into the tv. > >> > >> Again, I apologize if I'm posting this in the wrong place. > >> > >> Best regards. > > So nobody has any ideas at all? This is turning out to be a big > problem and I've discovered it will occur even without anything else > happening on the system. (originally I thought you needed to do some > other activity.) I've checked with some other devs I know and > everyone has said the problem is with alsa so it would seem there is a > serious bug somewhere since eventually those errors will happen and > the audio will start skipping like crazy. > > Please let me know if there's anything I can do within my ability to help. The first step would be to run vanilla 2.6.33 with Wei Ni's patch, in order to tell whether the bug is internal to the HDMI code, or is introduced by bleeding edge ALSA development. Thanks, Fengguang