Re: [PATCH v3 0/9] i2c: add support for filters
> > What is the plan for this patch series? > > I hope to review it this weekend and my hope it is good to go for 5.4. Series looks good basically. Just a few comments for some patches. See there. Thanks! signature.asc Description: PGP signature
Re: [PATCH v3 0/9] i2c: add support for filters
> > I don't know if it will fit other vendors need concerning the binding > > but for Microchip it sounds good. > > > > Acked-by: Ludovic Desroches > > for the whole serie. > > > > Regards > > > > Ludovic > > > > Hello Wolfram, > > What is the plan for this patch series? I hope to review it this weekend and my hope it is good to go for 5.4. signature.asc Description: PGP signature
Re: [PATCH v3 0/9] i2c: add support for filters
On 12.07.2019 11:20, Ludovic Desroches wrote: > On Tue, Jul 09, 2019 at 03:19:26PM +0200, Eugen Hristev - M18282 wrote: >> From: Eugen Hristev >> >> Hello, >> >> This series adds support for analog and digital filters for i2c controllers >> >> This series is based on the series: >> [PATCH v2 0/9] i2c: at91: filters support for at91 SoCs >> and enhanced to add the bindings for all controllers plus an extra binding >> for the width of the spikes in nanoseconds. >> >> First, bindings are created for >> 'i2c-ana-filter' >> 'i2c-dig-filter' >> 'i2c-filter-width-ns' >> >> The support is added in the i2c core to retrieve filter width and add it >> to the timings structure. >> Next, the at91 driver is enhanced for supporting digital filter, advanced >> digital filter (with selectable spike width) and the analog filter. >> >> Finally the device tree for two boards are modified to make use of the >> new properties. >> >> This series is the result of the comments on the ML in the direction >> requested: to make the bindings globally available for i2c drivers. >> >> Changes in v3: >> - made bindings global for i2c controllers and modified accordingly >> - gave up PADFCDF bit because it's a lack in datasheet >> - the computation on the width of the spike is based on periph clock as it >> is done for hold time. >> >> Changes in v2: >> - added device tree bindings and support for enable-ana-filt and >> enable-dig-filt >> - added the new properties to the DT for sama5d4_xplained/sama5d2_xplained >> >> Eugen Hristev (9): >>dt-bindings: i2c: at91: add new compatible >>dt-bindings: i2c: add bindings for i2c analog and digital filter >>i2c: add support for filter-width-ns optional property >>i2c: at91: add new platform support for sam9x60 >>i2c: at91: add support for digital filtering >>i2c: at91: add support for advanced digital filtering >>i2c: at91: add support for analog filtering >>ARM: dts: at91: sama5d2_xplained: add analog and digital filter for >> i2c >>ARM: dts: at91: sama5d4_xplained: add analog filter for i2c >> >> Documentation/devicetree/bindings/i2c/i2c-at91.txt | 3 +- >> Documentation/devicetree/bindings/i2c/i2c.txt | 11 + >> arch/arm/boot/dts/at91-sama5d2_xplained.dts| 6 +++ >> arch/arm/boot/dts/at91-sama5d4_xplained.dts| 1 + >> drivers/i2c/busses/i2c-at91-core.c | 38 + >> drivers/i2c/busses/i2c-at91-master.c | 49 >> -- >> drivers/i2c/busses/i2c-at91.h | 13 ++ >> drivers/i2c/i2c-core-base.c| 2 + >> include/linux/i2c.h| 2 + >> 9 files changed, 121 insertions(+), 4 deletions(-) > > Hi, > > I don't know if it will fit other vendors need concerning the binding > but for Microchip it sounds good. > > Acked-by: Ludovic Desroches > for the whole serie. > > Regards > > Ludovic > Hello Wolfram, What is the plan for this patch series? Thanks, Eugen
Re: [PATCH v3 0/9] i2c: add support for filters
On Tue, Jul 09, 2019 at 03:19:26PM +0200, Eugen Hristev - M18282 wrote: > From: Eugen Hristev > > Hello, > > This series adds support for analog and digital filters for i2c controllers > > This series is based on the series: > [PATCH v2 0/9] i2c: at91: filters support for at91 SoCs > and enhanced to add the bindings for all controllers plus an extra binding > for the width of the spikes in nanoseconds. > > First, bindings are created for > 'i2c-ana-filter' > 'i2c-dig-filter' > 'i2c-filter-width-ns' > > The support is added in the i2c core to retrieve filter width and add it > to the timings structure. > Next, the at91 driver is enhanced for supporting digital filter, advanced > digital filter (with selectable spike width) and the analog filter. > > Finally the device tree for two boards are modified to make use of the > new properties. > > This series is the result of the comments on the ML in the direction > requested: to make the bindings globally available for i2c drivers. > > Changes in v3: > - made bindings global for i2c controllers and modified accordingly > - gave up PADFCDF bit because it's a lack in datasheet > - the computation on the width of the spike is based on periph clock as it > is done for hold time. > > Changes in v2: > - added device tree bindings and support for enable-ana-filt and > enable-dig-filt > - added the new properties to the DT for sama5d4_xplained/sama5d2_xplained > > Eugen Hristev (9): > dt-bindings: i2c: at91: add new compatible > dt-bindings: i2c: add bindings for i2c analog and digital filter > i2c: add support for filter-width-ns optional property > i2c: at91: add new platform support for sam9x60 > i2c: at91: add support for digital filtering > i2c: at91: add support for advanced digital filtering > i2c: at91: add support for analog filtering > ARM: dts: at91: sama5d2_xplained: add analog and digital filter for > i2c > ARM: dts: at91: sama5d4_xplained: add analog filter for i2c > > Documentation/devicetree/bindings/i2c/i2c-at91.txt | 3 +- > Documentation/devicetree/bindings/i2c/i2c.txt | 11 + > arch/arm/boot/dts/at91-sama5d2_xplained.dts| 6 +++ > arch/arm/boot/dts/at91-sama5d4_xplained.dts| 1 + > drivers/i2c/busses/i2c-at91-core.c | 38 + > drivers/i2c/busses/i2c-at91-master.c | 49 > -- > drivers/i2c/busses/i2c-at91.h | 13 ++ > drivers/i2c/i2c-core-base.c| 2 + > include/linux/i2c.h| 2 + > 9 files changed, 121 insertions(+), 4 deletions(-) Hi, I don't know if it will fit other vendors need concerning the binding but for Microchip it sounds good. Acked-by: Ludovic Desroches for the whole serie. Regards Ludovic