Ptl i2srt5682 sdwrt722 #5166
Ptl i2srt5682 sdwrt722 #5166
Conversation
RDharageswari
requested review from
lgirdwood,
plbossart,
ranj063,
kv2019i,
dbaluta,
bardliao and
ujfalusi
as code owners
RDharageswari
force-pushed
the
ptl_i2srt5682_sdwrt711
branch
from
02d3945
to
8b129ba
Compare
RDharageswari
force-pushed
the
ptl_i2srt5682_sdwrt711
branch
from
8b129ba
to
2b8a24c
Compare
sound/soc/intel/boards/sof_sdw.c
Outdated
| DMI_MATCH(DMI_PRODUCT_FAMILY, "Intel_ptlrvp"), | ||
| }, | ||
| .driver_data = (void *)(SOF_SDW_TGL_HDMI | | ||
| SOF_SDW_PCH_DMIC), |
There was a problem hiding this comment.
Please align with the parentheses. I.e.
(void *) (SOF_SDW_TGL_HDMI |
SOF_SDW_PCH_DMIC),
|
SOFCI TEST |
There was a problem hiding this comment.
@RDharageswari SOF_SDW_PCH_DMIC has been changed to SOC_SDW_PCH_DMIC. That's why your PR can't build
RDharageswari
force-pushed
the
ptl_i2srt5682_sdwrt711
branch
from
2b8a24c
to
c2e7364
Compare
sound/soc/intel/boards/sof_sdw.c
Outdated
| .matches = { | ||
| DMI_MATCH(DMI_PRODUCT_FAMILY, "Intel_ptlrvp"), | ||
| }, | ||
| .driver_data = (void *)(SOF_SDW_TGL_HDMI | |
There was a problem hiding this comment.
no, all devices after TGL only use 3 HDMI ports, this will add an HDMI port for no reason.
sound/soc/intel/boards/sof_sdw.c
Outdated
| DMI_MATCH(DMI_PRODUCT_FAMILY, "Intel_ptlrvp"), | ||
| }, | ||
| .driver_data = (void *)(SOF_SDW_TGL_HDMI | | ||
| SOC_SDW_PCH_DMIC), |
There was a problem hiding this comment.
Also can you clarify why you need the PCH_DMIC? Is this a supported configuration? At the very least there should be an explanation of which microphone path is supported.
There was a problem hiding this comment.
hi Pierre,
This is the POR configuration.
There was a problem hiding this comment.
Humm, now I am completely lost. You list rt5682 in this PR, along with RT711. Neither are by default on the RVP, so this quirk is problematic indeed.
There was a problem hiding this comment.
Hi Pierre,
RT722 is the onboard codec and the patch adds RT722 to the ACPI table.
sorry for the wrong branch name.
RDharageswari
force-pushed
the
ptl_i2srt5682_sdwrt711
branch
from
c2e7364
to
ee35405
Compare
RDharageswari
force-pushed
the
ptl_i2srt5682_sdwrt711
branch
from
ee35405
to
86600d9
Compare
| .link_mask = BIT(1), | ||
| .links = ptl_rt722_l1, | ||
| .drv_name = "sof_sdw", | ||
| .sof_tplg_filename = "sof-ptl-rt722-4ch.tplg", |
There was a problem hiding this comment.
From where this topology is coming and what is the -4ch postfix means? Is it PCH-DMIC? Should not that be dynamically added to the base topology name based on the number of DMICs detected?
There was a problem hiding this comment.
Yes, the "-4ch" should be added automatically. See https://github.com/thesofproject/linux/blob/topic/sof-dev/sound/soc/sof/intel/hda.c#L1321. @RDharageswari Can you test if "-4ch" is added automatically on your device?
There was a problem hiding this comment.
as discussed on chat with Bard, -4ch .DMIC string can be appended through module params. Will post the topology changes for 4ch dmic soon.
There was a problem hiding this comment.
The topology PR is thesofproject/sof#9493
There was a problem hiding this comment.
the topology is wrong @bardliao, it relies on 2ch.
Enable on-board rt722 based sound card for PTL RVP. Signed-off-by: Dharageswari R <[email protected]>
This patch adds the driver data for rt5682 codec on SSP0 and max98360a speaker amplifiers on SSP1 for PTL platform. Signed-off-by: Dharageswari R <[email protected]>
RDharageswari
force-pushed
the
ptl_i2srt5682_sdwrt711
branch
from
86600d9
to
4bf72cd
Compare
|
SOFCI TEST |
|
The pause resume failures on https://sof-ci.01.org/linuxpr/PR5166/build4708/devicetest/index.html should not be caused by this PR. And we got approvals. Merging it. |
This PR adds the following:
RT722 SDCA card for PTL platform
Add machine and topology entries to the soc-acpi-intel-ptl-match for the I2S rt5682