From: Paul Menzel <pmenzel@molgen.mpg.de>
To: Guenter Roeck <linux@roeck-us.net>
Cc: linux-hwmon@vger.kernel.org, linux-i2c@vger.kernel.org,
linux-kernel@vger.kernel.org, devicetree@vger.kernel.org,
"Krzysztof Kozlowski" <krzk+dt@kernel.org>,
"Wolfram Sang" <wsa+renesas@sang-engineering.com>,
"René Rebe" <rene@exactcode.de>,
"Thomas Weißschuh" <linux@weissschuh.net>,
"Armin Wolf" <W_Armin@gmx.de>,
"Stephen Horvath" <s.horvath@outlook.com.au>
Subject: Re: [PATCH v4 5/6] i2c: smbus: Support DDR5 SPD EEPROMs
Date: Tue, 18 Jun 2024 15:51:15 +0200 [thread overview]
Message-ID: <8719fc64-2b51-4b79-ba52-0a3b9216f2db@molgen.mpg.de> (raw)
In-Reply-To: <975af7e5-b1b0-400e-a1c3-6d9140421f25@roeck-us.net>
Dear Guenter,
Am 18.06.24 um 15:32 schrieb Guenter Roeck:
> On 6/18/24 03:25, Paul Menzel wrote:
> [ ... ]
>>
>> $ ls -l /sys/bus/i2c/drivers/spd5118/0-0050/eeprom
>> -r--r--r-- 1 root root 1024 Jun 18 12:17 /sys/bus/i2c/drivers/spd5118/0-0050/eeprom
>> $ cp /sys/bus/i2c/drivers/spd5118/0-0050/eeprom /tmp
>> cp: error reading '/sys/bus/i2c/drivers/spd5118/0-0050/eeprom': No such device or address
>
> That suggests that the i801 driver got an error when trying some chip
> operation.
> Unfortunately I have no idea what that error or the failed operation
> might be.
>
>> $ od -t x1 /sys/bus/i2c/drivers/spd5118/0-0050/eeprom
>> od: /sys/bus/i2c/drivers/spd5118/0-0050/eeprom: read error: No such device or address
>> 0000000
>>
>>> sudo i2cdump -y -f 0 0x50
>>
>> $ sudo LD_LIBRARY_PATH=~/src/i2c-tools/lib tools/i2cdump -y -f 0 0x50
>> No size specified (using byte-data access)
>> Error: Could not open file `/dev/i2c-0' or `/dev/i2c/0': No such file or directory
>>
> This should work after you load the "i2c-dev" module.
Silly me. Thank you.
> If you get it to work, please provide the output. Maybe it helps
> tracking down the problem.
```
$ sudo LD_LIBRARY_PATH=~/src/i2c-tools/lib tools/i2cdump -y -f 0 0x50
No size specified (using byte-data access)
0 1 2 3 4 5 6 7 8 9 a b c d e f 0123456789abcdef
00: 51 18 0a 86 32 03 32 00 00 00 00 00 ff 01 00 00 Q???2?2......?..
10: 00 00 00 00 00 00 00 00 00 00 00 00 70 03 00 00 ............p?..
20: 50 05 00 00 00 00 00 00 00 00 00 00 00 00 00 00 P?..............
30: 00 58 01 00 00 00 00 00 00 00 00 00 00 00 00 00 .X?.............
40: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 ................
50: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 ................
60: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 ................
70: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 ................
80: 30 10 12 02 04 00 20 62 00 00 00 00 90 02 00 00 0????. b....??..
90: 00 00 00 00 a0 01 f2 03 7a 0d 00 00 00 00 80 3e ....????z?....?>
a0: 80 3e 80 3e 00 7d 80 bb 30 75 27 01 a0 00 82 00 ?>?>.}??0u'??.?.
b0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 ................
c0: 00 00 00 00 00 00 88 13 08 88 13 08 20 4e 20 10 ......?????? N ?
d0: 27 10 15 34 20 10 27 10 c4 09 04 4c 1d 0c 00 00 '??4 ?'????L??..
e0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 ................
f0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 ................
```
So (00,b) = 0x00 opposed to 0x07 in your example output.
$ sensors
[…]
Then there is no change:
```
$ sudo LD_LIBRARY_PATH=~/src/i2c-tools/lib tools/i2cdump -y -f 0 0x50
No size specified (using byte-data access)
0 1 2 3 4 5 6 7 8 9 a b c d e f 0123456789abcdef
00: 51 18 0a 86 32 03 32 00 00 00 00 00 ff 01 00 00 Q???2?2......?..
10: 00 00 00 00 00 00 00 00 00 00 00 00 70 03 00 00 ............p?..
20: 50 05 00 00 00 00 00 00 00 00 00 00 00 00 00 00 P?..............
30: 00 58 01 00 00 00 00 00 00 00 00 00 00 00 00 00 .X?.............
40: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 ................
50: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 ................
60: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 ................
70: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 ................
80: 30 10 12 02 04 00 20 62 00 00 00 00 90 02 00 00 0????. b....??..
90: 00 00 00 00 a0 01 f2 03 7a 0d 00 00 00 00 80 3e ....????z?....?>
a0: 80 3e 80 3e 00 7d 80 bb 30 75 27 01 a0 00 82 00 ?>?>.}??0u'??.?.
b0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 ................
c0: 00 00 00 00 00 00 88 13 08 88 13 08 20 4e 20 10 ......?????? N ?
d0: 27 10 15 34 20 10 27 10 c4 09 04 4c 1d 0c 00 00 '??4 ?'????L??..
e0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 ................
f0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 ................
```
Kind regards,
Paul
next prev parent reply other threads:[~2024-06-18 13:51 UTC|newest]
Thread overview: 50+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-06-04 4:02 [PATCH v4 0/6] hwmon: Add support for SPD5118 compliant chips Guenter Roeck
2024-06-04 4:02 ` [PATCH v4 1/6] dt-bindings: trivial-devices: Add jedec,spd5118 Guenter Roeck
2024-06-04 4:02 ` [PATCH v4 2/6] hwmon: Add support for SPD5118 compliant temperature sensors Guenter Roeck
2024-06-04 8:48 ` Stephen Horvath
2024-06-04 14:31 ` Guenter Roeck
2024-06-07 15:55 ` Armin Wolf
2024-06-04 4:02 ` [PATCH v4 3/6] hwmon: (spd5118) Add suspend/resume support Guenter Roeck
2024-06-04 8:45 ` Stephen Horvath
2024-06-04 14:31 ` Guenter Roeck
2024-06-07 15:57 ` Armin Wolf
2024-06-04 4:02 ` [PATCH v4 4/6] hwmon: (spd5118) Add support for reading SPD data Guenter Roeck
2024-06-04 11:58 ` Armin Wolf
2024-06-04 14:30 ` Guenter Roeck
2024-06-07 15:59 ` Armin Wolf
2024-06-04 4:02 ` [PATCH v4 5/6] i2c: smbus: Support DDR5 SPD EEPROMs Guenter Roeck
2024-06-04 7:32 ` Wolfram Sang
2024-06-05 12:21 ` Paul Menzel
2024-06-05 13:56 ` Guenter Roeck
2024-06-17 14:42 ` Paul Menzel
2024-06-17 15:49 ` Guenter Roeck
2024-06-18 10:25 ` Paul Menzel
2024-06-18 13:32 ` Guenter Roeck
2024-06-18 13:51 ` Paul Menzel [this message]
2024-06-18 14:23 ` Guenter Roeck
2024-06-18 14:59 ` Paul Menzel
2024-06-18 15:10 ` Guenter Roeck
2024-06-18 15:25 ` Paul Menzel
2024-06-18 15:43 ` Guenter Roeck
2024-06-18 18:16 ` Guenter Roeck
2024-06-18 18:59 ` Paul Menzel
2024-06-18 19:31 ` Guenter Roeck
2024-06-18 15:12 ` Guenter Roeck
2024-06-18 15:27 ` Paul Menzel
2024-06-07 16:06 ` Armin Wolf
2024-06-07 18:00 ` Wolfram Sang
2024-06-10 13:52 ` Guenter Roeck
2024-06-10 14:52 ` Wolfram Sang
2024-06-10 15:55 ` Guenter Roeck
2024-06-12 16:19 ` Wolfram Sang
2024-06-24 20:06 ` Heiner Kallweit
2024-06-24 20:30 ` Guenter Roeck
2024-06-04 4:02 ` [PATCH v4 6/6] hwmon: (spd5118) Add configuration option for auto-detection Guenter Roeck
2024-06-04 4:37 ` Thomas Weißschuh
2024-06-04 14:04 ` Guenter Roeck
2024-06-04 7:44 ` Wolfram Sang
2024-06-04 14:04 ` Guenter Roeck
2024-06-05 2:19 ` [PATCH v4a " Guenter Roeck
2024-06-05 9:22 ` Thomas Weißschuh
2024-06-05 14:04 ` Guenter Roeck
2024-06-07 16:08 ` Armin Wolf
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=8719fc64-2b51-4b79-ba52-0a3b9216f2db@molgen.mpg.de \
--to=pmenzel@molgen.mpg.de \
--cc=W_Armin@gmx.de \
--cc=devicetree@vger.kernel.org \
--cc=krzk+dt@kernel.org \
--cc=linux-hwmon@vger.kernel.org \
--cc=linux-i2c@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux@roeck-us.net \
--cc=linux@weissschuh.net \
--cc=rene@exactcode.de \
--cc=s.horvath@outlook.com.au \
--cc=wsa+renesas@sang-engineering.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 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).