All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 1/7] qed: select CONFIG_CRC32
@ 2021-01-03 21:36 Arnd Bergmann
  2021-01-03 21:36 ` [PATCH 2/7] phy: dp83640: " Arnd Bergmann
                   ` (5 more replies)
  0 siblings, 6 replies; 12+ messages in thread
From: Arnd Bergmann @ 2021-01-03 21:36 UTC (permalink / raw)
  To: David S. Miller, Jakub Kicinski, Mintz, Yuval
  Cc: Arnd Bergmann, netdev, linux-kernel

From: Arnd Bergmann <arnd@arndb.de>

Without this, the driver fails to link:

lpc_eth.c:(.text+0x1934): undefined reference to `crc32_le'
arm-linux-gnueabi-ld: drivers/net/ethernet/qlogic/qed/qed_debug.o: in function `qed_grc_dump':
qed_debug.c:(.text+0x4068): undefined reference to `crc32_le'
arm-linux-gnueabi-ld: drivers/net/ethernet/qlogic/qed/qed_debug.o: in function `qed_idle_chk_dump':
qed_debug.c:(.text+0x51fc): undefined reference to `crc32_le'
arm-linux-gnueabi-ld: drivers/net/ethernet/qlogic/qed/qed_debug.o: in function `qed_mcp_trace_dump':
qed_debug.c:(.text+0x6000): undefined reference to `crc32_le'
arm-linux-gnueabi-ld: drivers/net/ethernet/qlogic/qed/qed_debug.o: in function `qed_dbg_reg_fifo_dump':
qed_debug.c:(.text+0x66cc): undefined reference to `crc32_le'
arm-linux-gnueabi-ld: drivers/net/ethernet/qlogic/qed/qed_debug.o:qed_debug.c:(.text+0x6aa4): more undefined references to `crc32_le' follow

Fixes: 7a4b21b7d1f0 ("qed: Add nvram selftest")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
---
 drivers/net/ethernet/qlogic/Kconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/net/ethernet/qlogic/Kconfig b/drivers/net/ethernet/qlogic/Kconfig
index 4366c7a8de95..6b5ddb07ee83 100644
--- a/drivers/net/ethernet/qlogic/Kconfig
+++ b/drivers/net/ethernet/qlogic/Kconfig
@@ -78,6 +78,7 @@ config QED
 	depends on PCI
 	select ZLIB_INFLATE
 	select CRC8
+	select CRC32
 	select NET_DEVLINK
 	help
 	  This enables the support for Marvell FastLinQ adapters family.
-- 
2.29.2


^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [PATCH 2/7] phy: dp83640: select CONFIG_CRC32
  2021-01-03 21:36 [PATCH 1/7] qed: select CONFIG_CRC32 Arnd Bergmann
@ 2021-01-03 21:36 ` Arnd Bergmann
  2021-01-03 23:09   ` Andrew Lunn
  2021-01-04 13:03   ` Richard Cochran
  2021-01-03 21:36 ` [PATCH 3/7] can: kvaser_pciefd: " Arnd Bergmann
                   ` (4 subsequent siblings)
  5 siblings, 2 replies; 12+ messages in thread
From: Arnd Bergmann @ 2021-01-03 21:36 UTC (permalink / raw)
  To: Richard Cochran, David S. Miller
  Cc: Arnd Bergmann, Stefan Sørensen, netdev, linux-kernel

From: Arnd Bergmann <arnd@arndb.de>

Without crc32, this driver fails to link:

arm-linux-gnueabi-ld: drivers/net/phy/dp83640.o: in function `match':
dp83640.c:(.text+0x476c): undefined reference to `crc32_le'

Fixes: 539e44d26855 ("dp83640: Include hash in timestamp/packet matching")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
---
 drivers/ptp/Kconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/ptp/Kconfig b/drivers/ptp/Kconfig
index 476d7c7fe70a..d2bf05ccbbe2 100644
--- a/drivers/ptp/Kconfig
+++ b/drivers/ptp/Kconfig
@@ -64,6 +64,7 @@ config DP83640_PHY
 	depends on NETWORK_PHY_TIMESTAMPING
 	depends on PHYLIB
 	depends on PTP_1588_CLOCK
+	select CRC32
 	help
 	  Supports the DP83640 PHYTER with IEEE 1588 features.
 
-- 
2.29.2


^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [PATCH 3/7] can: kvaser_pciefd: select CONFIG_CRC32
  2021-01-03 21:36 [PATCH 1/7] qed: select CONFIG_CRC32 Arnd Bergmann
  2021-01-03 21:36 ` [PATCH 2/7] phy: dp83640: " Arnd Bergmann
@ 2021-01-03 21:36 ` Arnd Bergmann
  2021-01-04  8:43   ` Marc Kleine-Budde
  2021-01-03 21:36 ` [PATCH 4/7] wil6210: " Arnd Bergmann
                   ` (3 subsequent siblings)
  5 siblings, 1 reply; 12+ messages in thread
From: Arnd Bergmann @ 2021-01-03 21:36 UTC (permalink / raw)
  To: Wolfgang Grandegger, Marc Kleine-Budde, David S. Miller, Jakub Kicinski
  Cc: Arnd Bergmann, Henning Colliander, Christer Beskow,
	Jimmy Assarsson, linux-can, netdev, linux-kernel

From: Arnd Bergmann <arnd@arndb.de>

Without crc32, this driver fails to link:

arm-linux-gnueabi-ld: drivers/net/can/kvaser_pciefd.o: in function `kvaser_pciefd_probe':
kvaser_pciefd.c:(.text+0x2b0): undefined reference to `crc32_be'

Fixes: 26ad340e582d ("can: kvaser_pciefd: Add driver for Kvaser PCIEcan devices")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
---
 drivers/net/can/Kconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/net/can/Kconfig b/drivers/net/can/Kconfig
index 424970939fd4..1c28eade6bec 100644
--- a/drivers/net/can/Kconfig
+++ b/drivers/net/can/Kconfig
@@ -123,6 +123,7 @@ config CAN_JANZ_ICAN3
 config CAN_KVASER_PCIEFD
 	depends on PCI
 	tristate "Kvaser PCIe FD cards"
+	select CRC32
 	  help
 	  This is a driver for the Kvaser PCI Express CAN FD family.
 
-- 
2.29.2


^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [PATCH 4/7] wil6210: select CONFIG_CRC32
  2021-01-03 21:36 [PATCH 1/7] qed: select CONFIG_CRC32 Arnd Bergmann
  2021-01-03 21:36 ` [PATCH 2/7] phy: dp83640: " Arnd Bergmann
  2021-01-03 21:36 ` [PATCH 3/7] can: kvaser_pciefd: " Arnd Bergmann
@ 2021-01-03 21:36 ` Arnd Bergmann
  2021-01-11 11:47   ` Kalle Valo
  2021-01-14 18:45   ` Kalle Valo
  2021-01-03 21:36 ` [PATCH 5/7] cfg80211: " Arnd Bergmann
                   ` (2 subsequent siblings)
  5 siblings, 2 replies; 12+ messages in thread
From: Arnd Bergmann @ 2021-01-03 21:36 UTC (permalink / raw)
  To: Maya Erez, Kalle Valo, David S. Miller, Jakub Kicinski
  Cc: Arnd Bergmann, John W. Linville, Vladimir Kondratiev,
	linux-wireless, wil6210, netdev, linux-kernel

From: Arnd Bergmann <arnd@arndb.de>

Without crc32, the driver fails to link:

arm-linux-gnueabi-ld: drivers/net/wireless/ath/wil6210/fw.o: in function `wil_fw_verify':
fw.c:(.text+0x74c): undefined reference to `crc32_le'
arm-linux-gnueabi-ld: drivers/net/wireless/ath/wil6210/fw.o:fw.c:(.text+0x758): more undefined references to `crc32_le' follow

Fixes: 151a9706503f ("wil6210: firmware download")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
---
 drivers/net/wireless/ath/wil6210/Kconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/net/wireless/ath/wil6210/Kconfig b/drivers/net/wireless/ath/wil6210/Kconfig
index 6a95b199bf62..f074e9c31aa2 100644
--- a/drivers/net/wireless/ath/wil6210/Kconfig
+++ b/drivers/net/wireless/ath/wil6210/Kconfig
@@ -2,6 +2,7 @@
 config WIL6210
 	tristate "Wilocity 60g WiFi card wil6210 support"
 	select WANT_DEV_COREDUMP
+	select CRC32
 	depends on CFG80211
 	depends on PCI
 	default n
-- 
2.29.2


^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [PATCH 5/7] cfg80211: select CONFIG_CRC32
  2021-01-03 21:36 [PATCH 1/7] qed: select CONFIG_CRC32 Arnd Bergmann
                   ` (2 preceding siblings ...)
  2021-01-03 21:36 ` [PATCH 4/7] wil6210: " Arnd Bergmann
@ 2021-01-03 21:36 ` Arnd Bergmann
  2021-01-03 21:36 ` [PATCH 6/7] misdn: dsp: select CONFIG_BITREVERSE Arnd Bergmann
  2021-01-03 21:36 ` [PATCH 7/7] wan: ds26522: " Arnd Bergmann
  5 siblings, 0 replies; 12+ messages in thread
From: Arnd Bergmann @ 2021-01-03 21:36 UTC (permalink / raw)
  To: Johannes Berg, David S. Miller, Jakub Kicinski
  Cc: Arnd Bergmann, Tova Mussai, linux-wireless, netdev, linux-kernel

From: Arnd Bergmann <arnd@arndb.de>

Without crc32 support, this fails to link:

arm-linux-gnueabi-ld: net/wireless/scan.o: in function `cfg80211_scan_6ghz':
scan.c:(.text+0x928): undefined reference to `crc32_le'

Fixes: c8cb5b854b40 ("nl80211/cfg80211: support 6 GHz scanning")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
---
 net/wireless/Kconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/net/wireless/Kconfig b/net/wireless/Kconfig
index 27026f587fa6..f620acd2a0f5 100644
--- a/net/wireless/Kconfig
+++ b/net/wireless/Kconfig
@@ -21,6 +21,7 @@ config CFG80211
 	tristate "cfg80211 - wireless configuration API"
 	depends on RFKILL || !RFKILL
 	select FW_LOADER
+	select CRC32
 	# may need to update this when certificates are changed and are
 	# using a different algorithm, though right now they shouldn't
 	# (this is here rather than below to allow it to be a module)
-- 
2.29.2


^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [PATCH 6/7] misdn: dsp: select CONFIG_BITREVERSE
  2021-01-03 21:36 [PATCH 1/7] qed: select CONFIG_CRC32 Arnd Bergmann
                   ` (3 preceding siblings ...)
  2021-01-03 21:36 ` [PATCH 5/7] cfg80211: " Arnd Bergmann
@ 2021-01-03 21:36 ` Arnd Bergmann
  2021-01-03 21:36 ` [PATCH 7/7] wan: ds26522: " Arnd Bergmann
  5 siblings, 0 replies; 12+ messages in thread
From: Arnd Bergmann @ 2021-01-03 21:36 UTC (permalink / raw)
  To: Karsten Keil; +Cc: Arnd Bergmann, netdev, linux-kernel

From: Arnd Bergmann <arnd@arndb.de>

Without this, we run into a link error

arm-linux-gnueabi-ld: drivers/isdn/mISDN/dsp_audio.o: in function `dsp_audio_generate_law_tables':
(.text+0x30c): undefined reference to `byte_rev_table'
arm-linux-gnueabi-ld: drivers/isdn/mISDN/dsp_audio.o:(.text+0x5e4): more undefined references to `byte_rev_table' follow

Signed-off-by: Arnd Bergmann <arnd@arndb.de>
---
 drivers/isdn/mISDN/Kconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/isdn/mISDN/Kconfig b/drivers/isdn/mISDN/Kconfig
index 26cf0ac9c4ad..c9a53c222472 100644
--- a/drivers/isdn/mISDN/Kconfig
+++ b/drivers/isdn/mISDN/Kconfig
@@ -13,6 +13,7 @@ if MISDN != n
 config MISDN_DSP
 	tristate "Digital Audio Processing of transparent data"
 	depends on MISDN
+	select BITREVERSE
 	help
 	  Enable support for digital audio processing capability.
 
-- 
2.29.2


^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [PATCH 7/7] wan: ds26522: select CONFIG_BITREVERSE
  2021-01-03 21:36 [PATCH 1/7] qed: select CONFIG_CRC32 Arnd Bergmann
                   ` (4 preceding siblings ...)
  2021-01-03 21:36 ` [PATCH 6/7] misdn: dsp: select CONFIG_BITREVERSE Arnd Bergmann
@ 2021-01-03 21:36 ` Arnd Bergmann
  5 siblings, 0 replies; 12+ messages in thread
From: Arnd Bergmann @ 2021-01-03 21:36 UTC (permalink / raw)
  To: David S. Miller, Jakub Kicinski, Zhao Qiang
  Cc: Arnd Bergmann, Xie He, Masahiro Yamada, netdev, linux-kernel

From: Arnd Bergmann <arnd@arndb.de>

Without this, the driver runs into a link failure

arm-linux-gnueabi-ld: drivers/net/wan/slic_ds26522.o: in function `slic_ds26522_probe':
slic_ds26522.c:(.text+0x100c): undefined reference to `byte_rev_table'
arm-linux-gnueabi-ld: slic_ds26522.c:(.text+0x1cdc): undefined reference to `byte_rev_table'
arm-linux-gnueabi-ld: drivers/net/wan/slic_ds26522.o: in function `slic_write':
slic_ds26522.c:(.text+0x1e4c): undefined reference to `byte_rev_table'

