| Message ID | 1562678049-17581-1-git-send-email-eugen.hristev@microchip.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | i2c: add support for filters | expand |
On Tue, Jul 09, 2019 at 03:19:26PM +0200, Eugen Hristev - M18282 wrote: > From: Eugen Hristev <eugen.hristev@microchip.com> > > 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 <ludovic.desroches@microchip.com> for the whole serie. Regards Ludovic
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 <eugen.hristev@microchip.com> >> >> 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 <ludovic.desroches@microchip.com> > for the whole serie. > > Regards > > Ludovic > Hello Wolfram, What is the plan for this patch series? Thanks, Eugen
> > I don't know if it will fit other vendors need concerning the binding > > but for Microchip it sounds good. > > > > Acked-by: Ludovic Desroches <ludovic.desroches@microchip.com> > > 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.
> > 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!
From: Eugen Hristev <eugen.hristev@microchip.com> 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(-)