Message ID | 20250505-ddrbin-custom-v3-0-577b94c72035@cherry.de |
---|---|
Headers | show
Return-Path: <foss@0leil.net> X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from aws-us-west-2-korg-lkml-1.web.codeaurora.org (localhost.localdomain [127.0.0.1]) by smtp.lore.kernel.org (Postfix) with ESMTP id BF7C0C3ABB0 for <webhook@archiver.kernel.org>; Mon, 5 May 2025 14:20:57 +0000 (UTC) Received: from smtp-bc0e.mail.infomaniak.ch (smtp-bc0e.mail.infomaniak.ch [45.157.188.14]) by mx.groups.io with SMTP id smtpd.web11.51612.1746454851581079649 for <yocto-patches@lists.yoctoproject.org>; Mon, 05 May 2025 07:20:51 -0700 Authentication-Results: mx.groups.io; dkim=none (message not signed); spf=pass (domain: 0leil.net, ip: 45.157.188.14, mailfrom: foss+yocto@0leil.net) Received: from smtp-3-0000.mail.infomaniak.ch (unknown [IPv6:2001:1600:4:17::246b]) by smtp-3-3000.mail.infomaniak.ch (Postfix) with ESMTPS id 4ZrkG95rxwzFjJ; Mon, 5 May 2025 16:20:49 +0200 (CEST) Received: from unknown by smtp-3-0000.mail.infomaniak.ch (Postfix) with ESMTPA id 4ZrkG927YBzt1B; Mon, 5 May 2025 16:20:49 +0200 (CEST) From: Quentin Schulz <foss+yocto@0leil.net> Subject: [meta-rockchip PATCH v3 0/9] rkbin: factoring do_deploy, customize ddrbin and bump rkbin Date: Mon, 05 May 2025 16:20:36 +0200 Message-Id: <20250505-ddrbin-custom-v3-0-577b94c72035@cherry.de> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit X-B4-Tracking: v=1; b=H4sIADXJGGgC/3XNyw7CIBAF0F8xrMXwKJa68j+MC4RBJqaPQCU2T f9dwkZj4vLmzrmzkgQRIZHTbiURMiYchxLkfkdsMMMdKLqSiWBCMck5dS7ecKD2meaxp17Ljuu GHUEaUswUweOr7l1ID7OhcbQPG3Ai11IHLCou9Vvm9ejPcOaUUVC26xpjhQJ+tgFiXA4O6lIWH 91w/atF0S1jWmglWm/8t9627Q2VpSE49wAAAA== X-Change-ID: 20250311-ddrbin-custom-f83918406e3a To: yocto-patches@lists.yoctoproject.org Cc: Quentin Schulz <quentin.schulz@cherry.de> X-Mailer: b4 0.14.2 X-Infomaniak-Routing: alpha List-Id: <yocto-patches.lists.yoctoproject.org> X-Webhook-Received: from li982-79.members.linode.com [45.33.32.79] by aws-us-west-2-korg-lkml-1.web.codeaurora.org with HTTPS for <yocto-patches@lists.yoctoproject.org>; Mon, 05 May 2025 14:20:57 -0000 X-Groupsio-URL: https://lists.yoctoproject.org/g/yocto-patches/message/1502 |
Series |
rkbin: factoring do_deploy, customize ddrbin and bump rkbin
|
expand
|
This simplifies rockchip-rkbin-ddr, rockchip-rkbin-tf-a and rockchip-rkbin-optee-os do_deploy to only have one do_deploy task common to all supported SoC instead of having to add a new task override for each new SoC. It does require to set a few more variables though. Thanks to the variable abstraction, we can now move all of do_deploy into the rockchip-rkbin.inc file. The downside is that the version for OPTEE-OS and TF-A are now hardcoded in recipes, so a bump of rkbin requires to bump those as well. Additionally, this finally allows to customize the DDR binary blob using Rockchip's ddrbin_tool.py tool. This is typically useful if your board isn't following Rockchip's HW reference design, e.g. using a different UART controller or pins for serial output, or using a baudrate different from the default of 1500000. Finally it bumps OP-TEE OS, TF-A and DDR bin recipes to use the latest commit in rkbin git repo. I am not too fond of using a file as input for the customization, but this is what ddrbin_tool.py uses and there are possibly too many corner-cases to handle by using fine-grained variables. So for now, I decided to go with that but that is for sure not the only possible implementation! Note that the bump commit wasn't tested. Note that I plan on backporting all but the factoring out and rkbin bump patches to scarthgap (and I assume styhead too). The factoring out patches would break backward compatibility if someone appends to the do_deploy:<override> task. The last patch could be merged but there's no absolute need for it? Lastly, the patch about customizing DDR bin needs to be modified before being backported as it uses UNPACKDIR which doesn't exist in scarthgap. The logic should be safe to use with ${S} (for scarthgap) though! @Trevor, maybe you can check if that would fix your issue with the RK3308 DDR blob you wanted to keep on that outdated version which is one of the last versions where the UART you want to use exists? It'd be nice to get rid of the rk3308-specific recipes I believe :) Signed-off-by: Quentin Schulz <quentin.schulz@cherry.de> --- Changes in v3: - fixed tee- prefix instead of bl31- in rockchip-ddrbin-tf-a recipe, - build tested with bitbake virtual/bootloader, - Link to v2: https://lore.kernel.org/r/20250418-ddrbin-custom-v2-0-700828527faf@cherry.de Changes in v2: - factored out rockchip-rkbin-optee-os and rockchip-rkbin-tf-a as well, - moved do_deploy (and RKBIN_BINDIR) to rockchip-rkbin.inc, - removed bbfatal for the version variable as it's not used directly, only through another variable, which can be overwritten, so it doesn't make sense to enforce it if it's not necessarily used, - renamed variables to be recipe-agnostic (e.g. RKBIN_ instead of DDRBIN_ for the prefix) so they can be used from the .inc in the do_deploy task, - renamed ddrbin_params.txt into ddrbin_param.txt to match the original filename in rkbin git repo, - Link to v1: https://lore.kernel.org/r/20250311-ddrbin-custom-v1-0-e5c994ac25e1@cherry.de --- Quentin Schulz (9): bsp: rkbin: ddr: store directory path for the DDR bin blob in a variable bsp: rkbin: ddr: make deployed name configurable bsp: rkbin: add native recipe for tools (ddrbin_tool.py) bsp: rkbin: ddr: allow to customize DDR bin blob bsp: rkbin: ddr: factor out do_deploy to be SoC-agnostic bsp: rkbin: optee-os: factor out do_deploy to be SoC-agnostic bsp: rkbin: tf-a: factor out do_deploy to be SoC-agnostic bsp: rkbin: factor out do_deploy and RKBIN_BINDIR variable bsp: rkbin: bump to latest commit in master branch README | 20 +++++- recipes-bsp/rkbin/rockchip-rkbin-ddr_git.bb | 83 ++++++++++++++++-------- recipes-bsp/rkbin/rockchip-rkbin-native_git.bb | 21 ++++++ recipes-bsp/rkbin/rockchip-rkbin-optee-os_git.bb | 29 ++++----- recipes-bsp/rkbin/rockchip-rkbin-tf-a_git.bb | 29 ++++----- recipes-bsp/rkbin/rockchip-rkbin.inc | 21 +++++- 6 files changed, 141 insertions(+), 62 deletions(-) --- base-commit: 0aba79e47951b049bcdd39e07f660c665e257393 change-id: 20250311-ddrbin-custom-f83918406e3a Best regards,