Re: [PATCH v6 17/20] ARM: dts: aspeed: Add IBM Huygens BMC system

Subject: Re: [PATCH v6 17/20] ARM: dts: aspeed: Add IBM Huygens BMC system
From: Eddie James <eajames@xxxxxxxxxxxxx>
Date: Thu, 23 May 2024 14:00:22 -0500
Cc: LKML <linux-kernel@xxxxxxxxxxxxxxx>, Andi Shyti <andi.shyti@xxxxxxxxxx>, Andrew Jeffery <andrew@xxxxxxxxxxxxxxxxxxxx>, Conor Dooley <conor+dt@xxxxxxxxxx>, Joel Stanley <joel@xxxxxxxxx>, Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>, Lakshmi Yadlapati <lakshmiy@xxxxxxxxxx>, Mark Brown <broonie@xxxxxxxxxx>, Ninad Palsule <ninad@xxxxxxxxxxxxx>, Rob Herring <robh@xxxxxxxxxx>
In-reply-to: <2fe45df6-01a2-488b-99fb-5ee20491554c@web.de>
References: <20240522192524.3286237-18-eajames@linux.ibm.com> <2fe45df6-01a2-488b-99fb-5ee20491554c@web.de>
User-agent: Mozilla Thunderbird


On 5/23/24 13:45, Markus Elfring wrote:

The Huygens is a Rainier with modifed FSI wiring.

Will imperative wordings become helpful for a better commit message here?

This statement is a description of hardware. I cannot word thatimperatively. The commit message is imperative - "Add Huygens system".


Regards,
Markus

Follow-Ups:
- Re: [PATCH v6 17/20] ARM: dts: aspeed: Add IBM Huygens BMC system
  - From: Krzysztof Kozlowski
- Re: [v6 17/20] ARM: dts: aspeed: Add IBM Huygens BMC system
  - From: Markus Elfring

References:
- [PATCH v6 17/20] ARM: dts: aspeed: Add IBM Huygens BMC system
  - From: Eddie James
- Re: [PATCH v6 17/20] ARM: dts: aspeed: Add IBM Huygens BMC system
  - From: Markus Elfring