Fixes: c37d4a0085c5 ("Maxim/driver: Add driver for maxim ds26522")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
---
 drivers/net/wan/Kconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/net/wan/Kconfig b/drivers/net/wan/Kconfig
index 8931ef529065..a187c814a4a6 100644
--- a/drivers/net/wan/Kconfig
+++ b/drivers/net/wan/Kconfig
@@ -282,6 +282,7 @@ config SLIC_DS26522
 	tristate "Slic Maxim ds26522 card support"
 	depends on SPI
 	depends on FSL_SOC || ARCH_MXC || ARCH_LAYERSCAPE || COMPILE_TEST
+	select BITREVERSE
 	help
 	  This module initializes and configures the slic maxim card
 	  in T1 or E1 mode.
-- 
2.29.2


^ permalink raw reply related	[flat|nested] 12+ messages in thread

* Re: [PATCH 2/7] phy: dp83640: select CONFIG_CRC32
  2021-01-03 21:36 ` [PATCH 2/7] phy: dp83640: " Arnd Bergmann
@ 2021-01-03 23:09   ` Andrew Lunn
  2021-01-04 13:03   ` Richard Cochran
  1 sibling, 0 replies; 12+ messages in thread
From: Andrew Lunn @ 2021-01-03 23:09 UTC (permalink / raw)
  To: Arnd Bergmann
  Cc: Richard Cochran, David S. Miller, Arnd Bergmann,
	Stefan Sørensen, netdev, linux-kernel

On Sun, Jan 03, 2021 at 10:36:18PM +0100, Arnd Bergmann wrote:
> From: Arnd Bergmann <arnd@arndb.de>
> 
> Without crc32, this driver fails to link:
> 
> arm-linux-gnueabi-ld: drivers/net/phy/dp83640.o: in function `match':
> dp83640.c:(.text+0x476c): undefined reference to `crc32_le'
> 
> Fixes: 539e44d26855 ("dp83640: Include hash in timestamp/packet matching")
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>

Reviewed-by: Andrew Lunn <andrew@lunn.ch>

    Andrew

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH 3/7] can: kvaser_pciefd: select CONFIG_CRC32
  2021-01-03 21:36 ` [PATCH 3/7] can: kvaser_pciefd: " Arnd Bergmann
@ 2021-01-04  8:43   ` Marc Kleine-Budde
  0 siblings, 0 replies; 12+ messages in thread
From: Marc Kleine-Budde @ 2021-01-04  8:43 UTC (permalink / raw)
  To: Arnd Bergmann, Wolfgang Grandegger, David S. Miller, Jakub Kicinski
  Cc: Arnd Bergmann, Henning Colliander, Christer Beskow,
	Jimmy Assarsson, linux-can, netdev, linux-kernel


[-- Attachment #1.1: Type: text/plain, Size: 776 bytes --]

On 1/3/21 10:36 PM, Arnd Bergmann wrote:
> From: Arnd Bergmann <arnd@arndb.de>
> 
> Without crc32, this driver fails to link:
> 
> arm-linux-gnueabi-ld: drivers/net/can/kvaser_pciefd.o: in function `kvaser_pciefd_probe':
> kvaser_pciefd.c:(.text+0x2b0): undefined reference to `crc32_be'
> 
> Fixes: 26ad340e582d ("can: kvaser_pciefd: Add driver for Kvaser PCIEcan devices")
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>

Acked-by: Marc Kleine-Budde <mkl@pengutronix.de>

regrards,
Marc

-- 
Pengutronix e.K.                 | Marc Kleine-Budde           |
Embedded Linux                   | https://www.pengutronix.de  |
Vertretung West/Dortmund         | Phone: +49-231-2826-924     |
Amtsgericht Hildesheim, HRA 2686 | Fax:   +49-5121-206917-5555 |


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH 2/7] phy: dp83640: select CONFIG_CRC32
  2021-01-03 21:36 ` [PATCH 2/7] phy: dp83640: " Arnd Bergmann
  2021-01-03 23:09   ` Andrew Lunn
@ 2021-01-04 13:03   ` Richard Cochran
  1 sibling, 0 replies; 12+ messages in thread
From: Richard Cochran @ 2021-01-04 13:03 UTC (permalink / raw)
  To: Arnd Bergmann
  Cc: David S. Miller, Arnd Bergmann, Stefan Sørensen, netdev,
	linux-kernel

