[v3,0/5] Stripe control functionality

Message

Sameer Pujar Jan. 14, 2019, 6:21 p.m. UTC

Background
==========

Azalia Controller fetches command and audio data via DMA and send them
to codec through SDO(Serial Data Out) lines. SDO is for outboung and it
broadcasts to all codecs. There is atleast one SDO line present, but
there can be multiple SDO lines supported for extended bandwidth. This
is essential when a platform supports multiple hdmi/dp sinks and there
is a requirement for higher resolution audio playback. In such cases
simultaneous audio playback data can be striped across multiple SDOs.

Global Capabilities(GCAP) Register indicates the capabilities of the
controller. Bits 2:1 of GCAP can be read to know the number of supported
SDO lines (below is from HD audio spec)
  00: 1 SDO
  01: 2 SDO
  10: 4 SDO
  11: Reserved

Stripe control verb reports and controls the stripe capability of an
Audio Output Converter. This verb needs to be implemented only for an
audio output converter and only if the stripe bit of the Audio Widget
Capabilities parameter is 1. 
Stripe control: get code(0xf24) and set code(0x724)

Change log
==========

v1-->v2:
-------- 
  Patch 1: "ALSA: hda: add verbs for stripe control"
    * no change

  Patch 2: "ALSA: hda: Add api to program stripe control bits"
    * no change

  Patch 3: "ALSA: hda: add register offset for stripe control"
    * added mask for stripe control programming

  Patch 4: "ALSA: hda: program stripe bits for controller"
    * used stripe control mask instead of hard coded value

  Patch 5: "ALSA: hda: program stripe control for codec"
    * added conditional check to know if striping is supported.
      If supported, then only stripe verb is implemented.

v2-->v3:
-------- 
  Patch 1: "ALSA: hda: add verbs for stripe control"
    * no change
  Patch 2: "ALSA: hda: Add api to program stripe control bits"
    * added braces around stripe value calucation as per spec
  Patch 3: "ALSA: hda: add register offset for stripe control"
    * no change
  Patch 4: "ALSA: hda: program stripe bits for controller"
    * no change
  Patch 5: "ALSA: hda: program stripe control for codec"
    * no change

===========

Sameer Pujar (5):
  ALSA: hda: add verbs for stripe control
  ALSA: hda: Add api to program stripe control bits
  ALSA: hda: add register offset for stripe control
  ALSA: hda: program stripe bits for controller
  ALSA: hda: program stripe control for codec

 include/sound/hda_register.h |  2 ++
 include/sound/hda_verbs.h    |  2 ++
 include/sound/hdaudio.h      |  3 +++
 sound/hda/hdac_stream.c      | 40 ++++++++++++++++++++++++++++++++++++++++
 sound/pci/hda/patch_hdmi.c   | 10 +++++++++-
 5 files changed, 56 insertions(+), 1 deletion(-)

Comments

Takashi Iwai Jan. 14, 2019, 6:56 p.m. UTC | #1

On Mon, 14 Jan 2019 19:21:07 +0100,
Sameer Pujar wrote:
> 
> Background
> ==========
> 
> Azalia Controller fetches command and audio data via DMA and send them
> to codec through SDO(Serial Data Out) lines. SDO is for outboung and it
> broadcasts to all codecs. There is atleast one SDO line present, but
> there can be multiple SDO lines supported for extended bandwidth. This
> is essential when a platform supports multiple hdmi/dp sinks and there
> is a requirement for higher resolution audio playback. In such cases
> simultaneous audio playback data can be striped across multiple SDOs.
> 
> Global Capabilities(GCAP) Register indicates the capabilities of the
> controller. Bits 2:1 of GCAP can be read to know the number of supported
> SDO lines (below is from HD audio spec)
>   00: 1 SDO
>   01: 2 SDO
>   10: 4 SDO
>   11: Reserved
> 
> Stripe control verb reports and controls the stripe capability of an
> Audio Output Converter. This verb needs to be implemented only for an
> audio output converter and only if the stripe bit of the Audio Widget
> Capabilities parameter is 1. 
> Stripe control: get code(0xf24) and set code(0x724)
> 
> Change log
> ==========
> 
> v1-->v2:
> -------- 
>   Patch 1: "ALSA: hda: add verbs for stripe control"
>     * no change
> 
>   Patch 2: "ALSA: hda: Add api to program stripe control bits"
>     * no change
> 
>   Patch 3: "ALSA: hda: add register offset for stripe control"
>     * added mask for stripe control programming
> 
>   Patch 4: "ALSA: hda: program stripe bits for controller"
>     * used stripe control mask instead of hard coded value
> 
>   Patch 5: "ALSA: hda: program stripe control for codec"
>     * added conditional check to know if striping is supported.
>       If supported, then only stripe verb is implemented.
> 
> v2-->v3:
> -------- 
>   Patch 1: "ALSA: hda: add verbs for stripe control"
>     * no change
>   Patch 2: "ALSA: hda: Add api to program stripe control bits"
>     * added braces around stripe value calucation as per spec
>   Patch 3: "ALSA: hda: add register offset for stripe control"
>     * no change
>   Patch 4: "ALSA: hda: program stripe bits for controller"
>     * no change
>   Patch 5: "ALSA: hda: program stripe control for codec"
>     * no change
> 
> ===========
> 
> Sameer Pujar (5):
>   ALSA: hda: add verbs for stripe control
>   ALSA: hda: Add api to program stripe control bits
>   ALSA: hda: add register offset for stripe control
>   ALSA: hda: program stripe bits for controller
>   ALSA: hda: program stripe control for codec

Applied all five patches now.  Thanks.


Takashi