[<prev] [next>] [day] [month] [year] [list]
Message-ID: <20231019164844.4cc37f93@canb.auug.org.au>
Date: Thu, 19 Oct 2023 16:48:44 +1100
From: Stephen Rothwell <sfr@...b.auug.org.au>
To: Yury Norov <yury.norov@...il.com>, David Miller <davem@...emloft.net>,
Jakub Kicinski <kuba@...nel.org>, Paolo Abeni <pabeni@...hat.com>
Cc: Networking <netdev@...r.kernel.org>, Alexander Lobakin
<aleksander.lobakin@...el.com>, Linux Kernel Mailing List
<linux-kernel@...r.kernel.org>, Linux Next Mailing List
<linux-next@...r.kernel.org>, Paul Greenwalt <paul.greenwalt@...el.com>,
Pawel Chmielewski <pawel.chmielewski@...el.com>
Subject: linux-next: manual merge of the bitmap tree with the net-next tree
Hi all,
Today's linux-next merge of the bitmap tree got a conflict in:
include/linux/linkmode.h
between commit:
26c5334d344d ("ethtool: Add forced speed to supported link modes maps")
from the net-next tree and commit:
f849608560af ("linkmode: convert linkmode_{test,set,clear,mod}_bit() to macros")
from the bitmap tree.
I fixed it up (see below) and can carry the fix as necessary. This
is now fixed as far as linux-next is concerned, but any non trivial
conflicts should be mentioned to your upstream maintainer when your tree
is submitted for merging. You may also want to consider cooperating
with the maintainer of the conflicting tree to minimise any particularly
complex conflicts.
--
Cheers,
Stephen Rothwell
diff --cc include/linux/linkmode.h
index cd38f89553e6,f231e2edbfa5..000000000000
--- a/include/linux/linkmode.h
+++ b/include/linux/linkmode.h
@@@ -2,21 -2,6 +2,18 @@@
#define __LINKMODE_H
#include <linux/bitmap.h>
+
- static inline void linkmode_set_bit(int nr, volatile unsigned long *addr)
- {
- __set_bit(nr, addr);
- }
++#define linkmode_set_bit __set_bit
+
+static inline void linkmode_set_bit_array(const int *array, int array_size,
+ unsigned long *addr)
+{
+ int i;
+
+ for (i = 0; i < array_size; i++)
+ linkmode_set_bit(array[i], addr);
+}
+
#include <linux/ethtool.h>
#include <uapi/linux/ethtool.h>
@@@ -53,25 -38,20 +50,10 @@@ static inline int linkmode_andnot(unsig
return bitmap_andnot(dst, src1, src2, __ETHTOOL_LINK_MODE_MASK_NBITS);
}
- static inline void linkmode_clear_bit(int nr, volatile unsigned long *addr)
- {
- __clear_bit(nr, addr);
- }
-
- static inline void linkmode_mod_bit(int nr, volatile unsigned long *addr,
- int set)
- {
- if (set)
- linkmode_set_bit(nr, addr);
- else
- linkmode_clear_bit(nr, addr);
- }
-
- static inline int linkmode_test_bit(int nr, const volatile unsigned long *addr)
- {
- return test_bit(nr, addr);
- }
+ #define linkmode_test_bit test_bit
-#define linkmode_set_bit __set_bit
+ #define linkmode_clear_bit __clear_bit
+ #define linkmode_mod_bit __assign_bit
-static inline void linkmode_set_bit_array(const int *array, int array_size,
- unsigned long *addr)
-{
- int i;
-
- for (i = 0; i < array_size; i++)
- linkmode_set_bit(array[i], addr);
-}
-
static inline int linkmode_equal(const unsigned long *src1,
const unsigned long *src2)
{
Content of type "application/pgp-signature" skipped
Powered by blists - more mailing lists