From: Xilin Wu <wuxilin123@gmail.com>
To: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>, broonie@kernel.org
Cc: perex@perex.cz, tiwai@suse.com, lgirdwood@gmail.com,
alsa-devel@alsa-project.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v2 0/4] ASoC: qcom: display port changes
Date: Sat, 25 May 2024 15:12:28 +0800 [thread overview]
Message-ID: <21f0f2ac-dada-4e19-a0df-086021320101@gmail.com> (raw)
In-Reply-To: <44fad635-1b17-47bc-97f0-9a541e0b4f35@linaro.org>
Hi Srini,
On 2024/5/24 20:50, Srinivas Kandagatla wrote:
> Hi Xilin,
>
> On 23/05/2024 05:09, Xilin Wu wrote:
>>>
>>> Srinivas Kandagatla (4):
>>> ASoC: qcom: q6dsp: parse Display port tokens
>>> ASoC: qcom: common: add Display port Jack function
>>> ASoC: qcom: sc8280xp: add Display port Jack
>>> ASoC: qcom: sm8250: fix a typo in function name
>>>
>>> sound/soc/qcom/common.c | 29 +++++++++++++++++++++++++++++
>>> sound/soc/qcom/common.h | 3 +++
>>> sound/soc/qcom/qdsp6/topology.c | 26 ++++++++++++++++++++++++++
>>> sound/soc/qcom/sc8280xp.c | 14 ++++++++++++++
>>> sound/soc/qcom/sm8250.c | 4 ++--
>>> 5 files changed, 74 insertions(+), 2 deletions(-)
>>>
>>
>> Hi Srini,
>>
>> I tested this series on SM8550 with tplg in [1] and ucm in [2]. But
>> the kernel output errors attached below. Headphone does work properly
>> without DisplayPort in the ucm.
>>
>> What could be the possible cause of this? Is there any significant
>> change from sc8280xp to sm8550?
>>
>> --
>> Thanks,
>> Xilin Wu
>>
>> [1]
>> https://github.com/edk2-porting/audioreach-topology/blob/sakuramist/QCS8550-AYN-ODIN2.m4
>> [2]
>> https://github.com/strongtz/alsa-ucm-conf/blob/odin2/ucm2/Qualcomm/sm8550/HiFi.conf
>>
>> [ 1552.313713] qcom-apm gprsvc:service:2:1: Error (1) Processing
>> 0x01001000 cmd
>> [ 1552.313730] qcom-apm gprsvc:service:2:1: DSP returned error[1001000] 1
>> [ 1552.314455] qcom-apm gprsvc:service:2:1: Error (1) Processing
>
> Is the DP cable connected?
I'm sure that the cable is connected and I have desktop on external display.
If it's not connected, kernel gives the following error when using aplay:
hdmi-audio-codec hdmi-audio-codec.1.auto: ASoC: error at
snd_soc_dai_hw_params on i2s-hifi: -22
>
> if its not connected the dsp will throw this error.
>
> due to this issue I did workaround this issue by modeling it as
> conflicting device to Speaker in x13s ucm.
>
> I see in your ucm setup its not the case.
> which is why you might be hitting this issue.
>
> Can you try
> amixer -c 0 cset iface=MIXER,name='DISPLAY_PORT_RX_0 Audio Mixer
> MultiMedia2' 1
> aplay -D plughw:0,1 some-wav-file.wav
>
> both with and without display connected.
>
aplay always gives the following error:
Playing WAVE 'Summer.wav' : Signed 16 bit Little Endian, Rate 44100 Hz,
Stereo
aplay: set_params:1456: Unable to install hw params:
ACCESS: RW_INTERLEAVED
FORMAT: S16_LE
SUBFORMAT: STD
SAMPLE_BITS: 16
FRAME_BITS: 32
CHANNELS: 2
RATE: 44100
PERIOD_TIME: (42666 42667)
PERIOD_SIZE: (1881 1882)
PERIOD_BYTES: (7524 7528)
PERIODS: (3 5)
BUFFER_TIME: (170657 170658)
BUFFER_SIZE: 7526
BUFFER_BYTES: 30104
TICK_TIME: 0
and kernel gives the following when display is connected:
[drm:dp_catalog_audio_config_sdp] sdp_cfg = 0x100066
[drm:dp_catalog_audio_config_sdp] sdp_cfg2 = 0x1b800004
[drm:dp_audio_hw_params] Header Byte 1: value = 0xce020000, parity_byte
= 0xce
[drm:dp_audio_hw_params] Header Byte 2: value = 0x67010000, parity_byte
= 0x0
[drm:dp_audio_hw_params] Header Byte 3: value = 0x67010000, parity_byte
= 0x67
[drm:dp_audio_hw_params] Header Byte 1: value = 0x67010000, parity_byte
= 0x67
[drm:dp_audio_hw_params] Header Byte 2: value = 0x33443517, parity_byte
= 0x35
[drm:dp_audio_hw_params] Header Byte 3: value = 0x33443517, parity_byte
= 0x33
[drm:dp_audio_hw_params] Header Byte 1: value = 0x84840000, parity_byte
= 0x84
[drm:dp_audio_hw_params] Header Byte 2: value = 0x3344d71b, parity_byte
= 0xd7
[drm:dp_audio_hw_params] Header Byte 3: value = 0x44, parity_byte = 0x33
[drm:dp_audio_hw_params] Header Byte 1: value = 0xd8050000, parity_byte
= 0xd8
[drm:dp_audio_hw_params] Header Byte 2: value = 0x4b0f, parity_byte = 0x4b
[drm:dp_audio_hw_params] Header Byte 3: value = 0x4b0f, parity_byte = 0x0
[drm:dp_audio_hw_params] Header Byte 1: value = 0x71060000, parity_byte
= 0x71
[drm:dp_audio_hw_params] Header Byte 2: value = 0x4b0f, parity_byte = 0x4b
[drm:dp_catalog_audio_config_acr] select: 0x3, acr_ctrl: 0x80004130
[drm:dp_catalog_audio_sfe_level] mainlink_level = 0xa08,
safe_to_exit_level = 0x8
[drm:dp_catalog_audio_enable] dp_audio_cfg = 0xc1
qcom-apm gprsvc:service:2:1: Error (1) Processing 0x01001006 cmd
qcom-apm gprsvc:service:2:1: DSP returned error[1001006] 1
qcom-apm gprsvc:service:2:1: Error (1) Processing 0x01001006 cmd
qcom-apm gprsvc:service:2:1: DSP returned error[1001006] 1
qcom-apm gprsvc:service:2:1: Error (1) Processing 0x01001001 cmd
qcom-apm gprsvc:service:2:1: DSP returned error[1001001] 1
q6apm-lpass-dais 30000000.remoteproc:glink-edge:gpr:service@1:bedais:
Failed to prepare Graph -22
q6apm-lpass-dais 30000000.remoteproc:glink-edge:gpr:service@1:bedais:
ASoC: error at snd_soc_pcm_dai_prepare on DISPLAY_PORT_RX_0: -22
[drm:dp_catalog_audio_enable] dp_audio_cfg = 0xc0
>
> --srini
>
>
>
>> 0x01001006 cmd
>> [ 1552.314463] qcom-apm gprsvc:service:2:1: DSP returned error[1001006] 1
>> [ 1552.315496] qcom-apm gprsvc:service:2:1: Error (1) Processing
>> 0x01001006 cmd
>> [ 1552.315506] qcom-apm gprsvc:service:2:1: DSP returned error[1001006] 1
>> [ 1552.316033] qcom-apm gprsvc:service:2:1: Error (1) Processing
>> 0x01001001 cmd
>> [ 1552.316042] qcom-apm gprsvc:service:2:1: DSP returned error[1001001] 1
>> [ 1552.316045] q6apm-lpass-dais
>> 30000000.remoteproc:glink-edge:gpr:service@1:bedais: Failed to prepare
>> Graph -22
>> [ 1552.316047] q6apm-lpass-dais
>> 30000000.remoteproc:glink-edge:gpr:service@1:bedais: ASoC: error at
>> snd_soc_pcm_dai_prepare on DISPLAY_PORT_RX_0: -22
--
Thanks,
Xilin Wu
next prev parent reply other threads:[~2024-05-25 7:12 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-04-22 13:43 [PATCH v2 0/4] ASoC: qcom: display port changes srinivas.kandagatla
2024-04-22 13:43 ` [PATCH v2 1/4] ASoC: qcom: q6dsp: parse Display port tokens srinivas.kandagatla
2024-04-22 13:43 ` [PATCH v2 2/4] ASoC: qcom: common: add Display port Jack function srinivas.kandagatla
2024-04-23 11:44 ` Johan Hovold
2024-04-23 12:02 ` Johan Hovold
2024-04-23 15:55 ` Srinivas Kandagatla
2024-04-29 14:54 ` Johan Hovold
2024-05-09 8:59 ` Srinivas Kandagatla
2024-05-30 22:38 ` Dmitry Baryshkov
2024-05-31 6:44 ` Srinivas Kandagatla
2024-04-22 13:43 ` [PATCH v2 3/4] ASoC: qcom: sc8280xp: add Display port Jack srinivas.kandagatla
2024-04-23 11:48 ` Johan Hovold
2024-05-30 22:41 ` Dmitry Baryshkov
2024-04-22 13:43 ` [PATCH v2 4/4] ASoC: qcom: sm8250: fix a typo in function name srinivas.kandagatla
2024-04-23 11:59 ` [PATCH v2 0/4] ASoC: qcom: display port changes Johan Hovold
2024-04-23 12:38 ` Srinivas Kandagatla
2024-04-23 14:58 ` Johan Hovold
2024-04-23 15:59 ` Srinivas Kandagatla
2024-04-29 15:00 ` Johan Hovold
2024-05-30 22:56 ` Dmitry Baryshkov
2024-05-23 4:09 ` Xilin Wu
2024-05-24 12:50 ` Srinivas Kandagatla
2024-05-25 7:12 ` Xilin Wu [this message]
2024-06-06 9:18 ` Srinivas Kandagatla
2024-06-08 11:41 ` Xilin Wu
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=21f0f2ac-dada-4e19-a0df-086021320101@gmail.com \
--to=wuxilin123@gmail.com \
--cc=alsa-devel@alsa-project.org \
--cc=broonie@kernel.org \
--cc=lgirdwood@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=perex@perex.cz \
--cc=srinivas.kandagatla@linaro.org \
--cc=tiwai@suse.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox