diff mbox series

[4/5] ARM: dts: stm32: enable the analog filter for all I2C nodes in stm32mp151

Message ID 1612515104-838-5-git-send-email-alain.volmat@foss.st.com (mailing list archive)
State New, archived
Headers show
Series i2c: stm32: filter binding support & debug info | expand

Commit Message

Alain Volmat Feb. 5, 2021, 8:51 a.m. UTC
Enable the analog filter for all I2C nodes of the stm32mp151.

Signed-off-by: Alain Volmat <alain.volmat@foss.st.com>
---
 arch/arm/boot/dts/stm32mp151.dtsi | 6 ++++++
 1 file changed, 6 insertions(+)

Comments

Pierre Yves MORDRET Feb. 10, 2021, 8:39 a.m. UTC | #1
Hello

Looks good to me

Reviewed-by: Pierre-Yves MORDRET <pierre-yves.mordret@foss.st.com>

Thx
Regards


On 2/5/21 9:51 AM, Alain Volmat wrote:
> Enable the analog filter for all I2C nodes of the stm32mp151.
> 
> Signed-off-by: Alain Volmat <alain.volmat@foss.st.com>
> ---
>  arch/arm/boot/dts/stm32mp151.dtsi | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/stm32mp151.dtsi b/arch/arm/boot/dts/stm32mp151.dtsi
> index 3c75abacb374..558fc8fb38b6 100644
> --- a/arch/arm/boot/dts/stm32mp151.dtsi
> +++ b/arch/arm/boot/dts/stm32mp151.dtsi
> @@ -493,6 +493,7 @@
>  			#size-cells = <0>;
>  			st,syscfg-fmp = <&syscfg 0x4 0x1>;
>  			wakeup-source;
> +			i2c-analog-filter;
>  			status = "disabled";
>  		};
>  
> @@ -508,6 +509,7 @@
>  			#size-cells = <0>;
>  			st,syscfg-fmp = <&syscfg 0x4 0x2>;
>  			wakeup-source;
> +			i2c-analog-filter;
>  			status = "disabled";
>  		};
>  
> @@ -523,6 +525,7 @@
>  			#size-cells = <0>;
>  			st,syscfg-fmp = <&syscfg 0x4 0x4>;
>  			wakeup-source;
> +			i2c-analog-filter;
>  			status = "disabled";
>  		};
>  
> @@ -538,6 +541,7 @@
>  			#size-cells = <0>;
>  			st,syscfg-fmp = <&syscfg 0x4 0x10>;
>  			wakeup-source;
> +			i2c-analog-filter;
>  			status = "disabled";
>  		};
>  
> @@ -1533,6 +1537,7 @@
>  			#size-cells = <0>;
>  			st,syscfg-fmp = <&syscfg 0x4 0x8>;
>  			wakeup-source;
> +			i2c-analog-filter;
>  			status = "disabled";
>  		};
>  
> @@ -1570,6 +1575,7 @@
>  			#size-cells = <0>;
>  			st,syscfg-fmp = <&syscfg 0x4 0x20>;
>  			wakeup-source;
> +			i2c-analog-filter;
>  			status = "disabled";
>  		};
>  
>
Wolfram Sang March 18, 2021, 10:55 a.m. UTC | #2
On Fri, Feb 05, 2021 at 09:51:43AM +0100, Alain Volmat wrote:
> Enable the analog filter for all I2C nodes of the stm32mp151.
> 
> Signed-off-by: Alain Volmat <alain.volmat@foss.st.com>

I usually don't take DTS patches, but they can go in now via arm-soc as
I applied the patches to the driver.
Alexandre TORGUE March 18, 2021, 1:16 p.m. UTC | #3
Hi Wolfram

On 3/18/21 11:55 AM, Wolfram Sang wrote:
> On Fri, Feb 05, 2021 at 09:51:43AM +0100, Alain Volmat wrote:
>> Enable the analog filter for all I2C nodes of the stm32mp151.
>>
>> Signed-off-by: Alain Volmat <alain.volmat@foss.st.com>
> 
> I usually don't take DTS patches, but they can go in now via arm-soc as
> I applied the patches to the driver.
> 

I'll take it in my stm32 tree.

Thanks
Alex
Alexandre TORGUE March 29, 2021, 9:16 a.m. UTC | #4
Hi Alain

On 2/10/21 9:39 AM, Pierre Yves MORDRET wrote:
> Hello
> 
> Looks good to me
> 
> Reviewed-by: Pierre-Yves MORDRET <pierre-yves.mordret@foss.st.com>
> 
> Thx
> Regards
> 
> 

Applied on stm32-next.

Thanks.
Alex

> On 2/5/21 9:51 AM, Alain Volmat wrote:
>> Enable the analog filter for all I2C nodes of the stm32mp151.
>>
>> Signed-off-by: Alain Volmat <alain.volmat@foss.st.com>
>> ---
>>   arch/arm/boot/dts/stm32mp151.dtsi | 6 ++++++
>>   1 file changed, 6 insertions(+)
>>
>> diff --git a/arch/arm/boot/dts/stm32mp151.dtsi b/arch/arm/boot/dts/stm32mp151.dtsi
>> index 3c75abacb374..558fc8fb38b6 100644
>> --- a/arch/arm/boot/dts/stm32mp151.dtsi
>> +++ b/arch/arm/boot/dts/stm32mp151.dtsi
>> @@ -493,6 +493,7 @@
>>   			#size-cells = <0>;
>>   			st,syscfg-fmp = <&syscfg 0x4 0x1>;
>>   			wakeup-source;
>> +			i2c-analog-filter;
>>   			status = "disabled";
>>   		};
>>   
>> @@ -508,6 +509,7 @@
>>   			#size-cells = <0>;
>>   			st,syscfg-fmp = <&syscfg 0x4 0x2>;
>>   			wakeup-source;
>> +			i2c-analog-filter;
>>   			status = "disabled";
>>   		};
>>   
>> @@ -523,6 +525,7 @@
>>   			#size-cells = <0>;
>>   			st,syscfg-fmp = <&syscfg 0x4 0x4>;
>>   			wakeup-source;
>> +			i2c-analog-filter;
>>   			status = "disabled";
>>   		};
>>   
>> @@ -538,6 +541,7 @@
>>   			#size-cells = <0>;
>>   			st,syscfg-fmp = <&syscfg 0x4 0x10>;
>>   			wakeup-source;
>> +			i2c-analog-filter;
>>   			status = "disabled";
>>   		};
>>   
>> @@ -1533,6 +1537,7 @@
>>   			#size-cells = <0>;
>>   			st,syscfg-fmp = <&syscfg 0x4 0x8>;
>>   			wakeup-source;
>> +			i2c-analog-filter;
>>   			status = "disabled";
>>   		};
>>   
>> @@ -1570,6 +1575,7 @@
>>   			#size-cells = <0>;
>>   			st,syscfg-fmp = <&syscfg 0x4 0x20>;
>>   			wakeup-source;
>> +			i2c-analog-filter;
>>   			status = "disabled";
>>   		};
>>   
>>
>
diff mbox series

Patch

diff --git a/arch/arm/boot/dts/stm32mp151.dtsi b/arch/arm/boot/dts/stm32mp151.dtsi
index 3c75abacb374..558fc8fb38b6 100644
--- a/arch/arm/boot/dts/stm32mp151.dtsi
+++ b/arch/arm/boot/dts/stm32mp151.dtsi
@@ -493,6 +493,7 @@ 
 			#size-cells = <0>;
 			st,syscfg-fmp = <&syscfg 0x4 0x1>;
 			wakeup-source;
+			i2c-analog-filter;
 			status = "disabled";
 		};
 
@@ -508,6 +509,7 @@ 
 			#size-cells = <0>;
 			st,syscfg-fmp = <&syscfg 0x4 0x2>;
 			wakeup-source;
+			i2c-analog-filter;
 			status = "disabled";
 		};
 
@@ -523,6 +525,7 @@ 
 			#size-cells = <0>;
 			st,syscfg-fmp = <&syscfg 0x4 0x4>;
 			wakeup-source;
+			i2c-analog-filter;
 			status = "disabled";
 		};
 
@@ -538,6 +541,7 @@ 
 			#size-cells = <0>;
 			st,syscfg-fmp = <&syscfg 0x4 0x10>;
 			wakeup-source;
+			i2c-analog-filter;
 			status = "disabled";
 		};
 
@@ -1533,6 +1537,7 @@ 
 			#size-cells = <0>;
 			st,syscfg-fmp = <&syscfg 0x4 0x8>;
 			wakeup-source;
+			i2c-analog-filter;
 			status = "disabled";
 		};
 
@@ -1570,6 +1575,7 @@ 
 			#size-cells = <0>;
 			st,syscfg-fmp = <&syscfg 0x4 0x20>;
 			wakeup-source;
+			i2c-analog-filter;
 			status = "disabled";
 		};