From: Wolfram Sang <wsa+renesas@sang-engineering.com>
To: Paul Menzel <pmenzel@molgen.mpg.de>, Jean Delvare <jdelvare@suse.de>
Cc: linux-i2c@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] i2c: smbus: Add (LP)DDR5 types to `i2c_register_spd()`
Date: Fri, 26 Apr 2024 09:10:22 +0200 [thread overview]
Message-ID: <20240426071022.lqapnn6uzcypezrp@ninjato> (raw)
In-Reply-To: <20240327083356.74246-1-pmenzel@molgen.mpg.de>
[-- Attachment #1: Type: text/plain, Size: 1343 bytes --]
Hi Paul,
(adding Jean)
On Wed, Mar 27, 2024 at 09:33:55AM +0100, Paul Menzel wrote:
> On several systems Linux logs:
>
> i2c i2c-0: Memory type 0x22 not supported yet, not instantiating SPD
>
> 1. Supermicro Super Server/X13SAE, BIOS 2.0 10/17/2022
> 2. Dell Inc. Precision 3660/0PRR48, BIOS 2.9.3 11/22/2023
> 3. Dell Inc. OptiPlex SFF Plus 7010/0YGWFV, BIOS 1.7.1 08/11/2023
> 4. Run `git grep 'emory type.*supported yet, not instantiating SPD'` in
> the repository of dmesg reports for various computers collected by
> Linux users at https://linux-hardware.org. [1]
>
> Add 0x22 and 0x23 for DDR5 according to section 7.18.2 (Memory Device —
> Type), table 78 in *System Management BIOS (SMBIOS) Reference
> Specification*, version 3.6.0 [2].
>
> I use the same name as for DDR4 out of ignorance.
So, you didn't test it on DDR5 on your own? I'd like to have it tested.
The document you referenced does only mention I2C but the wikipedia
article [3] mentions I3C. It will probably support I2C fallback, but
I'd like to be sure.
[3] https://en.wikipedia.org/wiki/Serial_presence_detect
> [1]: https://www.dmtf.org/sites/default/files/standards/documents/DSP0134_3.6.0.pdf
> [2]: https://github.com/linuxhw/Dmesg
[1] and [2] are mixed up.
Thanks for the patch!
Wolfram
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
prev parent reply other threads:[~2024-04-26 7:10 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-03-27 8:33 [PATCH] i2c: smbus: Add (LP)DDR5 types to `i2c_register_spd()` Paul Menzel
2024-04-26 7:10 ` Wolfram Sang [this message]
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=20240426071022.lqapnn6uzcypezrp@ninjato \
--to=wsa+renesas@sang-engineering.com \
--cc=jdelvare@suse.de \
--cc=linux-i2c@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=pmenzel@molgen.mpg.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 read-only IMAP folder(s) and NNTP newsgroup(s).