Re: [meta-ti][dunfell/master PATCH] ti33x.inc: Use weak assignment for uboot board config

["Denys Dmytriyenko" <denys@ti.com>]

On Thu, Jun 11, 2020 at 10:13:53PM +0000, Ankur Tyagi wrote:
> ping

I haven't seen this patch on the list - are you sure you sent it properly? Can 
you please re-send it again?


> -----Original Message-----
> From: Ankur Tyagi <ankur.tyagi@gallagher.com> 
> Sent: Tuesday, 9 June 2020 4:02 PM
> To: meta-ti@lists.yoctoproject.org
> Cc: Ankur Tyagi <Ankur.Tyagi@gallagher.com>
> Subject: [meta-ti][dunfell/master PATCH] ti33x.inc: Use weak assignment for uboot board config
> 
> When using meta-ti BSP layer, user should be able to override default uboot
> board config with their custom board config as it is common for customers
> to design custom board based upon am335x-evm dev kit.

The idea is that if you are creating a custom board based on am335x-evm and 
adding new config to U-boot, you should consider adding a new OE machine. Even 
if your machine includes ti33x.inc for common SoC definitions, you can still 
overwrite UBOOT_MACHINE in your <custom-machine>.conf file...


> Signed-off-by: Ankur Tyagi <ankur.tyagi@gallagher.com>
> ---
>  conf/machine/include/ti33x.inc | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/conf/machine/include/ti33x.inc b/conf/machine/include/ti33x.inc
> index abfd7ec7..ff72a382 100644
> --- a/conf/machine/include/ti33x.inc
> +++ b/conf/machine/include/ti33x.inc
> @@ -39,7 +39,7 @@ KERNEL_DEVICETREE = " \
>  KERNEL_DEVICETREE += "${@oe.utils.conditional('ENABLE_TI_UIO_DEVICES', '1', 'am335x-icev2-pru-excl-uio.dtb', '', d)}"
>  
>  UBOOT_ARCH = "arm"
> -UBOOT_MACHINE = "am335x_evm_config"
> +UBOOT_MACHINE ?= "am335x_evm_config"
>  
>  UBOOT_ENTRYPOINT = "0x80008000"
>  UBOOT_LOADADDRESS = "0x80008000"
> -- 
> 2.17.1
> 

>