From: Wolfram Sang <wsa+renesas@sang-engineering.com>
To: Markus Stockhausen <markus.stockhausen@gmx.de>
Cc: andi.shyti@kernel.org, linusw@kernel.org, brgl@kernel.org,
linux-i2c@vger.kernel.org, linux-gpio@vger.kernel.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH v7 1/2] i2c: algo: bit: Allow to skip bit test
Date: Wed, 15 Jul 2026 22:31:44 +0200 [thread overview]
Message-ID: <alfuMKlJqXD-YEMb@ninjato> (raw)
In-Reply-To: <20260714162915.3018703-2-markus.stockhausen@gmx.de>
[-- Attachment #1: Type: text/plain, Size: 1028 bytes --]
On Tue, Jul 14, 2026 at 06:29:14PM +0200, Markus Stockhausen wrote:
> An I2C bus can make use of shared resources. E.g. two GPIO based buses
> that share a single SCL line. To synchronize access to the bus the driver
> might use locking with the help of i2c_lock_operations(). While this
> works for normal transfers it is not available during initialization.
>
> Especially if i2c-algo-bit module is loaded with parameter bit_test
> it will issue some basic sanity checks that will access the bus without
> locking. This might interfere badly with concurrent transfers. Even
> if these are well synchronized via locks.
>
> Allow the consumer of an algorithm to override if the bit_test is allowed
> or not. For this add a new boolean attribute named skip_bit_test to
> i2c_algo_bit_data. If set the test is not run.
>
> Signed-off-by: Markus Stockhausen <markus.stockhausen@gmx.de>
Reviewed-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
Tested-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
next prev parent reply other threads:[~2026-07-15 20:31 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-07-14 16:29 [PATCH v7 0/2] i2c: i2c-gpio: Enhance driver for buses with shared SCL Markus Stockhausen
2026-07-14 16:29 ` [PATCH v7 1/2] i2c: algo: bit: Allow to skip bit test Markus Stockhausen
2026-07-15 20:31 ` Wolfram Sang [this message]
2026-07-16 8:15 ` Bartosz Golaszewski
2026-07-14 16:29 ` [PATCH v7 2/2] i2c: i2c-gpio: Enhance driver for buses with shared SCL Markus Stockhausen
2026-07-15 20:36 ` Wolfram Sang
2026-07-28 19:25 ` Andi Shyti
2026-07-28 22:04 ` Wolfram Sang
2026-07-15 6:26 ` AW: [PATCH v7 0/2] " Markus Stockhausen
2026-07-15 6:44 ` 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=alfuMKlJqXD-YEMb@ninjato \
--to=wsa+renesas@sang-engineering.com \
--cc=andi.shyti@kernel.org \
--cc=brgl@kernel.org \
--cc=linusw@kernel.org \
--cc=linux-gpio@vger.kernel.org \
--cc=linux-i2c@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=markus.stockhausen@gmx.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 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.