[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <44232168-815a-eb68-9287-d83de910bd25@linaro.org>
Date: Wed, 11 Jan 2023 09:58:50 +0100
From: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
To: Hardevsinh Palaniya <hardevsinh.palaniya@...iconsignals.io>,
"robh+dt@...nel.org" <robh+dt@...nel.org>,
"krzysztof.kozlowski+dt@...aro.org"
<krzysztof.kozlowski+dt@...aro.org>,
"shawnguo@...nel.org" <shawnguo@...nel.org>,
"s.hauer@...gutronix.de" <s.hauer@...gutronix.de>,
"kernel@...gutronix.de" <kernel@...gutronix.de>,
"festevam@...il.com" <festevam@...il.com>,
"linux-imx@....com" <linux-imx@....com>,
"lgirdwood@...il.com" <lgirdwood@...il.com>,
"broonie@...nel.org" <broonie@...nel.org>,
"perex@...ex.cz" <perex@...ex.cz>,
"tiwai@...e.com" <tiwai@...e.com>,
"peng.fan@....com" <peng.fan@....com>,
"m.felsch@...gutronix.de" <m.felsch@...gutronix.de>,
"shengjiu.wang@....com" <shengjiu.wang@....com>,
"Markus.Niebel@...tq-group.com" <Markus.Niebel@...tq-group.com>,
"hongxing.zhu@....com" <hongxing.zhu@....com>,
"tommaso.merciai@...rulasolutions.com"
<tommaso.merciai@...rulasolutions.com>,
"jun.li@....com" <jun.li@....com>,
"ckeepax@...nsource.cirrus.com" <ckeepax@...nsource.cirrus.com>,
"rf@...nsource.cirrus.com" <rf@...nsource.cirrus.com>,
"povik+lin@...ebit.org" <povik+lin@...ebit.org>,
"pierre-louis.bossart@...ux.intel.com"
<pierre-louis.bossart@...ux.intel.com>,
"james.schulman@...rus.com" <james.schulman@...rus.com>,
"flatmax@...tmax.com" <flatmax@...tmax.com>,
"srinivas.kandagatla@...aro.org" <srinivas.kandagatla@...aro.org>,
"ryan.lee.analog@...il.com" <ryan.lee.analog@...il.com>,
"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
"linux-arm-kernel@...ts.infradead.org"
<linux-arm-kernel@...ts.infradead.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"alsa-devel@...a-project.org" <alsa-devel@...a-project.org>
Subject: Re: [PATCH] Support for MAX98090/91 codec in iMX8MM evk
On 11/01/2023 08:16, Hardevsinh Palaniya wrote:
> From: Hardevsinh Palaniya <hardevsinh.palaniya@...iconsignals.io>
> Date: Sat, 7 Jan 2023 17:08:28 +0530
> Subject: [PATCH] Support for MAX98090/91 codec in iMX8MM evk
Use subject prefixes matching the subsystem (which you can get for
example with `git log --oneline -- DIRECTORY_OR_FILE` on the directory
your patch is touching).
>
> - Adding node for MAX98090/91 in dts imx8mm-evk.dtsi
> - Adding tristate option in <sound/soc/codecs/Kconfig>
>
> Signed-off-by: Hardevsinh Palaniya <hardevsinh.palaniya@...iconsignals.io>
Your CC list insane. Use get_maintainers.pl.
>
> diff --git a/arch/arm64/boot/dts/freescale/imx8mm-evk.dtsi b/arch/arm64/boot/dts/freescale/imx8mm-evk.dtsi
> index e0b604ac0da4..58ff63cbc930 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mm-evk.dtsi
> +++ b/arch/arm64/boot/dts/freescale/imx8mm-evk.dtsi
> @@ -132,6 +132,32 @@ simple-audio-card,codec {
> clocks = <&clk IMX8MM_CLK_SAI3_ROOT>;
> };
> };
> +
> + sound-max98090 {
> + compatible = "simple-audio-card";
> + simple-audio-card,name = "max98090-audio";
> + simple-audio-card,format = "i2s";
> + simple-audio-card,frame-master = <&cpudai>;
> + simple-audio-card,bitclock-master = <&cpudai>;
> + simple-audio-card,widgets = "Speakers", "Speakers";
> + simple-audio-card,routing =
> + "Speakers", "SPKR",
> + "Speakers", "SPKL",
> + "IN1", "MICBIAS",
> + "MIC1","IN1",
> + "MIC2","IN1";
> +
> + cpudai: simple-audio-card,cpu {
> + sound-dai = <&sai5>;
> + dai-tdm-slot-num = <2>;
> + dai-tdm-slot-width = <32>;
> + };
> +
> + simple-audio-card,codec {
> + sound-dai = <&max98090>;
> + clocks = <&clk IMX8MM_CLK_SAI5_ROOT>;
> + };
> + };
> };
>
> &A53_0 {
> @@ -339,6 +365,14 @@ &i2c3 {
> pinctrl-0 = <&pinctrl_i2c3>;
> status = "okay";
>
> + max98090: max98090@10 {
Node names should be generic.
https://devicetree-specification.readthedocs.io/en/latest/chapter2-devicetree-basics.html#generic-names-recommendation
> + #sound-dai-cells = <0>;
> + compatible = "maxim,max98090";
compatible is first, then reg.
> + reg = <0x10>;
> + clocks = <&clk IMX8MM_CLK_SAI5_ROOT>;
> + clock-names = "mclk";
> + };
> +
> pca6416: gpio@20 {
> compatible = "ti,tca6416";
> reg = <0x20>;
> @@ -391,6 +425,20 @@ &sai3 {
> status = "okay";
> };
>
> +&sai5 {
> + pinctrl-names = "default";
> + pinctrl-0 = <&pinctrl_sai5>;
> + assigned-clocks = <&clk IMX8MM_CLK_SAI5>;
> + assigned-clock-parents = <&clk IMX8MM_AUDIO_PLL1_OUT>;
> + assigned-clock-rates = <24576000>;
> + clocks = <&clk IMX8MM_CLK_SAI5_IPG>, <&clk IMX8MM_CLK_DUMMY>,
> + <&clk IMX8MM_CLK_SAI5_ROOT>, <&clk IMX8MM_CLK_DUMMY>,
> + <&clk IMX8MM_CLK_DUMMY>, <&clk IMX8MM_AUDIO_PLL1_OUT>,
> + <&clk IMX8MM_AUDIO_PLL2_OUT>;
> + clock-names = "bus", "mclk0", "mclk1", "mclk2", "mclk3", "pll8k", "pll11k";
> + status = "okay";
> +};
> +
> &snvs_pwrkey {
> status = "okay";
> };
> @@ -552,6 +600,16 @@ MX8MM_IOMUXC_SAI3_TXD_SAI3_TX_DATA0 0xd6
> >;
> };
>
> + pinctrl_sai5: sai5grp {
> + fsl,pins = <
> + MX8MM_IOMUXC_SAI5_MCLK_SAI5_MCLK 0xd6
> + MX8MM_IOMUXC_SAI5_RXD2_SAI5_TX_BCLK 0xd6
> + MX8MM_IOMUXC_SAI5_RXD1_SAI5_TX_SYNC 0xd6
> + MX8MM_IOMUXC_SAI5_RXD0_SAI5_RX_DATA0 0xd6
> + MX8MM_IOMUXC_SAI5_RXD3_SAI5_TX_DATA0 0xd6
> + >;
> + }
> +
> pinctrl_typec1: typec1grp {
> fsl,pins = <
> MX8MM_IOMUXC_SD1_STROBE_GPIO2_IO11 0x159
> diff --git a/sound/soc/codecs/Kconfig b/sound/soc/codecs/Kconfig
> index 0f9d71490075..efef2df362a4 100644
> --- a/sound/soc/codecs/Kconfig
> +++ b/sound/soc/codecs/Kconfig
> @@ -1009,7 +1009,7 @@ config SND_SOC_MAX98088
> depends on I2C
>
> config SND_SOC_MAX98090
> - tristate
> + tristate "Maxim MAX98090/1, Stereo Audio Codec"
No, code cannot be mixed with DTS. This is unrelated and not explained
in commit msg.
> depends on I2C
>
> config SND_SOC_MAX98095
> --
> 2.25.1
Best regards,
Krzysztof
Powered by blists - more mailing lists