[PATCH] doc: icicle: Fix incorrect board name in HSS docs

[Padmarao Begari]

Hi Sean,

On Tue, Mar 16, 2021 at 9:52 AM Sean Anderson <seanga2 at gmail.com> wrote:

> On 3/16/21 12:05 AM, Padmarao Begari wrote:
> > Hi Sean,
> >
> > On Thu, Mar 11, 2021 at 7:18 AM Sean Anderson <seanga2 at gmail.com
> <mailto:seanga2 at gmail.com>> wrote:
> >
> >     The HSS source uses an "mpfs" prefix with the icicle board name.
> Change our
> >     documentation to match.
> >
> >     Signed-off-by: Sean Anderson <seanga2 at gmail.com <mailto:
> seanga2 at gmail.com>>
> >     ---
> >
> >       doc/board/microchip/mpfs_icicle.rst | 6 +++---
> >       1 file changed, 3 insertions(+), 3 deletions(-)
> >
> >     diff --git a/doc/board/microchip/mpfs_icicle.rst
> b/doc/board/microchip/mpfs_icicle.rst
> >     index c71c2f3cab..cd18472542 100644
> >     --- a/doc/board/microchip/mpfs_icicle.rst
> >     +++ b/doc/board/microchip/mpfs_icicle.rst
> >     @@ -63,15 +63,15 @@ Build the HSS (Hart Software Services) -
> Microchip boot-flow
> >
> >       .. code-block:: none
> >
> >     -   make BOARD=icicle-kit-es config
> >     +   make BOARD=mpfs-icicle-kit-es config
> >
> >       Alternatively, copy the default config for Microchip boot-flow.
> >
> >       .. code-block:: none
> >
> >     -   cp boards/icicle-kit-es/def_config .config
> >     +   cp boards/mpfs-icicle-kit-es/def_config .config
> >
> >     -2. make BOARD=icicle-kit-es
> >     +2. make BOARD=mpfs-icicle-kit-es
> >       3. In the Default subdirectory, the standard build will create
> hss.elf and
> >          various binary formats (hss.hex and hss.bin).
> >
> >
> > The "mpfs" prefix added only for Microchip boot flow of the HSS but not
> Custom boot flow of the HSS.
>
> Do you mean that
>
> * The "mpfs" prefix should be used for both microchip boot flow and
>    custom boot flow, this patch adds it only to the microchip boot flow,
>    and this patch should be modified to add it to both.
>

   The "mpfs" prefix should be used for both microchip boot flow and
   custom boot flow, this patch adds it only to the microchip boot flow,
   and this patch should be modified to add it to both.

Regards
Padmarao


> * The "mpfs" prefix should be used only for the microchip boot flow
>    and not be used for the custom boot flow, this patch adds it to both
>    the microchip boot flow and the custom boot flow, and this patch
>    should be modified to only add it to the custom boot flow.
>
> Or something else entirely?
>
> Thanks,
>
> --Sean
>
> >
> > Regards
> > Padmarao
> >
> >     --
> >     2.30.1
> >
>
>