Re: [meta-xilinx] [PATCH] machine-xilinx-default.inc: Default to u-boot for Zynq

[Manjukumar Harthikote Matha]

> -Original Message-
> From: Nathan Rossi [mailto:nat...@nathanrossi.com]
> Sent: Thursday, May 04, 2017 7:30 AM
> To: Manjukumar Harthikote Matha <manju...@xilinx.com>
> Cc: Philip Balister <phi...@balister.org>; meta-xil...@lists.yoctoproject.org
> Subject: Re: [meta-xilinx] [PATCH] machine-xilinx-default.inc: Default to 
> u-boot for
> Zynq
>
> On 2 May 2017 at 07:52, Manjukumar Harthikote Matha
> <manjukumar.harthikote-ma...@xilinx.com> wrote:
> >
> >
> >> -Original Message-
> >> From: Nathan Rossi [mailto:nat...@nathanrossi.com]
> >> Sent: Thursday, April 27, 2017 9:44 AM
> >> To: Manjukumar Harthikote Matha <manju...@xilinx.com>
> >> Cc: Philip Balister <phi...@balister.org>; 
> >> meta-xil...@lists.yoctoproject.org
> >> Subject: Re: [meta-xilinx] [PATCH] machine-xilinx-default.inc: Default to 
> >> u-boot
> for
> >> Zynq
> >>
> >> On 28 April 2017 at 01:36, Manjukumar Harthikote Matha
>  >> ma...@xilinx.com> wrote:
> >> >
> >> >
> >> >> -Original Message-
> >> >> From: Nathan Rossi [mailto:nat...@nathanrossi.com]
> >> >> Sent: Thursday, April 27, 2017 7:11 AM
> >> >> To: Manjukumar Harthikote Matha <manju...@xilinx.com>
> >> >> Cc: Philip Balister <phi...@balister.org>;
> >> >> meta-xil...@lists.yoctoproject.org
> >> >> Subject: Re: [meta-xilinx] [PATCH] machine-xilinx-default.inc:
> >> >> Default to u-boot for Zynq
> >> >>
> >> >> On 27 April 2017 at 08:21, Manjukumar Harthikote Matha
> >> >> <manjukumar.harthikote-ma...@xilinx.com> wrote:
> >> >> >
> >> >> >
> >> >> >> -Original Message-
> >> >> >> From: Philip Balister [mailto:phi...@balister.org]
> >> >> >> Sent: Wednesday, April 26, 2017 12:38 PM
> >> >> >> To: Manjukumar Harthikote Matha <manju...@xilinx.com>; Nathan
> >> >> >> Rossi <nat...@nathanrossi.com>
> >> >> >> Cc: meta-xil...@lists.yoctoproject.org
> >> >> >> Subject: Re: [meta-xilinx] [PATCH] machine-xilinx-default.inc:
> >> >> >> Default to u-boot
> >> >> for
> >> >> >> Zynq
> >> >> >>
> >> >> >> On 04/26/2017 03:06 PM, Manjukumar Harthikote Matha wrote:
> >> >> >> >
> >> >> >> >
> >> >> >> >> -Original Message-
> >> >> >> >> From: Nathan Rossi [mailto:nat...@nathanrossi.com]
> >> >> >> >> Sent: Wednesday, April 26, 2017 10:54 AM
> >> >> >> >> To: Manjukumar Harthikote Matha <manju...@xilinx.com>
> >> >> >> >> Cc: meta-xil...@lists.yoctoproject.org
> >> >> >> >> Subject: Re: [meta-xilinx] [PATCH] machine-xilinx-default.inc:
> >> >> >> >> Default to u-boot for Zynq
> >> >> >> >>
> >> >> >> >> On 27 April 2017 at 02:41, Manjukumar Harthikote Matha
> >> >> >> >>  wrote:
> >> >> >> >>>
> >> >> >> >>>
> >> >> >> >>>> -Original Message-
> >> >> >> >>>> From: meta-xilinx-boun...@yoctoproject.org
> >> >> >> >>>> [mailto:meta-xilinx- boun...@yoctoproject.org] On Behalf Of
> >> >> >> >>>> Nathan Rossi
> >> >> >> >>>> Sent: Wednesday, April 26, 2017 4:57 AM
> >> >> >> >>>> To: meta-xil...@lists.yoctoproject.org
> >> >> >> >>>> Subject: [meta-xilinx] [PATCH] machine-xilinx-default.inc:
> >> >> >> >>>> Default to u-boot for Zynq
> >> >> >> >>>>
> >> >> >> >>>> Upstream U-Boot provides an almost complete environment for
> >> >> >> >>>> the majority of Zynq targets and specifically covers all the
> >> >> >> >>>> boot functionality required for the boards in the meta-xilinx
> >> >> >> >>>> layer. As such default to
> >> >> >> >> the mainline version of U-Boot.
> >> >> >> >>>>
> >> >> >> >

Re: [meta-xilinx] [PATCH] machine-xilinx-default.inc: Default to u-boot for Zynq

[Nathan Rossi]

