From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 1013DC77B73 for ; Thu, 20 Apr 2023 19:13:57 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=8+qe8098MPlTukqq4cYRhzNIvSjZjH1n8lTJ7qj+MAU=; b=qWHpUvsZoP/Yk8 3X3uf5YqFeeBwm+6avftwKv21yO/TcoOC/II++mmxbCzcwTuzWEXeZ2ONBGmKzbdpHq9kXBTgI7/8 as8vq9wotRAbta/d3euT2GKn9gyf4ReU71z2vLtDqP64cL8b+0wQZb131lA1q5/Y5sDdvaA9laZdF ndhbQrJxkUT7ScSXa0lvv77ZdsHowqtCsElLiMSl2RRxn7KMUoMEN9HnHEEtZHZpumgf07NJe0wQM 08bPUoJ5X7Kk9l/bl5oj5XW9pUG44BZsORgXPypptxgZ/XNdR+tX9akjTAOWNL7aLUsy5SR8iKJei S9bppRovbbp6/WZr8r5g==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1ppZhm-008pot-0C; Thu, 20 Apr 2023 19:12:30 +0000 Received: from the.earth.li ([2a00:1098:86:4d:c0ff:ee:15:900d]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1ppZhk-008poI-0j for linux-arm-kernel@lists.infradead.org; Thu, 20 Apr 2023 19:12:29 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=earth.li; s=the; h=In-Reply-To:Content-Type:MIME-Version:References:Message-ID:Subject: Cc:To:From:Date:Sender:Reply-To:Content-Transfer-Encoding:Content-ID: Content-Description:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc :Resent-Message-ID:List-Id:List-Help:List-Unsubscribe:List-Subscribe: List-Post:List-Owner:List-Archive; bh=2+GOjpE5zP3l/Yj2zge3EDzVuoeZQl9/VD1vHaJbzx8=; b=onvz42aaty3fwxg9Ca5yT7zvbj ApCgI1YVT0f5FoEGBkWJxsWjY/I3p/YK6pev3OKfqIv4sdqdqnJdP8kYC0PMw7CZDVL50uMQx2gZU +DJWKf8jiwN9ZA9UTStq6aaGgInOXUrnbL31TGAs4xLkUAqBnOVgc8LQt5mrFwM++xLlu4Ejohbai Ra1YNhokSow9BBIo23A3W6ZKGDhQ649mvu1I4rYyvmgWLeL+0LO0aEiihsvfGJy+E+SsCuNTSlRCl wv4vbUjMxyYOHpEfWC3dv5a6PSwas1pRY6AyjBXe8g5DidDpAS+BLa0WswtgIvgQlWQGzsHasGfoL S6dHsp7Q==; Received: from noodles by the.earth.li with local (Exim 4.94.2) (envelope-from ) id 1ppZhb-002Tg3-Ac; Thu, 20 Apr 2023 20:12:19 +0100 Date: Thu, 20 Apr 2023 20:12:19 +0100 From: Jonathan McDowell To: Andre Przywara , Saravana Kannan , Greg Kroah-Hartman Cc: Rob Herring , Krzysztof Kozlowski , Chen-Yu Tsai , Jernej Skrabec , Samuel Holland , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-sunxi@lists.linux.dev, linux-kernel@vger.kernel.org Subject: Re: [PATCH 1/3] ARM: dts: sun5i: chip: Enable bluetooth Message-ID: References: <20230416012421.255bfd19@slackpad.lan> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20230416012421.255bfd19@slackpad.lan> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230420_121228_261457_BEAA929A X-CRM114-Status: GOOD ( 27.59 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Sun, Apr 16, 2023 at 01:24:21AM +0100, Andre Przywara wrote: > On Sat, 15 Apr 2023 18:46:03 +0100 > Jonathan McDowell wrote: > > > The C.H.I.P has an rtl8723bs device with the bluetooth interface hooked > > up on UART3. Support for this didn't exist in mainline when the DTS was > > initially added, but it does now, so enable it. > > > > Signed-off-by: Jonathan McDowell > > --- > > arch/arm/boot/dts/sun5i-r8-chip.dts | 4 ++++ > > 1 file changed, 4 insertions(+) > > > > diff --git a/arch/arm/boot/dts/sun5i-r8-chip.dts b/arch/arm/boot/dts/sun5i-r8-chip.dts > > index fd37bd1f3920..4d72a181d8aa 100644 > > --- a/arch/arm/boot/dts/sun5i-r8-chip.dts > > +++ b/arch/arm/boot/dts/sun5i-r8-chip.dts > > @@ -255,6 +255,10 @@ &uart3 { > > pinctrl-0 = <&uart3_pg_pins>, > > <&uart3_cts_rts_pg_pins>; > > status = "okay"; > > + > > + bluetooth { > > + compatible = "realtek,rtl8723bs-bt"; > > + } > > As the kernel test robot already pointed out, there is a semicolon > missing here. > Otherwise looks good (dt-validate passes), but don't know if there are > any wakeup GPIOs connected (can't seem to find a schematic?). So there are wakeups, but if I add: device-wake-gpios = <&axp_gpio 3 GPIO_ACTIVE_LOW>; host-wake-gpios = <&pio 1 3 GPIO_ACTIVE_HIGH>; /* PB3 */ then some odd sort of dependency issue happens where the serial port load is deferred waiting for the GPIO to appear, and then the device doesn't work. Error in dmesg is: serial serial0-0: deferred probe pending on 6.3-rc and on 6.1 I get: dw-apb-uart 1c28c00.serial: Failed to create device link (0x180) with axp20x-gpio I'm not clear why it's trying to link the serial port to the GPIO; it seems that it should be the bluetooth device that depends on both the UART and the GPIO, and that the GPIO is actually optional so shouldn't hold up loading, but I can't see how that should be represented. Adding Greg + Saravana in the hope they can tell me what I've done wrong here. The LED driver using a different GPIO line on the axp209 works fine, so the device is definitely loaded and working ok. > > }; > > > > &usb_otg { > J. -- Sex, truth and jellibabies. | .''`. Debian GNU/Linux Developer | : :' : Happy to accept PGP signed | `. `' or encrypted mail - RSA | `- key on the keyservers. _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel