[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <5e5c8bd2-4c4b-4371-a6b2-d4d4359a4233@nvidia.com>
Date: Tue, 5 Aug 2025 11:18:12 -0700
From: Donald Shannon <donalds@...dia.com>
To: Krzysztof Kozlowski <krzk@...nel.org>
Cc: robh@...nel.org, krzk+dt@...nel.org, conor+dt@...nel.org, joel@....id.au,
andrew@...econstruct.com.au, devicetree@...r.kernel.org,
linux-arm-kernel@...ts.infradead.org, linux-aspeed@...ts.ozlabs.org,
linux-kernel@...r.kernel.org, openbmc@...ts.ozlabs.org
Subject: Re: [PATCH v7 0/2] Adding device tree and binding for NVIDIA
GB200-UT3.0b
On 7/24/25 00:54, Krzysztof Kozlowski wrote:
> External email: Use caution opening links or attachments
>
>
> On Wed, Jul 23, 2025 at 03:23:48PM -0700, Donald Shannon wrote:
>> Patch 1 adds the binding for the NVIDIA GB200-UT3.0b platform.
>> Patch 2 adds the device tree for the NVIDIA GB200-UT3.0b platform.
>>
>> This is an Aspeed AST2600 based unit testing platform for GB200.
>> UT3.0b is different than nvidia-gb200nvl-bmc due to networking topology
>> differences, additional gpio expanders, and voltage regulator gating
>> some devices.
>>
>> Reference to Ast2600 SOC [1].
>> Reference to Blackwell GB200NVL Platform [2].
>>
>> Link: https://www.aspeedtech.com/server_ast2600/ [1]
>> Link: https://nvdam.widen.net/s/wwnsxrhm2w/blackwell-datasheet-3384703 [2]
>> Signed-off-by: Donald Shannon <donalds@...dia.com>
>> ---
>> Changes v1 -> v2:
>> - Changed phy-mode to rgmii-id [Lunn]
>> - Removed redundant max-speed for mac0 [Lunn]
>> - Fixed typo from gb200nvl to gb200 in Makefile
>> Changes v2 -> v3:
>> - Fixed whitespace issues [Krzysztof]
>> - Fixed schema validation issues from my end ( there are still issues
>> with the aspeed dtsi file that are not related to this new dts)
>> [Herring]
>> - Reordered to follow style guide [Krzysztof]
>> - Removed redundant status okays
>> - Changed vcc to vdd for the power gating on the gpio expanders
>> Changes v3 -> v4:
>> - Added changelog [Krzysztof]
>> - Added nvidia,gb200-ut30b board binding [Krzysztof]
>> - Removed unused imports
>> - Reordered a couple other style guide violations
>> - Added back in a couple needed "status okay"s
>> Changes v4 -> v5:
>> - Resumed my patch after a pause
>> - Don't plan to make this include of nvidia-gb200nvl-bmc due to some
>> platform differences
>> - Fixed io expanders that weren't gated by the 3.3V standby regulator
>> - Fixed incorrect interrupt pin for one IO expander
>> - Removed some IO expanders and I2C busses
>> Changes v5 -> v6:
>> - Fixed subject line
>> - Added missing gpio-key compatible type to buttons
>> Changes v6 -> v7:
>> - Removed Acked-by Krzysztof
> Why? You did not even give me chance to respond to your reply.
>
> Best regards,
> Krzysztof
Hi Krzysztof,
I will reply here to keep our discussion together.
I think that the issue was me using your kernel.org email and not your linaro.org email.
I will fix this in the next patch series along with some fixes for the warnings from the other thread.
Thanks,
Don
Powered by blists - more mailing lists