[Buildroot] [PATCH v3 1/2] package/microchip-hss-payload-generator: add host package
Thomas Petazzoni
thomas.petazzoni at bootlin.com
Tue Aug 8 21:56:27 UTC 2023
Hello Jamie,
On Wed, 12 Jul 2023 13:51:53 +0100
Jamie Gibbons via buildroot <buildroot at buildroot.org> wrote:
> The Buildroot icicle kit configuration uses the Hart Software Service's
> (HSS) payload generator tool. 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). Add the HSS payload generator as a
> host package to support this.
>
> Signed-off-by: Jamie Gibbons <jamie.gibbons at microchip.com>
> Reviewed-by: Valentina Fernandez <valentina.fernandezalanis at microchip.com>
> ---
> DEVELOPERS | 3 +++
> package/Config.in.host | 1 +
> .../Config.in.host | 11 ++++++++
> .../microchip-hss-payload-generator.mk | 25 +++++++++++++++++++
> 4 files changed, 40 insertions(+)
This was missing a hash file for the package, which provides the hash
of the tarball and the hash of the license file.
> +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
I was initially confused by this, because LICENSE.md also mentions than
the repo contains OpenSBI, which is under BSD-2-Clause. But in fact
this package only builds what's in tools/hss-payload-generator, so the
rest is irrelevant. Therefore, I have changed this to:
+# Some parts of the repository are under different licenses, but we
+# are only building/installing the code in
+# tools/hss-payload-generator/.
+HOST_MICROCHIP_HSS_PAYLOAD_GENERATOR_LICENSE = MIT or GPL-2.0+
+HOST_MICROCHIP_HSS_PAYLOAD_GENERATOR_LICENSE_FILES = tools/hss-payload-generator/LICENSE.md
Let me know if you think this is incorrect.
I've applied to our "next" branch with those changes. Thanks!
Thanks,
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