[U-Boot] [PATCHv4 3/3] ARM64: poplar: hi3798cv200: u-boot support for Poplar 96Boards

[Jorge Ramirez <jorge.ramirez-ortiz@linaro.org>]

On 05/29/2017 02:18 PM, Jorge Ramirez wrote:
> On 05/29/2017 01:57 PM, Tom Rini wrote:
>>> The issue is actually with serial-uclass.c when the kernel dts
>>> contains a chosen node that contains the stdout-path.
>>>      chosen {
>>>          stdout-path = "serial0:115200n8";
>>>      };
>>>
>>> Disabling uart0 (ie serial0) in u-boot.dtsi loses the console
>>> instead of probing the pl01x for the platform_data.
>>>
>>> is there a pre-defined way to work around this?
>> Provide a new stdout-path in the -u-boot.dtsi too?  Any changes you
>
> yes I obviously tried that but are you sure that that is allowed with 
> "chosen" it being kind of a special type of node?
> the compiler just cant find the label when added to u-boot.dtsi hence 
> why I was asking (sorry, I should have raised it upfront)
> ie:
>
> /*
>  * U-Boot addition to:
>  *  1) use platform data for the console
>  *  2) provide support for the generic-ehci USB driver currently not 
> available
>  *     in the linux kernel (8/May/2017).
>  *
>  * (C) Copyright 2017 Jorge Ramirez-Ortiz <jorge.ramirez-ortiz@linaro.org>
>  *
>  * SPDX-License-Identifier:     GPL-2.0+
>  */
>
> &soc {
>     usb2: ehci at 9890000 {
>         compatible = "generic-ehci";
>         reg = <0x9890000 0x100>;
>         status = "okay";
>     };
> };
>
> &uart0 {
>     status = "disabled";
> };
>
> &chosen {
>     stdout-path = &uart0;
>
> };


oops, wrong syntax. ok done.
will send v5 today

thanks!


>
>
> leads to
>
>   LD      u-boot
>   OBJCOPY u-boot.srec
>   OBJCOPY u-boot-nodtb.bin
>   SYM     u-boot.sym
> start=$(aarch64-linux-gnu-nm u-boot | grep __rel_dyn_start | cut -f 1 
> -d ' '); end=$(aarch64-linux-gnu-nm u-boot | grep __rel_dyn_end | cut 
> -f 1 -d ' '); tools/relocate-rela u-boot-nodtb.bin 0x37000000 $start $end
>   DTC     arch/arm/dts/hi3798cv200-poplar.dtb
> Error: ./arch/arm/dts/hi3798cv200-u-boot.dtsi:27.2-3 label or path, 
> 'chosen', not found
> FATAL ERROR: Syntax error parsing input tree
> scripts/Makefile.lib:322: recipe for target 
> 'arch/arm/dts/hi3798cv200-poplar.dtb' failed
> make[2]: *** [arch/arm/dts/hi3798cv200-poplar.dtb] Error 1
> dts/Makefile:32: recipe for target 
> 'arch/arm/dts/hi3798cv200-poplar.dtb' failed
> make[1]: *** [arch/arm/dts/hi3798cv200-poplar.dtb] Error 2
> Makefile:865: recipe for target 'dts/dt.dtb' failed
> make: *** [dts/dt.dtb] Error 2
>
>> could make to the main dts file so that this would work should be able
>> to be done in the -u-boot.dtsi too.
>
>
>
>
>
>
>
>
>
>
>