From: Sven Eckelmann <sven@narfation.org>
To: Andi Shyti <andi.shyti@kernel.org>,
Chris Packham <Chris.Packham@alliedtelesis.co.nz>
Cc: "linux-i2c@vger.kernel.org" <linux-i2c@vger.kernel.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
Jonas Jelonek <jelonek.jonas@gmail.com>,
Harshal Gohel <hg@simonwunderlich.de>,
Simon Wunderlich <sw@simonwunderlich.de>,
"stable@vger.kernel.org" <stable@vger.kernel.org>
Subject: Re: [PATCH v2 0/4] i2c: rtl9300: Fix multi-byte I2C operations
Date: Mon, 04 Aug 2025 10:35:35 +0200 [thread overview]
Message-ID: <10701742.nUPlyArG6x@ripper> (raw)
In-Reply-To: <4152a424-13ea-4437-b9e9-f1b5561cca9e@alliedtelesis.co.nz>
[-- Attachment #1: Type: text/plain, Size: 1743 bytes --]
On Monday, 4 August 2025 00:39:40 CEST Chris Packham wrote:
> For the series
>
> Reviewed-by: Chris Packham <chris.packham@alliedtelesis.co.nz>
> Tested-by: Chris Packham <chris.packham@alliedtelesis.co.nz>
Thank you.
> Note that I've only got the same simple eeprom devices that I did the
> initial development on so I don't think I've really exercised the block
> data paths but I can say the changes don't appear to have regressed
> anything.
I can understand this problem quite well. We can all only try our best and
then hope that someone with the actual HW can figure out the specific parts
which we didn't had access to.
> Is you series intended to apply on top of Jonas's? I'm trying to apply
> yours alone (for various reasons happens to be on top of net-next/main)
> and I'm getting conflicts.
No, I prepare something for downstream testing (with Jonas' patch):
https://github.com/openwrt/openwrt/pull/19577#discussion_r2248520949
> Conflict appears to be with
> https://lore.kernel.org/all/20250615235248.529019-1-alexguo1023@gmail.com/
Thanks, I was not aware of this specific one. I don't exactly know the repo
structure for I2C Host drivers. But
git://git.kernel.org/pub/scm/linux/kernel/git/andi.shyti/linux.git
i2c/i2c-host-fixes or i2c/i2c-host-next didn't had this patch. I've also
checked linux-next and couldn't find the patch at the moment.
I am guessing it is the best when I resent this patch as part of my patchset
and modify my patches accordingly. The resent will then be done this evening
(GMT+2). Preview can be found at
https://git.open-mesh.org/linux-merge.git/log/?h=b4/i2c-rtl9300-multi-byte
I've also checked linux-next and couldn't find the patch at the moment.
Kind regards,
Sven
[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
next prev parent reply other threads:[~2025-08-04 8:35 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-08-03 16:54 [PATCH v2 0/4] i2c: rtl9300: Fix multi-byte I2C operations Sven Eckelmann
2025-08-03 16:54 ` [PATCH v2 1/4] i2c: rtl9300: Fix multi-byte I2C write Sven Eckelmann
2025-08-03 16:54 ` [PATCH v2 2/4] i2c: rtl9300: Increase timeout for transfer polling Sven Eckelmann
2025-08-03 16:54 ` [PATCH v2 3/4] i2c: rtl9300: Add missing count byte for SMBus Block Ops Sven Eckelmann
2025-08-03 22:38 ` Chris Packham
2025-08-03 16:54 ` [PATCH v2 4/4] i2c: rtl9300: Implement I2C block read and write Sven Eckelmann
2025-08-03 22:39 ` [PATCH v2 0/4] i2c: rtl9300: Fix multi-byte I2C operations Chris Packham
2025-08-04 8:35 ` Sven Eckelmann [this message]
2025-08-04 20:45 ` Chris Packham
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=10701742.nUPlyArG6x@ripper \
--to=sven@narfation.org \
--cc=Chris.Packham@alliedtelesis.co.nz \
--cc=andi.shyti@kernel.org \
--cc=hg@simonwunderlich.de \
--cc=jelonek.jonas@gmail.com \
--cc=linux-i2c@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=stable@vger.kernel.org \
--cc=sw@simonwunderlich.de \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).