diff mbox series

[2/2] simatic-ipc227e: conf: Add WKS_FILE as x86_64-generic.wks

Message ID 20231010075014.3761016-2-nobuhiro1.iwamatsu@toshiba.co.jp (mailing list archive)
State New
Headers show
Series [1/2] wic: Rename wic/simatic-ipc227e.wks to wic/x86_64-generic.wks | expand

Commit Message

Nobuhiro Iwamatsu Oct. 10, 2023, 7:50 a.m. UTC
wic/simatic-ipc227e.wks has been changed to wic/x86_64-generic.wks. This
is set in the WKS_FILE variable.
This file format is different from conf/distro/cip-core-common.inc, so we need
to set WKS_FILE as x86_64-generic.wks in simatic-ipc227e.conf.
Add WKS_FILE to simatic-ipc227e.conf.

Signed-off-by: Nobuhiro Iwamatsu <nobuhiro1.iwamatsu@toshiba.co.jp>
---
 conf/machine/simatic-ipc227e.conf | 1 +
 1 file changed, 1 insertion(+)

Comments

Jan Kiszka Oct. 10, 2023, 7:53 a.m. UTC | #1
On 10.10.23 09:50, Nobuhiro Iwamatsu wrote:
> wic/simatic-ipc227e.wks has been changed to wic/x86_64-generic.wks. This
> is set in the WKS_FILE variable.
> This file format is different from conf/distro/cip-core-common.inc, so we need
> to set WKS_FILE as x86_64-generic.wks in simatic-ipc227e.conf.
> Add WKS_FILE to simatic-ipc227e.conf.
> 
> Signed-off-by: Nobuhiro Iwamatsu <nobuhiro1.iwamatsu@toshiba.co.jp>
> ---
>  conf/machine/simatic-ipc227e.conf | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/conf/machine/simatic-ipc227e.conf b/conf/machine/simatic-ipc227e.conf
> index a624fcd..8bb3e80 100644
> --- a/conf/machine/simatic-ipc227e.conf
> +++ b/conf/machine/simatic-ipc227e.conf
> @@ -9,6 +9,7 @@
>  DISTRO_ARCH = "amd64"
>  
>  IMAGE_FSTYPES ?= "wic"
> +WKS_FILE ?= "x86_64-generic.wks"
>  IMAGER_INSTALL += "${GRUB_BOOTLOADER_INSTALL}"
>  USE_CIP_KERNEL_CONFIG = "1"
>  KERNEL_DEFCONFIG = "cip-kernel-config/${KERNEL_DEFCONFIG_VERSION}/x86/siemens_ipc227e_defconfig"

Patch 1 and 2 need to be merged, or we have a build breakage after patch
1, don't we?

Jan
Nobuhiro Iwamatsu Oct. 10, 2023, 7:57 a.m. UTC | #2
Hi!

> -----Original Message-----
> From: Jan Kiszka <jan.kiszka@siemens.com>
> Sent: Tuesday, October 10, 2023 4:53 PM
> To: iwamatsu nobuhiro(岩松 信洋 ○DITC□DIT○OST)
> <nobuhiro1.iwamatsu@toshiba.co.jp>
> Cc: cip-dev@lists.cip-project.org
> Subject: Re: [PATCH 2/2] simatic-ipc227e: conf: Add WKS_FILE as
> x86_64-generic.wks
> 
> On 10.10.23 09:50, Nobuhiro Iwamatsu wrote:
> > wic/simatic-ipc227e.wks has been changed to wic/x86_64-generic.wks.
> > This is set in the WKS_FILE variable.
> > This file format is different from conf/distro/cip-core-common.inc, so
> > we need to set WKS_FILE as x86_64-generic.wks in simatic-ipc227e.conf.
> > Add WKS_FILE to simatic-ipc227e.conf.
> >
> > Signed-off-by: Nobuhiro Iwamatsu <nobuhiro1.iwamatsu@toshiba.co.jp>
> > ---
> >  conf/machine/simatic-ipc227e.conf | 1 +
> >  1 file changed, 1 insertion(+)
> >
> > diff --git a/conf/machine/simatic-ipc227e.conf
> > b/conf/machine/simatic-ipc227e.conf
> > index a624fcd..8bb3e80 100644
> > --- a/conf/machine/simatic-ipc227e.conf
> > +++ b/conf/machine/simatic-ipc227e.conf
> > @@ -9,6 +9,7 @@
> >  DISTRO_ARCH = "amd64"
> >
> >  IMAGE_FSTYPES ?= "wic"
> > +WKS_FILE ?= "x86_64-generic.wks"
> >  IMAGER_INSTALL += "${GRUB_BOOTLOADER_INSTALL}"
> >  USE_CIP_KERNEL_CONFIG = "1"
> >  KERNEL_DEFCONFIG =
> "cip-kernel-config/${KERNEL_DEFCONFIG_VERSION}/x86/siemens_ipc227
> e_defconfig"
> 
> Patch 1 and 2 need to be merged, or we have a build breakage after patch 1,
> don't we?

Correct. I will merge this series and re-send.

Best regards,
  Nobuhiro
diff mbox series

Patch

diff --git a/conf/machine/simatic-ipc227e.conf b/conf/machine/simatic-ipc227e.conf
index a624fcd..8bb3e80 100644
--- a/conf/machine/simatic-ipc227e.conf
+++ b/conf/machine/simatic-ipc227e.conf
@@ -9,6 +9,7 @@ 
 DISTRO_ARCH = "amd64"
 
 IMAGE_FSTYPES ?= "wic"
+WKS_FILE ?= "x86_64-generic.wks"
 IMAGER_INSTALL += "${GRUB_BOOTLOADER_INSTALL}"
 USE_CIP_KERNEL_CONFIG = "1"
 KERNEL_DEFCONFIG = "cip-kernel-config/${KERNEL_DEFCONFIG_VERSION}/x86/siemens_ipc227e_defconfig"