* [PATCH] hafnium: Fix build when _Noreturn is not available
@ 2022-04-20 5:42 Khem Raj
2022-04-20 8:32 ` [meta-arm] " Ross Burton
2022-05-11 15:41 ` Jon Mason
0 siblings, 2 replies; 4+ messages in thread
From: Khem Raj @ 2022-04-20 5:42 UTC (permalink / raw)
To: meta-arm; +Cc: Khem Raj, Jon Mason, Ross Burton
Signed-off-by: Khem Raj <raj.khem@gmail.com>
Cc: Jon Mason <jon.mason@arm.com>
Cc: Ross Burton <ross.burton@arm.com>
---
.../0001-define-_Noreturn-if-needed.patch | 33 +++++++++++++++++++
meta-arm/recipes-bsp/hafnium/hafnium_2.6.bb | 1 +
2 files changed, 34 insertions(+)
create mode 100644 meta-arm/recipes-bsp/hafnium/hafnium/0001-define-_Noreturn-if-needed.patch
diff --git a/meta-arm/recipes-bsp/hafnium/hafnium/0001-define-_Noreturn-if-needed.patch b/meta-arm/recipes-bsp/hafnium/hafnium/0001-define-_Noreturn-if-needed.patch
new file mode 100644
index 0000000..b73c533
--- /dev/null
+++ b/meta-arm/recipes-bsp/hafnium/hafnium/0001-define-_Noreturn-if-needed.patch
@@ -0,0 +1,33 @@
+From 0d941ba32a082023575fd0d14d52a12b7547b367 Mon Sep 17 00:00:00 2001
+From: Khem Raj <raj.khem@gmail.com>
+Date: Tue, 19 Apr 2022 22:32:56 -0700
+Subject: [PATCH] define _Noreturn if needed
+
+The new _Noreturn function specifier is not recognized by the parser and shows as a syntax error:
+
+Fixes
+../git/inc/hf/panic.h:13:1: error: '_Noreturn' is a C11 extension [-Werror,-Wc11-extensions]
+noreturn void panic(const char *fmt, ...);
+^
+
+Signed-off-by: Khem Raj <raj.khem@gmail.com>
+---
+ inc/hf/panic.h | 4 ++++
+ 1 file changed, 4 insertions(+)
+
+diff --git a/inc/hf/panic.h b/inc/hf/panic.h
+index ec864e4f..588f1193 100644
+--- a/inc/hf/panic.h
++++ b/inc/hf/panic.h
+@@ -10,4 +10,8 @@
+
+ #include <stdnoreturn.h>
+
++#ifndef _Noreturn
++#define _Noreturn __attribute__ ((noreturn))
++#endif
++
+ noreturn void panic(const char *fmt, ...);
+--
+2.36.0
+
diff --git a/meta-arm/recipes-bsp/hafnium/hafnium_2.6.bb b/meta-arm/recipes-bsp/hafnium/hafnium_2.6.bb
index 96e455f..a9f41b9 100644
--- a/meta-arm/recipes-bsp/hafnium/hafnium_2.6.bb
+++ b/meta-arm/recipes-bsp/hafnium/hafnium_2.6.bb
@@ -10,6 +10,7 @@ PACKAGE_ARCH = "${MACHINE_ARCH}"
inherit deploy python3native pkgconfig
SRC_URI = "gitsm://git.trustedfirmware.org/hafnium/hafnium.git;protocol=https;branch=master \
+ file://0001-define-_Noreturn-if-needed.patch \
file://host-ld.patch \
file://pkg-config-native.patch \
file://native-dtc.patch"
--
2.36.0
^ permalink raw reply related [flat|nested] 4+ messages in thread* Re: [meta-arm] [PATCH] hafnium: Fix build when _Noreturn is not available
2022-04-20 5:42 [PATCH] hafnium: Fix build when _Noreturn is not available Khem Raj
@ 2022-04-20 8:32 ` Ross Burton
2022-05-10 22:03 ` Khem Raj
2022-05-11 15:41 ` Jon Mason
1 sibling, 1 reply; 4+ messages in thread
From: Ross Burton @ 2022-04-20 8:32 UTC (permalink / raw)
To: Khem Raj; +Cc: meta-arm, Jon Mason, Ross Burton
Missing Upstream-Status.
However, I'd prefer to understand what the actual problem is first, as
this feels like a workaround not a fix.
Ross
On Wed, 20 Apr 2022 at 06:42, Khem Raj <raj.khem@gmail.com> wrote:
>
> Signed-off-by: Khem Raj <raj.khem@gmail.com>
> Cc: Jon Mason <jon.mason@arm.com>
> Cc: Ross Burton <ross.burton@arm.com>
> ---
> .../0001-define-_Noreturn-if-needed.patch | 33 +++++++++++++++++++
> meta-arm/recipes-bsp/hafnium/hafnium_2.6.bb | 1 +
> 2 files changed, 34 insertions(+)
> create mode 100644 meta-arm/recipes-bsp/hafnium/hafnium/0001-define-_Noreturn-if-needed.patch
>
> diff --git a/meta-arm/recipes-bsp/hafnium/hafnium/0001-define-_Noreturn-if-needed.patch b/meta-arm/recipes-bsp/hafnium/hafnium/0001-define-_Noreturn-if-needed.patch
> new file mode 100644
> index 0000000..b73c533
> --- /dev/null
> +++ b/meta-arm/recipes-bsp/hafnium/hafnium/0001-define-_Noreturn-if-needed.patch
> @@ -0,0 +1,33 @@
> +From 0d941ba32a082023575fd0d14d52a12b7547b367 Mon Sep 17 00:00:00 2001
> +From: Khem Raj <raj.khem@gmail.com>
> +Date: Tue, 19 Apr 2022 22:32:56 -0700
> +Subject: [PATCH] define _Noreturn if needed
> +
> +The new _Noreturn function specifier is not recognized by the parser and shows as a syntax error:
> +
> +Fixes
> +../git/inc/hf/panic.h:13:1: error: '_Noreturn' is a C11 extension [-Werror,-Wc11-extensions]
> +noreturn void panic(const char *fmt, ...);
> +^
> +
> +Signed-off-by: Khem Raj <raj.khem@gmail.com>
> +---
> + inc/hf/panic.h | 4 ++++
> + 1 file changed, 4 insertions(+)
> +
> +diff --git a/inc/hf/panic.h b/inc/hf/panic.h
> +index ec864e4f..588f1193 100644
> +--- a/inc/hf/panic.h
> ++++ b/inc/hf/panic.h
> +@@ -10,4 +10,8 @@
> +
> + #include <stdnoreturn.h>
> +
> ++#ifndef _Noreturn
> ++#define _Noreturn __attribute__ ((noreturn))
> ++#endif
> ++
> + noreturn void panic(const char *fmt, ...);
> +--
> +2.36.0
> +
> diff --git a/meta-arm/recipes-bsp/hafnium/hafnium_2.6.bb b/meta-arm/recipes-bsp/hafnium/hafnium_2.6.bb
> index 96e455f..a9f41b9 100644
> --- a/meta-arm/recipes-bsp/hafnium/hafnium_2.6.bb
> +++ b/meta-arm/recipes-bsp/hafnium/hafnium_2.6.bb
> @@ -10,6 +10,7 @@ PACKAGE_ARCH = "${MACHINE_ARCH}"
> inherit deploy python3native pkgconfig
>
> SRC_URI = "gitsm://git.trustedfirmware.org/hafnium/hafnium.git;protocol=https;branch=master \
> + file://0001-define-_Noreturn-if-needed.patch \
> file://host-ld.patch \
> file://pkg-config-native.patch \
> file://native-dtc.patch"
> --
> 2.36.0
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#3278): https://lists.yoctoproject.org/g/meta-arm/message/3278
> Mute This Topic: https://lists.yoctoproject.org/mt/90578787/1676615
> Group Owner: meta-arm+owner@lists.yoctoproject.org
> Unsubscribe: https://lists.yoctoproject.org/g/meta-arm/unsub [ross@burtonini.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [meta-arm] [PATCH] hafnium: Fix build when _Noreturn is not available
2022-04-20 8:32 ` [meta-arm] " Ross Burton
@ 2022-05-10 22:03 ` Khem Raj
0 siblings, 0 replies; 4+ messages in thread
From: Khem Raj @ 2022-05-10 22:03 UTC (permalink / raw)
To: Ross Burton; +Cc: meta-arm, Jon Mason, Ross Burton
On Wed, Apr 20, 2022 at 1:33 AM Ross Burton <ross@burtonini.com> wrote:
>
> Missing Upstream-Status.
>
> However, I'd prefer to understand what the actual problem is first, as
> this feels like a workaround not a fix.
ping, now that gcc12 is in master, it should be reproducible for you.
>
> Ross
>
> On Wed, 20 Apr 2022 at 06:42, Khem Raj <raj.khem@gmail.com> wrote:
> >
> > Signed-off-by: Khem Raj <raj.khem@gmail.com>
> > Cc: Jon Mason <jon.mason@arm.com>
> > Cc: Ross Burton <ross.burton@arm.com>
> > ---
> > .../0001-define-_Noreturn-if-needed.patch | 33 +++++++++++++++++++
> > meta-arm/recipes-bsp/hafnium/hafnium_2.6.bb | 1 +
> > 2 files changed, 34 insertions(+)
> > create mode 100644 meta-arm/recipes-bsp/hafnium/hafnium/0001-define-_Noreturn-if-needed.patch
> >
> > diff --git a/meta-arm/recipes-bsp/hafnium/hafnium/0001-define-_Noreturn-if-needed.patch b/meta-arm/recipes-bsp/hafnium/hafnium/0001-define-_Noreturn-if-needed.patch
> > new file mode 100644
> > index 0000000..b73c533
> > --- /dev/null
> > +++ b/meta-arm/recipes-bsp/hafnium/hafnium/0001-define-_Noreturn-if-needed.patch
> > @@ -0,0 +1,33 @@
> > +From 0d941ba32a082023575fd0d14d52a12b7547b367 Mon Sep 17 00:00:00 2001
> > +From: Khem Raj <raj.khem@gmail.com>
> > +Date: Tue, 19 Apr 2022 22:32:56 -0700
> > +Subject: [PATCH] define _Noreturn if needed
> > +
> > +The new _Noreturn function specifier is not recognized by the parser and shows as a syntax error:
> > +
> > +Fixes
> > +../git/inc/hf/panic.h:13:1: error: '_Noreturn' is a C11 extension [-Werror,-Wc11-extensions]
> > +noreturn void panic(const char *fmt, ...);
> > +^
> > +
> > +Signed-off-by: Khem Raj <raj.khem@gmail.com>
> > +---
> > + inc/hf/panic.h | 4 ++++
> > + 1 file changed, 4 insertions(+)
> > +
> > +diff --git a/inc/hf/panic.h b/inc/hf/panic.h
> > +index ec864e4f..588f1193 100644
> > +--- a/inc/hf/panic.h
> > ++++ b/inc/hf/panic.h
> > +@@ -10,4 +10,8 @@
> > +
> > + #include <stdnoreturn.h>
> > +
> > ++#ifndef _Noreturn
> > ++#define _Noreturn __attribute__ ((noreturn))
> > ++#endif
> > ++
> > + noreturn void panic(const char *fmt, ...);
> > +--
> > +2.36.0
> > +
> > diff --git a/meta-arm/recipes-bsp/hafnium/hafnium_2.6.bb b/meta-arm/recipes-bsp/hafnium/hafnium_2.6.bb
> > index 96e455f..a9f41b9 100644
> > --- a/meta-arm/recipes-bsp/hafnium/hafnium_2.6.bb
> > +++ b/meta-arm/recipes-bsp/hafnium/hafnium_2.6.bb
> > @@ -10,6 +10,7 @@ PACKAGE_ARCH = "${MACHINE_ARCH}"
> > inherit deploy python3native pkgconfig
> >
> > SRC_URI = "gitsm://git.trustedfirmware.org/hafnium/hafnium.git;protocol=https;branch=master \
> > + file://0001-define-_Noreturn-if-needed.patch \
> > file://host-ld.patch \
> > file://pkg-config-native.patch \
> > file://native-dtc.patch"
> > --
> > 2.36.0
> >
> >
> > -=-=-=-=-=-=-=-=-=-=-=-
> > Links: You receive all messages sent to this group.
> > View/Reply Online (#3278): https://lists.yoctoproject.org/g/meta-arm/message/3278
> > Mute This Topic: https://lists.yoctoproject.org/mt/90578787/1676615
> > Group Owner: meta-arm+owner@lists.yoctoproject.org
> > Unsubscribe: https://lists.yoctoproject.org/g/meta-arm/unsub [ross@burtonini.com]
> > -=-=-=-=-=-=-=-=-=-=-=-
> >
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] hafnium: Fix build when _Noreturn is not available
2022-04-20 5:42 [PATCH] hafnium: Fix build when _Noreturn is not available Khem Raj
2022-04-20 8:32 ` [meta-arm] " Ross Burton
@ 2022-05-11 15:41 ` Jon Mason
1 sibling, 0 replies; 4+ messages in thread
From: Jon Mason @ 2022-05-11 15:41 UTC (permalink / raw)
To: meta-arm, Khem Raj; +Cc: Ross Burton
On Tue, 19 Apr 2022 22:42:35 -0700, Khem Raj wrote:
>
Applied, thanks!
[1/1] hafnium: Fix build when _Noreturn is not available
commit: cb2eb8e3736be072735d35bb69f0e3be8432843c
Best regards,
--
Jon Mason <jon.mason@arm.com>
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2022-05-11 15:46 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-04-20 5:42 [PATCH] hafnium: Fix build when _Noreturn is not available Khem Raj
2022-04-20 8:32 ` [meta-arm] " Ross Burton
2022-05-10 22:03 ` Khem Raj
2022-05-11 15:41 ` Jon Mason
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.