On 2 May 2017 at 07:52, Manjukumar Harthikote Matha
<manjukumar.harthikote-ma...@xilinx.com> wrote:
>
>
>> -Original Message-
>> From: Nathan Rossi [mailto:nat...@nathanrossi.com]
>> Sent: Thursday, April 27, 2017 9:44 AM
>> To: Manjukumar Harthikote Matha <manju...@xilinx.com>
>> Cc: Philip Balister <phi...@balister.org>; meta-xil...@lists.yoctoproject.org
>> Subject: Re: [meta-xilinx] [PATCH] machine-xilinx-default.inc: Default to 
>> u-boot for
>> Zynq
>>
>> On 28 April 2017 at 01:36, Manjukumar Harthikote Matha 
>> > ma...@xilinx.com> wrote:
>> >
>> >
>> >> -Original Message-
>> >> From: Nathan Rossi [mailto:nat...@nathanrossi.com]
>> >> Sent: Thursday, April 27, 2017 7:11 AM
>> >> To: Manjukumar Harthikote Matha <manju...@xilinx.com>
>> >> Cc: Philip Balister <phi...@balister.org>;
>> >> meta-xil...@lists.yoctoproject.org
>> >> Subject: Re: [meta-xilinx] [PATCH] machine-xilinx-default.inc:
>> >> Default to u-boot for Zynq
>> >>
>> >> On 27 April 2017 at 08:21, Manjukumar Harthikote Matha
>> >> <manjukumar.harthikote-ma...@xilinx.com> wrote:
>> >> >
>> >> >
>> >> >> -Original Message-
>> >> >> From: Philip Balister [mailto:phi...@balister.org]
>> >> >> Sent: Wednesday, April 26, 2017 12:38 PM
>> >> >> To: Manjukumar Harthikote Matha <manju...@xilinx.com>; Nathan
>> >> >> Rossi <nat...@nathanrossi.com>
>> >> >> Cc: meta-xil...@lists.yoctoproject.org
>> >> >> Subject: Re: [meta-xilinx] [PATCH] machine-xilinx-default.inc:
>> >> >> Default to u-boot
>> >> for
>> >> >> Zynq
>> >> >>
>> >> >> On 04/26/2017 03:06 PM, Manjukumar Harthikote Matha wrote:
>> >> >> >
>> >> >> >
>> >> >> >> -Original Message-
>> >> >> >> From: Nathan Rossi [mailto:nat...@nathanrossi.com]
>> >> >> >> Sent: Wednesday, April 26, 2017 10:54 AM
>> >> >> >> To: Manjukumar Harthikote Matha <manju...@xilinx.com>
>> >> >> >> Cc: meta-xil...@lists.yoctoproject.org
>> >> >> >> Subject: Re: [meta-xilinx] [PATCH] machine-xilinx-default.inc:
>> >> >> >> Default to u-boot for Zynq
>> >> >> >>
>> >> >> >> On 27 April 2017 at 02:41, Manjukumar Harthikote Matha
>> >> >> >>  wrote:
>> >> >> >>>
>> >> >> >>>
>> >> >> >>>> -Original Message-
>> >> >> >>>> From: meta-xilinx-boun...@yoctoproject.org
>> >> >> >>>> [mailto:meta-xilinx- boun...@yoctoproject.org] On Behalf Of
>> >> >> >>>> Nathan Rossi
>> >> >> >>>> Sent: Wednesday, April 26, 2017 4:57 AM
>> >> >> >>>> To: meta-xil...@lists.yoctoproject.org
>> >> >> >>>> Subject: [meta-xilinx] [PATCH] machine-xilinx-default.inc:
>> >> >> >>>> Default to u-boot for Zynq
>> >> >> >>>>
>> >> >> >>>> Upstream U-Boot provides an almost complete environment for
>> >> >> >>>> the majority of Zynq targets and specifically covers all the
>> >> >> >>>> boot functionality required for the boards in the meta-xilinx
>> >> >> >>>> layer. As such default to
>> >> >> >> the mainline version of U-Boot.
>> >> >> >>>>
>> >> >> >>>> For users that require or prefer to use u-boot-xlnx this can
>> >> >> >>>> be selected on a per- machine basis using:
>> >> >> >>>>
>> >> >> >>>>   PREFERRED_PROVIDER_virtual/bootloader = "u-boot-xlnx"
>> >> >> >>>>
>> >> >> >>>> Signed-off-by: Nathan Rossi <nat...@nathanrossi.com>
>> >> >> >>>> ---
>> >> >> >>>>  conf/machine/include/machine-xilinx-default.inc | 1 +
>> >> >> >>>>  1 file changed, 1 insertion(+)
>> >> >> >>>>
>> >> >> >>>&g

Re: [meta-xilinx] [PATCH] machine-xilinx-default.inc: Default to u-boot for Zynq

[Nathan Rossi]

On 28 April 2017 at 05:13, Philip Balister  wrote:
> On 04/27/2017 12:59 PM, Mike Looijmans wrote:
>> On 27-04-17 16:11, Nathan Rossi wrote:
>> ...
>>> Looking at the differences, ignoring the obvious zynqmp/microblaze
>>> changes there are only a few features that are available in
>>> u-boot-xlnx that are not in mainline:
>>>  * partial bitstream loading support
>>>  * secure/encrypted bitstream loading support
>>>  * support for rsa verification of images
>>>  * board configs/devicetrees for various Xilinx internal boards
>>
>> You missed one: Dual-QSPI flash support. That only exists on
>> u-boot-xlnx, it never got implemented in a way acceptable to upstream.

Ah yes I did miss that, my bad. Fooled myself thinking it was working
when testing u-boot on a zc706.

>>
>> The same goes for the kernel. The only way to get dual-qspi flash to
>> work is to use the xlnx bootloader and kernel.
>
> But surely this could be handled as patches adding this feature to
> mainline, and not carrying a custom u-boot source tree forever.

There was a series sent for u-boot that adds this support, so applying
that series would work (even though the changes were not accepted).
(https://patchwork.ozlabs.org/patch/650100/)

Regards,
Nathan
-- 
___
meta-xilinx mailing list
meta-xilinx@yoctoproject.org
https://lists.yoctoproject.org/listinfo/meta-xilinx

Re: [meta-xilinx] [PATCH] machine-xilinx-default.inc: Default to u-boot for Zynq

[Philip Balister]

On 04/27/2017 12:59 PM, Mike Looijmans wrote:
> On 27-04-17 16:11, Nathan Rossi wrote:
> ...
>> Looking at the differences, ignoring the obvious zynqmp/microblaze
>> changes there are only a few features that are available in
>> u-boot-xlnx that are not in mainline:
>>  * partial bitstream loading support
>>  * secure/encrypted bitstream loading support
>>  * support for rsa verification of images
>>  * board configs/devicetrees for various Xilinx internal boards
> 
> You missed one: Dual-QSPI flash support. That only exists on
> u-boot-xlnx, it never got implemented in a way acceptable to upstream.
> 
> The same goes for the kernel. The only way to get dual-qspi flash to
> work is to use the xlnx bootloader and kernel.

But surely this could be handled as patches adding this feature to
mainline, and not carrying a custom u-boot source tree forever.

Philip


> 
> 
> 
-- 
___
meta-xilinx mailing list
meta-xilinx@yoctoproject.org
https://lists.yoctoproject.org/listinfo/meta-xilinx

Re: [meta-xilinx] [PATCH] machine-xilinx-default.inc: Default to u-boot for Zynq

[Mike Looijmans]

On 27-04-17 16:11, Nathan Rossi wrote:
...

Looking at the differences, ignoring the obvious zynqmp/microblaze
changes there are only a few features that are available in
u-boot-xlnx that are not in mainline:
 * partial bitstream loading support
 * secure/encrypted bitstream loading support
 * support for rsa verification of images
 * board configs/devicetrees for various Xilinx internal boards


You missed one: Dual-QSPI flash support. That only exists on 
u-boot-xlnx, it never got implemented in a way acceptable to upstream.


The same goes for the kernel. The only way to get dual-qspi flash to 
work is to use the xlnx bootloader and kernel.




--
Mike Looijmans


Kind regards,

Mike Looijmans
System Expert

TOPIC Products
Materiaalweg 4, NL-5681 RJ Best
Postbus 440, NL-5680 AK Best
Telefoon: +31 (0) 499 33 69 79
E-mail: mike.looijm...@topicproducts.com
Website: www.topicproducts.com

Please consider the environment before printing this e-mail





--
___
meta-xilinx mailing list
meta-xilinx@yoctoproject.org
https://lists.yoctoproject.org/listinfo/meta-xilinx

Re: [meta-xilinx] [PATCH] machine-xilinx-default.inc: Default to u-boot for Zynq

[Nathan Rossi]

On 28 April 2017 at 01:36, Manjukumar Harthikote Matha
<manjukumar.harthikote-ma...@xilinx.com> wrote:
>
>
>> -Original Message-
>> From: Nathan Rossi [mailto:nat...@nathanrossi.com]
>> Sent: Thursday, April 27, 2017 7:11 AM
>> To: Manjukumar Harthikote Matha <manju...@xilinx.com>
>> Cc: Philip Balister <phi...@balister.org>; meta-xil...@lists.yoctoproject.org
>> Subject: Re: [meta-xilinx] [PATCH] machine-xilinx-default.inc: Default to 
>> u-boot for
>> Zynq
>>
>> On 27 April 2017 at 08:21, Manjukumar Harthikote Matha
>> <manjukumar.harthikote-ma...@xilinx.com> wrote:
>> >
>> >
>> >> -Original Message-
>> >> From: Philip Balister [mailto:phi...@balister.org]
>> >> Sent: Wednesday, April 26, 2017 12:38 PM
>> >> To: Manjukumar Harthikote Matha <manju...@xilinx.com>; Nathan Rossi
>> >> <nat...@nathanrossi.com>
>> >> Cc: meta-xil...@lists.yoctoproject.org
>> >> Subject: Re: [meta-xilinx] [PATCH] machine-xilinx-default.inc: Default to 
>> >> u-boot
>> for
>> >> Zynq
>> >>
>> >> On 04/26/2017 03:06 PM, Manjukumar Harthikote Matha wrote:
>> >> >
>> >> >
>> >> >> -Original Message-----
>> >> >> From: Nathan Rossi [mailto:nat...@nathanrossi.com]
>> >> >> Sent: Wednesday, April 26, 2017 10:54 AM
>> >> >> To: Manjukumar Harthikote Matha <manju...@xilinx.com>
>> >> >> Cc: meta-xil...@lists.yoctoproject.org
>> >> >> Subject: Re: [meta-xilinx] [PATCH] machine-xilinx-default.inc:
>> >> >> Default to u-boot for Zynq
>> >> >>
>> >> >> On 27 April 2017 at 02:41, Manjukumar Harthikote Matha
>> >> >>  wrote:
>> >> >>>
>> >> >>>
>> >> >>>> -Original Message-
>> >> >>>> From: meta-xilinx-boun...@yoctoproject.org [mailto:meta-xilinx-
>> >> >>>> boun...@yoctoproject.org] On Behalf Of Nathan Rossi
>> >> >>>> Sent: Wednesday, April 26, 2017 4:57 AM
>> >> >>>> To: meta-xil...@lists.yoctoproject.org
>> >> >>>> Subject: [meta-xilinx] [PATCH] machine-xilinx-default.inc: Default
>> >> >>>> to u-boot for Zynq
>> >> >>>>
>> >> >>>> Upstream U-Boot provides an almost complete environment for the
>> >> >>>> majority of Zynq targets and specifically covers all the boot
>> >> >>>> functionality required for the boards in the meta-xilinx layer. As
>> >> >>>> such default to
>> >> >> the mainline version of U-Boot.
>> >> >>>>
>> >> >>>> For users that require or prefer to use u-boot-xlnx this can be
>> >> >>>> selected on a per- machine basis using:
>> >> >>>>
>> >> >>>>   PREFERRED_PROVIDER_virtual/bootloader = "u-boot-xlnx"
>> >> >>>>
>> >> >>>> Signed-off-by: Nathan Rossi <nat...@nathanrossi.com>
>> >> >>>> ---
>> >> >>>>  conf/machine/include/machine-xilinx-default.inc | 1 +
>> >> >>>>  1 file changed, 1 insertion(+)
>> >> >>>>
>> >> >>>> diff --git a/conf/machine/include/machine-xilinx-default.inc
>> >> >>>> b/conf/machine/include/machine-xilinx-default.inc
>> >> >>>> index 13e4df5746..f9e7e3a33f 100644
>> >> >>>> --- a/conf/machine/include/machine-xilinx-default.inc
>> >> >>>> +++ b/conf/machine/include/machine-xilinx-default.inc
>> >> >>>> @@ -19,6 +19,7 @@ PREFERRED_VERSION_linux-xlnx ?= "4.6-xilinx-
>> v2016.4%"
>> >> >>>>
>> >> >>>>  # U-Boot Configuration
>> >> >>>>  XILINX_DEFAULT_UBOOT := "u-boot-xlnx"
>> >> >>>> +XILINX_DEFAULT_UBOOT_zynq := "u-boot"
>> >> >>>>  XILINX_DEFAULT_UBOOT_zynqmp := "u-boot"
>> >> >>>
>> >> >>> Why have any preferred_provider? Distro can provide these settings.
>> >> >>> For ex: meta-petalinux can provide u-boot-xlnx, oe-core can provide
>> >> >>> u-boot
>> >> >>>
>&

Re: [meta-xilinx] [PATCH] machine-xilinx-default.inc: Default to u-boot for Zynq

[Manjukumar Harthikote Matha]

> -Original Message-
> From: Nathan Rossi [mailto:nat...@nathanrossi.com]
> Sent: Thursday, April 27, 2017 7:11 AM
> To: Manjukumar Harthikote Matha <manju...@xilinx.com>
> Cc: Philip Balister <phi...@balister.org>; meta-xil...@lists.yoctoproject.org
> Subject: Re: [meta-xilinx] [PATCH] machine-xilinx-default.inc: Default to 
> u-boot for
> Zynq
>
> On 27 April 2017 at 08:21, Manjukumar Harthikote Matha
> <manjukumar.harthikote-ma...@xilinx.com> wrote:
> >
> >
> >> -Original Message-
> >> From: Philip Balister [mailto:phi...@balister.org]
> >> Sent: Wednesday, April 26, 2017 12:38 PM
> >> To: Manjukumar Harthikote Matha <manju...@xilinx.com>; Nathan Rossi
> >> <nat...@nathanrossi.com>
> >> Cc: meta-xil...@lists.yoctoproject.org
> >> Subject: Re: [meta-xilinx] [PATCH] machine-xilinx-default.inc: Default to 
> >> u-boot
> for
> >> Zynq
> >>
> >> On 04/26/2017 03:06 PM, Manjukumar Harthikote Matha wrote:
> >> >
> >> >
> >> >> -Original Message-
> >> >> From: Nathan Rossi [mailto:nat...@nathanrossi.com]
> >> >> Sent: Wednesday, April 26, 2017 10:54 AM
> >> >> To: Manjukumar Harthikote Matha <manju...@xilinx.com>
> >> >> Cc: meta-xil...@lists.yoctoproject.org
> >> >> Subject: Re: [meta-xilinx] [PATCH] machine-xilinx-default.inc:
> >> >> Default to u-boot for Zynq
> >> >>
> >> >> On 27 April 2017 at 02:41, Manjukumar Harthikote Matha
> >> >>  wrote:
> >> >>>
> >> >>>
> >> >>>> -Original Message-
> >> >>>> From: meta-xilinx-boun...@yoctoproject.org [mailto:meta-xilinx-
> >> >>>> boun...@yoctoproject.org] On Behalf Of Nathan Rossi
> >> >>>> Sent: Wednesday, April 26, 2017 4:57 AM
> >> >>>> To: meta-xil...@lists.yoctoproject.org
> >> >>>> Subject: [meta-xilinx] [PATCH] machine-xilinx-default.inc: Default
> >> >>>> to u-boot for Zynq
> >> >>>>
> >> >>>> Upstream U-Boot provides an almost complete environment for the
> >> >>>> majority of Zynq targets and specifically covers all the boot
> >> >>>> functionality required for the boards in the meta-xilinx layer. As
> >> >>>> such default to
> >> >> the mainline version of U-Boot.
> >> >>>>
> >> >>>> For users that require or prefer to use u-boot-xlnx this can be
> >> >>>> selected on a per- machine basis using:
> >> >>>>
> >> >>>>   PREFERRED_PROVIDER_virtual/bootloader = "u-boot-xlnx"
> >> >>>>
> >> >>>> Signed-off-by: Nathan Rossi <nat...@nathanrossi.com>
> >> >>>> ---
> >> >>>>  conf/machine/include/machine-xilinx-default.inc | 1 +
> >> >>>>  1 file changed, 1 insertion(+)
> >> >>>>
> >> >>>> diff --git a/conf/machine/include/machine-xilinx-default.inc
> >> >>>> b/conf/machine/include/machine-xilinx-default.inc
> >> >>>> index 13e4df5746..f9e7e3a33f 100644
> >> >>>> --- a/conf/machine/include/machine-xilinx-default.inc
> >> >>>> +++ b/conf/machine/include/machine-xilinx-default.inc
> >> >>>> @@ -19,6 +19,7 @@ PREFERRED_VERSION_linux-xlnx ?= "4.6-xilinx-
> v2016.4%"
> >> >>>>
> >> >>>>  # U-Boot Configuration
> >> >>>>  XILINX_DEFAULT_UBOOT := "u-boot-xlnx"
> >> >>>> +XILINX_DEFAULT_UBOOT_zynq := "u-boot"
> >> >>>>  XILINX_DEFAULT_UBOOT_zynqmp := "u-boot"
> >> >>>
> >> >>> Why have any preferred_provider? Distro can provide these settings.
> >> >>> For ex: meta-petalinux can provide u-boot-xlnx, oe-core can provide
> >> >>> u-boot
> >> >>>
> >> >>> Any thoughts?
> >> >>
> >> >> Unfortunately not picking a provider for "virtual/bootloader" will
> >> >> result in bitbake attempting to build all providers (since
> >> >> EXTRA_IMAGEDEPENDS is depending on virtual/bootloader), as bitbake
> >> >> has no idea which one is desired (and they are all compatible).
> >> >>
> &g

Re: [meta-xilinx] [PATCH] machine-xilinx-default.inc: Default to u-boot for Zynq

[Nathan Rossi]

On 27 April 2017 at 08:21, Manjukumar Harthikote Matha
<manjukumar.harthikote-ma...@xilinx.com> wrote:
>
>
>> -Original Message-
>> From: Philip Balister [mailto:phi...@balister.org]
>> Sent: Wednesday, April 26, 2017 12:38 PM
>> To: Manjukumar Harthikote Matha <manju...@xilinx.com>; Nathan Rossi
>> <nat...@nathanrossi.com>
>> Cc: meta-xil...@lists.yoctoproject.org
>> Subject: Re: [meta-xilinx] [PATCH] machine-xilinx-default.inc: Default to 
>> u-boot for
>> Zynq
>>
>> On 04/26/2017 03:06 PM, Manjukumar Harthikote Matha wrote:
>> >
>> >
>> >> -Original Message-
>> >> From: Nathan Rossi [mailto:nat...@nathanrossi.com]
>> >> Sent: Wednesday, April 26, 2017 10:54 AM
>> >> To: Manjukumar Harthikote Matha <manju...@xilinx.com>
>> >> Cc: meta-xil...@lists.yoctoproject.org
>> >> Subject: Re: [meta-xilinx] [PATCH] machine-xilinx-default.inc:
>> >> Default to u-boot for Zynq
>> >>
>> >> On 27 April 2017 at 02:41, Manjukumar Harthikote Matha
>> >>  wrote:
>> >>>
>> >>>
>> >>>> -----Original Message-
>> >>>> From: meta-xilinx-boun...@yoctoproject.org [mailto:meta-xilinx-
>> >>>> boun...@yoctoproject.org] On Behalf Of Nathan Rossi
>> >>>> Sent: Wednesday, April 26, 2017 4:57 AM
>> >>>> To: meta-xil...@lists.yoctoproject.org
>> >>>> Subject: [meta-xilinx] [PATCH] machine-xilinx-default.inc: Default
>> >>>> to u-boot for Zynq
>> >>>>
>> >>>> Upstream U-Boot provides an almost complete environment for the
>> >>>> majority of Zynq targets and specifically covers all the boot
>> >>>> functionality required for the boards in the meta-xilinx layer. As
>> >>>> such default to
>> >> the mainline version of U-Boot.
>> >>>>
>> >>>> For users that require or prefer to use u-boot-xlnx this can be
>> >>>> selected on a per- machine basis using:
>> >>>>
>> >>>>   PREFERRED_PROVIDER_virtual/bootloader = "u-boot-xlnx"
>> >>>>
>> >>>> Signed-off-by: Nathan Rossi <nat...@nathanrossi.com>
>> >>>> ---
>> >>>>  conf/machine/include/machine-xilinx-default.inc | 1 +
>> >>>>  1 file changed, 1 insertion(+)
>> >>>>
>> >>>> diff --git a/conf/machine/include/machine-xilinx-default.inc
>> >>>> b/conf/machine/include/machine-xilinx-default.inc
>> >>>> index 13e4df5746..f9e7e3a33f 100644
>> >>>> --- a/conf/machine/include/machine-xilinx-default.inc
>> >>>> +++ b/conf/machine/include/machine-xilinx-default.inc
>> >>>> @@ -19,6 +19,7 @@ PREFERRED_VERSION_linux-xlnx ?= "4.6-xilinx-v2016.4%"
>> >>>>
>> >>>>  # U-Boot Configuration
>> >>>>  XILINX_DEFAULT_UBOOT := "u-boot-xlnx"
>> >>>> +XILINX_DEFAULT_UBOOT_zynq := "u-boot"
>> >>>>  XILINX_DEFAULT_UBOOT_zynqmp := "u-boot"
>> >>>
>> >>> Why have any preferred_provider? Distro can provide these settings.
>> >>> For ex: meta-petalinux can provide u-boot-xlnx, oe-core can provide
>> >>> u-boot
>> >>>
>> >>> Any thoughts?
>> >>
>> >> Unfortunately not picking a provider for "virtual/bootloader" will
>> >> result in bitbake attempting to build all providers (since
>> >> EXTRA_IMAGEDEPENDS is depending on virtual/bootloader), as bitbake
>> >> has no idea which one is desired (and they are all compatible).
>> >>
>> >> --
>> >> NOTE: multiple providers are available for virtual/bootloader
>> >> (u-boot-xlnx, u-boot- xlnx-dev, u-boot)
>> >> NOTE: consider defining a PREFERRED_PROVIDER entry to match
>> >> virtual/bootloader ...
>> >> ERROR: Multiple .bb files are due to be built which each provide
>> >> virtual/bootloader ...
>> >> 
>> >> --
>> >>
>> >> Also note, meta-petalinux or other layers should already be able to
>> >> override this default (by setting with ??=) either in a distro conf
>> >> or in a machine conf e.g. zynq- generic (for meta-petalinux).
>> >>
>> > If this is the case then why meta-xilinx sho

Re: [meta-xilinx] [PATCH] machine-xilinx-default.inc: Default to u-boot for Zynq

[Jean-Francois Dagenais]

> On Apr 26, 2017, at 15:37, Philip Balister  wrote:
> 
> I'd be happiest if meta-xilinx used upstream u-boot and a bbappend to
> add patches that are not upstream yet. This way we (the consumers) know
> exactly what the delta is to upstream.

Just for reference, you could also:

git clone -o origin upstream git://git.denx.de/u-boot.git
cd u-boot
git remote add xilinx https://github.com/Xilinx/u-boot-xlnx.git
git fetch --all
UPSTREAM_SHA1=$(grep "^SRCREV\ \=" ../whatever/u-boot_20xy.z.bb | sed 
"s/.*\"\(.*\)\".*/\1/g")
git co $UPSTREAM_SHA1
XLNX_SHA1=$(grep "^SRCREV\ \=" ../whatever/u-boot-xlnx_20xy.z.bb | sed 
"s/.*\"\(.*\)\".*/\1/g")
git merge --no-commit --no-ff $XLNX_SHA1
[... inspect ...]
git merge --abort

;) Sorry, not trying to be a smart ass, just documenting for posterity.
-- 
___
meta-xilinx mailing list
meta-xilinx@yoctoproject.org
https://lists.yoctoproject.org/listinfo/meta-xilinx

Re: [meta-xilinx] [PATCH] machine-xilinx-default.inc: Default to u-boot for Zynq

[Philip Balister]

On 04/26/2017 03:06 PM, Manjukumar Harthikote Matha wrote:
> 
> 
>> -Original Message-
>> From: Nathan Rossi [mailto:nat...@nathanrossi.com]
>> Sent: Wednesday, April 26, 2017 10:54 AM
>> To: Manjukumar Harthikote Matha <manju...@xilinx.com>
>> Cc: meta-xil...@lists.yoctoproject.org
>> Subject: Re: [meta-xilinx] [PATCH] machine-xilinx-default.inc: Default to 
>> u-boot for
>> Zynq
>>
>> On 27 April 2017 at 02:41, Manjukumar Harthikote Matha 
>> > ma...@xilinx.com> wrote:
>>>
>>>
>>>> -Original Message-
>>>> From: meta-xilinx-boun...@yoctoproject.org [mailto:meta-xilinx-
>>>> boun...@yoctoproject.org] On Behalf Of Nathan Rossi
>>>> Sent: Wednesday, April 26, 2017 4:57 AM
>>>> To: meta-xil...@lists.yoctoproject.org
>>>> Subject: [meta-xilinx] [PATCH] machine-xilinx-default.inc: Default to
>>>> u-boot for Zynq
>>>>
>>>> Upstream U-Boot provides an almost complete environment for the
>>>> majority of Zynq targets and specifically covers all the boot
>>>> functionality required for the boards in the meta-xilinx layer. As such 
>>>> default to
>> the mainline version of U-Boot.
>>>>
>>>> For users that require or prefer to use u-boot-xlnx this can be
>>>> selected on a per- machine basis using:
>>>>
>>>>   PREFERRED_PROVIDER_virtual/bootloader = "u-boot-xlnx"
>>>>
>>>> Signed-off-by: Nathan Rossi <nat...@nathanrossi.com>
>>>> ---
>>>>  conf/machine/include/machine-xilinx-default.inc | 1 +
>>>>  1 file changed, 1 insertion(+)
>>>>
>>>> diff --git a/conf/machine/include/machine-xilinx-default.inc
>>>> b/conf/machine/include/machine-xilinx-default.inc
>>>> index 13e4df5746..f9e7e3a33f 100644
>>>> --- a/conf/machine/include/machine-xilinx-default.inc
>>>> +++ b/conf/machine/include/machine-xilinx-default.inc
>>>> @@ -19,6 +19,7 @@ PREFERRED_VERSION_linux-xlnx ?= "4.6-xilinx-v2016.4%"
>>>>
>>>>  # U-Boot Configuration
>>>>  XILINX_DEFAULT_UBOOT := "u-boot-xlnx"
>>>> +XILINX_DEFAULT_UBOOT_zynq := "u-boot"
>>>>  XILINX_DEFAULT_UBOOT_zynqmp := "u-boot"
>>>
>>> Why have any preferred_provider? Distro can provide these settings.
>>> For ex: meta-petalinux can provide u-boot-xlnx, oe-core can provide
>>> u-boot
>>>
>>> Any thoughts?
>>
>> Unfortunately not picking a provider for "virtual/bootloader" will result in 
>> bitbake
>> attempting to build all providers (since EXTRA_IMAGEDEPENDS is depending on
>> virtual/bootloader), as bitbake has no idea which one is desired (and they 
>> are all
>> compatible).
>>
>> --
>> NOTE: multiple providers are available for virtual/bootloader (u-boot-xlnx, 
>> u-boot-
>> xlnx-dev, u-boot)
>> NOTE: consider defining a PREFERRED_PROVIDER entry to match 
>> virtual/bootloader
>> ...
>> ERROR: Multiple .bb files are due to be built which each provide 
>> virtual/bootloader
>> ...
>> 
>> --
>>
>> Also note, meta-petalinux or other layers should already be able to override 
>> this
>> default (by setting with ??=) either in a distro conf or in a machine conf 
>> e.g. zynq-
>> generic (for meta-petalinux).
>>
> If this is the case then why meta-xilinx should peg to upstream u-boot by 
> default? It should peg to u-boot-xlnx or linux-xlnx by default. We know that 
> there are patches/drivers which are not upstreamed yet, and only available in 
> Xilinx specific tree. This layer is specific to Xilinx updates and should 
> stick to defaults supported by Xilinx. Other distros or layer stack which use 
> meta-xilinx should override depending on their requirements not the other way 
> round.
> PREFERRED_PROVIDER_virtual/bootloader = "u-boot" should be specific to boards 
> other than the Xilinx eval boards, for example in zybo/microzed.

I'd be happiest if meta-xilinx used upstream u-boot and a bbappend to
add patches that are not upstream yet. This way we (the consumers) know
exactly what the delta is to upstream.

In this day and age, using non-mainline u-boot and kernel leads to
future pain. (Yes, I have some old pain from using linux-xlnx)

Philip


> 
> Thanks
> Manju
> 
> 
>> Regards,
>> Nathan
> 
> 
> This email and any attachments are intended for the sole use of the named 
> recipient(s) and contain(s) confidential information that may be proprietary, 
> privileged or copyrighted under applicable law. If you are not the intended 
> recipient, do not read, copy, or forward this email message or any 
> attachments. Delete this email message and any attachments immediately.
> 
-- 
___
meta-xilinx mailing list
meta-xilinx@yoctoproject.org
https://lists.yoctoproject.org/listinfo/meta-xilinx

Re: [meta-xilinx] [PATCH] machine-xilinx-default.inc: Default to u-boot for Zynq

[Manjukumar Harthikote Matha]

> -Original Message-
> From: Nathan Rossi [mailto:nat...@nathanrossi.com]
> Sent: Wednesday, April 26, 2017 10:54 AM
> To: Manjukumar Harthikote Matha <manju...@xilinx.com>
> Cc: meta-xil...@lists.yoctoproject.org
> Subject: Re: [meta-xilinx] [PATCH] machine-xilinx-default.inc: Default to 
> u-boot for
> Zynq
>
> On 27 April 2017 at 02:41, Manjukumar Harthikote Matha  ma...@xilinx.com> wrote:
> >
> >
> >> -Original Message-
> >> From: meta-xilinx-boun...@yoctoproject.org [mailto:meta-xilinx-
> >> boun...@yoctoproject.org] On Behalf Of Nathan Rossi
> >> Sent: Wednesday, April 26, 2017 4:57 AM
> >> To: meta-xil...@lists.yoctoproject.org
> >> Subject: [meta-xilinx] [PATCH] machine-xilinx-default.inc: Default to
> >> u-boot for Zynq
> >>
> >> Upstream U-Boot provides an almost complete environment for the
> >> majority of Zynq targets and specifically covers all the boot
> >> functionality required for the boards in the meta-xilinx layer. As such 
> >> default to
> the mainline version of U-Boot.
> >>
> >> For users that require or prefer to use u-boot-xlnx this can be
> >> selected on a per- machine basis using:
> >>
> >>   PREFERRED_PROVIDER_virtual/bootloader = "u-boot-xlnx"
> >>
> >> Signed-off-by: Nathan Rossi <nat...@nathanrossi.com>
> >> ---
> >>  conf/machine/include/machine-xilinx-default.inc | 1 +
> >>  1 file changed, 1 insertion(+)
> >>
> >> diff --git a/conf/machine/include/machine-xilinx-default.inc
> >> b/conf/machine/include/machine-xilinx-default.inc
> >> index 13e4df5746..f9e7e3a33f 100644
> >> --- a/conf/machine/include/machine-xilinx-default.inc
> >> +++ b/conf/machine/include/machine-xilinx-default.inc
> >> @@ -19,6 +19,7 @@ PREFERRED_VERSION_linux-xlnx ?= "4.6-xilinx-v2016.4%"
> >>
> >>  # U-Boot Configuration
> >>  XILINX_DEFAULT_UBOOT := "u-boot-xlnx"
> >> +XILINX_DEFAULT_UBOOT_zynq := "u-boot"
> >>  XILINX_DEFAULT_UBOOT_zynqmp := "u-boot"
> >
> > Why have any preferred_provider? Distro can provide these settings.
> > For ex: meta-petalinux can provide u-boot-xlnx, oe-core can provide
> > u-boot
> >
> > Any thoughts?
>
> Unfortunately not picking a provider for "virtual/bootloader" will result in 
> bitbake
> attempting to build all providers (since EXTRA_IMAGEDEPENDS is depending on
> virtual/bootloader), as bitbake has no idea which one is desired (and they 
> are all
> compatible).
>
> --
> NOTE: multiple providers are available for virtual/bootloader (u-boot-xlnx, 
> u-boot-
> xlnx-dev, u-boot)
> NOTE: consider defining a PREFERRED_PROVIDER entry to match virtual/bootloader
> ...
> ERROR: Multiple .bb files are due to be built which each provide 
> virtual/bootloader
> ...
> 
> --
>
> Also note, meta-petalinux or other layers should already be able to override 
> this
> default (by setting with ??=) either in a distro conf or in a machine conf 
> e.g. zynq-
> generic (for meta-petalinux).
>
If this is the case then why meta-xilinx should peg to upstream u-boot by 
default? It should peg to u-boot-xlnx or linux-xlnx by default. We know that 
there are patches/drivers which are not upstreamed yet, and only available in 
Xilinx specific tree. This layer is specific to Xilinx updates and should stick 
to defaults supported by Xilinx. Other distros or layer stack which use 
meta-xilinx should override depending on their requirements not the other way 
round.
PREFERRED_PROVIDER_virtual/bootloader = "u-boot" should be specific to boards 
other than the Xilinx eval boards, for example in zybo/microzed.

