Thread (6 messages) 6 messages, 5 authors, 2021-05-26

Re: [PATCH] nvme-tcp: remove incorrect Kconfig dep in BLK_DEV_NVME

From: Keith Busch <kbusch@kernel.org>
Date: 2021-05-24 21:37:05

On Fri, May 21, 2021 at 02:51:15PM -0700, Sagi Grimberg wrote:
We need to select NVME_CORE.
This looks fine and matches what the other drivers are doing, but I
think a more appropriate dependency should have NVME_FABRICS select
NVME_CORE.
quoted hunk ↗ jump to hunk
Signed-off-by: Sagi Grimberg <sagi@grimberg.me>
---
 drivers/nvme/host/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/nvme/host/Kconfig b/drivers/nvme/host/Kconfig
index a44d49d63968..c0826b3c617c 100644
--- a/drivers/nvme/host/Kconfig
+++ b/drivers/nvme/host/Kconfig
@@ -71,7 +71,7 @@ config NVME_FC
 config NVME_TCP
 	tristate "NVM Express over Fabrics TCP host driver"
 	depends on INET
-	depends on BLK_DEV_NVME
+	select NVME_CORE
 	select NVME_FABRICS
 	select CRYPTO
 	select CRYPTO_CRC32C
-- 
2.27.0
_______________________________________________
Linux-nvme mailing list
Linux-nvme@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-nvme
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help