From: Jeremy Kerr <jk@codeconstruct.com.au>
To: openembedded-devel@lists.openembedded.org
Subject: [meta-networking][PATCH 1/2] mctp: upgrade to 18e45b42
Date: Thu, 16 Dec 2021 16:31:14 +0800 [thread overview]
Message-ID: <20211216083115.417282-1-jk@codeconstruct.com.au> (raw)
We now have better handling of local stack configuration at runtime, and
the 5.15 kernel header change has been integrated.
Signed-off-by: Jeremy Kerr <jk@codeconstruct.com.au>
---
.../0001-build-Adjust-for-kernel-mctp.h.patch | 84 -------------------
.../recipes-support/mctp/mctp_git.bb | 6 +-
2 files changed, 2 insertions(+), 88 deletions(-)
delete mode 100644 meta-networking/recipes-support/mctp/mctp/0001-build-Adjust-for-kernel-mctp.h.patch
diff --git a/meta-networking/recipes-support/mctp/mctp/0001-build-Adjust-for-kernel-mctp.h.patch b/meta-networking/recipes-support/mctp/mctp/0001-build-Adjust-for-kernel-mctp.h.patch
deleted file mode 100644
index 83bf19770..000000000
--- a/meta-networking/recipes-support/mctp/mctp/0001-build-Adjust-for-kernel-mctp.h.patch
+++ /dev/null
@@ -1,84 +0,0 @@
-From 653eccc918ef993186c312694c288b2a5b39248e Mon Sep 17 00:00:00 2001
-From: Khem Raj <raj.khem@gmail.com>
-Date: Fri, 5 Nov 2021 13:14:55 -0700
-Subject: [PATCH] build: Adjust for kernel mctp.h
-
-Starting 5.15 kernel has mctp.h as well and provides some of the
-defintions, therefore move the ones we need to provide in this header
-outside the conditional check so its always provided regardless of
-header coming from kernel or not.
-
-Include linux/if_link.h so it can pre-define IFLA_MCTP_UNSPEC and other
-defines and there is a check for conditionally providing them here but
-kernel does not have such a check, so if linux/if_link.h is included
-after this header then we end up with duplicate defines
-
-linux/if_link.h:1267:2: error: redefinition of enumerator 'IFLA_MCTP_UNSPEC'
- IFLA_MCTP_UNSPEC,
- ^
-../git/src/mctp.h:74:2: note: previous definition is here
- IFLA_MCTP_UNSPEC,
- ^
-
-Upstream-Status: Submitted [https://github.com/CodeConstruct/mctp/pull/2]
-Signed-off-by: Khem Raj <raj.khem@gmail.com>
----
- src/mctp.h | 21 +++++++++++----------
- 1 file changed, 11 insertions(+), 10 deletions(-)
-
-diff --git a/src/mctp.h b/src/mctp.h
-index 4f73def..e602168 100644
---- a/src/mctp.h
-+++ b/src/mctp.h
-@@ -19,6 +19,7 @@
- #if HAVE_LINUX_MCTP_H
- #include <linux/mctp.h>
- #endif
-+#include <linux/if_link.h>
-
- #ifndef AF_MCTP
- #define AF_MCTP 45
-@@ -33,10 +34,6 @@
-
- #ifndef MCTP_NET_ANY
-
--#ifndef MAX_ADDR_LEN
--#define MAX_ADDR_LEN 32
--#endif
--
- typedef uint8_t mctp_eid_t;
-
- struct mctp_addr {
-@@ -53,6 +50,16 @@ struct sockaddr_mctp {
- uint8_t __smctp_pad1;
- };
-
-+#define MCTP_NET_ANY 0
-+#define MCTP_ADDR_ANY 0xff
-+#define MCTP_TAG_OWNER 0x08
-+
-+#endif /* MCTP_NET_ANY */
-+
-+#ifndef MAX_ADDR_LEN
-+#define MAX_ADDR_LEN 32
-+#endif
-+
- struct sockaddr_mctp_ext {
- struct sockaddr_mctp smctp_base;
- int smctp_ifindex;
-@@ -61,12 +68,6 @@ struct sockaddr_mctp_ext {
- unsigned char smctp_haddr[MAX_ADDR_LEN];
- };
-
--#define MCTP_NET_ANY 0
--#define MCTP_ADDR_ANY 0xff
--#define MCTP_TAG_OWNER 0x08
--
--#endif /* MCTP_NET_ANY */
--
- #ifndef IFLA_MCTP_MAX
-
- /* From if_link.h */
---
-2.33.1
-
diff --git a/meta-networking/recipes-support/mctp/mctp_git.bb b/meta-networking/recipes-support/mctp/mctp_git.bb
index 50d84d204..6cc5b1480 100644
--- a/meta-networking/recipes-support/mctp/mctp_git.bb
+++ b/meta-networking/recipes-support/mctp/mctp_git.bb
@@ -7,11 +7,9 @@ LIC_FILES_CHKSUM = "file://LICENSE;md5=4cc91856b08b094b4f406a29dc61db21"
PV = "0.1+git${SRCPV}"
-SRCREV = "072bafe725c50329f99cf9d3b2624e8799e8163a"
+SRCREV = "18e45b42e672a652c067cbf66098df9022103f23"
-SRC_URI = "git://github.com/CodeConstruct/mctp;branch=main;protocol=https \
- file://0001-build-Adjust-for-kernel-mctp.h.patch \
- "
+SRC_URI = "git://github.com/CodeConstruct/mctp;branch=main;protocol=https"
S = "${WORKDIR}/git"
--
2.33.0
next reply other threads:[~2021-12-16 8:32 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-12-16 8:31 Jeremy Kerr [this message]
2021-12-16 8:31 ` [meta-networking][PATCH 2/2] mctp: install dbus service configuration Jeremy Kerr
2021-12-16 17:52 ` [oe] " Khem Raj
2021-12-17 1:57 ` Jeremy Kerr
2021-12-17 2:28 ` Khem Raj
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20211216083115.417282-1-jk@codeconstruct.com.au \
--to=jk@codeconstruct.com.au \
--cc=openembedded-devel@lists.openembedded.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.