Thanks
Manju


> Regards,
> Nathan


This email and any attachments are intended for the sole use of the named 
recipient(s) and contain(s) confidential information that may be proprietary, 
privileged or copyrighted under applicable law. If you are not the intended 
recipient, do not read, copy, or forward this email message or any attachments. 
Delete this email message and any attachments immediately.

-- 
___
meta-xilinx mailing list
meta-xilinx@yoctoproject.org
https://lists.yoctoproject.org/listinfo/meta-xilinx

Re: [meta-xilinx] [PATCH] machine-xilinx-default.inc: Default to u-boot for Zynq

[Manjukumar Harthikote Matha]

> -Original Message-
> From: meta-xilinx-boun...@yoctoproject.org [mailto:meta-xilinx-
> boun...@yoctoproject.org] On Behalf Of Nathan Rossi
> Sent: Wednesday, April 26, 2017 4:57 AM
> To: meta-xil...@lists.yoctoproject.org
> Subject: [meta-xilinx] [PATCH] machine-xilinx-default.inc: Default to u-boot 
> for Zynq
>
> Upstream U-Boot provides an almost complete environment for the majority of 
> Zynq
> targets and specifically covers all the boot functionality required for the 
> boards in
> the meta-xilinx layer. As such default to the mainline version of U-Boot.
>
> For users that require or prefer to use u-boot-xlnx this can be selected on a 
> per-
> machine basis using:
>
>   PREFERRED_PROVIDER_virtual/bootloader = "u-boot-xlnx"
>
> Signed-off-by: Nathan Rossi <nat...@nathanrossi.com>
> ---
>  conf/machine/include/machine-xilinx-default.inc | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/conf/machine/include/machine-xilinx-default.inc
> b/conf/machine/include/machine-xilinx-default.inc
> index 13e4df5746..f9e7e3a33f 100644
> --- a/conf/machine/include/machine-xilinx-default.inc
> +++ b/conf/machine/include/machine-xilinx-default.inc
> @@ -19,6 +19,7 @@ PREFERRED_VERSION_linux-xlnx ?= "4.6-xilinx-v2016.4%"
>
>  # U-Boot Configuration
>  XILINX_DEFAULT_UBOOT := "u-boot-xlnx"
> +XILINX_DEFAULT_UBOOT_zynq := "u-boot"
>  XILINX_DEFAULT_UBOOT_zynqmp := "u-boot"

Why have any preferred_provider? Distro can provide these settings.
For ex: meta-petalinux can provide u-boot-xlnx, oe-core can provide u-boot

Any thoughts?

Thanks
Manju

>  PREFERRED_PROVIDER_virtual/bootloader ??= "${XILINX_DEFAULT_UBOOT}"
>
> --
> 2.11.0
>
> --
> ___
> meta-xilinx mailing list
> meta-xilinx@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-xilinx


This email and any attachments are intended for the sole use of the named 
recipient(s) and contain(s) confidential information that may be proprietary, 
privileged or copyrighted under applicable law. If you are not the intended 
recipient, do not read, copy, or forward this email message or any attachments. 
Delete this email message and any attachments immediately.

-- 
___
meta-xilinx mailing list
meta-xilinx@yoctoproject.org
https://lists.yoctoproject.org/listinfo/meta-xilinx

Re: [meta-xilinx] [PATCH] machine-xilinx-default.inc: Default to u-boot for Zynq

[Nathan Rossi]

On 27 April 2017 at 02:41, Manjukumar Harthikote Matha
<manjukumar.harthikote-ma...@xilinx.com> wrote:
>
>
>> -Original Message-
>> From: meta-xilinx-boun...@yoctoproject.org [mailto:meta-xilinx-
>> boun...@yoctoproject.org] On Behalf Of Nathan Rossi
>> Sent: Wednesday, April 26, 2017 4:57 AM
>> To: meta-xil...@lists.yoctoproject.org
>> Subject: [meta-xilinx] [PATCH] machine-xilinx-default.inc: Default to u-boot 
>> for Zynq
>>
>> Upstream U-Boot provides an almost complete environment for the majority of 
>> Zynq
>> targets and specifically covers all the boot functionality required for the 
>> boards in
>> the meta-xilinx layer. As such default to the mainline version of U-Boot.
>>
>> For users that require or prefer to use u-boot-xlnx this can be selected on 
>> a per-
>> machine basis using:
>>
>>   PREFERRED_PROVIDER_virtual/bootloader = "u-boot-xlnx"
>>
>> Signed-off-by: Nathan Rossi <nat...@nathanrossi.com>
>> ---
>>  conf/machine/include/machine-xilinx-default.inc | 1 +
>>  1 file changed, 1 insertion(+)
>>
>> diff --git a/conf/machine/include/machine-xilinx-default.inc
>> b/conf/machine/include/machine-xilinx-default.inc
>> index 13e4df5746..f9e7e3a33f 100644
>> --- a/conf/machine/include/machine-xilinx-default.inc
>> +++ b/conf/machine/include/machine-xilinx-default.inc
>> @@ -19,6 +19,7 @@ PREFERRED_VERSION_linux-xlnx ?= "4.6-xilinx-v2016.4%"
>>
>>  # U-Boot Configuration
>>  XILINX_DEFAULT_UBOOT := "u-boot-xlnx"
>> +XILINX_DEFAULT_UBOOT_zynq := "u-boot"
>>  XILINX_DEFAULT_UBOOT_zynqmp := "u-boot"
>
> Why have any preferred_provider? Distro can provide these settings.
> For ex: meta-petalinux can provide u-boot-xlnx, oe-core can provide u-boot
>
> Any thoughts?

