* v3.15-rc1: CORB reset timeout#1, CORBRP = 0 @ 2014-04-16 8:30 Ronald 2014-04-28 15:34 ` Takashi Iwai 0 siblings, 1 reply; 6+ messages in thread From: Ronald @ 2014-04-16 8:30 UTC (permalink / raw) To: alsa-devel Just rebuild v3.15. The message in the subject appears to be at the error level. Sound plays fine. Everything seems to work as intended. [ 6.779741] snd_hda_intel 0000:00:10.1: Disabling MSI [ 6.781978] ALSA device list: [ 6.781980] No soundcards found. [ 6.785907] snd_hda_intel 0000:00:10.1: CORB reset timeout#1, CORBRP = 0 [ 7.670040] sound hdaudioC0D0: autoconfig: line_outs=1 (0x10/0x0/0x0/0x0/0x0) type:speaker [ 7.670046] sound hdaudioC0D0: speaker_outs=0 (0x0/0x0/0x0/0x0/0x0) [ 7.670049] sound hdaudioC0D0: hp_outs=1 (0x11/0x0/0x0/0x0/0x0) [ 7.670052] sound hdaudioC0D0: mono: mono_out=0x0 [ 7.670055] sound hdaudioC0D0: dig-out=0x13/0x0 [ 7.670057] sound hdaudioC0D0: inputs: [ 7.670060] sound hdaudioC0D0: Mic=0x14 [ 8.730734] input: HDA Digital PCBeep as /devices/pci0000:00/0000:00:10.1/sound/card0/hdaudioC0D0/input10 [ 8.731256] input: HDA NVidia Front Headphone as /devices/pci0000:00/0000:00:10.1/sound/card0/input11 Is this reason for concern? ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: v3.15-rc1: CORB reset timeout#1, CORBRP = 0 2014-04-16 8:30 v3.15-rc1: CORB reset timeout#1, CORBRP = 0 Ronald @ 2014-04-28 15:34 ` Takashi Iwai 2014-04-29 15:08 ` Ronald 0 siblings, 1 reply; 6+ messages in thread From: Takashi Iwai @ 2014-04-28 15:34 UTC (permalink / raw) To: Ronald; +Cc: alsa-devel, David Henningsson At Wed, 16 Apr 2014 10:30:13 +0200, Ronald wrote: > > Just rebuild v3.15. The message in the subject appears to be at the > error level. Sound plays fine. Everything seems to work as intended. > > [ 6.779741] snd_hda_intel 0000:00:10.1: Disabling MSI > [ 6.781978] ALSA device list: > [ 6.781980] No soundcards found. > [ 6.785907] snd_hda_intel 0000:00:10.1: CORB reset timeout#1, CORBRP = 0 > [ 7.670040] sound hdaudioC0D0: autoconfig: line_outs=1 > (0x10/0x0/0x0/0x0/0x0) type:speaker > [ 7.670046] sound hdaudioC0D0: speaker_outs=0 (0x0/0x0/0x0/0x0/0x0) > [ 7.670049] sound hdaudioC0D0: hp_outs=1 (0x11/0x0/0x0/0x0/0x0) > [ 7.670052] sound hdaudioC0D0: mono: mono_out=0x0 > [ 7.670055] sound hdaudioC0D0: dig-out=0x13/0x0 > [ 7.670057] sound hdaudioC0D0: inputs: > [ 7.670060] sound hdaudioC0D0: Mic=0x14 > [ 8.730734] input: HDA Digital PCBeep as > /devices/pci0000:00/0000:00:10.1/sound/card0/hdaudioC0D0/input10 > [ 8.731256] input: HDA NVidia Front Headphone as > /devices/pci0000:00/0000:00:10.1/sound/card0/input11 > > Is this reason for concern? This comes from the commit ca460f86521ed515d17dd1314f7b95183866f681 ALSA: hda - Fix CORB reset to follow specification I guess it's a non-standard behavior of Nvidia controller. The write to these bits are likely ignored, so it's mostly harmless. For avoiding the confusion, we may need to paper over the changed reset procedure for such chips. Could you give the specific of your hardware, especially about this HD-audio controller chip? thanks, Takashi ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: v3.15-rc1: CORB reset timeout#1, CORBRP = 0 2014-04-28 15:34 ` Takashi Iwai @ 2014-04-29 15:08 ` Ronald 2014-04-29 15:24 ` Takashi Iwai 0 siblings, 1 reply; 6+ messages in thread From: Ronald @ 2014-04-29 15:08 UTC (permalink / raw) To: Takashi Iwai; +Cc: alsa-devel, David Henningsson I take it you want the vendor:device id, but since you have requested additional info I have provided some more data. Please let me know if I missed anything you require. Thank you. Below you will find lspci. I have used the alsa-info.sh script for the rest (how convenient!): http://www.alsa-project.org/db/?f=daf13e17122d2db2851591b4d79d3ac34fabf864 00:10.1 Audio device [0403]: NVIDIA Corporation MCP51 High Definition Audio [10de:026c] (rev a2) Subsystem: Hewlett-Packard Company Presario V6133CL [103c:30b7] Control: I/O- Mem+ BusMaster+ SpecCycle- MemWINV- VGASnoop- ParErr- Stepping- SERR- FastB2B- DisINTx- Status: Cap+ 66MHz+ UDF- FastB2B+ ParErr- DEVSEL=fast >TAbort- <TAbort- <MAbort- >SERR- <PERR- INTx- Latency: 0 (500ns min, 1250ns max) Interrupt: pin B routed to IRQ 21 Region 0: Memory at b0000000 (32-bit, non-prefetchable) [size=16K] Capabilities: [44] Power Management version 2 Flags: PMEClk- DSI- D1- D2- AuxCurrent=0mA PME(D0-,D1-,D2-,D3hot+,D3cold+) Status: D0 NoSoftRst- PME-Enable- DSel=0 DScale=0 PME- Capabilities: [50] MSI: Enable- Count=1/1 Maskable+ 64bit+ Address: 0000000000000000 Data: 0000 Masking: 00000000 Pending: 00000000 Capabilities: [6c] HyperTransport: MSI Mapping Enable- Fixed+ Kernel driver in use: snd_hda_intel 2014-04-28 17:34 GMT+02:00 Takashi Iwai <tiwai@suse.de>: > At Wed, 16 Apr 2014 10:30:13 +0200, > Ronald wrote: >> >> Just rebuild v3.15. The message in the subject appears to be at the >> error level. Sound plays fine. Everything seems to work as intended. >> >> [ 6.779741] snd_hda_intel 0000:00:10.1: Disabling MSI >> [ 6.781978] ALSA device list: >> [ 6.781980] No soundcards found. >> [ 6.785907] snd_hda_intel 0000:00:10.1: CORB reset timeout#1, CORBRP = 0 >> [ 7.670040] sound hdaudioC0D0: autoconfig: line_outs=1 >> (0x10/0x0/0x0/0x0/0x0) type:speaker >> [ 7.670046] sound hdaudioC0D0: speaker_outs=0 (0x0/0x0/0x0/0x0/0x0) >> [ 7.670049] sound hdaudioC0D0: hp_outs=1 (0x11/0x0/0x0/0x0/0x0) >> [ 7.670052] sound hdaudioC0D0: mono: mono_out=0x0 >> [ 7.670055] sound hdaudioC0D0: dig-out=0x13/0x0 >> [ 7.670057] sound hdaudioC0D0: inputs: >> [ 7.670060] sound hdaudioC0D0: Mic=0x14 >> [ 8.730734] input: HDA Digital PCBeep as >> /devices/pci0000:00/0000:00:10.1/sound/card0/hdaudioC0D0/input10 >> [ 8.731256] input: HDA NVidia Front Headphone as >> /devices/pci0000:00/0000:00:10.1/sound/card0/input11 >> >> Is this reason for concern? > > This comes from the commit ca460f86521ed515d17dd1314f7b95183866f681 > ALSA: hda - Fix CORB reset to follow specification > > I guess it's a non-standard behavior of Nvidia controller. The write > to these bits are likely ignored, so it's mostly harmless. > > For avoiding the confusion, we may need to paper over the changed > reset procedure for such chips. Could you give the specific of your > hardware, especially about this HD-audio controller chip? > > > thanks, > > Takashi ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: v3.15-rc1: CORB reset timeout#1, CORBRP = 0 2014-04-29 15:08 ` Ronald @ 2014-04-29 15:24 ` Takashi Iwai 2014-04-29 16:31 ` Ronald 0 siblings, 1 reply; 6+ messages in thread From: Takashi Iwai @ 2014-04-29 15:24 UTC (permalink / raw) To: Ronald; +Cc: alsa-devel, David Henningsson At Tue, 29 Apr 2014 17:08:25 +0200, Ronald wrote: > > I take it you want the vendor:device id, but since you have requested > additional info I have provided some more data. Please let me know if > I missed anything you require. Thank you. > > Below you will find lspci. I have used the alsa-info.sh script for the > rest (how convenient!): > > http://www.alsa-project.org/db/?f=daf13e17122d2db2851591b4d79d3ac34fabf864 > > 00:10.1 Audio device [0403]: NVIDIA Corporation MCP51 High Definition > Audio [10de:026c] (rev a2) > Subsystem: Hewlett-Packard Company Presario V6133CL [103c:30b7] > Control: I/O- Mem+ BusMaster+ SpecCycle- MemWINV- VGASnoop- > ParErr- Stepping- SERR- FastB2B- DisINTx- > Status: Cap+ 66MHz+ UDF- FastB2B+ ParErr- DEVSEL=fast >TAbort- > <TAbort- <MAbort- >SERR- <PERR- INTx- > Latency: 0 (500ns min, 1250ns max) > Interrupt: pin B routed to IRQ 21 > Region 0: Memory at b0000000 (32-bit, non-prefetchable) [size=16K] > Capabilities: [44] Power Management version 2 > Flags: PMEClk- DSI- D1- D2- AuxCurrent=0mA > PME(D0-,D1-,D2-,D3hot+,D3cold+) > Status: D0 NoSoftRst- PME-Enable- DSel=0 DScale=0 PME- > Capabilities: [50] MSI: Enable- Count=1/1 Maskable+ 64bit+ > Address: 0000000000000000 Data: 0000 > Masking: 00000000 Pending: 00000000 > Capabilities: [6c] HyperTransport: MSI Mapping Enable- Fixed+ > Kernel driver in use: snd_hda_intel Thanks. Could you check whether the patch below works? Takashi --- diff --git a/sound/pci/hda/hda_controller.c b/sound/pci/hda/hda_controller.c index 248b90abb882..480bbddbd801 100644 --- a/sound/pci/hda/hda_controller.c +++ b/sound/pci/hda/hda_controller.c @@ -1059,24 +1059,26 @@ static void azx_init_cmd_io(struct azx *chip) /* reset the corb hw read pointer */ azx_writew(chip, CORBRP, ICH6_CORBRP_RST); - for (timeout = 1000; timeout > 0; timeout--) { - if ((azx_readw(chip, CORBRP) & ICH6_CORBRP_RST) == ICH6_CORBRP_RST) - break; - udelay(1); - } - if (timeout <= 0) - dev_err(chip->card->dev, "CORB reset timeout#1, CORBRP = %d\n", - azx_readw(chip, CORBRP)); + if (!(chip->driver_caps & AZX_DCAPS_CORBRP_SELF_CLEAR)) { + for (timeout = 1000; timeout > 0; timeout--) { + if ((azx_readw(chip, CORBRP) & ICH6_CORBRP_RST) == ICH6_CORBRP_RST) + break; + udelay(1); + } + if (timeout <= 0) + dev_err(chip->card->dev, "CORB reset timeout#1, CORBRP = %d\n", + azx_readw(chip, CORBRP)); - azx_writew(chip, CORBRP, 0); - for (timeout = 1000; timeout > 0; timeout--) { - if (azx_readw(chip, CORBRP) == 0) - break; - udelay(1); + azx_writew(chip, CORBRP, 0); + for (timeout = 1000; timeout > 0; timeout--) { + if (azx_readw(chip, CORBRP) == 0) + break; + udelay(1); + } + if (timeout <= 0) + dev_err(chip->card->dev, "CORB reset timeout#2, CORBRP = %d\n", + azx_readw(chip, CORBRP)); } - if (timeout <= 0) - dev_err(chip->card->dev, "CORB reset timeout#2, CORBRP = %d\n", - azx_readw(chip, CORBRP)); /* enable corb dma */ azx_writeb(chip, CORBCTL, ICH6_CORBCTL_RUN); diff --git a/sound/pci/hda/hda_intel.c b/sound/pci/hda/hda_intel.c index d6bca62ef387..b540ad71eb0d 100644 --- a/sound/pci/hda/hda_intel.c +++ b/sound/pci/hda/hda_intel.c @@ -249,7 +249,8 @@ enum { /* quirks for Nvidia */ #define AZX_DCAPS_PRESET_NVIDIA \ (AZX_DCAPS_NVIDIA_SNOOP | AZX_DCAPS_RIRB_DELAY | AZX_DCAPS_NO_MSI |\ - AZX_DCAPS_ALIGN_BUFSIZE | AZX_DCAPS_NO_64BIT) + AZX_DCAPS_ALIGN_BUFSIZE | AZX_DCAPS_NO_64BIT |\ + AZX_DCAPS_CORBRP_SELF_CLEAR) #define AZX_DCAPS_PRESET_CTHDA \ (AZX_DCAPS_NO_MSI | AZX_DCAPS_POSFIX_LPIB | AZX_DCAPS_4K_BDLE_BOUNDARY) diff --git a/sound/pci/hda/hda_priv.h b/sound/pci/hda/hda_priv.h index ba38b819f984..4a7cb01fa912 100644 --- a/sound/pci/hda/hda_priv.h +++ b/sound/pci/hda/hda_priv.h @@ -189,6 +189,7 @@ enum { SDI0, SDI1, SDI2, SDI3, SDO0, SDO1, SDO2, SDO3 }; #define AZX_DCAPS_COUNT_LPIB_DELAY (1 << 25) /* Take LPIB as delay */ #define AZX_DCAPS_PM_RUNTIME (1 << 26) /* runtime PM support */ #define AZX_DCAPS_I915_POWERWELL (1 << 27) /* HSW i915 powerwell support */ +#define AZX_DCAPS_CORBRP_SELF_CLEAR (1 << 28) /* CORBRP clears itself after reset */ /* position fix mode */ enum { ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: v3.15-rc1: CORB reset timeout#1, CORBRP = 0 2014-04-29 15:24 ` Takashi Iwai @ 2014-04-29 16:31 ` Ronald 2014-04-29 16:42 ` Takashi Iwai 0 siblings, 1 reply; 6+ messages in thread From: Ronald @ 2014-04-29 16:31 UTC (permalink / raw) To: Takashi Iwai; +Cc: alsa-devel, David Henningsson It 'works' as in: the message did not reappear. Thanks a ton! 2014-04-29 17:24 GMT+02:00 Takashi Iwai <tiwai@suse.de>: > At Tue, 29 Apr 2014 17:08:25 +0200, > Ronald wrote: >> >> I take it you want the vendor:device id, but since you have requested >> additional info I have provided some more data. Please let me know if >> I missed anything you require. Thank you. >> >> Below you will find lspci. I have used the alsa-info.sh script for the >> rest (how convenient!): >> >> http://www.alsa-project.org/db/?f=daf13e17122d2db2851591b4d79d3ac34fabf864 >> >> 00:10.1 Audio device [0403]: NVIDIA Corporation MCP51 High Definition >> Audio [10de:026c] (rev a2) >> Subsystem: Hewlett-Packard Company Presario V6133CL [103c:30b7] >> Control: I/O- Mem+ BusMaster+ SpecCycle- MemWINV- VGASnoop- >> ParErr- Stepping- SERR- FastB2B- DisINTx- >> Status: Cap+ 66MHz+ UDF- FastB2B+ ParErr- DEVSEL=fast >TAbort- >> <TAbort- <MAbort- >SERR- <PERR- INTx- >> Latency: 0 (500ns min, 1250ns max) >> Interrupt: pin B routed to IRQ 21 >> Region 0: Memory at b0000000 (32-bit, non-prefetchable) [size=16K] >> Capabilities: [44] Power Management version 2 >> Flags: PMEClk- DSI- D1- D2- AuxCurrent=0mA >> PME(D0-,D1-,D2-,D3hot+,D3cold+) >> Status: D0 NoSoftRst- PME-Enable- DSel=0 DScale=0 PME- >> Capabilities: [50] MSI: Enable- Count=1/1 Maskable+ 64bit+ >> Address: 0000000000000000 Data: 0000 >> Masking: 00000000 Pending: 00000000 >> Capabilities: [6c] HyperTransport: MSI Mapping Enable- Fixed+ >> Kernel driver in use: snd_hda_intel > > Thanks. Could you check whether the patch below works? > > > Takashi > > --- > diff --git a/sound/pci/hda/hda_controller.c b/sound/pci/hda/hda_controller.c > index 248b90abb882..480bbddbd801 100644 > --- a/sound/pci/hda/hda_controller.c > +++ b/sound/pci/hda/hda_controller.c > @@ -1059,24 +1059,26 @@ static void azx_init_cmd_io(struct azx *chip) > > /* reset the corb hw read pointer */ > azx_writew(chip, CORBRP, ICH6_CORBRP_RST); > - for (timeout = 1000; timeout > 0; timeout--) { > - if ((azx_readw(chip, CORBRP) & ICH6_CORBRP_RST) == ICH6_CORBRP_RST) > - break; > - udelay(1); > - } > - if (timeout <= 0) > - dev_err(chip->card->dev, "CORB reset timeout#1, CORBRP = %d\n", > - azx_readw(chip, CORBRP)); > + if (!(chip->driver_caps & AZX_DCAPS_CORBRP_SELF_CLEAR)) { > + for (timeout = 1000; timeout > 0; timeout--) { > + if ((azx_readw(chip, CORBRP) & ICH6_CORBRP_RST) == ICH6_CORBRP_RST) > + break; > + udelay(1); > + } > + if (timeout <= 0) > + dev_err(chip->card->dev, "CORB reset timeout#1, CORBRP = %d\n", > + azx_readw(chip, CORBRP)); > > - azx_writew(chip, CORBRP, 0); > - for (timeout = 1000; timeout > 0; timeout--) { > - if (azx_readw(chip, CORBRP) == 0) > - break; > - udelay(1); > + azx_writew(chip, CORBRP, 0); > + for (timeout = 1000; timeout > 0; timeout--) { > + if (azx_readw(chip, CORBRP) == 0) > + break; > + udelay(1); > + } > + if (timeout <= 0) > + dev_err(chip->card->dev, "CORB reset timeout#2, CORBRP = %d\n", > + azx_readw(chip, CORBRP)); > } > - if (timeout <= 0) > - dev_err(chip->card->dev, "CORB reset timeout#2, CORBRP = %d\n", > - azx_readw(chip, CORBRP)); > > /* enable corb dma */ > azx_writeb(chip, CORBCTL, ICH6_CORBCTL_RUN); > diff --git a/sound/pci/hda/hda_intel.c b/sound/pci/hda/hda_intel.c > index d6bca62ef387..b540ad71eb0d 100644 > --- a/sound/pci/hda/hda_intel.c > +++ b/sound/pci/hda/hda_intel.c > @@ -249,7 +249,8 @@ enum { > /* quirks for Nvidia */ > #define AZX_DCAPS_PRESET_NVIDIA \ > (AZX_DCAPS_NVIDIA_SNOOP | AZX_DCAPS_RIRB_DELAY | AZX_DCAPS_NO_MSI |\ > - AZX_DCAPS_ALIGN_BUFSIZE | AZX_DCAPS_NO_64BIT) > + AZX_DCAPS_ALIGN_BUFSIZE | AZX_DCAPS_NO_64BIT |\ > + AZX_DCAPS_CORBRP_SELF_CLEAR) > > #define AZX_DCAPS_PRESET_CTHDA \ > (AZX_DCAPS_NO_MSI | AZX_DCAPS_POSFIX_LPIB | AZX_DCAPS_4K_BDLE_BOUNDARY) > diff --git a/sound/pci/hda/hda_priv.h b/sound/pci/hda/hda_priv.h > index ba38b819f984..4a7cb01fa912 100644 > --- a/sound/pci/hda/hda_priv.h > +++ b/sound/pci/hda/hda_priv.h > @@ -189,6 +189,7 @@ enum { SDI0, SDI1, SDI2, SDI3, SDO0, SDO1, SDO2, SDO3 }; > #define AZX_DCAPS_COUNT_LPIB_DELAY (1 << 25) /* Take LPIB as delay */ > #define AZX_DCAPS_PM_RUNTIME (1 << 26) /* runtime PM support */ > #define AZX_DCAPS_I915_POWERWELL (1 << 27) /* HSW i915 powerwell support */ > +#define AZX_DCAPS_CORBRP_SELF_CLEAR (1 << 28) /* CORBRP clears itself after reset */ > > /* position fix mode */ > enum { ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: v3.15-rc1: CORB reset timeout#1, CORBRP = 0 2014-04-29 16:31 ` Ronald @ 2014-04-29 16:42 ` Takashi Iwai 0 siblings, 0 replies; 6+ messages in thread From: Takashi Iwai @ 2014-04-29 16:42 UTC (permalink / raw) To: Ronald; +Cc: alsa-devel, David Henningsson At Tue, 29 Apr 2014 18:31:44 +0200, Ronald wrote: > > It 'works' as in: the message did not reappear. Thanks a ton! Good to hear. Now I applied the patch. thanks, Takashi > > 2014-04-29 17:24 GMT+02:00 Takashi Iwai <tiwai@suse.de>: > > At Tue, 29 Apr 2014 17:08:25 +0200, > > Ronald wrote: > >> > >> I take it you want the vendor:device id, but since you have requested > >> additional info I have provided some more data. Please let me know if > >> I missed anything you require. Thank you. > >> > >> Below you will find lspci. I have used the alsa-info.sh script for the > >> rest (how convenient!): > >> > >> http://www.alsa-project.org/db/?f=daf13e17122d2db2851591b4d79d3ac34fabf864 > >> > >> 00:10.1 Audio device [0403]: NVIDIA Corporation MCP51 High Definition > >> Audio [10de:026c] (rev a2) > >> Subsystem: Hewlett-Packard Company Presario V6133CL [103c:30b7] > >> Control: I/O- Mem+ BusMaster+ SpecCycle- MemWINV- VGASnoop- > >> ParErr- Stepping- SERR- FastB2B- DisINTx- > >> Status: Cap+ 66MHz+ UDF- FastB2B+ ParErr- DEVSEL=fast >TAbort- > >> <TAbort- <MAbort- >SERR- <PERR- INTx- > >> Latency: 0 (500ns min, 1250ns max) > >> Interrupt: pin B routed to IRQ 21 > >> Region 0: Memory at b0000000 (32-bit, non-prefetchable) [size=16K] > >> Capabilities: [44] Power Management version 2 > >> Flags: PMEClk- DSI- D1- D2- AuxCurrent=0mA > >> PME(D0-,D1-,D2-,D3hot+,D3cold+) > >> Status: D0 NoSoftRst- PME-Enable- DSel=0 DScale=0 PME- > >> Capabilities: [50] MSI: Enable- Count=1/1 Maskable+ 64bit+ > >> Address: 0000000000000000 Data: 0000 > >> Masking: 00000000 Pending: 00000000 > >> Capabilities: [6c] HyperTransport: MSI Mapping Enable- Fixed+ > >> Kernel driver in use: snd_hda_intel > > > > Thanks. Could you check whether the patch below works? > > > > > > Takashi > > > > --- > > diff --git a/sound/pci/hda/hda_controller.c b/sound/pci/hda/hda_controller.c > > index 248b90abb882..480bbddbd801 100644 > > --- a/sound/pci/hda/hda_controller.c > > +++ b/sound/pci/hda/hda_controller.c > > @@ -1059,24 +1059,26 @@ static void azx_init_cmd_io(struct azx *chip) > > > > /* reset the corb hw read pointer */ > > azx_writew(chip, CORBRP, ICH6_CORBRP_RST); > > - for (timeout = 1000; timeout > 0; timeout--) { > > - if ((azx_readw(chip, CORBRP) & ICH6_CORBRP_RST) == ICH6_CORBRP_RST) > > - break; > > - udelay(1); > > - } > > - if (timeout <= 0) > > - dev_err(chip->card->dev, "CORB reset timeout#1, CORBRP = %d\n", > > - azx_readw(chip, CORBRP)); > > + if (!(chip->driver_caps & AZX_DCAPS_CORBRP_SELF_CLEAR)) { > > + for (timeout = 1000; timeout > 0; timeout--) { > > + if ((azx_readw(chip, CORBRP) & ICH6_CORBRP_RST) == ICH6_CORBRP_RST) > > + break; > > + udelay(1); > > + } > > + if (timeout <= 0) > > + dev_err(chip->card->dev, "CORB reset timeout#1, CORBRP = %d\n", > > + azx_readw(chip, CORBRP)); > > > > - azx_writew(chip, CORBRP, 0); > > - for (timeout = 1000; timeout > 0; timeout--) { > > - if (azx_readw(chip, CORBRP) == 0) > > - break; > > - udelay(1); > > + azx_writew(chip, CORBRP, 0); > > + for (timeout = 1000; timeout > 0; timeout--) { > > + if (azx_readw(chip, CORBRP) == 0) > > + break; > > + udelay(1); > > + } > > + if (timeout <= 0) > > + dev_err(chip->card->dev, "CORB reset timeout#2, CORBRP = %d\n", > > + azx_readw(chip, CORBRP)); > > } > > - if (timeout <= 0) > > - dev_err(chip->card->dev, "CORB reset timeout#2, CORBRP = %d\n", > > - azx_readw(chip, CORBRP)); > > > > /* enable corb dma */ > > azx_writeb(chip, CORBCTL, ICH6_CORBCTL_RUN); > > diff --git a/sound/pci/hda/hda_intel.c b/sound/pci/hda/hda_intel.c > > index d6bca62ef387..b540ad71eb0d 100644 > > --- a/sound/pci/hda/hda_intel.c > > +++ b/sound/pci/hda/hda_intel.c > > @@ -249,7 +249,8 @@ enum { > > /* quirks for Nvidia */ > > #define AZX_DCAPS_PRESET_NVIDIA \ > > (AZX_DCAPS_NVIDIA_SNOOP | AZX_DCAPS_RIRB_DELAY | AZX_DCAPS_NO_MSI |\ > > - AZX_DCAPS_ALIGN_BUFSIZE | AZX_DCAPS_NO_64BIT) > > + AZX_DCAPS_ALIGN_BUFSIZE | AZX_DCAPS_NO_64BIT |\ > > + AZX_DCAPS_CORBRP_SELF_CLEAR) > > > > #define AZX_DCAPS_PRESET_CTHDA \ > > (AZX_DCAPS_NO_MSI | AZX_DCAPS_POSFIX_LPIB | AZX_DCAPS_4K_BDLE_BOUNDARY) > > diff --git a/sound/pci/hda/hda_priv.h b/sound/pci/hda/hda_priv.h > > index ba38b819f984..4a7cb01fa912 100644 > > --- a/sound/pci/hda/hda_priv.h > > +++ b/sound/pci/hda/hda_priv.h > > @@ -189,6 +189,7 @@ enum { SDI0, SDI1, SDI2, SDI3, SDO0, SDO1, SDO2, SDO3 }; > > #define AZX_DCAPS_COUNT_LPIB_DELAY (1 << 25) /* Take LPIB as delay */ > > #define AZX_DCAPS_PM_RUNTIME (1 << 26) /* runtime PM support */ > > #define AZX_DCAPS_I915_POWERWELL (1 << 27) /* HSW i915 powerwell support */ > > +#define AZX_DCAPS_CORBRP_SELF_CLEAR (1 << 28) /* CORBRP clears itself after reset */ > > > > /* position fix mode */ > > enum { > ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2014-04-29 16:42 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2014-04-16 8:30 v3.15-rc1: CORB reset timeout#1, CORBRP = 0 Ronald 2014-04-28 15:34 ` Takashi Iwai 2014-04-29 15:08 ` Ronald 2014-04-29 15:24 ` Takashi Iwai 2014-04-29 16:31 ` Ronald 2014-04-29 16:42 ` Takashi Iwai
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox