Re: [PATCH 1/2] dt-bindings: arm: aspeed: add ASUS X4TF board

From: Zev Weiss
Date: Fri Feb 23 2024 - 17:53:11 EST


On Thu, Feb 22, 2024 at 08:02:07AM PST, Conor Dooley wrote:
>On Thu, Feb 22, 2024 at 04:59:13PM +0800, Kelly Hung wrote:
>> Document the new compatibles used on ASUS X4TF.
>
>It would be good to mention here what the x4tf is - is it a bmc or
>an sbc etc.
>

..and if, as a quick web search indicates is likely the case, it's a
BMC for a server (or family of servers), following the existing naming
convention with something more like "asus,x4tf-bmc" might be nice.

Also, Kelly -- when sending revised versions of patches please use the
'-v' flag to git format-patch to include a version number in the subject
line so it's clearer what's going on. (So when posting the next version
incorporating the feedback on this one use '-v3', since this appears to
be a v2.)

Zev