2021-07-27 13:44:38

by Codrin Ciubotariu

Subject: [PATCH v2] ARM: dts: at91: sama5d2_icp: enable digital filter for I2C nodes

SAMA5D2's I2C controller supports digital filter, so let's enable it.

Signed-off-by: Codrin Ciubotariu <[email protected]>
---

Changes in v2:
- removed Acked-by tag

arch/arm/boot/dts/at91-sama5d2_icp.dts | 6 ++++++
1 file changed, 6 insertions(+)

diff --git a/arch/arm/boot/dts/at91-sama5d2_icp.dts b/arch/arm/boot/dts/at91-sama5d2_icp.dts
index bd64721fa23c..1c235fc5f788 100644
--- a/arch/arm/boot/dts/at91-sama5d2_icp.dts
+++ b/arch/arm/boot/dts/at91-sama5d2_icp.dts
@@ -184,6 +184,8 @@ i2c6: i2c@600 {
dmas = <0>, <0>;
pinctrl-names = "default";
pinctrl-0 = <&pinctrl_flx4_default>;
+ i2c-digital-filter;
+ i2c-digital-filter-width-ns = <35>;
status = "okay";

mcp16502@5b {
@@ -307,6 +309,8 @@ regulator-state-mem {
&i2c0 { /* mikrobus i2c */
pinctrl-names = "default";
pinctrl-0 = <&pinctrl_mikrobus_i2c>;
+ i2c-digital-filter;
+ i2c-digital-filter-width-ns = <35>;
status = "okay";
};

@@ -314,6 +318,8 @@ &i2c1 {
dmas = <0>, <0>;
pinctrl-names = "default";
pinctrl-0 = <&pinctrl_i2c1_default>;
+ i2c-digital-filter;
+ i2c-digital-filter-width-ns = <35>;
status = "okay";

eeprom@50 {
--
2.30.2

2021-07-27 14:50:51

by Nicolas Ferre

Subject: Re: [PATCH v2] ARM: dts: at91: sama5d2_icp: enable digital filter for I2C nodes

On 27/07/2021 at 15:41, Codrin Ciubotariu wrote:
> SAMA5D2's I2C controller supports digital filter, so let's enable it.
>
> Signed-off-by: Codrin Ciubotariu <[email protected]>
> ---
>
> Changes in v2:
> - removed Acked-by tag

And I add it back here ;-) :
Acked-by: Nicolas Ferre <[email protected]>

I queue the patch in at91-dt for 5.15.

Thanks Codrin, best regards,
Nicolas

>
> arch/arm/boot/dts/at91-sama5d2_icp.dts | 6 ++++++
> 1 file changed, 6 insertions(+)
>
> diff --git a/arch/arm/boot/dts/at91-sama5d2_icp.dts b/arch/arm/boot/dts/at91-sama5d2_icp.dts
> index bd64721fa23c..1c235fc5f788 100644
> --- a/arch/arm/boot/dts/at91-sama5d2_icp.dts
> +++ b/arch/arm/boot/dts/at91-sama5d2_icp.dts
> @@ -184,6 +184,8 @@ i2c6: i2c@600 {
> dmas = <0>, <0>;
> pinctrl-names = "default";
> pinctrl-0 = <&pinctrl_flx4_default>;
> + i2c-digital-filter;
> + i2c-digital-filter-width-ns = <35>;
> status = "okay";
>
> mcp16502@5b {
> @@ -307,6 +309,8 @@ regulator-state-mem {
> &i2c0 { /* mikrobus i2c */
> pinctrl-names = "default";
> pinctrl-0 = <&pinctrl_mikrobus_i2c>;
> + i2c-digital-filter;
> + i2c-digital-filter-width-ns = <35>;
> status = "okay";
> };
>
> @@ -314,6 +318,8 @@ &i2c1 {
> dmas = <0>, <0>;
> pinctrl-names = "default";
> pinctrl-0 = <&pinctrl_i2c1_default>;
> + i2c-digital-filter;
> + i2c-digital-filter-width-ns = <35>;
> status = "okay";
>
> eeprom@50 {
>


--
Nicolas Ferre