19 Jul, 2011

1 commit


17 Jul, 2011

1 commit

  • The WM8994 and WM8958 series of devices have two MICBIAS supplies rather
    than one, the current widget actually manages the microphone detection
    control register bit (which is managed separately by the relevant API).

    Fix this, hooking the relevant supplies up to the MICBIAS1 and MICBIAS2
    widgets.

    Signed-off-by: Mark Brown
    Cc: stable@kernel.org

    Mark Brown
     

15 Jul, 2011

1 commit


09 Jul, 2011

6 commits


08 Jul, 2011

1 commit


07 Jul, 2011

1 commit


06 Jul, 2011

1 commit


05 Jul, 2011

3 commits


04 Jul, 2011

2 commits

  • This delay is very conservative.

    Signed-off-by: Mark Brown
    Acked-by: Liam Girdwood
    Cc: stable@kernel.org

    Mark Brown
     
  • The I2S controller needs a clock to respond to register writes. Without
    this, register writes will at worst hang the CPU. In practice, I've only
    observed writes being dropped.

    Luckily, the dropped register writes historically had no effect:

    TEGRA_I2S_TIMING: The value we wrote was the reset default.

    TEGRA_I2S_FIFO_SCR: The default was for the FIFOs to request more data
    when one slot was empty. The requested value was for the FIFOs to request
    when four slots were empty. The DMA controller in the mainline kernel is
    configured to burst a single entry at a time into the FIFO, hence there
    was no issue. The only negative effect was on bus efficiency losses due
    to an increased number of arbitration attempts.

    However, in various non-upstream changes, the DMA controller now bursts
    four entries at a time into the FIFO. If there is only space for one
    entry, the data is simply dropped. In practice, this resulted in 3/4 of
    samples being dropped, and playback at 4x the expected rate and pitch.
    By fixing the clocking issue, this is solved.

    Signed-off-by: Stephen Warren
    Acked-by: Liam Girdwood
    Signed-off-by: Mark Brown

    Stephen Warren
     

30 Jun, 2011

3 commits

  • The char can be unsigned on some architectures. Since the code checks
    the negative values, they should be declared as signed char explicitly.

    sound/pci/rme9652/hdspm.c:5449: warning: comparison is always false due to limited range of data type
    sound/pci/rme9652/hdspm.c:5462: warning: comparison is always false due to limited range of data type

    Signed-off-by: Takashi Iwai

    Takashi Iwai
     
  • Fix the wrongly converted short values:
    sound/pci/cs5535audio/cs5535audio_pcm.c:152: warning: large integer implicitly truncated to unsigned type
    sound/pci/cs5535audio/cs5535audio_pcm.c:160: warning: large integer implicitly truncated to unsigned type

    Signed-off-by: Takashi Iwai

    Takashi Iwai
     
  • The Blackfin DMA controller can report one frame beyond the end of the
    buffer in the wraparound case but ALSA requires that the pointer always
    be in the buffer. Do the wraparound to handle this. A similar bug is
    likely to apply to the other Blackfin PCM drivers but the code is less
    obvious to inspection and I don't have a user to test.

    Reported-by: Kieran O'Leary
    Acked-by: Liam Girdwood
    Signed-off-by: Mark Brown
    Cc: stable@kernel.org

    Mark Brown
     

29 Jun, 2011

1 commit

  • I noticed that the last character of the ELD monitor name is lost,
    this fixes the issue.

    This fix should be confirming to the HDA spec, and works together with
    the DRM part of the ELD patch.

    The HDA spec does not mention that Monitor_Name_String is an '\0'
    ending string, and it allows NML to be 1, which is only valid when MNL
    does not count the possible ending '\0'.

    Signed-off-by: Wu Fengguang
    Signed-off-by: Takashi Iwai

    Wu Fengguang
     

28 Jun, 2011

3 commits


24 Jun, 2011

5 commits


23 Jun, 2011

1 commit


21 Jun, 2011

4 commits


20 Jun, 2011

1 commit

  • The reporter, who is running kernel 2.6.38, reports that
    he needs to set model=auto for the headphone output to work
    correctly.

    BugLink: http://bugs.launchpad.net/bugs/761022
    Cc: stable@kernel.org (v2.6.38+)
    Reported-by: Jo
    Signed-off-by: David Henningsson
    Signed-off-by: Takashi Iwai

    David Henningsson
     

17 Jun, 2011

3 commits

  • The via driver spews warnigs like
    hda-codec: no NID for mapping control Independent HP:0:0
    with some codecs because snd_hda_add_nid() is called with nid=0.
    This patch fixes it by skipping the call when no corresponding widget
    is found.

    Signed-off-by: Takashi Iwai

    Takashi Iwai
     
  • Since commit 13882a82ee16 (optimize iso queueing by setting
    wake only after the last packet), drivers are required to call
    fw_iso_context_queue_flush() after queueing a batch of packets.
    The missing call would have an effect only if the controller
    queue underruns, but then the DMA would stop completely.

    Signed-off-by: Clemens Ladisch
    Signed-off-by: Takashi Iwai

    Clemens Ladisch
     
  • Fixed remaining issues of the signedness bug discovered by Dan Carpenter.
    A check was remaining that tests if unsigned rt->rate is >= 0.
    Changed that so that rt->rate now consistently uses ARRAY_SIZE(rates)
    as invalid rate value and not -1.

    Signed-off-by: Torsten Schenk
    Signed-off-by: Takashi Iwai

    Torsten Schenk
     

15 Jun, 2011

1 commit


14 Jun, 2011

1 commit