[OE-core] [PATCH] u-boot: Upgrade to 2018.03 release

Martin Hundebøll mnhu at prevas.dk
Wed Apr 11 08:42:17 UTC 2018 

Hi,

On 2018-04-11 04:28, yamada.masahiro at socionext.com wrote:
> Hi.
> 
>>> I had to patch up our own u-boot recipe as shown in the attached patch
>>> to make v2018.03 compile for qemu-x86.
>>>
>>> The thing is that the build of pylibfdt became unconditional since
>>> 15b97f5c5e ('pylibfdt: move pylibfdt to scripts/dtc/pylibfdt and
>>> refactor makefile')
>>>
>>> In my case u-boot/pylibfdt failed to find the correct (native) headers,
>>> because python setuptools / distutils looks into STAGING_{LIB,INC}DIR
>>> when compiling the native extension for libfdt. I didn't find any other
>>> way to specificy this in either the u-boot Makefile or some other magic
>>> environment variable.
>>
>> CCing Yamada-san, maybe he has an idea.
>> Also, do not top post.
> 
> 
> 
> The build of pylibfdt is conditional.
> It is built only when CONFIG_PYLIBFDT=y,
> which is selected by CONFIG_DTOC.
> 
> If you really do not need pylibfdt,
> you can disable it by tweaking the configuration.
> 
> If you need pylibfdt but you cannot build it,
> it is a different problem.
> 
> Thanks.

Correct. But I was building u-boot.rom for qemu-x86, which depends on 
binman and thus pylibfdt. Before the mentioned commit, one could avoid 
the building of pylibfdt by installing it on the host, as the makerule 
checked if python could already import pylibfdt. This check is now removed.

Or am I missing something?

// Martin

> 
>> -----Original Message-----
>> From: Marek Vasut [mailto:marex at denx.de]
>> Sent: Tuesday, April 10, 2018 6:55 PM
>> To: Martin Hundebøll <mnhu at prevas.dk>; Khem Raj <raj.khem at gmail.com>
>> Cc: Otavio Salvador <otavio at ossystems.com.br>;
>> openembedded-core at lists.openembedded.org; Yamada, Masahiro/山田 真弘
>> <yamada.masahiro at socionext.com>
>> Subject: Re: [OE-core] [PATCH] u-boot: Upgrade to 2018.03 release
>>
>> On 04/10/2018 11:49 AM, Martin Hundebøll wrote:
>>> Hi,
>>
>> Hi,
>>
>>> I had to patch up our own u-boot recipe as shown in the attached patch
>>> to make v2018.03 compile for qemu-x86.
>>>
>>> The thing is that the build of pylibfdt became unconditional since
>>> 15b97f5c5e ('pylibfdt: move pylibfdt to scripts/dtc/pylibfdt and
>>> refactor makefile')
>>>
>>> In my case u-boot/pylibfdt failed to find the correct (native) headers,
>>> because python setuptools / distutils looks into STAGING_{LIB,INC}DIR
>>> when compiling the native extension for libfdt. I didn't find any other
>>> way to specificy this in either the u-boot Makefile or some other magic
>>> environment variable.
>>
>> CCing Yamada-san, maybe he has an idea.
>> Also, do not top post.
>>
>>> // Martin
>>>
>>> On 2018-04-09 02:54, Khem Raj wrote:
>>>>
>>>> On Sun, Apr 8, 2018 at 4:10 PM Marek Vasut <marex at denx.de
>>>> <mailto:marex at denx.de>> wrote:
>>>>
>>>>      This upgrades the U-Boot from 2018.01 to 2018.03 release and drops
>>>>      patches accepted upstream, getting the patch count to zero.
>>>>
>>>>
>>>>
>>>> It fails for me if I have libfdt installed on build host
>>>> It seems build is doing the include dance which goes all the way into
>>>> hosts system include directories
>>>>
>>>>
>>>>
>>>>      Signed-off-by: Marek Vasut <marex at denx.de <mailto:marex at denx.de>>
>>>>      Cc: Otavio Salvador <otavio at ossystems.com.br
>>>>      <mailto:otavio at ossystems.com.br>>
>>>>      Cc: Ross Burton <ross.burton at intel.com
>>>> <mailto:ross.burton at intel.com>>
>>>>      Cc: Richard Purdie <richard.purdie at linuxfoundation.org
>>>>      <mailto:richard.purdie at linuxfoundation.org>>
>>>>      ---
>>>>        .../u-boot/files/MPC8315ERDB-enable-DHCP.patch        | 19
>>>>      -------------------
>>>>        ...t-common_2018.01.inc => u-boot-common_2018.03.inc} |  6
>> ++----
>>>>        ...fw-utils_2018.01.bb <http://fw-utils_2018.01.bb> =>
>>>>      u-boot-fw-utils_2018.03.bb <http://u-boot-fw-utils_2018.03.bb>}
>> |  0
>>>>        ...t-mkimage_2018.01.bb <http://t-mkimage_2018.01.bb> =>
>>>>      u-boot-mkimage_2018.03.bb <http://u-boot-mkimage_2018.03.bb>}
>> |  0
>>>>        .../u-boot/{u-boot_2018.01.bb <http://u-boot_2018.01.bb> =>
>>>>      u-boot_2018.03.bb <http://u-boot_2018.03.bb>}   |  0
>>>>        5 files changed, 2 insertions(+), 23 deletions(-)
>>>>        delete mode 100644
>>>>      meta/recipes-bsp/u-boot/files/MPC8315ERDB-enable-DHCP.patch
>>>>        rename meta/recipes-bsp/u-boot/{u-boot-common_2018.01.inc =>
>>>>      u-boot-common_2018.03.inc} (68%)
>>>>        rename meta/recipes-bsp/u-boot/{u-boot-fw-utils_2018.01.bb
>>>>      <http://u-boot-fw-utils_2018.01.bb> =>
>> u-boot-fw-utils_2018.03.bb
>>>>      <http://u-boot-fw-utils_2018.03.bb>} (100%)
>>>>        rename meta/recipes-bsp/u-boot/{u-boot-mkimage_2018.01.bb
>>>>      <http://u-boot-mkimage_2018.01.bb> => u-boot-mkimage_2018.03.bb
>>>>      <http://u-boot-mkimage_2018.03.bb>} (100%)
>>>>        rename meta/recipes-bsp/u-boot/{u-boot_2018.01.bb
>>>>      <http://u-boot_2018.01.bb> => u-boot_2018.03.bb
>>>>      <http://u-boot_2018.03.bb>} (100%)
>>>>
>>>>      diff --git
>>>>      a/meta/recipes-bsp/u-boot/files/MPC8315ERDB-enable-DHCP.patch
>>>>      b/meta/recipes-bsp/u-boot/files/MPC8315ERDB-enable-DHCP.patch
>>>>      deleted file mode 100644
>>>>      index ecaa1796a4..0000000000
>>>>      ---
>> a/meta/recipes-bsp/u-boot/files/MPC8315ERDB-enable-DHCP.patch
>>>>      +++ /dev/null
>>>>      @@ -1,19 +0,0 @@
>>>>      -Enabled dhcp client functionality for Yocto reference
>>>>      -hardware MPC8315E-RDB.
>>>>      -
>>>>      -Upstream-Status: Pending
>>>>      -
>>>>      -Signed-off-by: Ed Bartosh <ed.bartosh at linux.intel.com
>>>>      <mailto:ed.bartosh at linux.intel.com>>
>>>>      -
>>>>      -Index: git/configs/MPC8315ERDB_defconfig
>>>>      -==============================================================
>> =====
>>>>      ---- git.orig/configs/MPC8315ERDB_defconfig
>>>>      -+++ git/configs/MPC8315ERDB_defconfig
>>>>      -@@ -12,6 +12,7 @@ CONFIG_CMD_PCI=y
>>>>      - CONFIG_CMD_SATA=y
>>>>      - CONFIG_CMD_USB=y
>>>>      - # CONFIG_CMD_SETEXPR is not set
>>>>      -+CONFIG_CMD_DHCP=y
>>>>      - CONFIG_CMD_MII=y
>>>>      - CONFIG_CMD_PING=y
>>>>      - CONFIG_CMD_DATE=y
>>>>      diff --git a/meta/recipes-bsp/u-boot/u-boot-common_2018.01.inc
>>>>      b/meta/recipes-bsp/u-boot/u-boot-common_2018.03.inc
>>>>      similarity index 68%
>>>>      rename from meta/recipes-bsp/u-boot/u-boot-common_2018.01.inc
>>>>      rename to meta/recipes-bsp/u-boot/u-boot-common_2018.03.inc
>>>>      index d2073ea0c7..8ffae365a9 100644
>>>>      --- a/meta/recipes-bsp/u-boot/u-boot-common_2018.01.inc
>>>>      +++ b/meta/recipes-bsp/u-boot/u-boot-common_2018.03.inc
>>>>      @@ -7,10 +7,8 @@ PE = "1"
>>>>
>>>>        # We use the revision in order to avoid having to fetch it from
>> the
>>>>        # repo during parse
>>>>      -SRCREV = "f3dd87e0b98999a78e500e8c6d2b063ebadf535a"
>>>>      +SRCREV = "f95ab1fb6e37f0601f397091bb011edf7a98b890"
>>>>
>>>>      -SRC_URI = "git://git.denx.de/u-boot.git
>>>>      <http://git.denx.de/u-boot.git> \
>>>>      -    file://MPC8315ERDB-enable-DHCP.patch \
>>>>      -"
>>>>      +SRC_URI = "git://git.denx.de/u-boot.git
>>>>      <http://git.denx.de/u-boot.git>"
>>>>
>>>>        S = "${WORKDIR}/git"
>>>>      diff --git a/meta/recipes-bsp/u-boot/u-boot-fw-utils_2018.01.bb
>>>>      <http://u-boot-fw-utils_2018.01.bb>
>>>>      b/meta/recipes-bsp/u-boot/u-boot-fw-utils_2018.03.bb
>>>>      <http://u-boot-fw-utils_2018.03.bb>
>>>>      similarity index 100%
>>>>      rename from meta/recipes-bsp/u-boot/u-boot-fw-utils_2018.01.bb
>>>>      <http://u-boot-fw-utils_2018.01.bb>
>>>>      rename to meta/recipes-bsp/u-boot/u-boot-fw-utils_2018.03.bb
>>>>      <http://u-boot-fw-utils_2018.03.bb>
>>>>      diff --git a/meta/recipes-bsp/u-boot/u-boot-mkimage_2018.01.bb
>>>>      <http://u-boot-mkimage_2018.01.bb>
>>>>      b/meta/recipes-bsp/u-boot/u-boot-mkimage_2018.03.bb
>>>>      <http://u-boot-mkimage_2018.03.bb>
>>>>      similarity index 100%
>>>>      rename from meta/recipes-bsp/u-boot/u-boot-mkimage_2018.01.bb
>>>>      <http://u-boot-mkimage_2018.01.bb>
>>>>      rename to meta/recipes-bsp/u-boot/u-boot-mkimage_2018.03.bb
>>>>      <http://u-boot-mkimage_2018.03.bb>
>>>>      diff --git a/meta/recipes-bsp/u-boot/u-boot_2018.01.bb
>>>>      <http://u-boot_2018.01.bb>
>>>>      b/meta/recipes-bsp/u-boot/u-boot_2018.03.bb
>>>> <http://u-boot_2018.03.bb>
>>>>      similarity index 100%
>>>>      rename from meta/recipes-bsp/u-boot/u-boot_2018.01.bb
>>>>      <http://u-boot_2018.01.bb>
>>>>      rename to meta/recipes-bsp/u-boot/u-boot_2018.03.bb
>>>>      <http://u-boot_2018.03.bb>
>>>>      --
>>>>      2.16.2
>>>>
>>>>      --
>>>>      _______________________________________________
>>>>      Openembedded-core mailing list
>>>>      Openembedded-core at lists.openembedded.org
>>>>      <mailto:Openembedded-core at lists.openembedded.org>
>>>>      http://lists.openembedded.org/mailman/listinfo/openembedded-cor
>> e
>>>>
>>>>
>>>>
>>
>>
>> --
>> Best regards,
>> Marek Vasut

More information about the Openembedded-core mailing list