Re: [PATCH 2/2] sandbox: add cpuinfo command

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Ahmad,

On 16/10/2024 10:55, Ahmad Fatoum wrote:
> We have stack dump support in sandbox via AddressSanitizer, if it's
> compiled in. To make it easier to test proper operation, let's a cpuinfo
Small typo in the commit msg
> command like we already do on ARM and give it the same -s option.
>
> Signed-off-by: Ahmad Fatoum <a.fatoum@xxxxxxxxxxxxxx>
> ---
>
> +BAREBOX_CMD_START(cpuinfo)
> +	.cmd            = do_cpuinfo,
> +	BAREBOX_CMD_DESC("show info about CPU")
> +	BAREBOX_CMD_OPTS("[-s]")

It seems the -s option here is mandatory, maybe remove the [ ] brackets? Not sure if it means "optional" here...

Regards,

-- 

Yann










[Index of Archives]     [Linux Embedded]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux