Re: [PATCH 2/2] docs/system/arm: Add quanta-gbs-bmc reference

[Cédric Le Goater]

On 6/21/21 3:31 PM, Peter Maydell wrote:
> On Tue, 15 Jun 2021 at 20:29, Patrick Venture <venture@google.com> wrote:
>>
>> Add line item reference to quanta-gbs-bmc machine.
>>
>> Signed-off-by: Patrick Venture <venture@google.com>
>> ---
>>  docs/system/arm/nuvoton.rst | 3 ++-
>>  1 file changed, 2 insertions(+), 1 deletion(-)
>>
>> diff --git a/docs/system/arm/nuvoton.rst b/docs/system/arm/nuvoton.rst
>> index ca011bd479..197d149f66 100644
>> --- a/docs/system/arm/nuvoton.rst
>> +++ b/docs/system/arm/nuvoton.rst
>> @@ -1,4 +1,4 @@
>> -Nuvoton iBMC boards (``npcm750-evb``, ``quanta-gsj``)
>> +Nuvoton iBMC boards (``*-bmc``, ``npcm750-evb``, ``quanta-gsj``)
>>  =====================================================
>>
>>  The `Nuvoton iBMC`_ chips (NPCM7xx) are a family of ARM-based SoCs that are
>> @@ -18,6 +18,7 @@ segment. The following machines are based on this chip :
>>  The NPCM730 SoC has two Cortex-A9 cores and is targeted for Data Center and
>>  Hyperscale applications. The following machines are based on this chip :
>>
>> +- ``quanta-gbs-bmc``    Quanta GBS server BMC
>>  - ``quanta-gsj``        Quanta GSJ server BMC
> 
> Why aren't we consistent about the names of the machines here?
> This suggests that either we should have called the new board
> 'quanta-gbs', not 'quanta-gbs-bmc', or we should deprecate
> 'quanta-gsj' and rename it 'quanta-gsj-bmc' ...

Indeed, the latter would be the best option. 

Thanks,

C.