linux-kernel - Re: [PATCH v4 5/6] i2c: smbus: Support DDR5 SPD EEPROMs

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <b2fe83e6-8ebc-42f7-ba14-fbc1806a90f9@roeck-us.net>
Date: Tue, 18 Jun 2024 11:16:35 -0700
From: Guenter Roeck <linux@...ck-us.net>
To: Paul Menzel <pmenzel@...gen.mpg.de>
Cc: linux-hwmon@...r.kernel.org, linux-i2c@...r.kernel.org,
 linux-kernel@...r.kernel.org, devicetree@...r.kernel.org,
 Krzysztof Kozlowski <krzk+dt@...nel.org>,
 Wolfram Sang <wsa+renesas@...g-engineering.com>,
 René Rebe <rene@...ctcode.de>,
 Thomas Weißschuh <linux@...ssschuh.net>,
 Armin Wolf <W_Armin@....de>, Stephen Horvath <s.horvath@...look.com.au>
Subject: Re: [PATCH v4 5/6] i2c: smbus: Support DDR5 SPD EEPROMs

On 6/18/24 08:25, Paul Menzel wrote:
> Dear Guenter,
> 
> 
> Am 18.06.24 um 17:10 schrieb Guenter Roeck:
>> On 6/18/24 07:59, Paul Menzel wrote:
>> [ ... ]
>>
>>> I did
>>>
>>>      $ tail -3 /etc/sensors3.conf
>>>      chip "spd5118-*"
>>>          set temp1_max 56000
>>>          set temp1_crit 84000
>>>
>>> but it stays with the defaults:
>>>
>>> ```
>>> $ sensors
>>> spd5118-i2c-0-53
>>> Adapter: SMBus I801 adapter at efa0
>>> temp1:        +20.8°C  (low  =  +0.0°C, high = +55.0°C)
>>>                         (crit low =  +0.0°C, crit = +85.0°C)
>>>
>>
>> You'd have to write directly into the attribute files.
>> For example, if you have
>>
>> $ grep . /sys/class/hwmon/*/name
>> /sys/class/hwmon/hwmon0/name:nvme
>> /sys/class/hwmon/hwmon1/name:nct6687
>> /sys/class/hwmon/hwmon2/name:k10temp
>> /sys/class/hwmon/hwmon3/name:spd5118
>> /sys/class/hwmon/hwmon4/name:spd5118
>> /sys/class/hwmon/hwmon5/name:spd5118
>> /sys/class/hwmon/hwmon6/name:spd5118
>> /sys/class/hwmon/hwmon7/name:mt7921_phy0
>> /sys/class/hwmon/hwmon8/name:amdgpu
>>
>> you could run
>>
>> sudo bash -c 'echo 56000 > /sys/class/hwmon/hwmon3/temp1_max'
> 
>      $ sudo bash -c 'echo 56000 > /sys/class/hwmon/hwmon3/temp1_max'
>      bash: line 1: echo: write error: No such device or address
> 

Please add

	.use_single_write = true,

to the regmap configuration (spd5118_regmap_config) to see if it helps.

Thanks,
Guenter