[Buildroot] [PATCH 1/2] package/python-pillow: enable host package

[Buildroot] [PATCH 1/2] package/python-pillow: enable host package

[Kory Maincent via buildroot]

From: Kory Maincent <kory.maincent@bootlin.com>

Enable host package, needed for the build of specific optee-os package
version.

Signed-off-by: Kory Maincent <kory.maincent@bootlin.com>
---
 package/python-pillow/python-pillow.mk | 1 +
 1 file changed, 1 insertion(+)

diff --git a/package/python-pillow/python-pillow.mk b/package/python-pillow/python-pillow.mk
index 1105326799..2abe5e04ef 100644
--- a/package/python-pillow/python-pillow.mk
+++ b/package/python-pillow/python-pillow.mk
@@ -72,3 +72,4 @@ PYTHON_PILLOW_BUILD_OPTS += --disable-webp --disable-webpmux
 endif
 
 $(eval $(python-package))
+$(eval $(host-python-package))
-- 
2.25.1

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

[Buildroot] [PATCH 2/2] boot/optee-os: add support to build with python-pillow

[Kory Maincent via buildroot]

From: Kory Maincent <kory.maincent@bootlin.com>

SOC constructor version of optee-os may uses python-pillow to build the
Trusted User Interface feature.
Add support to build these specific versions by adding a new
BR2_TARGET_OPTEE_OS_NEEDS_PYTHON_PILLOW option which will
select host-python-pillow dependency.

Signed-off-by: Kory Maincent <kory.maincent@bootlin.com>
---
 boot/optee-os/Config.in   | 7 +++++++
 boot/optee-os/optee-os.mk | 4 ++++
 2 files changed, 11 insertions(+)

diff --git a/boot/optee-os/Config.in b/boot/optee-os/Config.in
index fb8a189a75..b459393191 100644
--- a/boot/optee-os/Config.in
+++ b/boot/optee-os/Config.in
@@ -72,6 +72,13 @@ config BR2_TARGET_OPTEE_OS_NEEDS_PYTHON_CRYPTOGRAPHY
 	  python-cryptography. Select this option if optee-os needs
 	  python-cryptography to be built.
 
+config BR2_TARGET_OPTEE_OS_NEEDS_PYTHON_PILLOW
+	bool "OP-TEE OS needs host-python-pillow"
+	help
+	  OP-TEE OS version from SOC manufacturer may use python-pillow
+	  to build the Trusted User Interface feature.
+	  Select this option if optee-os need python-pillow to be built.
+
 config BR2_TARGET_OPTEE_OS_CORE
 	bool "Build core"
 	default y
diff --git a/boot/optee-os/optee-os.mk b/boot/optee-os/optee-os.mk
index 5313a1badc..88f14b48e4 100644
--- a/boot/optee-os/optee-os.mk
+++ b/boot/optee-os/optee-os.mk
@@ -29,6 +29,10 @@ else
 OPTEE_OS_DEPENDENCIES += host-python-pycryptodomex
 endif
 
+ifeq ($(BR2_TARGET_OPTEE_OS_NEEDS_PYTHON_PILLOW),y)
+OPTEE_OS_DEPENDENCIES += host-python-pillow
+endif
+
 ifeq ($(BR2_TARGET_OPTEE_OS_NEEDS_DTC),y)
 OPTEE_OS_DEPENDENCIES += host-dtc
 endif
-- 
2.25.1

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH 2/2] boot/optee-os: add support to build with python-pillow

[Etienne Carriere]

Hello Kory,

On Thu, 2 Jun 2022 at 11:00, <kory.maincent@bootlin.com> wrote:
>
> From: Kory Maincent <kory.maincent@bootlin.com>
>
> SOC constructor version of optee-os may uses python-pillow to build the

s/uses/use/

> Trusted User Interface feature.
> Add support to build these specific versions by adding a new
> BR2_TARGET_OPTEE_OS_NEEDS_PYTHON_PILLOW option which will
> select host-python-pillow dependency.
>
> Signed-off-by: Kory Maincent <kory.maincent@bootlin.com>
> ---

LGTM (Acked-by: Etienne Carriere <etienne.carriere@linaro.org>) with 2
minor spelling issues.



>  boot/optee-os/Config.in   | 7 +++++++
>  boot/optee-os/optee-os.mk | 4 ++++
>  2 files changed, 11 insertions(+)
>
> diff --git a/boot/optee-os/Config.in b/boot/optee-os/Config.in
> index fb8a189a75..b459393191 100644
> --- a/boot/optee-os/Config.in
> +++ b/boot/optee-os/Config.in
> @@ -72,6 +72,13 @@ config BR2_TARGET_OPTEE_OS_NEEDS_PYTHON_CRYPTOGRAPHY
>           python-cryptography. Select this option if optee-os needs
>           python-cryptography to be built.
>
> +config BR2_TARGET_OPTEE_OS_NEEDS_PYTHON_PILLOW
> +       bool "OP-TEE OS needs host-python-pillow"
> +       help
> +         OP-TEE OS version from SOC manufacturer may use python-pillow
> +         to build the Trusted User Interface feature.
> +         Select this option if optee-os need python-pillow to be built.

s/need/needs/

> +
>  config BR2_TARGET_OPTEE_OS_CORE
>         bool "Build core"
>         default y
> diff --git a/boot/optee-os/optee-os.mk b/boot/optee-os/optee-os.mk
> index 5313a1badc..88f14b48e4 100644
> --- a/boot/optee-os/optee-os.mk
> +++ b/boot/optee-os/optee-os.mk
> @@ -29,6 +29,10 @@ else
>  OPTEE_OS_DEPENDENCIES += host-python-pycryptodomex
>  endif
>
> +ifeq ($(BR2_TARGET_OPTEE_OS_NEEDS_PYTHON_PILLOW),y)
> +OPTEE_OS_DEPENDENCIES += host-python-pillow
> +endif
> +
>  ifeq ($(BR2_TARGET_OPTEE_OS_NEEDS_DTC),y)
>  OPTEE_OS_DEPENDENCIES += host-dtc
>  endif
> --
> 2.25.1
>
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH 1/2] package/python-pillow: enable host package

[Thomas Petazzoni via buildroot]

On Thu,  2 Jun 2022 11:00:11 +0200
Kory Maincent via buildroot <buildroot@buildroot.org> wrote:

> From: Kory Maincent <kory.maincent@bootlin.com>
> 
> Enable host package, needed for the build of specific optee-os package
> version.
> 
> Signed-off-by: Kory Maincent <kory.maincent@bootlin.com>
> ---
>  package/python-pillow/python-pillow.mk | 1 +
>  1 file changed, 1 insertion(+)

Both applied to master, with improved commit logs, and an improved
Config.in help text on PATCH 2/2.

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot