From: 'Andy Shevchenko' <andriy.shevchenko@linux.intel.com>
To: Jarkko Nikula <jarkko.nikula@linux.intel.com>
Cc: Jiawen Wu <jiawenwu@trustnetic.com>,
linux-i2c@vger.kernel.org, 'Andi Shyti' <andi.shyti@kernel.org>,
'Mika Westerberg' <mika.westerberg@linux.intel.com>,
'Jan Dabros' <jsd@semihalf.com>,
'Sanket Goswami' <Sanket.Goswami@amd.com>,
'Basavaraj Natikar' <Basavaraj.Natikar@amd.com>,
michael.j.ruhl@intel.com
Subject: Re: [PATCH v3 0/6] i2c: designware: Generic polling mode code
Date: Fri, 23 Feb 2024 17:20:01 +0200 [thread overview]
Message-ID: <Zdi3oQ87rfvIMUsH@smile.fi.intel.com> (raw)
In-Reply-To: <c813d6d1-e37f-44d5-be40-67976f49cdb7@linux.intel.com>
On Fri, Feb 23, 2024 at 05:09:37PM +0200, Jarkko Nikula wrote:
> On 2/18/24 11:19, Jiawen Wu wrote:
> > On Tue, Feb 13, 2024 8:49 PM, Jarkko Nikula wrote:
> > > Hi Jiawen, Sanket and Basavaraj
> > >
> > > I hope you could give a quick test to this patchset to see it won't cause
> > > regression on your HW. This is mostly the same than the first version what
> > > Jiawen tested earlier but in this 3rd version I decided to drop semphore
> > > touching code patches from v2 due they being out of scope and fix a few
> > > things noted by Andy.
> > >
> > > Changes are in the patch 3/6 with two minor fixes to comments while moving
> > > them and the patch 6/6 where i2c_dw_wait_transfer() is slightly modified
> > > by Andy's suggestion plus I decided to remove one comment which became
> > > unclear after this patch.
> >
> > Hi Jarkko,
> >
> > I've been testing the v3 patch series on Wangxun 10Gb NIC, it works well, too.
> > Thanks!
> >
> > Tested-by: Jiawen Wu <jiawenwu@trustnetic.com>
> >
> Andy: Was this set ok in your point of view now?
Yes, I hoped that Andi applies it earlier last week or so.
--
With Best Regards,
Andy Shevchenko
next prev parent reply other threads:[~2024-02-23 15:20 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-02-13 12:48 [PATCH v3 0/6] i2c: designware: Generic polling mode code Jarkko Nikula
2024-02-13 12:48 ` [PATCH v3 1/6] i2c: designware: Uniform initialization flow for polling mode Jarkko Nikula
2024-02-13 12:48 ` [PATCH v3 2/6] i2c: designware: Do not enable interrupts shortly in " Jarkko Nikula
2024-02-13 12:48 ` [PATCH v3 3/6] i2c: designware: Use accessors to DW_IC_INTR_MASK register Jarkko Nikula
2024-02-13 12:48 ` [PATCH v3 4/6] i2c: designware: Move interrupt handling functions before i2c_dw_xfer() Jarkko Nikula
2024-02-13 12:48 ` [PATCH v3 5/6] i2c: designware: Fix RX FIFO depth define on Wangxun 10Gb NIC Jarkko Nikula
2024-02-13 12:48 ` [PATCH v3 6/6] i2c: designware: Implement generic polling mode code for " Jarkko Nikula
2024-02-18 9:19 ` [PATCH v3 0/6] i2c: designware: Generic polling mode code Jiawen Wu
2024-02-23 15:09 ` Jarkko Nikula
2024-02-23 15:20 ` 'Andy Shevchenko' [this message]
2024-02-27 1:21 ` Andi Shyti
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=Zdi3oQ87rfvIMUsH@smile.fi.intel.com \
--to=andriy.shevchenko@linux.intel.com \
--cc=Basavaraj.Natikar@amd.com \
--cc=Sanket.Goswami@amd.com \
--cc=andi.shyti@kernel.org \
--cc=jarkko.nikula@linux.intel.com \
--cc=jiawenwu@trustnetic.com \
--cc=jsd@semihalf.com \
--cc=linux-i2c@vger.kernel.org \
--cc=michael.j.ruhl@intel.com \
--cc=mika.westerberg@linux.intel.com \
/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: link
Be 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.