From: Serge Semin <Sergey.Semin@baikalelectronics.ru> To: Mark Brown <broonie@kernel.org>, Ramil Zaripov <Ramil.Zaripov@baikalelectronics.ru> Cc: Serge Semin <Sergey.Semin@baikalelectronics.ru>, Serge Semin <fancer.lancer@gmail.com>, kernel test robot <lkp@intel.com>, Randy Dunlap <rdunlap@infradead.org>, <linux-spi@vger.kernel.org>, <linux-kernel@vger.kernel.org> Subject: [PATCH v2] spi: dw-bt1: Fix undefined devm_mux_control_get symbol Date: Fri, 27 Nov 2020 17:46:11 +0300 [thread overview] Message-ID: <20201127144612.4204-1-Sergey.Semin@baikalelectronics.ru> (raw) In-Reply-To: <20201116040721.8001-1-rdunlap@infradead.org> I mistakenly added the select attributes to the SPI_DW_BT1_DIRMAP config instead of having them defined in SPI_DW_BT1. If the kernel doesn't have the MULTIPLEXER and MUX_MMIO configs manually enabled and the SPI_DW_BT1_DIRMAP config hasn't been selected, Baikal-T1 SPI device will always fail to be probed by the driver. Fix that and the error reported by the test robot: >> ld.lld: error: undefined symbol: devm_mux_control_get >>> referenced by spi-dw-bt1.c >>> spi/spi-dw-bt1.o:(dw_spi_bt1_sys_init) in archive drivers/built-in.a by moving the MULTIPLEXER/MUX_MMIO configs selection to the SPI_DW_BT1 config. Link: https://lore.kernel.org/lkml/202011161745.uYRlekse-lkp@intel.com/ Link: https://lore.kernel.org/linux-spi/20201116040721.8001-1-rdunlap@infradead.org/ Fixes: abf00907538e ("spi: dw: Add Baikal-T1 SPI Controller glue driver") Reported-by: kernel test robot <lkp@intel.com> Signed-off-by: Serge Semin <Sergey.Semin@baikalelectronics.ru> Cc: Randy Dunlap <rdunlap@infradead.org> Cc: Ramil Zaripov <Ramil.Zaripov@baikalelectronics.ru> --- Mark, I see you haven't applied the previous version of the patch yet. So please replace it with this one. Randy, sorry for resending the patch. I've just realized that your solution doesn't completely fix the problem. Yeah, the kernel build won't fail to be linked after your patch being merged, but the driver still will fail to probe the device if the MULTIPLEXER and MUX_MMIO haven't been selected for the kernel. So the select attributes need to be moved from SPI_DW_BT1_DIRMAP to the SPI_DW_BT1 config. --- drivers/spi/Kconfig | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/spi/Kconfig b/drivers/spi/Kconfig index 415d57b2057f..0707068ffe47 100644 --- a/drivers/spi/Kconfig +++ b/drivers/spi/Kconfig @@ -255,6 +255,8 @@ config SPI_DW_MMIO config SPI_DW_BT1 tristate "Baikal-T1 SPI driver for DW SPI core" depends on MIPS_BAIKAL_T1 || COMPILE_TEST + select MULTIPLEXER + select MUX_MMIO help Baikal-T1 SoC is equipped with three DW APB SSI-based MMIO SPI controllers. Two of them are pretty much normal: with IRQ, DMA, @@ -268,8 +270,6 @@ config SPI_DW_BT1 config SPI_DW_BT1_DIRMAP bool "Directly mapped Baikal-T1 Boot SPI flash support" depends on SPI_DW_BT1 - select MULTIPLEXER - select MUX_MMIO help Directly mapped SPI flash memory is an interface specific to the Baikal-T1 System Boot Controller. It is a 16MB MMIO region, which -- 2.29.2
next prev parent reply other threads:[~2020-11-27 14:53 UTC|newest] Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-11-16 4:07 [PATCH] spi: dw: fix build error by selecting MULTIPLEXER Randy Dunlap 2020-11-16 19:52 ` Serge Semin 2020-11-27 14:46 ` Serge Semin [this message] 2020-11-27 16:20 ` [PATCH v2] spi: dw-bt1: Fix undefined devm_mux_control_get symbol Mark Brown
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=20201127144612.4204-1-Sergey.Semin@baikalelectronics.ru \ --to=sergey.semin@baikalelectronics.ru \ --cc=Ramil.Zaripov@baikalelectronics.ru \ --cc=broonie@kernel.org \ --cc=fancer.lancer@gmail.com \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-spi@vger.kernel.org \ --cc=lkp@intel.com \ --cc=rdunlap@infradead.org \ --subject='Re: [PATCH v2] spi: dw-bt1: Fix undefined devm_mux_control_get symbol' \ /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
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).