Re: [PATCH v2 2/2] ARM: dts: aspeed: Add NVIDIA MGX Vera C2 BMC
From: Andrew Jeffery
Date: Wed Nov 11 2026 - 18:46:15 EST
On Wed, 2026-11-11 at 17:32 +0800, Jacky Huang wrote:
> Hi Andrew,
>
> Thanks for taking the time to review this.
>
> > Can you please expand the description of the platform? Why was it
> > designed and built? What's its purpose?
>
> The NVIDIA MGX Vera C2 is a Vera CPU-based server platform. I will
> expand the description in v3 to clarify its purpose.
Thanks. Just to make sure we're on the same page, I feel stating that
'NVIDIA MGX Vera C2' is 'Vera-based' doesn't add any new information as
it's already in the name. Describing the purpose would give better
insight.
I'm after a helpful description in the commit message for the purpose
of summarising the content of the branch when I do a pull-request. 'Add
support for more platforms' doesn't really cut it there.
If the platform is described on the web by e.g. marketing material,
please include the link.
Andrew