From nobody Thu Dec 18 04:44:51 2025 Received: from michel.telenet-ops.be (michel.telenet-ops.be [195.130.137.88]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 86E9D125DF for ; Wed, 26 Mar 2025 14:22:28 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=195.130.137.88 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1742998952; cv=none; b=jFk9T+0/KGXhnwr1LbrNSQ3DMAr8/Hlkx8YMIIstkwHbUdaXFuzdGovzDCPnEvXzWeAegRecCYU5OG5FTslI//cIYlJqcF6daCUJ21ukFa+5zX2k74OpUoumQ7SmvT+aa6QCPz97yXDMVowql2oDNPx5aJ19Jt+QnBhGZi0EWMg= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1742998952; c=relaxed/simple; bh=TKxti6j94LxS0EU/52dbpDWY1Jx1BVAWz3BmQzJD2tA=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=NVIitNOWM9CBI0kGPWVjct4dZLlrrej9zghD4i7OQ656vG3lf4I7ZeRsuYG+G+88oau5zVjQIF6ljVN+vPBQXFw+vqvAtZUUBqvTyNeQXGcYMfY6Dvck3kT/RQBYwCwCbPdLv4ixib1QXHRPs3/O+SXffVKaHXjfDubfNWnsuzM= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=glider.be; spf=none smtp.mailfrom=linux-m68k.org; arc=none smtp.client-ip=195.130.137.88 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=glider.be Authentication-Results: smtp.subspace.kernel.org; spf=none smtp.mailfrom=linux-m68k.org Received: from ramsan.of.borg ([IPv6:2a02:1810:ac12:ed80:b1ef:107c:7814:6642]) by michel.telenet-ops.be with cmsmtp id VSNL2E0085Szt1p06SNLFp; Wed, 26 Mar 2025 15:22:21 +0100 Received: from rox.of.borg ([192.168.97.57]) by ramsan.of.borg with esmtp (Exim 4.97) (envelope-from ) id 1txRdf-0000000FjIr-0Skx; Wed, 26 Mar 2025 15:22:20 +0100 Received: from geert by rox.of.borg with local (Exim 4.97) (envelope-from ) id 1txRe8-0000000Gytg-2oz1; Wed, 26 Mar 2025 15:22:20 +0100 From: Geert Uytterhoeven To: Mark Brown , Sricharan Ramabadhran , Md Sadre Alam , Varadarajan Narayanan Cc: linux-spi@vger.kernel.org, linux-kernel@vger.kernel.org, Geert Uytterhoeven Subject: [PATCH] spi: SPI_QPIC_SNAND should be tristate and depend on MTD Date: Wed, 26 Mar 2025 15:22:19 +0100 Message-ID: X-Mailer: git-send-email 2.43.0 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" SPI_QPIC_SNAND is the only driver that selects MTD instead of depending on it, which could lead to circular dependencies. Moreover, as SPI_QPIC_SNAND is bool, this forces MTD (and various related symbols) to be built-in, as can be seen in an allmodconfig kernel. Except for a missing semicolon, there is no reason why SPI_QPIC_SNAND cannot be tristate; all MODULE_*() boilerplate is already present. Hence make SPI_QPIC_SNAND tristate, let it depend on MTD, and add the missing semicolon. Fixes: 7304d1909080ef0c ("spi: spi-qpic: add driver for QCOM SPI NAND flash= Interface") Signed-off-by: Geert Uytterhoeven --- Compile-tested only. --- drivers/spi/Kconfig | 4 ++-- drivers/spi/spi-qpic-snand.c | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/spi/Kconfig b/drivers/spi/Kconfig index c18bb46ed13fea9a..0121ecb13d541799 100644 --- a/drivers/spi/Kconfig +++ b/drivers/spi/Kconfig @@ -945,9 +945,9 @@ config SPI_QCOM_QSPI QSPI(Quad SPI) driver for Qualcomm QSPI controller. =20 config SPI_QPIC_SNAND - bool "QPIC SNAND controller" + tristate "QPIC SNAND controller" depends on ARCH_QCOM || COMPILE_TEST - select MTD + depends on MTD help QPIC_SNAND (QPIC SPI NAND) driver for Qualcomm QPIC controller. QPIC controller supports both parallel nand and serial nand. diff --git a/drivers/spi/spi-qpic-snand.c b/drivers/spi/spi-qpic-snand.c index fbba7741a9bf336d..17eb67e19132612c 100644 --- a/drivers/spi/spi-qpic-snand.c +++ b/drivers/spi/spi-qpic-snand.c @@ -1614,7 +1614,7 @@ static const struct of_device_id qcom_snandc_of_match= [] =3D { .data =3D &ipq9574_snandc_props, }, {} -} +}; MODULE_DEVICE_TABLE(of, qcom_snandc_of_match); =20 static struct platform_driver qcom_spi_driver =3D { --=20 2.43.0