Re: [PATCH v2 13/13] arm64: defconfig: Enable I3C and SPD5118 hwmon

On 11/04/2026 07:34, Akhil R wrote:
> On Fri, 10 Apr 2026 11:57:11 +0200, Krzysztof Kozlowski wrote:
>> On 10/04/2026 10:37, Akhil R wrote:
>>> On Fri, 10 Apr 2026 09:18:48 +0200, Krzysztof Kozlowski wrote:
>>>> On 10/04/2026 08:57, Guenter Roeck wrote:
>>>>> On 4/9/26 23:39, Krzysztof Kozlowski wrote:
>>>>>> On 09/04/2026 12:57, Akhil R wrote:
>>>>>>> Add I3C subsystem support, DesignWare I3C master controller, and
>>>>>>> SPD5118 hwmon sensor as modules to the defconfig and therefore
>>>>>>> enable the support for SPD5118 sensor on SOCAMM found in NVIDIA
>>>>>>> Vera platforms.
>>>>>>
>>>>>> git grep for "Vera" gave me zero results. Are you sure this is an
>>>>>> upstream platform? Please point the DTS using this.
>>>>>>
>>>>>
>>>>> I think this is an ACPI based system, or at least that is what Google search
>>>>> tells me.
>>>>
>>>> Thanks. Following Google Vera is either a "CPU" or entire architecture
>>>> (at least that's how they call it), so it does not have SPD5118 sensor.
>>>
>>> SOCAMM is a Memory Module. SPD5118, as it's Kconfig mentions, is a sensor
>>> found within such memory modules. I didn't quite get why would you state
>>> that the SOCAMM present in Vera architecture (or CPU) does not have
>>> SPD5118 in it.
>>
>> I said that CPU or entire architecture does not have it.
>>
>> Commit is pretty vague in helping me to figure out the things I asked
>> for in last email.
>>
>>
>>>
>>> Pasting the below from the Vera Rubin product page [1] -
>>> "NVIDIA Vera CPUs add enhanced serviceability with small-outline
>>> compression-attached memory modules (SOCAMM) LPDDR5X and in-system tests
>>> for the CPU cores."
>>>
>>> [1]: https://www.nvidia.com/en-us/data-center/technologies/rubin/
>>
>> So this is for Vera Rubin? For what is this exactly?
> 
> SOCAMM is with the Vera CPU. Any Vera based platform would have this module.
> Vera Rubin is one such platform.
> 
> SPD5118 is within the SOCAMM.

So just mention this, instead of giving imprecise "Vera".

Best regards,
Krzysztof