[PATCH 0/2] Updates docs/system/arm to include new bmc machines

[PATCH 0/2] Updates docs/system/arm to include new bmc machines

[Patrick Venture]

Patrick Venture (2):
  docs/system/arm: Add quanta-q7l1-bmc reference
  docs/system/arm: Add quanta-gbs-bmc reference

 docs/system/arm/aspeed.rst  | 1 +
 docs/system/arm/nuvoton.rst | 3 ++-
 2 files changed, 3 insertions(+), 1 deletion(-)

-- 
2.32.0.272.g935e593368-goog

[PATCH 1/2] docs/system/arm: Add quanta-q7l1-bmc reference

[Patrick Venture]

Adds a line-item reference to the supported quanta-q71l-bmc aspeed
entry.

Signed-off-by: Patrick Venture <venture@google.com>
---
 docs/system/arm/aspeed.rst | 1 +
 1 file changed, 1 insertion(+)

diff --git a/docs/system/arm/aspeed.rst b/docs/system/arm/aspeed.rst
index 57ee2bd94f..cec87e3743 100644
--- a/docs/system/arm/aspeed.rst
+++ b/docs/system/arm/aspeed.rst
@@ -13,6 +13,7 @@ etc.
 AST2400 SoC based machines :
 
 - ``palmetto-bmc``         OpenPOWER Palmetto POWER8 BMC
+- ``quanta-q71l-bmc``      OpenBMC Quanta BMC
 
 AST2500 SoC based machines :
 
-- 
2.32.0.272.g935e593368-goog

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

[Cédric Le Goater]

On 6/15/21 9:28 PM, Patrick Venture wrote:
> Adds a line-item reference to the supported quanta-q71l-bmc aspeed
> entry.
> 
> Signed-off-by: Patrick Venture <venture@google.com>

Reviewed-by: Cédric Le Goater <clg@kaod.org>

Thanks,

C.

> ---
>  docs/system/arm/aspeed.rst | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/docs/system/arm/aspeed.rst b/docs/system/arm/aspeed.rst
> index 57ee2bd94f..cec87e3743 100644
> --- a/docs/system/arm/aspeed.rst
> +++ b/docs/system/arm/aspeed.rst
> @@ -13,6 +13,7 @@ etc.
>  AST2400 SoC based machines :
>  
>  - ``palmetto-bmc``         OpenPOWER Palmetto POWER8 BMC
> +- ``quanta-q71l-bmc``      OpenBMC Quanta BMC
>  
>  AST2500 SoC based machines :
>  
> 

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

[Patrick Venture]

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
 
 There are also two more SoCs, NPCM710 and NPCM705, which are single-core
-- 
2.32.0.272.g935e593368-goog

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

[Cédric Le Goater]

On 6/15/21 9:28 PM, Patrick Venture wrote:
> Add line item reference to quanta-gbs-bmc machine.
> 
> Signed-off-by: Patrick Venture <venture@google.com>

Reviewed-by: Cédric Le Goater <clg@kaod.org>

Thanks,

C.

> ---
>  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
>  
>  There are also two more SoCs, NPCM710 and NPCM705, which are single-core
> 

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

[Peter Maydell]

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' ...

thanks
-- PMM

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. 

Re: [PATCH 0/2] Updates docs/system/arm to include new bmc machines

[Peter Maydell]

On Tue, 15 Jun 2021 at 20:28, Patrick Venture <venture@google.com> wrote:
>
>
> Patrick Venture (2):
>   docs/system/arm: Add quanta-q7l1-bmc reference
>   docs/system/arm: Add quanta-gbs-bmc reference
>
>  docs/system/arm/aspeed.rst  | 1 +
>  docs/system/arm/nuvoton.rst | 3 ++-
>  2 files changed, 3 insertions(+), 1 deletion(-)



Applied to target-arm.next, thanks.

-- PMM

Re: [PATCH 0/2] Updates docs/system/arm to include new bmc machines

[Peter Maydell]

On Mon, 28 Jun 2021 at 15:07, Peter Maydell <peter.maydell@linaro.org> wrote:
>
> On Tue, 15 Jun 2021 at 20:28, Patrick Venture <venture@google.com> wrote:
> >
> >
> > Patrick Venture (2):
> >   docs/system/arm: Add quanta-q7l1-bmc reference
> >   docs/system/arm: Add quanta-gbs-bmc reference
> >
> >  docs/system/arm/aspeed.rst  | 1 +
> >  docs/system/arm/nuvoton.rst | 3 ++-
> >  2 files changed, 3 insertions(+), 1 deletion(-)
>
>
>
> Applied to target-arm.next, thanks.

Just noticed the docs don't actually build:

Warning, treated as error:
/home/petmay01/linaro/qemu-from-laptop/qemu/docs/system/arm/nuvoton.rst:2:Title
underline too short.

Nuvoton iBMC boards (``*-bmc``, ``npcm750-evb``, ``quanta-gsj``)
=====================================================


But the fix is trivial, so I will make it locally.

Passing configure '--enable-docs' will enforce that it complains if you
don't have enough prerequisites to build the docs, rather than silently
skipping them.

thanks
-- PMM