All of lore.kernel.org
 help / color / mirror / Atom feed
From: Wolfram Sang <wsa+renesas@sang-engineering.com>
To: Brigham Campbell <me@brighamcampbell.com>,
	Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Cc: "Stephen Horvath" <s.horvath@outlook.com.au>,
	linux-i2c@vger.kernel.org, "Jean Delvare" <jdelvare@suse.com>,
	"Benoît Monin" <benoit.monin@bootlin.com>
Subject: Re: [PATCH i2c-tools] TODO: add file and describe items for the 4.5 release
Date: Wed, 8 Jul 2026 10:38:55 +0200	[thread overview]
Message-ID: <ak4Mn4NOfIqCQTO3@shikoro> (raw)
In-Reply-To: <DJSVN4OMOMB7.EBB9SV2PLWTR@brighamcampbell.com>

[-- Attachment #1: Type: text/plain, Size: 1713 bytes --]


@Andy: you know more about laptops and I2C than I do. The initial issue
was reported here:

https://lore.kernel.org/all/DJRA1PB8LKKJ.2RTZWEP2LVRFF@brighamcampbell.com/

It would be great to get that bus working because Brigham likely could
then test the new DDR5 patches for decode-dimms.

> It looks like it's bound to i2c_hid_acpi. No ddr5 temperature sensors in
> sight...

Well, it was worth a try. Thanks.

> > How many? If it is in the 120 range, then every access from i2cdetect
> > fails with it.
> 
> When I run i2cdetect on the i801 controller, on one invocation I count
> 111 repetitions of the "SMBus is busy, can't use it!" error once, then
> on another invocation I count 112 repetitions.

Okay, as expected, this is more or less one BUSY condition per address
scanned.

> When the kernel's i2c bus arbitration retry logic kicks in, it appears
> to be able to eventually complete the operation successfully before the
> i2c core driver code gives up because otherwise I would expect i2cdetect
> to exit prematurely and report an error.

It would mark that address as "XX" if there was an error. As it prints
"--", it indeed indicates that the retry could have succeeded.

> Wolfram, if you or any other kernel developers think this may be
> indicative of some underlying issue, I'm happy to investigate further
> and report back. I'm using a Dell XPS 14 DA14260 laptop.

Thank you, but sadly I am not the right person to ask. My expertise is
the embedded sector. I don't know much about laptops, ACPI, and how the
management processors play in the game. My assumption would be that the
BUSY state has something to do with the latter. But maybe Andy knows?


[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

  reply	other threads:[~2026-07-08  8:38 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-07-05 18:10 [PATCH i2c-tools] TODO: add file and describe items for the 4.5 release Wolfram Sang
2026-07-06  4:45 ` Brigham Campbell
2026-07-06  5:19   ` Stephen Horvath
2026-07-06  6:33     ` Brigham Campbell
2026-07-07 12:16       ` Wolfram Sang
2026-07-08  3:41         ` Brigham Campbell
2026-07-08  8:38           ` Wolfram Sang [this message]
2026-07-08 11:10             ` Andy Shevchenko
2026-07-06  8:04 ` Benoît Monin
2026-07-07 11:51 ` 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=ak4Mn4NOfIqCQTO3@shikoro \
    --to=wsa+renesas@sang-engineering.com \
    --cc=andriy.shevchenko@linux.intel.com \
    --cc=benoit.monin@bootlin.com \
    --cc=jdelvare@suse.com \
    --cc=linux-i2c@vger.kernel.org \
    --cc=me@brighamcampbell.com \
    --cc=s.horvath@outlook.com.au \
    /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.