[U-Boot] [PATCH v3 07/22] pcm052: board: Remove "m4go" command as it is superseded by "bootaux"

Lukasz Majewski lukma at denx.de
Wed Feb 13 15:01:47 UTC 2019


Hi Marcel,

> Hi Lukasz
> 
> On Sun, 2019-02-03 at 00:02 +0100, Lukasz Majewski wrote:
> > The "m4go" provides exactly the same functionality as the IMX
> > generic "bootaux" command. Remove it to not duplicate the code.  
> 
> Makes sense but as far as I can tell from its Kconfig so far bootaux
> is only for i.MX 7 and i.MX 6. Should I send a patch for this?

Yes, please prepare a patch on top of this series.

(I'm now build testing v4 of this patch series, and plan to post it
tonight).

> Plus
> you would also have to enable bootaux support in your defconfig.

It turned out that BK4 is finally not using it, so I don't need to
enable it.

> 
> Cheers
> 
> Marcel
> 
> > Signed-off-by: Lukasz Majewski <lukma at denx.de>
> > ---
> > 
> > Changes in v3: None
> > Changes in v2: None
> > 
> >  board/phytec/pcm052/pcm052.c | 38
> > ---------------------------------- ----
> >  1 file changed, 38 deletions(-)
> > 
> > diff --git a/board/phytec/pcm052/pcm052.c
> > b/board/phytec/pcm052/pcm052.c
> > index cfc8009102..4e4b870304 100644
> > --- a/board/phytec/pcm052/pcm052.c
> > +++ b/board/phytec/pcm052/pcm052.c
> > @@ -577,41 +577,3 @@ int checkboard(void)
> >  
> >  	return 0;
> >  }
> > -
> > -static int do_m4go(cmd_tbl_t *cmdtp, int flag, int argc,
> > -		       char * const argv[])
> > -{
> > -	ulong addr;
> > -
> > -	/* Consume 'm4go' */
> > -	argc--; argv++;
> > -
> > -	/*
> > -	 * Parse provided address - default to load_addr in case
> > not provided.
> > -	 */
> > -
> > -	if (argc)
> > -		addr = simple_strtoul(argv[0], NULL, 16);
> > -	else
> > -		addr = load_addr;
> > -
> > -	/*
> > -	 * Write boot address in PERSISTENT_ENTRY1[31:0] aka
> > SRC_GPR2[31:0]
> > -	 */
> > -	writel(addr + 0x401, 0x4006E028);
> > -
> > -	/*
> > -	 * Start secondary processor by enabling its clock
> > -	 */
> > -	writel(0x15a5a, 0x4006B08C);
> > -
> > -	return 1;
> > -}
> > -
> > -U_BOOT_CMD(
> > -	m4go, 2 /* one arg max */, 1 /* repeatable */, do_m4go,
> > -	"start the secondary Cortex-M4 from scatter file image",
> > -	"[<addr>]\n"
> > -	"    - start secondary Cortex-M4 core using a scatter file
> > image\n"
> > -	"The argument needs to be a scatter file\n"
> > -);  




Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma at denx.de
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 488 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20190213/04bf87ea/attachment.sig>


More information about the U-Boot mailing list