[Buildroot] [PATCH v2 1/2] package/microchip-hss-payload-generator: add host package
Thomas Petazzoni
thomas.petazzoni at bootlin.com
Wed Jul 12 11:49:33 UTC 2023
Hello Jamie,
So very minor nits below.
On Wed, 12 Jul 2023 09:34:54 +0100
Jamie Gibbons via buildroot <buildroot at buildroot.org> wrote:
> diff --git a/package/microchip-hss-payload-generator/Config.in.host b/package/microchip-hss-payload-generator/Config.in.host
> new file mode 100644
> index 0000000000..1e21fbbb5d
> --- /dev/null
> +++ b/package/microchip-hss-payload-generator/Config.in.host
> @@ -0,0 +1,10 @@
> +config BR2_PACKAGE_HOST_MICROCHIP_HSS_PAYLOAD_GENERATOR
> + bool "HSS Payload Generator"
Should be:
bool "host microchip-hss-payload-generator"
> + help
> + Microchip PolarFire SoC Payload Generator. This tool creates a formatted
> + payload image for the HSS zero-stage bootloader on PolarFire SoC, given a
> + configuration file and a set of ELF binaries. The configuration file is
> + used to map the ELF binaries or binary blobs to the individual application
> + harts (U54s).
Is "make check-package" running fine? These lines look a bit long to
me, but "make check-package" will tell for sure.
> diff --git a/package/microchip-hss-payload-generator/microchip-hss-payload-generator.mk b/package/microchip-hss-payload-generator/microchip-hss-payload-generator.mk
> new file mode 100644
> index 0000000000..7344d208c1
> --- /dev/null
> +++ b/package/microchip-hss-payload-generator/microchip-hss-payload-generator.mk
> @@ -0,0 +1,24 @@
> +################################################################################
> +#
> +# microchip-hss-payload-generator
> +#
> +################################################################################
Missing empty line between the silly comment header above and the first
variable definition.
> +HOST_MICROCHIP_HSS_PAYLOAD_GENERATOR_VERSION = 2023.06
> +HOST_MICROCHIP_HSS_PAYLOAD_GENERATOR_SITE = $(call github,polarfire-soc,hart-software-services,v$(HOST_MICROCHIP_HSS_PAYLOAD_GENERATOR_VERSION))
> +HOST_MICROCHIP_HSS_PAYLOAD_GENERATOR_LICENSE = MIT
> +HOST_MICROCHIP_HSS_PAYLOAD_GENERATOR_LICENSE_FILES = LICENSE.md
> +HOST_MICROCHIP_HSS_PAYLOAD_GENERATOR_DEPENDENCIES = host-elfutils host-libyaml
No other comments, the rest looks fine to me, of course except the
host-openssl missing dependency pointed out by Giulio.
Thomas
--
Thomas Petazzoni, co-owner and CEO, Bootlin
Embedded Linux and Kernel engineering and training
https://bootlin.com
More information about the buildroot
mailing list