On Sun, Jan 03, 2021 at 10:36:18PM +0100, Arnd Bergmann wrote:
> From: Arnd Bergmann <arnd@arndb.de>
> 
> Without crc32, this driver fails to link:
> 
> arm-linux-gnueabi-ld: drivers/net/phy/dp83640.o: in function `match':
> dp83640.c:(.text+0x476c): undefined reference to `crc32_le'
> 
> Fixes: 539e44d26855 ("dp83640: Include hash in timestamp/packet matching")
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>

Acked-by: Richard Cochran <richardcochran@gmail.com>

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH 4/7] wil6210: select CONFIG_CRC32
  2021-01-03 21:36 ` [PATCH 4/7] wil6210: " Arnd Bergmann
@ 2021-01-11 11:47   ` Kalle Valo
  2021-01-14 18:45   ` Kalle Valo
  1 sibling, 0 replies; 12+ messages in thread
From: Kalle Valo @ 2021-01-11 11:47 UTC (permalink / raw)
  To: Arnd Bergmann
  Cc: Maya Erez, David S. Miller, Jakub Kicinski, Arnd Bergmann,
	John W. Linville, Vladimir Kondratiev, linux-wireless, wil6210,
	netdev, linux-kernel

Arnd Bergmann <arnd@kernel.org> writes:

> From: Arnd Bergmann <arnd@arndb.de>
>
> Without crc32, the driver fails to link:
>
> arm-linux-gnueabi-ld: drivers/net/wireless/ath/wil6210/fw.o: in function `wil_fw_verify':
> fw.c:(.text+0x74c): undefined reference to `crc32_le'
> arm-linux-gnueabi-ld: drivers/net/wireless/ath/wil6210/fw.o:fw.c:(.text+0x758): more undefined references to `crc32_le' follow
>
> Fixes: 151a9706503f ("wil6210: firmware download")
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>

I'll queue this to v5.11.

-- 
https://patchwork.kernel.org/project/linux-wireless/list/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH 4/7] wil6210: select CONFIG_CRC32
  2021-01-03 21:36 ` [PATCH 4/7] wil6210: " Arnd Bergmann
  2021-01-11 11:47   ` Kalle Valo
@ 2021-01-14 18:45   ` Kalle Valo
  1 sibling, 0 replies; 12+ messages in thread
From: Kalle Valo @ 2021-01-14 18:45 UTC (permalink / raw)
  To: Arnd Bergmann
  Cc: Maya Erez, David S. Miller, Jakub Kicinski, Arnd Bergmann,
	John W. Linville, Vladimir Kondratiev, linux-wireless, wil6210,
	netdev, linux-kernel

Arnd Bergmann <arnd@kernel.org> wrote:

> From: Arnd Bergmann <arnd@arndb.de>
> 
> Without crc32, the driver fails to link:
> 
> arm-linux-gnueabi-ld: drivers/net/wireless/ath/wil6210/fw.o: in function `wil_fw_verify':
> fw.c:(.text+0x74c): undefined reference to `crc32_le'
> arm-linux-gnueabi-ld: drivers/net/wireless/ath/wil6210/fw.o:fw.c:(.text+0x758): more undefined references to `crc32_le' follow
> 
> Fixes: 151a9706503f ("wil6210: firmware download")
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>

Dave had already applied this so dropping from my queue.

Patch set to Not Applicable.

-- 
https://patchwork.kernel.org/project/linux-wireless/patch/20210103213645.1994783-4-arnd@kernel.org/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches


^ permalink raw reply	[flat|nested] 12+ messages in thread

end of thread, other threads:[~2021-01-14 18:46 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-01-03 21:36 [PATCH 1/7] qed: select CONFIG_CRC32 Arnd Bergmann
2021-01-03 21:36 ` [PATCH 2/7] phy: dp83640: " Arnd Bergmann
2021-01-03 23:09   ` Andrew Lunn
2021-01-04 13:03   ` Richard Cochran
2021-01-03 21:36 ` [PATCH 3/7] can: kvaser_pciefd: " Arnd Bergmann
2021-01-04  8:43   ` Marc Kleine-Budde
2021-01-03 21:36 ` [PATCH 4/7] wil6210: " Arnd Bergmann
2021-01-11 11:47   ` Kalle Valo
2021-01-14 18:45   ` Kalle Valo
2021-01-03 21:36 ` [PATCH 5/7] cfg80211: " Arnd Bergmann
2021-01-03 21:36 ` [PATCH 6/7] misdn: dsp: select CONFIG_BITREVERSE Arnd Bergmann
2021-01-03 21:36 ` [PATCH 7/7] wan: ds26522: " Arnd Bergmann

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.