* [PATCH v1 0/2] Add sound card support for lemans-evk and monaco-evk
@ 2025-08-08 5:29 Mohammad Rafi Shaik
2025-08-08 5:29 ` [PATCH v1 1/2] ASoC: dt-bindings: qcom,sm8250: Add lemans-evk and monaco-evk sound card Mohammad Rafi Shaik
2025-08-08 5:29 ` [PATCH v1 2/2] ASoC: qcom: sc8280xp: Add sound card support for lemans-evk and monaco-evk Mohammad Rafi Shaik
0 siblings, 2 replies; 5+ messages in thread
From: Mohammad Rafi Shaik @ 2025-08-08 5:29 UTC (permalink / raw)
To: Srinivas Kandagatla, Liam Girdwood, Mark Brown, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Jaroslav Kysela, Takashi Iwai
Cc: linux-sound, linux-arm-msm, devicetree, linux-kernel, kernel
This patchset adds support for sound card on Qualcomm LEMANS-EVK and
MONACO-EVK boards.
Mohammad Rafi Shaik (2):
ASoC: dt-bindings: qcom,sm8250: Add lemans-evk and monaco-evk sound
card
ASoC: qcom: sc8280xp: Add sound card support for lemans-evk and
monaco-evk
Documentation/devicetree/bindings/sound/qcom,sm8250.yaml | 2 ++
sound/soc/qcom/sc8280xp.c | 2 ++
2 files changed, 4 insertions(+)
base-commit: b7d4e259682caccb51a25283655f2c8f02e32d23
--
2.34.1
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH v1 1/2] ASoC: dt-bindings: qcom,sm8250: Add lemans-evk and monaco-evk sound card
2025-08-08 5:29 [PATCH v1 0/2] Add sound card support for lemans-evk and monaco-evk Mohammad Rafi Shaik
@ 2025-08-08 5:29 ` Mohammad Rafi Shaik
2025-08-08 6:14 ` Krzysztof Kozlowski
2025-08-08 5:29 ` [PATCH v1 2/2] ASoC: qcom: sc8280xp: Add sound card support for lemans-evk and monaco-evk Mohammad Rafi Shaik
1 sibling, 1 reply; 5+ messages in thread
From: Mohammad Rafi Shaik @ 2025-08-08 5:29 UTC (permalink / raw)
To: Srinivas Kandagatla, Liam Girdwood, Mark Brown, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Jaroslav Kysela, Takashi Iwai
Cc: linux-sound, linux-arm-msm, devicetree, linux-kernel, kernel
Document the bindings for the Qualcomm LEMANS-EVK and MONACO-EVK
board specific sound card.
The bindings are the same as for other newer Qualcomm ADSP sound cards,
thus keep them in existing qcom,sm8250.yaml file, even though Linux driver
is separate.
Signed-off-by: Mohammad Rafi Shaik <mohammad.rafi.shaik@oss.qualcomm.com>
---
Documentation/devicetree/bindings/sound/qcom,sm8250.yaml | 2 ++
1 file changed, 2 insertions(+)
diff --git a/Documentation/devicetree/bindings/sound/qcom,sm8250.yaml b/Documentation/devicetree/bindings/sound/qcom,sm8250.yaml
index 5d3dbb6cb1ae..c63bfe031b57 100644
--- a/Documentation/devicetree/bindings/sound/qcom,sm8250.yaml
+++ b/Documentation/devicetree/bindings/sound/qcom,sm8250.yaml
@@ -31,6 +31,8 @@ properties:
- fairphone,fp4-sndcard
- fairphone,fp5-sndcard
- qcom,apq8096-sndcard
+ - qcom,lemans-evk-sndcard
+ - qcom,monaco-evk-sndcard
- qcom,qcm6490-idp-sndcard
- qcom,qcs6490-rb3gen2-sndcard
- qcom,qcs8275-sndcard
--
2.34.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH v1 2/2] ASoC: qcom: sc8280xp: Add sound card support for lemans-evk and monaco-evk
2025-08-08 5:29 [PATCH v1 0/2] Add sound card support for lemans-evk and monaco-evk Mohammad Rafi Shaik
2025-08-08 5:29 ` [PATCH v1 1/2] ASoC: dt-bindings: qcom,sm8250: Add lemans-evk and monaco-evk sound card Mohammad Rafi Shaik
@ 2025-08-08 5:29 ` Mohammad Rafi Shaik
1 sibling, 0 replies; 5+ messages in thread
From: Mohammad Rafi Shaik @ 2025-08-08 5:29 UTC (permalink / raw)
To: Srinivas Kandagatla, Liam Girdwood, Mark Brown, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Jaroslav Kysela, Takashi Iwai
Cc: linux-sound, linux-arm-msm, devicetree, linux-kernel, kernel
Add compatibles for sound card on Qualcomm LEMANS-EVK and
MONACO-EVK boards.
Signed-off-by: Mohammad Rafi Shaik <mohammad.rafi.shaik@oss.qualcomm.com>
---
sound/soc/qcom/sc8280xp.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/sound/soc/qcom/sc8280xp.c b/sound/soc/qcom/sc8280xp.c
index 73f9f82c4e25..386e671d17f7 100644
--- a/sound/soc/qcom/sc8280xp.c
+++ b/sound/soc/qcom/sc8280xp.c
@@ -184,6 +184,8 @@ static int sc8280xp_platform_probe(struct platform_device *pdev)
}
static const struct of_device_id snd_sc8280xp_dt_match[] = {
+ {.compatible = "qcom,lemans-evk-sndcard", "lemans"},
+ {.compatible = "qcom,monaco-evk-sndcard", "monaco"},
{.compatible = "qcom,qcm6490-idp-sndcard", "qcm6490"},
{.compatible = "qcom,qcs6490-rb3gen2-sndcard", "qcs6490"},
{.compatible = "qcom,qcs8275-sndcard", "qcs8275"},
--
2.34.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH v1 1/2] ASoC: dt-bindings: qcom,sm8250: Add lemans-evk and monaco-evk sound card
2025-08-08 5:29 ` [PATCH v1 1/2] ASoC: dt-bindings: qcom,sm8250: Add lemans-evk and monaco-evk sound card Mohammad Rafi Shaik
@ 2025-08-08 6:14 ` Krzysztof Kozlowski
2025-08-08 9:15 ` Mohammad Rafi Shaik
0 siblings, 1 reply; 5+ messages in thread
From: Krzysztof Kozlowski @ 2025-08-08 6:14 UTC (permalink / raw)
To: Mohammad Rafi Shaik, Srinivas Kandagatla, Liam Girdwood,
Mark Brown, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Jaroslav Kysela, Takashi Iwai
Cc: linux-sound, linux-arm-msm, devicetree, linux-kernel, kernel
On 08/08/2025 07:29, Mohammad Rafi Shaik wrote:
> Document the bindings for the Qualcomm LEMANS-EVK and MONACO-EVK
> board specific sound card.
>
> The bindings are the same as for other newer Qualcomm ADSP sound cards,
> thus keep them in existing qcom,sm8250.yaml file, even though Linux driver
> is separate.
>
> Signed-off-by: Mohammad Rafi Shaik <mohammad.rafi.shaik@oss.qualcomm.com>
> ---
> Documentation/devicetree/bindings/sound/qcom,sm8250.yaml | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/sound/qcom,sm8250.yaml b/Documentation/devicetree/bindings/sound/qcom,sm8250.yaml
> index 5d3dbb6cb1ae..c63bfe031b57 100644
> --- a/Documentation/devicetree/bindings/sound/qcom,sm8250.yaml
> +++ b/Documentation/devicetree/bindings/sound/qcom,sm8250.yaml
> @@ -31,6 +31,8 @@ properties:
> - fairphone,fp4-sndcard
> - fairphone,fp5-sndcard
> - qcom,apq8096-sndcard
> + - qcom,lemans-evk-sndcard
Lemans is already there under one of the QCS cards. I was told it is the
same, so you do not get a new compatible.
Monaco maybe as well.
We also name standard cards per SoC, not per board, so evk should be
dropped or commit msg should explain why this is different.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v1 1/2] ASoC: dt-bindings: qcom,sm8250: Add lemans-evk and monaco-evk sound card
2025-08-08 6:14 ` Krzysztof Kozlowski
@ 2025-08-08 9:15 ` Mohammad Rafi Shaik
0 siblings, 0 replies; 5+ messages in thread
From: Mohammad Rafi Shaik @ 2025-08-08 9:15 UTC (permalink / raw)
To: Krzysztof Kozlowski, Srinivas Kandagatla, Liam Girdwood,
Mark Brown, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Jaroslav Kysela, Takashi Iwai
Cc: linux-sound, linux-arm-msm, devicetree, linux-kernel, kernel
On 8/8/2025 11:44 AM, Krzysztof Kozlowski wrote:
> On 08/08/2025 07:29, Mohammad Rafi Shaik wrote:
>> Document the bindings for the Qualcomm LEMANS-EVK and MONACO-EVK
>> board specific sound card.
>>
>> The bindings are the same as for other newer Qualcomm ADSP sound cards,
>> thus keep them in existing qcom,sm8250.yaml file, even though Linux driver
>> is separate.
>>
>> Signed-off-by: Mohammad Rafi Shaik <mohammad.rafi.shaik@oss.qualcomm.com>
>> ---
>> Documentation/devicetree/bindings/sound/qcom,sm8250.yaml | 2 ++
>> 1 file changed, 2 insertions(+)
>>
>> diff --git a/Documentation/devicetree/bindings/sound/qcom,sm8250.yaml b/Documentation/devicetree/bindings/sound/qcom,sm8250.yaml
>> index 5d3dbb6cb1ae..c63bfe031b57 100644
>> --- a/Documentation/devicetree/bindings/sound/qcom,sm8250.yaml
>> +++ b/Documentation/devicetree/bindings/sound/qcom,sm8250.yaml
>> @@ -31,6 +31,8 @@ properties:
>> - fairphone,fp4-sndcard
>> - fairphone,fp5-sndcard
>> - qcom,apq8096-sndcard
>> + - qcom,lemans-evk-sndcard
>
>
> Lemans is already there under one of the QCS cards. I was told it is the
> same, so you do not get a new compatible.
>
> Monaco maybe as well.
>
> We also name standard cards per SoC, not per board, so evk should be
> dropped or commit msg should explain why this is different.
>
Ack,
will remove the EVK and keep the soc compatible.
For Lemans will re-use the qcs9100-sndcard compatible and
for monaco will add the new soc compatible (qcs8300-sndcard) in the same
series with updated cover-letter and drop the Lemans related change.
Thanks & regards,
Rafi.
>
> Best regards,
> Krzysztof
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2025-08-08 9:15 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-08-08 5:29 [PATCH v1 0/2] Add sound card support for lemans-evk and monaco-evk Mohammad Rafi Shaik
2025-08-08 5:29 ` [PATCH v1 1/2] ASoC: dt-bindings: qcom,sm8250: Add lemans-evk and monaco-evk sound card Mohammad Rafi Shaik
2025-08-08 6:14 ` Krzysztof Kozlowski
2025-08-08 9:15 ` Mohammad Rafi Shaik
2025-08-08 5:29 ` [PATCH v1 2/2] ASoC: qcom: sc8280xp: Add sound card support for lemans-evk and monaco-evk Mohammad Rafi Shaik
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).