[Buildroot] [PATCH/next 1/5] armada-firmware: new package
Baruch Siach
baruch at tkos.co.il
Mon Nov 13 07:30:24 UTC 2017
Hi Sergey,
On Sun, Nov 12, 2017 at 11:16:42PM +0300, Sergey Matyukevich wrote:
> This package adds SCP BL2 firmware for Marvell Armada 7040 and 8040 SoCs.
>
> Signed-off-by: Sergey Matyukevich <geomatsi at gmail.com>
> ---
> boot/Config.in | 1 +
> boot/armada-firmware/Config.in | 19 +++++++++++++++++++
> boot/armada-firmware/armada-firmware.hash | 2 ++
> boot/armada-firmware/armada-firmware.mk | 18 ++++++++++++++++++
> 4 files changed, 40 insertions(+)
> create mode 100644 boot/armada-firmware/Config.in
> create mode 100644 boot/armada-firmware/armada-firmware.hash
> create mode 100644 boot/armada-firmware/armada-firmware.mk
>
> diff --git a/boot/Config.in b/boot/Config.in
> index 2f46c8546e..0ffbd7288b 100644
> --- a/boot/Config.in
> +++ b/boot/Config.in
> @@ -17,5 +17,6 @@ source "boot/ts4800-mbrboot/Config.in"
> source "boot/uboot/Config.in"
> source "boot/vexpress-firmware/Config.in"
> source "boot/xloader/Config.in"
> +source "boot/armada-firmware/Config.in"
>
> endmenu
> diff --git a/boot/armada-firmware/Config.in b/boot/armada-firmware/Config.in
> new file mode 100644
> index 0000000000..530b5622c8
> --- /dev/null
> +++ b/boot/armada-firmware/Config.in
> @@ -0,0 +1,19 @@
> +config BR2_TARGET_ARMADA_FIRMWARE
> + bool "armada-firmware"
> + depends on BR2_aarch64
> + help
> + Marvell Armada SCP BL2 firmware images.
> +
> +if BR2_TARGET_ARMADA_FIRMWARE
> +
> +config BR2_TARGET_ARMADA_FIRMWARE_IMAGE
> + string "Armada SCP BL2 image name"
> + help
> + Armada SCP BL2 firmware image name.
This symbol is not used here, which is quite unusual. Instead, you set this
symbol in the defconfig, and use it in the ATF package. But since we'll use
only one of the 7k/8k images (right?), I suggest to add a 7k/8k choice, and
make BR2_TARGET_ARMADA_FIRMWARE_IMAGE a blink symbol that is set (default)
according to the choice.
Something like (abbreviated; untested):
choice
prompt "Armada platform"
config BR2_PACKAGE_ARMADA_8040
bool "8040"
config BR2_PACKAGE_ARMADA_7040
bool "7040"
endchoice
config BR2_TARGET_ARMADA_FIRMWARE_IMAGE
string
default "mrvl_scp_bl2_8040.img" if BR2_PACKAGE_ARMADA_8040
default "mrvl_scp_bl2_7040.img" if BR2_PACKAGE_ARMADA_7040
> +config BR2_TARGET_ARMADA_FIRMWARE_VERSION
> + string "Armada SCP BL2 image version"
> + help
> + Armada SCP BL2 firmware image version.
> +
> +endif
> diff --git a/boot/armada-firmware/armada-firmware.hash b/boot/armada-firmware/armada-firmware.hash
> new file mode 100644
> index 0000000000..5eeca9d6ea
> --- /dev/null
> +++ b/boot/armada-firmware/armada-firmware.hash
> @@ -0,0 +1,2 @@
> +# Locally calculated
> +sha256 b310443c0d51d07b7c11597548685ae608b8478eee7095925427da6ab71e0168 armada-firmware-binaries-marvell-armada-17.10.tar.gz
> diff --git a/boot/armada-firmware/armada-firmware.mk b/boot/armada-firmware/armada-firmware.mk
> new file mode 100644
> index 0000000000..d75dc7cb6c
> --- /dev/null
> +++ b/boot/armada-firmware/armada-firmware.mk
> @@ -0,0 +1,18 @@
> +################################################################################
> +#
> +# Marvell Armada SCP BL2 firmware images
> +#
> +################################################################################
> +
> +ARMADA_FIRMWARE_VERSION = $(call qstrip,$(BR2_TARGET_ARMADA_FIRMWARE_VERSION))
Having a configurable version is an exception of very few packages (linux,
uboot, ATF). Other packages should set to a static version string or a commit
id when there is no official version release.
> +ARMADA_FIRMWARE_SITE = $(call github,MarvellEmbeddedProcessors,binaries-marvell,$(ARMADA_FIRMWARE_VERSION))
> +ARMADA_FIRMWARE_LICENSE = Proprietary
These binaries come with no license, so you should also add
ARMADA_FIRMWARE_REDISTRIBUTE = NO.
> +
> +ARMADA_FIRMWARE_INSTALL_IMAGES = YES
> +
> +define ARMADA_FIRMWARE_INSTALL_IMAGES_CMDS
> + $(INSTALL) -D -m 0644 $(@D)/mrvl_scp_bl2_7040.img $(BINARIES_DIR)/mrvl_scp_bl2_7040.img
> + $(INSTALL) -D -m 0644 $(@D)/mrvl_scp_bl2_8040.img $(BINARIES_DIR)/mrvl_scp_bl2_8040.img
> +endef
> +
> +$(eval $(generic-package))
baruch
--
http://baruch.siach.name/blog/ ~. .~ Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
- baruch at tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il -
More information about the buildroot
mailing list