On Wed, May 18, 2016 at 04:18:38PM +0200, Heiko Schocher wrote:

> U-Boot SPL 2016.03-rc3-00019-g6dfb4c2-dirty (Mar 09 2016 - 07:40:06)
> SHC C3-Sample
> MPU reference clock runs at 6 MHz
> Setting MPU clock to 594 MHz
> Enabling Spread Spectrum of 18 permille for MPU
> Trying to boot from MMC
> reading u-boot.img
> reading u-boot.img
> 
> U-Boot 2016.03-rc3-00019-g6dfb4c2-dirty (Mar 09 2016 - 07:05:35 +0100)
> 
>        Watchdog enabled
> I2C:   ready
> DRAM:  512 MiB
> reloc off 1f783000
> MMC:   OMAP SD/MMC: 0, OMAP SD/MMC: 1
> Net:   cpsw
> U-Boot#
> 
> Signed-off-by: Heiko Schocher <h...@denx.de>
> 
> ---
> This patch drops the following checkpatch warnings:
> 
> warning: please write a paragraph that describes the config symbol fully
>   I wrote a help ... I do not understand, whats missing here...
> 
> warning: line over 80 characters
>   On places where I think this does not disturb...
> 
>   Patch:
>   http://permalink.gmane.org/gmane.comp.boot-loaders.u-boot/259522
>   is needed for clean compiling.

We need to find some other way to solve this or move all of the
partition stuff to Kconfig as well so we can default some stuff on and
then turn it off in cases.

> +config CONS_INDEX
> +     int "UART used for console"
> +     range 1 6
> +     default 1
> +     help
> +       The AM335x SoC has a total of 6 UARTs (UART0 to UART5 as referenced
> +       in documentation, etc) available to it. The SHC board uses UART1
> +       for the console.

Can you really have console on the other locations?  On the AM335x EVM,
it was possible, and doubly so with people making RS232 capes for
Beaglebones.  That's why it's exposed there.

> +int do_panic(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
> +{
> +     hang_bosch("u-boot: uImage not found", 4);
> +
> +     /*NOTREACHED*/
> +     return 0;
> +}
> +
> +U_BOOT_CMD(
> +     panic, 1, 0,    do_panic,
> +     "Hang the CPU - Panic!",
> +     ""
> +);

No more commands in board directories, and we should be able to do this
generically.  In fact, we should already be able to make fail to boot
Linux fatal which I gather is what this is used for.

-- 
Tom

Attachment: signature.asc
Description: Digital signature

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to