Re: [yocto] [meta-rockchip][PATCH 3/3] wic/wks cleanup

2021-06-30 Thread Trevor Woerner
On Mon 2021-06-28 @ 11:15:41 AM, Trevor Woerner wrote:
> By exporting a couple more variables the wks file for every rockchip device
> can be built from one template instead of having separate wks files for each
> board and platform.
> 
> The following BSP variables were checked before and after this change to make
> sure they remained valid/sensible:
> - WKS_FILE
> - UBOOT_SUFFIX
> - SPL_BINARY
> - IMAGE_FSTYPES
> 
> Built-tested for every MACHINE in this BSP.
> 
> Run-tested on the following devices to ensure they continue to boot correctly
> to a cmdline (core-image-base):
> - tinker-board
> - rock-pi-e
> - rock-pi-4b
> - rock64
> - nanopi-m4-2gb
> 
> Signed-off-by: Trevor Woerner 
> ---
>  conf/machine/firefly-rk3288.conf  |  2 --
>  conf/machine/include/nanopi-m4.inc|  1 -
>  conf/machine/include/rk3288.inc   |  3 +--
>  conf/machine/include/rk3328.inc   |  1 -
>  conf/machine/include/rk3399.inc   |  2 --
>  conf/machine/include/rock-pi-4.inc|  1 -
>  conf/machine/include/rockchip-wic.inc |  5 +
>  conf/machine/include/tinker.inc   |  2 --
>  conf/machine/rock-pi-e.conf   |  2 --
>  conf/machine/rock64.conf  |  2 --
>  conf/machine/vyasa-rk3288.conf|  1 -
>  wic/firefly-rk3288.wks|  7 ---
>  wic/rk3288-boot.wks   | 24 
>  wic/rk3399-boot.wks   | 24 
>  wic/rock-pi-4.wks |  7 ---
>  wic/rock-pi-e.wks |  4 
>  wic/{rk3328-boot.wks => rockchip.wks} |  9 ++---
>  wic/tinker-board.wks  |  8 
>  wic/vyasa-rk3288.wks  |  8 
>  19 files changed, 12 insertions(+), 101 deletions(-)
>  delete mode 100644 wic/firefly-rk3288.wks
>  delete mode 100644 wic/rk3288-boot.wks
>  delete mode 100644 wic/rk3399-boot.wks
>  delete mode 100644 wic/rock-pi-4.wks
>  delete mode 100644 wic/rock-pi-e.wks
>  rename wic/{rk3328-boot.wks => rockchip.wks} (64%)
>  delete mode 100644 wic/tinker-board.wks
>  delete mode 100644 wic/vyasa-rk3288.wks
> 
> diff --git a/conf/machine/firefly-rk3288.conf 
> b/conf/machine/firefly-rk3288.conf
> index 138e840..3270bb9 100644

Applied to meta-rockchip master.

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#54026): https://lists.yoctoproject.org/g/yocto/message/54026
Mute This Topic: https://lists.yoctoproject.org/mt/83847095/21656
Group Owner: yocto+ow...@lists.yoctoproject.org
Unsubscribe: https://lists.yoctoproject.org/g/yocto/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[yocto] [meta-rockchip][PATCH 3/3] wic/wks cleanup

2021-06-28 Thread Trevor Woerner
By exporting a couple more variables the wks file for every rockchip device
can be built from one template instead of having separate wks files for each
board and platform.

The following BSP variables were checked before and after this change to make
sure they remained valid/sensible:
- WKS_FILE
- UBOOT_SUFFIX
- SPL_BINARY
- IMAGE_FSTYPES

Built-tested for every MACHINE in this BSP.

Run-tested on the following devices to ensure they continue to boot correctly
to a cmdline (core-image-base):
- tinker-board
- rock-pi-e
- rock-pi-4b
- rock64
- nanopi-m4-2gb

Signed-off-by: Trevor Woerner 
---
 conf/machine/firefly-rk3288.conf  |  2 --
 conf/machine/include/nanopi-m4.inc|  1 -
 conf/machine/include/rk3288.inc   |  3 +--
 conf/machine/include/rk3328.inc   |  1 -
 conf/machine/include/rk3399.inc   |  2 --
 conf/machine/include/rock-pi-4.inc|  1 -
 conf/machine/include/rockchip-wic.inc |  5 +
 conf/machine/include/tinker.inc   |  2 --
 conf/machine/rock-pi-e.conf   |  2 --
 conf/machine/rock64.conf  |  2 --
 conf/machine/vyasa-rk3288.conf|  1 -
 wic/firefly-rk3288.wks|  7 ---
 wic/rk3288-boot.wks   | 24 
 wic/rk3399-boot.wks   | 24 
 wic/rock-pi-4.wks |  7 ---
 wic/rock-pi-e.wks |  4 
 wic/{rk3328-boot.wks => rockchip.wks} |  9 ++---
 wic/tinker-board.wks  |  8 
 wic/vyasa-rk3288.wks  |  8 
 19 files changed, 12 insertions(+), 101 deletions(-)
 delete mode 100644 wic/firefly-rk3288.wks
 delete mode 100644 wic/rk3288-boot.wks
 delete mode 100644 wic/rk3399-boot.wks
 delete mode 100644 wic/rock-pi-4.wks
 delete mode 100644 wic/rock-pi-e.wks
 rename wic/{rk3328-boot.wks => rockchip.wks} (64%)
 delete mode 100644 wic/tinker-board.wks
 delete mode 100644 wic/vyasa-rk3288.wks

diff --git a/conf/machine/firefly-rk3288.conf b/conf/machine/firefly-rk3288.conf
index 138e840..3270bb9 100644
--- a/conf/machine/firefly-rk3288.conf
+++ b/conf/machine/firefly-rk3288.conf
@@ -11,5 +11,3 @@ require conf/machine/include/rockchip-wic.inc
 
 KERNEL_DEVICETREE = "rk3288-firefly.dtb"
 UBOOT_MACHINE = "firefly-rk3288_defconfig"
-
-WKS_FILE ?= "firefly-rk3288.wks"
diff --git a/conf/machine/include/nanopi-m4.inc 
b/conf/machine/include/nanopi-m4.inc
index b5251a1..ac6479d 100644
--- a/conf/machine/include/nanopi-m4.inc
+++ b/conf/machine/include/nanopi-m4.inc
@@ -9,4 +9,3 @@ KMACHINE = "nanopi-m4"
 KERNEL_DEVICETREE = "rockchip/rk3399-nanopi-m4.dtb"
 
 RK_BOOT_DEVICE = "mmcblk1"
-WKS_FILE ?= "rock-pi-4.wks"
diff --git a/conf/machine/include/rk3288.inc b/conf/machine/include/rk3288.inc
index a109f26..6b5f70a 100644
--- a/conf/machine/include/rk3288.inc
+++ b/conf/machine/include/rk3288.inc
@@ -13,5 +13,4 @@ KBUILD_DEFCONFIG ?= "multi_v7_defconfig"
 KERNEL_IMAGETYPE = "zImage"
 
 PREFERRED_PROVIDER_virtual/bootloader ?= "u-boot"
-SPL_BINARY ?= "idbloader.img"
-
+UBOOT_SUFFIX ?= "bin"
diff --git a/conf/machine/include/rk3328.inc b/conf/machine/include/rk3328.inc
index b0cafb5..70261a0 100644
--- a/conf/machine/include/rk3328.inc
+++ b/conf/machine/include/rk3328.inc
@@ -21,4 +21,3 @@ UBOOT_SUFFIX ?= "itb"
 UBOOT_ENTRYPOINT ?= "0x0600"
 
 PREFERRED_PROVIDER_virtual/bootloader ?= "u-boot"
-SPL_BINARY ?= "idbloader.img"
diff --git a/conf/machine/include/rk3399.inc b/conf/machine/include/rk3399.inc
index 79e83e2..3fc712f 100644
--- a/conf/machine/include/rk3399.inc
+++ b/conf/machine/include/rk3399.inc
@@ -21,5 +21,3 @@ UBOOT_SUFFIX ?= "itb"
 UBOOT_ENTRYPOINT ?= "0x0600"
 
 PREFERRED_PROVIDER_virtual/bootloader ?= "u-boot"
-SPL_BINARY ?= "idbloader.img"
-
diff --git a/conf/machine/include/rock-pi-4.inc 
b/conf/machine/include/rock-pi-4.inc
index 92fc330..b6fb3dd 100644
--- a/conf/machine/include/rock-pi-4.inc
+++ b/conf/machine/include/rock-pi-4.inc
@@ -4,6 +4,5 @@ MACHINEOVERRIDES =. "rock-pi-4:"
 require conf/machine/include/rk3399.inc
 
 RK_BOOT_DEVICE = "mmcblk1"
-WKS_FILE ?= "rock-pi-4.wks"
 
 MACHINE_EXTRA_RRECOMMENDS += "kernel-modules"
diff --git a/conf/machine/include/rockchip-wic.inc 
b/conf/machine/include/rockchip-wic.inc
index 0ee8c0e..61d9f3d 100644
--- a/conf/machine/include/rockchip-wic.inc
+++ b/conf/machine/include/rockchip-wic.inc
@@ -1,6 +1,9 @@
 # common meta-rockchip wic/wks items
 
+SPL_BINARY ?= "idbloader.img"
+
 IMAGE_FSTYPES += "wic wic.bmap"
+WKS_FILE = "rockchip.wks"
 WKS_FILE_DEPENDS ?= " \
mtools-native \
dosfstools-native \
@@ -24,4 +27,6 @@ WICVARS_append = " \
RK_BOOT_DEVICE \
RK_CONSOLE_BAUD \
RK_CONSOLE_DEVICE \
+   SPL_BINARY \
+   UBOOT_SUFFIX \
"
diff --git a/conf/machine/include/tinker.inc b/conf/machine/include/tinker.inc
index eaeb564..2d05bef 100644
--- a/conf/machine/include/tinker.inc
+++ b/conf/machine/include/tinker.inc
@@ -1,4 +1,2 @@
 require conf/machine/include/rk3288.inc