[meta-rockchip] [PATCH 0/3] fixup u-boot compiling error

[meta-rockchip] [PATCH 0/3] fixup u-boot compiling error

[Randy 'ayaka' Li]

It would be better that Rockchip adds a new tag for the recently
u-boot release. Anyway, I just sent my recipes for building the
latest u-boot. You may not merge the last patch.

Randy 'ayaka' Li (2):
  recipes-bsp/u-boot: fixup build for gcc7
  recipes-bsp/u-boot: update build rules

Randy Li (1):
  recipes-bsp/u-boot: update to the latest version

 recipes-bsp/u-boot/u-boot-rockchip.inc        |  1 +
 .../u-boot/u-boot-rockchip/gcc7_fixup.patch   | 38 +++++++++++++++++++
 .../u-boot/u-boot-rockchip_20171218.bb        |  3 ++
 recipes-bsp/u-boot/u-boot-rockchip_git.bb     | 18 +++++++++
 4 files changed, 60 insertions(+)
 create mode 100644 recipes-bsp/u-boot/u-boot-rockchip/gcc7_fixup.patch
 create mode 100644 recipes-bsp/u-boot/u-boot-rockchip_git.bb

-- 
2.20.1

[PATCH 1/3] recipes-bsp/u-boot: fixup build for gcc7

[Randy 'ayaka' Li]

Signed-off-by: Randy Li <randy.li@rock-chips.com>
Signed-off-by: Randy 'ayaka' Li <ayaka@soulik.info>
---
 .../u-boot/u-boot-rockchip/gcc7_fixup.patch   | 38 +++++++++++++++++++
 1 file changed, 38 insertions(+)
 create mode 100644 recipes-bsp/u-boot/u-boot-rockchip/gcc7_fixup.patch

diff --git a/recipes-bsp/u-boot/u-boot-rockchip/gcc7_fixup.patch b/recipes-bsp/u-boot/u-boot-rockchip/gcc7_fixup.patch
new file mode 100644
index 0000000..ccb709a
--- /dev/null
+++ b/recipes-bsp/u-boot/u-boot-rockchip/gcc7_fixup.patch
@@ -0,0 +1,38 @@
+diff --git a/include/linux/log2.h b/include/linux/log2.h
+index aa1de63090..5526af036d 100644
+--- a/include/linux/log2.h
++++ b/include/linux/log2.h
+@@ -12,12 +12,6 @@
+ #include <linux/types.h>
+ #include <linux/bitops.h>
+ 
+-/*
+- * deal with unrepresentable constant logarithms
+- */
+-extern __attribute__((const, noreturn))
+-int ____ilog2_NaN(void);
+-
+ /*
+  * non-constant log of base 2 calculators
+  * - the arch may override these in asm/bitops.h if they can be implemented
+@@ -82,7 +76,7 @@ unsigned long __rounddown_pow_of_two(unsigned long n)
+ #define ilog2(n)				\
+ (						\
+ 	__builtin_constant_p(n) ? (		\
+-		(n) < 1 ? ____ilog2_NaN() :	\
++		(n) < 2 ? 0 :			\
+ 		(n) & (1ULL << 63) ? 63 :	\
+ 		(n) & (1ULL << 62) ? 62 :	\
+ 		(n) & (1ULL << 61) ? 61 :	\
+@@ -145,10 +139,7 @@ unsigned long __rounddown_pow_of_two(unsigned long n)
+ 		(n) & (1ULL <<  4) ?  4 :	\
+ 		(n) & (1ULL <<  3) ?  3 :	\
+ 		(n) & (1ULL <<  2) ?  2 :	\
+-		(n) & (1ULL <<  1) ?  1 :	\
+-		(n) & (1ULL <<  0) ?  0 :	\
+-		____ilog2_NaN()			\
+-				   ) :		\
++		1 ) :				\
+ 	(sizeof(n) <= 4) ?			\
+ 	__ilog2_u32(n) :			\
+ 	__ilog2_u64(n)				\
-- 
2.20.1

[PATCH 2/3] recipes-bsp/u-boot: update build rules

[Randy 'ayaka' Li]

Adding the missing recipes header file.
Fixing the compiler error after oe-core update.

Signed-off-by: Randy 'ayaka' Li <ayaka@soulik.info>
---
 recipes-bsp/u-boot/u-boot-rockchip.inc         | 1 +
 recipes-bsp/u-boot/u-boot-rockchip_20171218.bb | 3 +++
 2 files changed, 4 insertions(+)

diff --git a/recipes-bsp/u-boot/u-boot-rockchip.inc b/recipes-bsp/u-boot/u-boot-rockchip.inc
index 5d89cd6..b495dbe 100644
--- a/recipes-bsp/u-boot/u-boot-rockchip.inc
+++ b/recipes-bsp/u-boot/u-boot-rockchip.inc
@@ -3,6 +3,7 @@
 # Released under the MIT license (see COPYING.MIT for the terms)
 
 require recipes-bsp/u-boot/u-boot.inc
+require recipes-bsp/u-boot/u-boot-common.inc
 
 DESCRIPTION = "Rockchip next-dev U-Boot"
 LICENSE = "GPLv2+"
diff --git a/recipes-bsp/u-boot/u-boot-rockchip_20171218.bb b/recipes-bsp/u-boot/u-boot-rockchip_20171218.bb
index 4545f13..e14d948 100644
--- a/recipes-bsp/u-boot/u-boot-rockchip_20171218.bb
+++ b/recipes-bsp/u-boot/u-boot-rockchip_20171218.bb
@@ -9,5 +9,8 @@ TAG = "release-${PV}"
 SRC_URI = " \
 	git://github.com/rockchip-linux/u-boot.git;tag=${TAG};nobranch=1; \
 	file://binutils-2.28-ld-fix.patch \
+	file://gcc7_fixup.patch \
 "
 S = "${WORKDIR}/git"
+
+SRCREV = "release-20171218"
-- 
2.20.1

[PATCH 3/3] recipes-bsp/u-boot: update to the latest version

[Randy 'ayaka' Li]

From: Randy Li <ayaka@soulik.info>

Signed-off-by: Randy Li <ayaka@soulik.info>
---
 recipes-bsp/u-boot/u-boot-rockchip_git.bb | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)
 create mode 100644 recipes-bsp/u-boot/u-boot-rockchip_git.bb

diff --git a/recipes-bsp/u-boot/u-boot-rockchip_git.bb b/recipes-bsp/u-boot/u-boot-rockchip_git.bb
new file mode 100644
index 0000000..aa4b84d
--- /dev/null
+++ b/recipes-bsp/u-boot/u-boot-rockchip_git.bb
@@ -0,0 +1,18 @@
+# Copyright (C) 2019 SUMOMO Computer Assocation
+# Released under the MIT license (see COPYING.MIT for the terms)
+FILESEXTRAPATHS_prepend := "${THISDIR}/${PN}:"
+
+include u-boot-rockchip.inc
+
+DESCRIPTION = "Rockchip U-Boot"
+
+LIC_FILES_CHKSUM = "file://Licenses/README;md5=a2c678cfd4a4d97135585cad908541c6"
+
+SRC_URI = " \
+	git://github.com/rockchip-linux/u-boot.git;branch=release; \
+	file://binutils-2.28-ld-fix.patch \
+	file://gcc7_fixup.patch \
+	"
+
+S = "${WORKDIR}/git"
+PV = "v2019.01"
-- 
2.20.1

Re: [PATCH 1/3] recipes-bsp/u-boot: fixup build for gcc7

[Trevor Woerner]

On Wed 2019-04-24 @ 10:59:16 PM, Randy 'ayaka' Li wrote:
> Signed-off-by: Randy Li <randy.li@rock-chips.com>
> Signed-off-by: Randy 'ayaka' Li <ayaka@soulik.info>
> ---
>  .../u-boot/u-boot-rockchip/gcc7_fixup.patch   | 38 +++++++++++++++++++

Why do we care about gcc7 on master? Master only has support for gcc8.

Re: [PATCH 3/3] recipes-bsp/u-boot: update to the latest version

[Trevor Woerner]

On Wed 2019-04-24 @ 10:59:18 PM, Randy 'ayaka' Li wrote:
> From: Randy Li <ayaka@soulik.info>
> 
> Signed-off-by: Randy Li <ayaka@soulik.info>
> ---
>  recipes-bsp/u-boot/u-boot-rockchip_git.bb | 18 ++++++++++++++++++
>  1 file changed, 18 insertions(+)
>  create mode 100644 recipes-bsp/u-boot/u-boot-rockchip_git.bb
> 
> diff --git a/recipes-bsp/u-boot/u-boot-rockchip_git.bb b/recipes-bsp/u-boot/u-boot-rockchip_git.bb
> new file mode 100644
> index 0000000..aa4b84d
> --- /dev/null
> +++ b/recipes-bsp/u-boot/u-boot-rockchip_git.bb
> @@ -0,0 +1,18 @@
> +# Copyright (C) 2019 SUMOMO Computer Assocation
> +# Released under the MIT license (see COPYING.MIT for the terms)
> +FILESEXTRAPATHS_prepend := "${THISDIR}/${PN}:"
> +
> +include u-boot-rockchip.inc
> +
> +DESCRIPTION = "Rockchip U-Boot"
> +
> +LIC_FILES_CHKSUM = "file://Licenses/README;md5=a2c678cfd4a4d97135585cad908541c6"
> +
> +SRC_URI = " \
> +	git://github.com/rockchip-linux/u-boot.git;branch=release; \
> +	file://binutils-2.28-ld-fix.patch \
> +	file://gcc7_fixup.patch \
> +	"
> +
> +S = "${WORKDIR}/git"
> +PV = "v2019.01"

This doesn't even get past the fetching stage:

	WARNING: u-boot-rockchip-1_v2019.01-r0 do_fetch: Failed to fetch URL git://github.com/rockchip-linux/u-boot.git;branch=release;, attempting MIRRORS if available
	ERROR: u-boot-rockchip-1_v2019.01-r0 do_fetch: Fetcher failure: Unable to find revision 3c99166441bf3ea325af2da83cfe65430b49c066 in branch release even from upstream
	ERROR: u-boot-rockchip-1_v2019.01-r0 do_fetch: Fetcher failure for URL: 'git://github.com/rockchip-linux/u-boot.git;branch=release;'. Unable to fetch URL from any source.
	ERROR: u-boot-rockchip-1_v2019.01-r0 do_fetch:
	ERROR: u-boot-rockchip-1_v2019.01-r0 do_fetch: Function failed: base_do_fetch
	ERROR: Task (/opt/oe/configs/z/build-master/meta-rockchip/layers/meta-rockchip/recipes-bsp/u-boot/u-boot-rockchip_git.bb:do_fetch) failed with exit code '1'

Re: [PATCH 3/3] recipes-bsp/u-boot: update to the latest version

[ayaka]

On 5/4/19 11:06 AM, Trevor Woerner wrote:
> On Wed 2019-04-24 @ 10:59:18 PM, Randy 'ayaka' Li wrote:
>> From: Randy Li <ayaka@soulik.info>
>>
>> Signed-off-by: Randy Li <ayaka@soulik.info>
>> ---
>>   recipes-bsp/u-boot/u-boot-rockchip_git.bb | 18 ++++++++++++++++++
>>   1 file changed, 18 insertions(+)
>>   create mode 100644 recipes-bsp/u-boot/u-boot-rockchip_git.bb
>>
>> diff --git a/recipes-bsp/u-boot/u-boot-rockchip_git.bb b/recipes-bsp/u-boot/u-boot-rockchip_git.bb
>> new file mode 100644
>> index 0000000..aa4b84d
>> --- /dev/null
>> +++ b/recipes-bsp/u-boot/u-boot-rockchip_git.bb
>> @@ -0,0 +1,18 @@
>> +# Copyright (C) 2019 SUMOMO Computer Assocation
>> +# Released under the MIT license (see COPYING.MIT for the terms)
>> +FILESEXTRAPATHS_prepend := "${THISDIR}/${PN}:"
>> +
>> +include u-boot-rockchip.inc
>> +
>> +DESCRIPTION = "Rockchip U-Boot"
>> +
>> +LIC_FILES_CHKSUM = "file://Licenses/README;md5=a2c678cfd4a4d97135585cad908541c6"
>> +
>> +SRC_URI = " \
>> +	git://github.com/rockchip-linux/u-boot.git;branch=release; \
>> +	file://binutils-2.28-ld-fix.patch \
>> +	file://gcc7_fixup.patch \
>> +	"
>> +
>> +S = "${WORKDIR}/git"
>> +PV = "v2019.01"
> This doesn't even get past the fetching stage:
I am sorry, I forget that ssh git protocol only work for those users who 
uploaded their key to the github, I would change it back to https.
>
> 	WARNING: u-boot-rockchip-1_v2019.01-r0 do_fetch: Failed to fetch URL git://github.com/rockchip-linux/u-boot.git;branch=release;, attempting MIRRORS if available
> 	ERROR: u-boot-rockchip-1_v2019.01-r0 do_fetch: Fetcher failure: Unable to find revision 3c99166441bf3ea325af2da83cfe65430b49c066 in branch release even from upstream
> 	ERROR: u-boot-rockchip-1_v2019.01-r0 do_fetch: Fetcher failure for URL: 'git://github.com/rockchip-linux/u-boot.git;branch=release;'. Unable to fetch URL from any source.
> 	ERROR: u-boot-rockchip-1_v2019.01-r0 do_fetch:
> 	ERROR: u-boot-rockchip-1_v2019.01-r0 do_fetch: Function failed: base_do_fetch
> 	ERROR: Task (/opt/oe/configs/z/build-master/meta-rockchip/layers/meta-rockchip/recipes-bsp/u-boot/u-boot-rockchip_git.bb:do_fetch) failed with exit code '1'

Re: [PATCH 1/3] recipes-bsp/u-boot: fixup build for gcc7

[ayaka]

On 5/4/19 11:02 AM, Trevor Woerner wrote:
> On Wed 2019-04-24 @ 10:59:16 PM, Randy 'ayaka' Li wrote:
>> Signed-off-by: Randy Li <randy.li@rock-chips.com>
>> Signed-off-by: Randy 'ayaka' Li <ayaka@soulik.info>
>> ---
>>   .../u-boot/u-boot-rockchip/gcc7_fixup.patch   | 38 +++++++++++++++++++
> Why do we care about gcc7 on master? Master only has support for gcc8.

Anyway, the u-boot won't work after gcc6. At that time I make this this, 
yocto still use the gcc7.

If you need, I can change this name.

Re: [PATCH 3/3] recipes-bsp/u-boot: update to the latest version

[Trevor Woerner]

On Sat 2019-05-04 @ 06:43:34 PM, ayaka wrote:
> 
> On 5/4/19 11:06 AM, Trevor Woerner wrote:
> > On Wed 2019-04-24 @ 10:59:18 PM, Randy 'ayaka' Li wrote:
> > > From: Randy Li <ayaka@soulik.info>
> > > 
> > > Signed-off-by: Randy Li <ayaka@soulik.info>
> > > ---
> > >   recipes-bsp/u-boot/u-boot-rockchip_git.bb | 18 ++++++++++++++++++
> > >   1 file changed, 18 insertions(+)
> > >   create mode 100644 recipes-bsp/u-boot/u-boot-rockchip_git.bb
> > > 
> > > diff --git a/recipes-bsp/u-boot/u-boot-rockchip_git.bb b/recipes-bsp/u-boot/u-boot-rockchip_git.bb
> > > new file mode 100644
> > > index 0000000..aa4b84d
> > > --- /dev/null
> > > +++ b/recipes-bsp/u-boot/u-boot-rockchip_git.bb
> > > @@ -0,0 +1,18 @@
> > > +# Copyright (C) 2019 SUMOMO Computer Assocation
> > > +# Released under the MIT license (see COPYING.MIT for the terms)
> > > +FILESEXTRAPATHS_prepend := "${THISDIR}/${PN}:"
> > > +
> > > +include u-boot-rockchip.inc
> > > +
> > > +DESCRIPTION = "Rockchip U-Boot"
> > > +
> > > +LIC_FILES_CHKSUM = "file://Licenses/README;md5=a2c678cfd4a4d97135585cad908541c6"
> > > +
> > > +SRC_URI = " \
> > > +	git://github.com/rockchip-linux/u-boot.git;branch=release; \
> > > +	file://binutils-2.28-ld-fix.patch \
> > > +	file://gcc7_fixup.patch \
> > > +	"
> > > +
> > > +S = "${WORKDIR}/git"
> > > +PV = "v2019.01"
> > This doesn't even get past the fetching stage:
> I am sorry, I forget that ssh git protocol only work for those users who
> uploaded their key to the github, I would change it back to https.
> > 
> > 	WARNING: u-boot-rockchip-1_v2019.01-r0 do_fetch: Failed to fetch URL git://github.com/rockchip-linux/u-boot.git;branch=release;, attempting MIRRORS if available
> > 	ERROR: u-boot-rockchip-1_v2019.01-r0 do_fetch: Fetcher failure: Unable to find revision 3c99166441bf3ea325af2da83cfe65430b49c066 in branch release even from upstream
> > 	ERROR: u-boot-rockchip-1_v2019.01-r0 do_fetch: Fetcher failure for URL: 'git://github.com/rockchip-linux/u-boot.git;branch=release;'. Unable to fetch URL from any source.
> > 	ERROR: u-boot-rockchip-1_v2019.01-r0 do_fetch:
> > 	ERROR: u-boot-rockchip-1_v2019.01-r0 do_fetch: Function failed: base_do_fetch
> > 	ERROR: Task (/opt/oe/configs/z/build-master/meta-rockchip/layers/meta-rockchip/recipes-bsp/u-boot/u-boot-rockchip_git.bb:do_fetch) failed with exit code '1'

The problem is that it's trying to fetch commit 3c9916 which doesn't exist on
github.com/rockchip-linux/u-boot.git. That revision is on upstream U-Boot.

Re: [PATCH 3/3] recipes-bsp/u-boot: update to the latest version

[ayaka]

On 5/6/19 9:31 PM, Trevor Woerner wrote:
> On Sat 2019-05-04 @ 06:43:34 PM, ayaka wrote:
>> On 5/4/19 11:06 AM, Trevor Woerner wrote:
>>> On Wed 2019-04-24 @ 10:59:18 PM, Randy 'ayaka' Li wrote:
>>>> From: Randy Li <ayaka@soulik.info>
>>>>
>>>> Signed-off-by: Randy Li <ayaka@soulik.info>
>>>> ---
>>>>    recipes-bsp/u-boot/u-boot-rockchip_git.bb | 18 ++++++++++++++++++
>>>>    1 file changed, 18 insertions(+)
>>>>    create mode 100644 recipes-bsp/u-boot/u-boot-rockchip_git.bb
>>>>
>>>> diff --git a/recipes-bsp/u-boot/u-boot-rockchip_git.bb b/recipes-bsp/u-boot/u-boot-rockchip_git.bb
>>>> new file mode 100644
>>>> index 0000000..aa4b84d
>>>> --- /dev/null
>>>> +++ b/recipes-bsp/u-boot/u-boot-rockchip_git.bb
>>>> @@ -0,0 +1,18 @@
>>>> +# Copyright (C) 2019 SUMOMO Computer Assocation
>>>> +# Released under the MIT license (see COPYING.MIT for the terms)
>>>> +FILESEXTRAPATHS_prepend := "${THISDIR}/${PN}:"
>>>> +
>>>> +include u-boot-rockchip.inc
>>>> +
>>>> +DESCRIPTION = "Rockchip U-Boot"
>>>> +
>>>> +LIC_FILES_CHKSUM = "file://Licenses/README;md5=a2c678cfd4a4d97135585cad908541c6"
>>>> +
>>>> +SRC_URI = " \
>>>> +	git://github.com/rockchip-linux/u-boot.git;branch=release; \
>>>> +	file://binutils-2.28-ld-fix.patch \
>>>> +	file://gcc7_fixup.patch \
>>>> +	"
>>>> +
>>>> +S = "${WORKDIR}/git"
>>>> +PV = "v2019.01"
>>> This doesn't even get past the fetching stage:
>> I am sorry, I forget that ssh git protocol only work for those users who
>> uploaded their key to the github, I would change it back to https.
>>> 	WARNING: u-boot-rockchip-1_v2019.01-r0 do_fetch: Failed to fetch URL git://github.com/rockchip-linux/u-boot.git;branch=release;, attempting MIRRORS if available
>>> 	ERROR: u-boot-rockchip-1_v2019.01-r0 do_fetch: Fetcher failure: Unable to find revision 3c99166441bf3ea325af2da83cfe65430b49c066 in branch release even from upstream
>>> 	ERROR: u-boot-rockchip-1_v2019.01-r0 do_fetch: Fetcher failure for URL: 'git://github.com/rockchip-linux/u-boot.git;branch=release;'. Unable to fetch URL from any source.
>>> 	ERROR: u-boot-rockchip-1_v2019.01-r0 do_fetch:
>>> 	ERROR: u-boot-rockchip-1_v2019.01-r0 do_fetch: Function failed: base_do_fetch
>>> 	ERROR: Task (/opt/oe/configs/z/build-master/meta-rockchip/layers/meta-rockchip/recipes-bsp/u-boot/u-boot-rockchip_git.bb:do_fetch) failed with exit code '1'
> The problem is that it's trying to fetch commit 3c9916 which doesn't exist on
> github.com/rockchip-linux/u-boot.git. That revision is on upstream U-Boot.
How comes? I didn't write any commit id there, it is based on the branch 
at all.

Re: [PATCH 3/3] recipes-bsp/u-boot: update to the latest version

[Trevor Woerner]

[-- Attachment #1: Type: text/plain, Size: 3582 bytes --]

a previous patch of yours did a:
    require recipes-bsp/u-boot/u-boot-common.inc
which defines this SRCREV

my subtle point being: I don't get the impression you're testing your
patches, or at least not testing them correctly.

the fact you're CCing Romain suggests your either using a very old branch
of https://git.yoctoproject.org/cgit/cgit.cgi/meta-rockchip/ or that you're
submitting patches to me for https://github.com/rockchip-linux/meta-rockchip
?
Unfortunately for me, when https://github.com/rockchip-linux/meta-rockchip
did a clone of https://git.yoctoproject.org/cgit/cgit.cgi/meta-rockchip/,
they left my and Romain's names as maintainers in the README (along with
instructions for github)

On Wed, May 8, 2019 at 12:19 PM ayaka <ayaka@soulik.info> wrote:

>
> On 5/6/19 9:31 PM, Trevor Woerner wrote:
> > On Sat 2019-05-04 @ 06:43:34 PM, ayaka wrote:
> >> On 5/4/19 11:06 AM, Trevor Woerner wrote:
> >>> On Wed 2019-04-24 @ 10:59:18 PM, Randy 'ayaka' Li wrote:
> >>>> From: Randy Li <ayaka@soulik.info>
> >>>>
> >>>> Signed-off-by: Randy Li <ayaka@soulik.info>
> >>>> ---
> >>>>    recipes-bsp/u-boot/u-boot-rockchip_git.bb | 18 ++++++++++++++++++
> >>>>    1 file changed, 18 insertions(+)
> >>>>    create mode 100644 recipes-bsp/u-boot/u-boot-rockchip_git.bb
> >>>>
> >>>> diff --git a/recipes-bsp/u-boot/u-boot-rockchip_git.bb
> b/recipes-bsp/u-boot/u-boot-rockchip_git.bb
> >>>> new file mode 100644
> >>>> index 0000000..aa4b84d
> >>>> --- /dev/null
> >>>> +++ b/recipes-bsp/u-boot/u-boot-rockchip_git.bb
> >>>> @@ -0,0 +1,18 @@
> >>>> +# Copyright (C) 2019 SUMOMO Computer Assocation
> >>>> +# Released under the MIT license (see COPYING.MIT for the terms)
> >>>> +FILESEXTRAPATHS_prepend := "${THISDIR}/${PN}:"
> >>>> +
> >>>> +include u-boot-rockchip.inc
> >>>> +
> >>>> +DESCRIPTION = "Rockchip U-Boot"
> >>>> +
> >>>> +LIC_FILES_CHKSUM =
> "file://Licenses/README;md5=a2c678cfd4a4d97135585cad908541c6"
> >>>> +
> >>>> +SRC_URI = " \
> >>>> +  git://github.com/rockchip-linux/u-boot.git;branch=release; \
> >>>> +  file://binutils-2.28-ld-fix.patch \
> >>>> +  file://gcc7_fixup.patch \
> >>>> +  "
> >>>> +
> >>>> +S = "${WORKDIR}/git"
> >>>> +PV = "v2019.01"
> >>> This doesn't even get past the fetching stage:
> >> I am sorry, I forget that ssh git protocol only work for those users who
> >> uploaded their key to the github, I would change it back to https.
> >>>     WARNING: u-boot-rockchip-1_v2019.01-r0 do_fetch: Failed to fetch
> URL git://github.com/rockchip-linux/u-boot.git;branch=release;,
> attempting MIRRORS if available
> >>>     ERROR: u-boot-rockchip-1_v2019.01-r0 do_fetch: Fetcher failure:
> Unable to find revision 3c99166441bf3ea325af2da83cfe65430b49c066 in branch
> release even from upstream
> >>>     ERROR: u-boot-rockchip-1_v2019.01-r0 do_fetch: Fetcher failure for
> URL: 'git://github.com/rockchip-linux/u-boot.git;branch=release;'. Unable
> to fetch URL from any source.
> >>>     ERROR: u-boot-rockchip-1_v2019.01-r0 do_fetch:
> >>>     ERROR: u-boot-rockchip-1_v2019.01-r0 do_fetch: Function failed:
> base_do_fetch
> >>>     ERROR: Task
> (/opt/oe/configs/z/build-master/meta-rockchip/layers/meta-rockchip/recipes-bsp/u-boot/u-boot-rockchip_git.bb:do_fetch)
> failed with exit code '1'
> > The problem is that it's trying to fetch commit 3c9916 which doesn't
> exist on
> > github.com/rockchip-linux/u-boot.git. That revision is on upstream
> U-Boot.
> How comes? I didn't write any commit id there, it is based on the branch
> at all.
>

[-- Attachment #2: Type: text/html, Size: 5897 bytes --]

Re: [PATCH 3/3] recipes-bsp/u-boot: update to the latest version

[ayaka]

On 09/05/2019 01:30, Trevor Woerner wrote:
> a previous patch of yours did a:
>     require recipes-bsp/u-boot/u-boot-common.inc
> which defines this SRCREV
>
> my subtle point being: I don't get the impression you're testing your 
> patches, or at least not testing them correctly.

I did the test before but I forget to remove the variables in local.conf

Anyway I will start a clean build this time.

Now I met a problem with new gcc 9 on the latest oe, I will fix them 
then flush new patches later.

>
> the fact you're CCing Romain suggests your either using a very old 
> branch of https://git.yoctoproject.org/cgit/cgit.cgi/meta-rockchip/ or 
> that you're submitting patches to me for 
> https://github.com/rockchip-linux/meta-rockchip ?
> Unfortunately for me, when 
> https://github.com/rockchip-linux/meta-rockchip did a clone of 
> https://git.yoctoproject.org/cgit/cgit.cgi/meta-rockchip/, they left 
> my and Romain's names as maintainers in the README (along with 
> instructions for github)
>
> On Wed, May 8, 2019 at 12:19 PM ayaka <ayaka@soulik.info 
> <mailto:ayaka@soulik.info>> wrote:
>
>
>     On 5/6/19 9:31 PM, Trevor Woerner wrote:
>     > On Sat 2019-05-04 @ 06:43:34 PM, ayaka wrote:
>     >> On 5/4/19 11:06 AM, Trevor Woerner wrote:
>     >>> On Wed 2019-04-24 @ 10:59:18 PM, Randy 'ayaka' Li wrote:
>     >>>> From: Randy Li <ayaka@soulik.info <mailto:ayaka@soulik.info>>
>     >>>>
>     >>>> Signed-off-by: Randy Li <ayaka@soulik.info
>     <mailto:ayaka@soulik.info>>
>     >>>> ---
>     >>>>    recipes-bsp/u-boot/u-boot-rockchip_git.bb
>     <http://u-boot-rockchip_git.bb> | 18 ++++++++++++++++++
>     >>>>    1 file changed, 18 insertions(+)
>     >>>>    create mode 100644
>     recipes-bsp/u-boot/u-boot-rockchip_git.bb
>     <http://u-boot-rockchip_git.bb>
>     >>>>
>     >>>> diff --git a/recipes-bsp/u-boot/u-boot-rockchip_git.bb
>     <http://u-boot-rockchip_git.bb>
>     b/recipes-bsp/u-boot/u-boot-rockchip_git.bb
>     <http://u-boot-rockchip_git.bb>
>     >>>> new file mode 100644
>     >>>> index 0000000..aa4b84d
>     >>>> --- /dev/null
>     >>>> +++ b/recipes-bsp/u-boot/u-boot-rockchip_git.bb
>     <http://u-boot-rockchip_git.bb>
>     >>>> @@ -0,0 +1,18 @@
>     >>>> +# Copyright (C) 2019 SUMOMO Computer Assocation
>     >>>> +# Released under the MIT license (see COPYING.MIT for the terms)
>     >>>> +FILESEXTRAPATHS_prepend := "${THISDIR}/${PN}:"
>     >>>> +
>     >>>> +include u-boot-rockchip.inc
>     >>>> +
>     >>>> +DESCRIPTION = "Rockchip U-Boot"
>     >>>> +
>     >>>> +LIC_FILES_CHKSUM =
>     "file://Licenses/README;md5=a2c678cfd4a4d97135585cad908541c6"
>     >>>> +
>     >>>> +SRC_URI = " \
>     >>>> +  git://github.com/rockchip-linux/u-boot.git;branch=release
>     <http://github.com/rockchip-linux/u-boot.git;branch=release>; \
>     >>>> +  file://binutils-2.28-ld-fix.patch \
>     >>>> +  file://gcc7_fixup.patch \
>     >>>> +  "
>     >>>> +
>     >>>> +S = "${WORKDIR}/git"
>     >>>> +PV = "v2019.01"
>     >>> This doesn't even get past the fetching stage:
>     >> I am sorry, I forget that ssh git protocol only work for those
>     users who
>     >> uploaded their key to the github, I would change it back to https.
>     >>>     WARNING: u-boot-rockchip-1_v2019.01-r0 do_fetch: Failed to
>     fetch URL
>     git://github.com/rockchip-linux/u-boot.git;branch=release
>     <http://github.com/rockchip-linux/u-boot.git;branch=release>;,
>     attempting MIRRORS if available
>     >>>     ERROR: u-boot-rockchip-1_v2019.01-r0 do_fetch: Fetcher
>     failure: Unable to find revision
>     3c99166441bf3ea325af2da83cfe65430b49c066 in branch release even
>     from upstream
>     >>>     ERROR: u-boot-rockchip-1_v2019.01-r0 do_fetch: Fetcher
>     failure for URL:
>     'git://github.com/rockchip-linux/u-boot.git;branch=release;
>     <http://github.com/rockchip-linux/u-boot.git;branch=release;>'.
>     Unable to fetch URL from any source.
>     >>>     ERROR: u-boot-rockchip-1_v2019.01-r0 do_fetch:
>     >>>     ERROR: u-boot-rockchip-1_v2019.01-r0 do_fetch: Function
>     failed: base_do_fetch
>     >>>     ERROR: Task
>     (/opt/oe/configs/z/build-master/meta-rockchip/layers/meta-rockchip/recipes-bsp/u-boot/u-boot-rockchip_git.bb:do_fetch)
>     failed with exit code '1'
>     > The problem is that it's trying to fetch commit 3c9916 which
>     doesn't exist on
>     > github.com/rockchip-linux/u-boot.git
>     <http://github.com/rockchip-linux/u-boot.git>. That revision is on
>     upstream U-Boot.
>     How comes? I didn't write any commit id there, it is based on the
>     branch
>     at all.
>