From: Nicolas Ferre <nicolas.ferre@microchip.com> To: Wolfram Sang <wsa@kernel.org>, Codrin Ciubotariu <codrin.ciubotariu@microchip.com>, <linux-i2c@vger.kernel.org>, <linux-arm-kernel@lists.infradead.org>, <linux-kernel@vger.kernel.org>, <alexandre.belloni@bootlin.com>, <ludovic.desroches@microchip.com>, <andrew@sanpeople.com>, <mhoffman@lightlink.com>, <khali@linux-fr.org>, <peda@axentia.se>, Claudiu Beznea <Claudiu.Beznea@microchip.com> Cc: Cristian Birsan <Cristian.Birsan@microchip.com> Subject: Re: [RESEND 0/3] i2c: at91: Fixes and updates Date: Thu, 30 Jun 2022 09:41:53 +0200 [thread overview] Message-ID: <176099e2-cbff-1987-f59a-2ca618a9c92a@microchip.com> (raw) In-Reply-To: <YrysFU0fP7X1wkBd@shikoro> Hi Wolfram, On 29/06/2022 at 21:46, Wolfram Sang wrote: > On Tue, Jun 14, 2022 at 01:13:44PM +0300, Codrin Ciubotariu wrote: >> Resend due to the fact that the i2c list no longer accepts replys on the >> old patch-set. Comments not addressed yet. >> >> The first patch is mostly a prerequisite for the second one. It only >> moves the i2c_recover_bus() out of the actual transfer function. This >> helps the second patch disable the controller before using GPIO >> recovery. The second patch will keep the controller enabled when a >> transfer occurs. Before using GPIO recovery, the controller must be >> disabled, to ignore potential glitches. However, the controller must be >> enabled for HW recovery (bus CLEAR command). The third and last patch >> adds advanced digital filtering support for SAMA5D4. The TWI IP found in >> SAMA5D4 supports advanced digital filtering, even if, at the moment of >> this patch, the SAMA5D4 datasheet does not mention it. > > So, can you guys read this mail? It should be possible somehow that I > can read your remarks to my remarks, no? Codrin is not with Microchip anymore and we have to re-distribute the maintenance of this driver internally. His email address will probably "bounce" soon. So, forgive us if there's a lack of response for some time ;-) Best regards, Nicolas -- Nicolas Ferre
WARNING: multiple messages have this Message-ID (diff)
From: Nicolas Ferre <nicolas.ferre@microchip.com> To: Wolfram Sang <wsa@kernel.org>, Codrin Ciubotariu <codrin.ciubotariu@microchip.com>, <linux-i2c@vger.kernel.org>, <linux-arm-kernel@lists.infradead.org>, <linux-kernel@vger.kernel.org>, <alexandre.belloni@bootlin.com>, <ludovic.desroches@microchip.com>, <andrew@sanpeople.com>, <mhoffman@lightlink.com>, <khali@linux-fr.org>, <peda@axentia.se>, Claudiu Beznea <Claudiu.Beznea@microchip.com> Cc: Cristian Birsan <Cristian.Birsan@microchip.com> Subject: Re: [RESEND 0/3] i2c: at91: Fixes and updates Date: Thu, 30 Jun 2022 09:41:53 +0200 [thread overview] Message-ID: <176099e2-cbff-1987-f59a-2ca618a9c92a@microchip.com> (raw) In-Reply-To: <YrysFU0fP7X1wkBd@shikoro> Hi Wolfram, On 29/06/2022 at 21:46, Wolfram Sang wrote: > On Tue, Jun 14, 2022 at 01:13:44PM +0300, Codrin Ciubotariu wrote: >> Resend due to the fact that the i2c list no longer accepts replys on the >> old patch-set. Comments not addressed yet. >> >> The first patch is mostly a prerequisite for the second one. It only >> moves the i2c_recover_bus() out of the actual transfer function. This >> helps the second patch disable the controller before using GPIO >> recovery. The second patch will keep the controller enabled when a >> transfer occurs. Before using GPIO recovery, the controller must be >> disabled, to ignore potential glitches. However, the controller must be >> enabled for HW recovery (bus CLEAR command). The third and last patch >> adds advanced digital filtering support for SAMA5D4. The TWI IP found in >> SAMA5D4 supports advanced digital filtering, even if, at the moment of >> this patch, the SAMA5D4 datasheet does not mention it. > > So, can you guys read this mail? It should be possible somehow that I > can read your remarks to my remarks, no? Codrin is not with Microchip anymore and we have to re-distribute the maintenance of this driver internally. His email address will probably "bounce" soon. So, forgive us if there's a lack of response for some time ;-) Best regards, Nicolas -- Nicolas Ferre _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2022-06-30 7:42 UTC|newest] Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top 2022-06-14 10:13 [RESEND 0/3] i2c: at91: Fixes and updates Codrin Ciubotariu 2022-06-14 10:13 ` Codrin Ciubotariu 2022-06-14 10:13 ` [RESEND 1/3] i2c: at91: move i2c_recover_bus() outside of at91_do_twi_transfer() Codrin Ciubotariu 2022-06-14 10:13 ` Codrin Ciubotariu 2022-06-14 10:13 ` [RESEND 2/3] i2c: at91: keep the controller disabled when it is not used Codrin Ciubotariu 2022-06-14 10:13 ` Codrin Ciubotariu 2022-06-14 10:13 ` [RESEND 3/3] i2c: at91: add advanced digital filtering support for SAMA5D4 Codrin Ciubotariu 2022-06-14 10:13 ` Codrin Ciubotariu 2022-06-29 19:46 ` [RESEND 0/3] i2c: at91: Fixes and updates Wolfram Sang 2022-06-29 19:46 ` Wolfram Sang 2022-06-30 7:41 ` Nicolas Ferre [this message] 2022-06-30 7:41 ` Nicolas Ferre 2022-06-30 8:09 ` Wolfram Sang 2022-06-30 8:09 ` Wolfram Sang
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=176099e2-cbff-1987-f59a-2ca618a9c92a@microchip.com \ --to=nicolas.ferre@microchip.com \ --cc=Claudiu.Beznea@microchip.com \ --cc=Cristian.Birsan@microchip.com \ --cc=alexandre.belloni@bootlin.com \ --cc=andrew@sanpeople.com \ --cc=codrin.ciubotariu@microchip.com \ --cc=khali@linux-fr.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-i2c@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=ludovic.desroches@microchip.com \ --cc=mhoffman@lightlink.com \ --cc=peda@axentia.se \ --cc=wsa@kernel.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.