Unfortunately not picking a provider for "virtual/bootloader" will
result in bitbake attempting to build all providers (since
EXTRA_IMAGEDEPENDS is depending on virtual/bootloader), as bitbake has
no idea which one is desired (and they are all compatible).

--
NOTE: multiple providers are available for virtual/bootloader
(u-boot-xlnx, u-boot-xlnx-dev, u-boot)
NOTE: consider defining a PREFERRED_PROVIDER entry to match virtual/bootloader
...
ERROR: Multiple .bb files are due to be built which each provide
virtual/bootloader
...

--

Also note, meta-petalinux or other layers should already be able to
override this default (by setting with ??=) either in a distro conf or
in a machine conf e.g. zynq-generic (for meta-petalinux).

Regards,
Nathan
-- 
___
meta-xilinx mailing list
meta-xilinx@yoctoproject.org
https://lists.yoctoproject.org/listinfo/meta-xilinx

[meta-xilinx] [PATCH] machine-xilinx-default.inc: Default to u-boot for Zynq

[Nathan Rossi]

Upstream U-Boot provides an almost complete environment for the majority
of Zynq targets and specifically covers all the boot functionality
required for the boards in the meta-xilinx layer. As such default to the
mainline version of U-Boot.

For users that require or prefer to use u-boot-xlnx this can be selected
on a per-machine basis using:

  PREFERRED_PROVIDER_virtual/bootloader = "u-boot-xlnx"

Signed-off-by: Nathan Rossi 
---
 conf/machine/include/machine-xilinx-default.inc | 1 +
 1 file changed, 1 insertion(+)

diff --git a/conf/machine/include/machine-xilinx-default.inc 
b/conf/machine/include/machine-xilinx-default.inc
index 13e4df5746..f9e7e3a33f 100644
--- a/conf/machine/include/machine-xilinx-default.inc
+++ b/conf/machine/include/machine-xilinx-default.inc
@@ -19,6 +19,7 @@ PREFERRED_VERSION_linux-xlnx ?= "4.6-xilinx-v2016.4%"
 
 # U-Boot Configuration
 XILINX_DEFAULT_UBOOT := "u-boot-xlnx"
+XILINX_DEFAULT_UBOOT_zynq := "u-boot"
 XILINX_DEFAULT_UBOOT_zynqmp := "u-boot"
 PREFERRED_PROVIDER_virtual/bootloader ??= "${XILINX_DEFAULT_UBOOT}"
 
-- 
2.11.0

-- 
___
meta-xilinx mailing list
meta-xilinx@yoctoproject.org
https://lists.yoctoproject.org/listinfo/meta-xilinx