At Wed, 22 Jan 2014 13:25:34 +0100,
David Henningsson wrote:
> 
> On 01/22/2014 12:40 PM, Takashi Iwai wrote:
> > At Wed, 22 Jan 2014 12:32:28 +0100,
> > David Henningsson wrote:
> >>
> >> From: Niraj <[email protected]>
> >>
> >> Apparently this board seems to need some extra initialization before a 
> >> proper
> >> codec probe value can be read.
> >>
> >> BugLink: https://bugs.launchpad.net/bugs/1155202
> >> Signed-off-by: David Henningsson <[email protected]>
> >> ---
> >>  sound/pci/hda/hda_intel.c |    6 ++++++
> >>  1 file changed, 6 insertions(+)
> >>
> >> Hi Takashi,
> >>
> >> Here's a patch from Niraj, which has found this code below to make audio 
> >> work
> >> on the Intel DZ77SL-50K board. Without it, no codecs will be detected.
> >>
> >> I'm not sure if you like it this way or if you want me or Niraj to change
> >> anything, or debug further. Or maybe add a new quirk for this particular 
> >> device?
> > 
> > Of course, I don't like this :)
> > 
> > The patch essentially disables the codec reset in *all* cases after a
> > codec is probed.  This smells badly of a future regression.
> 
> Hmm, but I don't see how the patch *disables* any codec reset. It only
> does an *extra* controller reset in some cases.

Ah, yes.  I misread the code.

> > FWIW, you can set a forced codec probe mask in probe_mask_list[].
> > The bit 0x100 means to force the detection.
> 
> Yes, we tried this and it did not help. If the codecs do not show up
> naturally in STATETS, then they cannot be communicated with either.

OK.

> > But, before going that direction, could you clarify how this fixes the
> > probing at all?  Which azx_init_chip() calls hinders the probing of an
> > active codec?
> 
> It seems more like, the particular chip or board requires a special
> quirky initialization sequence.

The problem is that it doesn't explain exactly what this special
initialization does.  How does this magic sequence come?  For example,
why does it need to set UNSOL flag at first?  Is the while loop
guaranteed to quit for a reasonable time?  Can't it be simply calling
azx_reset() twice?

And, yet another question comes up: doesn't S4 need the similar
workaround?  If S4 works, what's the difference?


Takashi

> > 
> > thanks,
> > 
> > Takashi
> > 
> >>
> >> diff --git a/sound/pci/hda/hda_intel.c b/sound/pci/hda/hda_intel.c
> >> index fa2879a..2bf46e9 100644
> >> --- a/sound/pci/hda/hda_intel.c
> >> +++ b/sound/pci/hda/hda_intel.c
> >> @@ -1306,6 +1306,12 @@ static void azx_init_chip(struct azx *chip, int 
> >> full_reset)
> >>  
> >>    /* reset controller */
> >>    azx_reset(chip, full_reset);
> >> +  if (!chip->codec_mask) {
> >> +          azx_writel(chip, GCTL, 0x100);
> >> +          while(azx_readl(chip, GCTL) != 0x0);
> >> +          azx_writel(chip, GCTL, 0x1);
> >> +          azx_reset(chip, full_reset);
> >> +  }
> >>  
> >>    /* initialize interrupts */
> >>    azx_int_clear(chip);
> >> -- 
> >> 1.7.9.5
> >>
> > 
> 
> 
> 
> -- 
> David Henningsson, Canonical Ltd.
> https://launchpad.net/~diwic
>

-- 
You received this bug notification because you are a member of Desktop
Packages, which is subscribed to alsa-driver in Ubuntu.
https://bugs.launchpad.net/bugs/1155202

Title:
  [Intel DZ77SL-50K, Intel PantherPoint HDMI, Digital Out, HDMI] No
  sound at all

Status in “alsa-driver” package in Ubuntu:
  Incomplete

Bug description:
  I am using ubuntu 10.04

  I expected sound

  Got no sound at all

  I am very new to ubuntu so I have no idea how to find the version of
  the package I am using

  ProblemType: Bug
  DistroRelease: Ubuntu 12.04
  Package: alsa-base 1.0.25+dfsg-0ubuntu1.1
  ProcVersionSignature: Ubuntu 3.2.0-38.61-generic-pae 3.2.37
  Uname: Linux 3.2.0-38-generic-pae i686
  AlsaVersion: Advanced Linux Sound Architecture Driver Version 1.0.24.
  AplayDevices:
   **** List of PLAYBACK Hardware Devices ****
   card 0: PCH [HDA Intel PCH], device 3: HDMI 0 [HDMI 0]
     Subdevices: 1/1
     Subdevice #0: subdevice #0
  ApportVersion: 2.0.1-0ubuntu17.1
  Architecture: i386
  ArecordDevices: **** List of CAPTURE Hardware Devices ****
  AudioDevicesInUse:
   USER        PID ACCESS COMMAND
   /dev/snd/controlC0:  jan        1750 F.... pulseaudio
  Card0.Amixer.info:
   Card hw:0 'PCH'/'HDA Intel PCH at 0xf7c30000 irq 45'
     Mixer name : 'Intel PantherPoint HDMI'
     Components : 'HDA:80862806,80860101,00100000'
     Controls      : 7
     Simple ctrls  : 2
  CurrentDmesg: [   24.291482] eth0: no IPv6 routers present
  Date: Thu Mar 14 08:53:13 2013
  InstallationMedia: Ubuntu 12.04.1 LTS "Precise Pangolin" - Release i386 
(20120817.3)
  MarkForUpload: True
  PackageArchitecture: all
  ProcEnviron:
   TERM=xterm
   PATH=(custom, no user)
   LANG=en_US.UTF-8
   SHELL=/bin/bash
  SourcePackage: alsa-driver
  Symptom: audio
  Symptom_AlsaPlaybackTest: ALSA playback test through plughw:PCH failed
  Symptom_Card: Built-in Audio - HDA Intel PCH
  Symptom_DevicesInUse:
   1750  jan       F.... pulseaudio
       PID ACCESS COMMAND
  Symptom_Jack: Digital Out, HDMI
  Symptom_Type: No sound at all
  Title: [, Intel PantherPoint HDMI, Digital Out, HDMI] No sound at all
  UpgradeStatus: No upgrade log present (probably fresh install)
  dmi.bios.date: 03/19/2012
  dmi.bios.vendor: Intel Corporation
  dmi.bios.version: SLZ7710H.86A.0055.2012.0319.2140
  dmi.board.name: DZ77SL-50K
  dmi.board.vendor: Intel Corporation
  dmi.board.version: AAG55115-300
  dmi.chassis.type: 3
  dmi.modalias: 
dmi:bvnIntelCorporation:bvrSLZ7710H.86A.0055.2012.0319.2140:bd03/19/2012:svn:pn:pvr:rvnIntelCorporation:rnDZ77SL-50K:rvrAAG55115-300:cvn:ct3:cvr:

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/alsa-driver/+bug/1155202/+subscriptions

-- 
Mailing list: https://launchpad.net/~desktop-packages
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~desktop-packages
More help   : https://help.launchpad.net/ListHelp

Reply via email to