2019-08-01 07:53:51

by Olivier Moysan

Subject: [PATCH] ARM: dts: stm32: add DFSDM pins to stm32mp157c

Add DFSDM pins to stm32mp157c.

Signed-off-by: Olivier Moysan <[email protected]>
---
arch/arm/boot/dts/stm32mp157-pinctrl.dtsi | 39 +++++++++++++++++++++++++++++++
1 file changed, 39 insertions(+)

diff --git a/arch/arm/boot/dts/stm32mp157-pinctrl.dtsi b/arch/arm/boot/dts/stm32mp157-pinctrl.dtsi
index 9eaec9bf8cb8..f96a928cbc49 100644
--- a/arch/arm/boot/dts/stm32mp157-pinctrl.dtsi
+++ b/arch/arm/boot/dts/stm32mp157-pinctrl.dtsi
@@ -230,6 +230,45 @@
};
};

+ dfsdm_clkout_pins_a: dfsdm-clkout-pins-0 {
+ pins {
+ pinmux = <STM32_PINMUX('B', 13, AF3)>; /* DFSDM_CKOUT */
+ bias-disable;
+ drive-push-pull;
+ slew-rate = <0>;
+ };
+ };
+
+ dfsdm_clkout_sleep_pins_a: dfsdm-clkout-sleep-pins-0 {
+ pins {
+ pinmux = <STM32_PINMUX('B', 13, ANALOG)>; /* DFSDM_CKOUT */
+ };
+ };
+
+ dfsdm_data1_pins_a: dfsdm-data1-pins-0 {
+ pins {
+ pinmux = <STM32_PINMUX('C', 3, AF3)>; /* DFSDM_DATA1 */
+ };
+ };
+
+ dfsdm_data1_sleep_pins_a: dfsdm-data1-sleep-pins-0 {
+ pins {
+ pinmux = <STM32_PINMUX('C', 3, ANALOG)>; /* DFSDM_DATA1 */
+ };
+ };
+
+ dfsdm_data3_pins_a: dfsdm-data3-pins-0 {
+ pins {
+ pinmux = <STM32_PINMUX('F', 13, AF6)>; /* DFSDM_DATA3 */
+ };
+ };
+
+ dfsdm_data3_sleep_pins_a: dfsdm-data3-sleep-pins-0 {
+ pins {
+ pinmux = <STM32_PINMUX('F', 13, ANALOG)>; /* DFSDM_DATA3 */
+ };
+ };
+
ethernet0_rgmii_pins_a: rgmii-0 {
pins1 {
pinmux = <STM32_PINMUX('G', 5, AF11)>, /* ETH_RGMII_CLK125 */
--
2.7.4

2019-08-02 12:57:34

by Alexandre Torgue

Subject: Re: [PATCH] ARM: dts: stm32: add DFSDM pins to stm32mp157c

Hi Olivier

On 8/1/19 9:46 AM, Olivier Moysan wrote:
> Add DFSDM pins to stm32mp157c.
>
> Signed-off-by: Olivier Moysan <[email protected]>
> ---
> arch/arm/boot/dts/stm32mp157-pinctrl.dtsi | 39 +++++++++++++++++++++++++++++++
> 1 file changed, 39 insertions(+)
>
> diff --git a/arch/arm/boot/dts/stm32mp157-pinctrl.dtsi b/arch/arm/boot/dts/stm32mp157-pinctrl.dtsi
> index 9eaec9bf8cb8..f96a928cbc49 100644
> --- a/arch/arm/boot/dts/stm32mp157-pinctrl.dtsi
> +++ b/arch/arm/boot/dts/stm32mp157-pinctrl.dtsi
> @@ -230,6 +230,45 @@
> };
> };
>

I use to only take pinconfig which are used in board. So please resend
with the "board patch".

regards
Alex


> + dfsdm_clkout_pins_a: dfsdm-clkout-pins-0 {
> + pins {
> + pinmux = <STM32_PINMUX('B', 13, AF3)>; /* DFSDM_CKOUT */
> + bias-disable;
> + drive-push-pull;
> + slew-rate = <0>;
> + };
> + };
> +
> + dfsdm_clkout_sleep_pins_a: dfsdm-clkout-sleep-pins-0 {
> + pins {
> + pinmux = <STM32_PINMUX('B', 13, ANALOG)>; /* DFSDM_CKOUT */
> + };
> + };
> +
> + dfsdm_data1_pins_a: dfsdm-data1-pins-0 {
> + pins {
> + pinmux = <STM32_PINMUX('C', 3, AF3)>; /* DFSDM_DATA1 */
> + };
> + };
> +
> + dfsdm_data1_sleep_pins_a: dfsdm-data1-sleep-pins-0 {
> + pins {
> + pinmux = <STM32_PINMUX('C', 3, ANALOG)>; /* DFSDM_DATA1 */
> + };
> + };
> +
> + dfsdm_data3_pins_a: dfsdm-data3-pins-0 {
> + pins {
> + pinmux = <STM32_PINMUX('F', 13, AF6)>; /* DFSDM_DATA3 */
> + };
> + };
> +
> + dfsdm_data3_sleep_pins_a: dfsdm-data3-sleep-pins-0 {
> + pins {
> + pinmux = <STM32_PINMUX('F', 13, ANALOG)>; /* DFSDM_DATA3 */
> + };
> + };
> +
> ethernet0_rgmii_pins_a: rgmii-0 {
> pins1 {
> pinmux = <STM32_PINMUX('G', 5, AF11)>, /* ETH_RGMII_CLK125 */
>

2019-08-02 13:22:42

by Olivier Moysan

Subject: Re: [PATCH] ARM: dts: stm32: add DFSDM pins to stm32mp157c

Hi ALex,

On 8/2/19 10:09 AM, Alexandre Torgue wrote:
> Hi Olivier
>
> On 8/1/19 9:46 AM, Olivier Moysan wrote:
>> Add DFSDM pins to stm32mp157c.
>>
>> Signed-off-by: Olivier Moysan <[email protected]>
>> ---
>> arch/arm/boot/dts/stm32mp157-pinctrl.dtsi | 39 +++++++++++++++++++++++++++++++
>> 1 file changed, 39 insertions(+)
>>
>> diff --git a/arch/arm/boot/dts/stm32mp157-pinctrl.dtsi b/arch/arm/boot/dts/stm32mp157-pinctrl.dtsi
>> index 9eaec9bf8cb8..f96a928cbc49 100644
>> --- a/arch/arm/boot/dts/stm32mp157-pinctrl.dtsi
>> +++ b/arch/arm/boot/dts/stm32mp157-pinctrl.dtsi
>> @@ -230,6 +230,45 @@
>> };
>> };
>>
>
> I use to only take pinconfig which are used in board. So please resend
> with the "board patch".
>

The DFSDM is one of the interface used in the STM32MP15 soundcard.
This soundcard also uses the Wolfson wm8994 audio codec.
The wm8994 codec driver requires adaptations, and the upstream of
these changes is not planned today.
So, the related board patches cannot be sent.

BRs
Olivier

> regards
> Alex
>
>
>> + dfsdm_clkout_pins_a: dfsdm-clkout-pins-0 {
>> + pins {
>> + pinmux = <STM32_PINMUX('B', 13, AF3)>; /* DFSDM_CKOUT */
>> + bias-disable;
>> + drive-push-pull;
>> + slew-rate = <0>;
>> + };
>> + };
>> +
>> + dfsdm_clkout_sleep_pins_a: dfsdm-clkout-sleep-pins-0 {
>> + pins {
>> + pinmux = <STM32_PINMUX('B', 13, ANALOG)>; /* DFSDM_CKOUT */
>> + };
>> + };
>> +
>> + dfsdm_data1_pins_a: dfsdm-data1-pins-0 {
>> + pins {
>> + pinmux = <STM32_PINMUX('C', 3, AF3)>; /* DFSDM_DATA1 */
>> + };
>> + };
>> +
>> + dfsdm_data1_sleep_pins_a: dfsdm-data1-sleep-pins-0 {
>> + pins {
>> + pinmux = <STM32_PINMUX('C', 3, ANALOG)>; /* DFSDM_DATA1 */
>> + };
>> + };
>> +
>> + dfsdm_data3_pins_a: dfsdm-data3-pins-0 {
>> + pins {
>> + pinmux = <STM32_PINMUX('F', 13, AF6)>; /* DFSDM_DATA3 */
>> + };
>> + };
>> +
>> + dfsdm_data3_sleep_pins_a: dfsdm-data3-sleep-pins-0 {
>> + pins {
>> + pinmux = <STM32_PINMUX('F', 13, ANALOG)>; /* DFSDM_DATA3 */
>> + };
>> + };
>> +
>> ethernet0_rgmii_pins_a: rgmii-0 {
>> pins1 {
>> pinmux = <STM32_PINMUX('G', 5, AF11)>, /* ETH_RGMII_CLK125 */
>>