[PATCHv3 09/12] doc: cmd: add documentation for cpu command

[Z.Q. Hou]

Hi Simon,

> -----Original Message-----
> From: Simon Glass <sjg at chromium.org>
> Sent: Monday, July 29, 2024 3:36 AM
> To: Z.Q. Hou <zhiqiang.hou at nxp.com>; Heinrich Schuchardt
> <xypron.glpk at gmx.de>
> Cc: u-boot at lists.denx.de; trini at konsulko.com; Peng Fan
> <peng.fan at nxp.com>; festevam at gmail.com
> Subject: Re: [PATCHv3 09/12] doc: cmd: add documentation for cpu command
> 
> +Heinrich Schuchardt for review too
> 
> Hi Zhiqiang,
> 
> On Fri, 26 Jul 2024 at 11:30, Zhiqiang Hou <Zhiqiang.Hou at nxp.com> wrote:
> >
> > From: Hou Zhiqiang <Zhiqiang.Hou at nxp.com>
> >
> > Add documentation for the 'cpu' command, taking NXP i.MX 8M Plus as a
> > example.
> >
> > Signed-off-by: Hou Zhiqiang <Zhiqiang.Hou at nxp.com>
> > ---
> > V3:
> >  - New patch.
> >
> >  doc/usage/cmd/cpu.rst | 82
> > +++++++++++++++++++++++++++++++++++++++++++
> >  1 file changed, 82 insertions(+)
> >  create mode 100644 doc/usage/cmd/cpu.rst
> >
> 
> I think you have left behind the review tags I sent on the previous version. You
> should pick those up for the next version ('patman status' can do it for you).

Will add in v4.

> 
> > diff --git a/doc/usage/cmd/cpu.rst b/doc/usage/cmd/cpu.rst new file
> > mode 100644 index 00000000000..0970e57d5b5
> > --- /dev/null
> > +++ b/doc/usage/cmd/cpu.rst
> > @@ -0,0 +1,82 @@
> > +.. SPDX-License-Identifier: GPL-2.0+
> > +.. Copyright 2024 NXP
> > +
> > +.. index::
> > +   single: cpu (command)
> > +
> > +cpu command
> > +===========
> > +
> > +Synopsis
> > +--------
> > +
> > +::
> > +
> > +    cpu list
> > +    cpu detail
> > +    cpu release <core ID> <addr>
> > +
> > +Description
> > +-----------
> > +
> > +The *cpu* command prints information about the CPUs, and release a
> > +CPU core to a given address to run applications.
> > +
> > +Example
> > +-------
> 
> We normally put the examples at the bottom, after all subcommands have
> been explained.

Will add brief description for each subcommand.

> 
> > +
> > +cpu list
> > +~~~~~~~~
> > +List and print brief information of all the CPU cores On i.MX8M Plus EVK:
> > +::
> > +
> > +    u-boot=> cpu list
> > +      0: cpu at 0      NXP i.MX8MP Rev1.1 A53 at 1200 MHz at 31C
> > +
> 
> Are there blank lines here? It seems like there should not be?

Yes, copied from the test logs.

> 
> > +      1: cpu at 1      NXP i.MX8MP Rev1.1 A53 at 1200 MHz at 30C
> > +
> > +      2: cpu at 2      NXP i.MX8MP Rev1.1 A53 at 1200 MHz at 31C
> > +
> > +      3: cpu at 3      NXP i.MX8MP Rev1.1 A53 at 1200 MHz at 31C
> > +
> > +cpu detail
> > +~~~~~~~~~~
> > +Print more details of all the CPU cores On i.MX8M Plus EVK:
> > +::
> > +
> > +    u-boot=> cpu detail
> > +      0: cpu at 0      NXP i.MX8MP Rev1.1 A53 at 1200 MHz at 31C
> > +
> > +            ID = 0, freq = 1.2 GHz: L1 cache, MMU
> > +      1: cpu at 1      NXP i.MX8MP Rev1.1 A53 at 1200 MHz at 30C
> > +
> > +            ID = 0, freq = 1.2 GHz: L1 cache, MMU
> > +      2: cpu at 2      NXP i.MX8MP Rev1.1 A53 at 1200 MHz at 31C
> > +
> > +            ID = 0, freq = 1.2 GHz: L1 cache, MMU
> > +      3: cpu at 3      NXP i.MX8MP Rev1.1 A53 at 1200 MHz at 31C
> > +
> > +            ID = 0, freq = 1.2 GHz: L1 cache, MMU
> > +
> > +cpu release
> > +~~~~~~~~~~~
> > +On i.MX8M Plus EVK, release the LAST core to run a RTOS application,
> > +the <core ID> is the sequence number refer to the 'cpu list':
> > +::
> > +
> > +     u-boot=> load mmc 1:2 C0000000 /hello_world.bin
> > +     66008 bytes read in 5 ms (12.6 MiB/s)
> > +     u-boot=> dcache flush; icache flush
> > +     u-boot=> cpu release 3 C0000000
> 
> lower-case c

Will change in v4.

Thanks,
Zhiqiang