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

From: Krzysztof Kozlowski <krzk@kernel.org>
To: Akhil R <akhilrajeev@nvidia.com>
Cc: Frank.Li@nxp.com, acpica-devel@lists.linux.dev,
	alexandre.belloni@bootlin.com, conor+dt@kernel.org,
	devicetree@vger.kernel.org, ebiggers@kernel.org,
	krzk+dt@kernel.org, lenb@kernel.org, linux-acpi@vger.kernel.org,
	linux-hwmon@vger.kernel.org, linux-i3c@lists.infradead.org,
	linux-kernel@vger.kernel.org, linux@roeck-us.net,
	miquel.raynal@bootlin.com, p.zabel@pengutronix.de,
	rafael@kernel.org, robh@kernel.org, sakari.ailus@linux.intel.com,
	wsa+renesas@sang-engineering.com
Subject: Re: [PATCH v2 13/13] arm64: defconfig: Enable I3C and SPD5118 hwmon
Date: Sun, 12 Apr 2026 15:21:36 +0200	[thread overview]
Message-ID: <5751aa6f-ee7a-4031-99a8-b64f45e9084b@kernel.org> (raw)
In-Reply-To: <20260411053433.49655-1-akhilrajeev@nvidia.com>

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

-- 
linux-i3c mailing list
linux-i3c@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-i3c