From patchwork Thu Mar 15 15:42:29 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Arnd Bergmann X-Patchwork-Id: 131807 Delivered-To: patch@linaro.org Received: by 10.46.84.17 with SMTP id i17csp1274566ljb; Thu, 15 Mar 2018 08:43:26 -0700 (PDT) X-Google-Smtp-Source: AG47ELt2n0+69xlILbiynuOMRfqZYN+6q1gYwdchvhcU4nnSildfUyDid2J94e3UnBcoXeVtBF2B X-Received: by 10.98.211.198 with SMTP id z67mr8221032pfk.0.1521128606456; Thu, 15 Mar 2018 08:43:26 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1521128606; cv=none; d=google.com; s=arc-20160816; b=Q+ASTXai5YhlXxoHhsWkA1q9oPnLk0/LMoXuO3ygGOxB0+OC30rP+h5uIktT3P1Gqg L7FnQNHhX8vIG6EnGcmuOeMvIDYPgQKjfmGymM3NkydGiB9CMEGca4YVEMVX3S5Z0It+ uaWtmDn54QgHMHaN3uMx8nTl2qzyh7go2766NcxrPtZANr5xTHDUBqWZPkJRip2FLI4D EHxw6NU1SoK3/aQZXVd8LZeH4IaAHbVeBFKHB2N8WXcNR2iEspZq14yUrB74RwJae2mf fj/fLQcTZo8R7WrNFdltQcF+8S6PAfslmjsL8kDRo3Aq9cJE6GDPAGr+wv1ngqqjbrGi qIWw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:message-id:date:subject:cc:to:from :arc-authentication-results; bh=YR8auM2t+AqsI3Ea90RFUFlYlpJpOGY0B+baFFHP5LY=; b=TCtQe2gfnq73JtrBpdvdqfKq5pFlD4eoOvwEC+coQy7kv5R6xM8//X/ZuBmiiXCKO9 F2934oYAek4KLX3JEHnOGPvaSKJladHRybOtklt9TkNQXj+2g01IqdkjqMWqN77kuKCC Ua26pfEgT+Qp2K7r1V7UqQicur5VuAxfUs2VxWkpKit5f0lt0PN7jIcQNkMxHaJUkEuM /zQUlyLdvnSkBIjW6p5mP+ksaUlyOPmfbQNiRRmrqC285X7JUHnIbcqhTIY5shjfQo/b mdtZTOVcqUADEYswDQykOox7r7RN3+B8trw8JxifXR1R7RnX9oJmIMYHPbeQHFQuNovm 4BTg== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id g11-v6si4138478plt.580.2018.03.15.08.43.26; Thu, 15 Mar 2018 08:43:26 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932638AbeCOPnX (ORCPT + 28 others); Thu, 15 Mar 2018 11:43:23 -0400 Received: from mout.kundenserver.de ([212.227.126.131]:44115 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752693AbeCOPnT (ORCPT ); Thu, 15 Mar 2018 11:43:19 -0400 Received: from wuerfel.lan ([95.208.111.237]) by mrelayeu.kundenserver.de (mreue006 [212.227.15.129]) with ESMTPA (Nemesis) id 0LbCMI-1eBeyB3nOk-00kvzC; Thu, 15 Mar 2018 16:43:14 +0100 From: Arnd Bergmann To: Marcel Holtmann , Johan Hedberg , Kalle Valo , Arnd Bergmann Cc: linux-bluetooth@vger.kernel.org, linux-kernel@vger.kernel.org, linux-wireless@vger.kernel.org, netdev@vger.kernel.org Subject: [PATCH] Bluetooth: btrsi: rework dependencies Date: Thu, 15 Mar 2018 16:42:29 +0100 Message-Id: <20180315154312.3941446-1-arnd@arndb.de> X-Mailer: git-send-email 2.9.0 X-Provags-ID: V03:K0:A069dhHsn5QT0JH5T0EdNEXwfFHbmPAZAOqZ7n8MImkPRn3PHJg uWn6JuVyfv9re7y+Sn4Y2m64SHnOgkII6BKa4rUFO7QjzF6cufV6yaeZrtyfvUgP7807Mda e0spKpGwVR7cJ6KNqizAlaHfbmi6Zr0BpZ2d+Y4yvRpNrutU68t9meUO694VpRqPqIkeyXb lk211T5f6ttJ6zKZ2B6yg== X-UI-Out-Filterresults: notjunk:1; V01:K0:6ep7Hhoaweo=:wpMQ1CUUWBgPQGalUlg3rL GazXc0x6kcy3SHoku7Cl0iZUAgxWth2FcRH3WP3W1JGhNjJBZbUdjyU/elU0zb1BlKz7uauT5 dEdjv8NmEAVvL1z+bthwuyHtOphBTdE+jS5DbMCGbW/ovIX+IXjzX/XHbfKh1+N3HaohYkEo0 wFFPCHs3Vl+epJLTSkXwQmqmxDvPmU5sIi6azpcTlsTNuwkjgplS30HlTcRkxDSNzhfqa+bIx MBnjialJrwpK4NzMSZFjE7Wludp0FAh/xcxnsySl6KMFPC/BD9ch/3UGydZ9lQywNEcBhTy/h h+C2BxXSm1F9wgIxftQA/3fKWvCGU0LNW+OyOtBSO3HX7g2EIIFfxC5FRNDjFGsJBrCMxem+r tC4NYFQk4lBKxKLZIq3UO2l/8T3Nbg5UMWpEXqT7XG6Z40DTt1wlgdYdtRQo5FkqJEcAOWHcM LgABV7U28g7ZNDibU+tw8BD/eSxsO9e/2eqe1qcUgHo4Uo63cdkQYnzI8EDsboS1glvtESlIq ik5y9ElmnTOnsNx6zUNbEEOJAlOeew40x9wnBUeKmA8iu15BUtDJnIHgi2aWwqTUOAm5oya4b tllIzYQQPx8itonJm17SKe8FDVrv09QRuQRTgcRBlxphIm13DJeCwTazvjb+1/t5Nfjt1Dhf2 wBcH7DDVxqXqCoKh6TbRqpnRZqcAAPHxu9NH15JKx4zMHc/W4Gdp47MIBtqBB51vKR0cmebL8 2sPVul1Rf0TZl5aMNXeRjaSaDB2PfhuatA0TUQ== Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org The linkage between the bluetooth driver and the wireless driver is not defined properly, leading to build problems such as: warning: (BT_HCIRSI) selects RSI_COEX which has unmet direct dependencies (NETDEVICES && WLAN && WLAN_VENDOR_RSI && BT_HCIRSI && RSI_91X) drivers/net/wireless/rsi/rsi_91x_main.o: In function `rsi_read_pkt': (.text+0x205): undefined reference to `rsi_bt_ops' To actually make it work, we need the following steps: - remove the bogus 'select RSI_COEX', this is already covered by the default - change RSI_COEX to a 'bool' symbol so that the #ifdefs work all the time - ensure that BT_HCIRSI is built-in whenever RSI_91X is built-in - prevent BT_HCIRSI from being enabled when CONFIG_BT=m and RSI_91X=y, as this cannot work Fixes: 38aa4da50483 ("Bluetooth: btrsi: add new rsi bluetooth driver") Signed-off-by: Arnd Bergmann --- drivers/bluetooth/Kconfig | 9 ++++++++- drivers/bluetooth/Makefile | 2 +- drivers/net/wireless/rsi/Kconfig | 6 +----- 3 files changed, 10 insertions(+), 7 deletions(-) -- 2.9.0 diff --git a/drivers/bluetooth/Kconfig b/drivers/bluetooth/Kconfig index d8bbd661dbdb..6b3e8bf69d07 100644 --- a/drivers/bluetooth/Kconfig +++ b/drivers/bluetooth/Kconfig @@ -394,8 +394,9 @@ config BT_QCOMSMD config BT_HCIRSI tristate "Redpine HCI support" + depends on RSI_91X + depends on !(BT=m && RSI_91X=y) default n - select RSI_COEX help Redpine BT driver. This driver handles BT traffic from upper layers and pass @@ -404,4 +405,10 @@ config BT_HCIRSI Say Y here to compile support for HCI over Redpine into the kernel or say M to compile as a module. +config BT_HCIRSI_MODULE + tristate + # ensure that the BT_HCIRSI driver is visible to the core + default y if BT_HCIRSI=m && RSI_91X=y + default BT_HCIRSI + endmenu diff --git a/drivers/bluetooth/Makefile b/drivers/bluetooth/Makefile index 03cfc1b20c4a..9e8d22712ff3 100644 --- a/drivers/bluetooth/Makefile +++ b/drivers/bluetooth/Makefile @@ -28,7 +28,7 @@ obj-$(CONFIG_BT_QCA) += btqca.o obj-$(CONFIG_BT_HCIUART_NOKIA) += hci_nokia.o -obj-$(CONFIG_BT_HCIRSI) += btrsi.o +obj-$(CONFIG_BT_HCIRSI_MODULE) += btrsi.o btmrvl-y := btmrvl_main.o btmrvl-$(CONFIG_DEBUG_FS) += btmrvl_debugfs.o diff --git a/drivers/net/wireless/rsi/Kconfig b/drivers/net/wireless/rsi/Kconfig index f004be33fcfa..bc6195767c61 100644 --- a/drivers/net/wireless/rsi/Kconfig +++ b/drivers/net/wireless/rsi/Kconfig @@ -43,12 +43,8 @@ config RSI_USB Select M (recommended), if you have a RSI 1x1 wireless module. config RSI_COEX - bool "Redpine Signals WLAN BT Coexistence support" - depends on BT_HCIRSI && RSI_91X - default y + def_bool BT_HCIRSI && RSI_91X ---help--- This option enables the WLAN BT coex support in rsi drivers. - Select M (recommended), if you have want to use this feature - and you have RS9113 module. endif # WLAN_VENDOR_RSI