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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id DF585C4332F for ; Wed, 28 Dec 2022 19:21:43 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232864AbiL1TVm (ORCPT ); Wed, 28 Dec 2022 14:21:42 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:45284 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232797AbiL1TVl (ORCPT ); Wed, 28 Dec 2022 14:21:41 -0500 Received: from gloria.sntech.de (gloria.sntech.de [185.11.138.130]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 2FBB0B7FA; Wed, 28 Dec 2022 11:21:39 -0800 (PST) Received: from [80.120.31.194] (helo=phil.localnet) by gloria.sntech.de with esmtpsa (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1pAbzZ-000631-1G; Wed, 28 Dec 2022 20:21:33 +0100 From: Heiko Stuebner To: krzysztof.kozlowski+dt@linaro.org, piotr.oniszczuk@gmail.com, robh+dt@kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-rockchip@lists.infradead.org, Andy Yan Cc: Andy Yan Subject: Re: [PATCH 2/2] arm64: dts: rockchip: Enable wifi module AP6398s for rk3566 box demo Date: Wed, 28 Dec 2022 20:21:32 +0100 Message-ID: <17226287.uLZWGnKmhe@phil> In-Reply-To: <20221225114953.3518595-1-andyshrk@163.com> References: <20221225114915.3518502-1-andyshrk@163.com> <20221225114953.3518595-1-andyshrk@163.com> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org Hi Andy, Am Sonntag, 25. Dezember 2022, 12:49:53 CET schrieb Andy Yan: > There is a AP6398s wifi/bt module on this board. > Fix the sdmmc1 dt node to make wifi work. > > Fixes: 2e0537b16b25 ("arm64: dts: rockchip: Add dts for rockchip rk3566 box demo board") > > Signed-off-by: Andy Yan > --- > > .../boot/dts/rockchip/rk3566-box-demo.dts | 26 ++++++++++++++++--- > 1 file changed, 22 insertions(+), 4 deletions(-) > > diff --git a/arch/arm64/boot/dts/rockchip/rk3566-box-demo.dts b/arch/arm64/boot/dts/rockchip/rk3566-box-demo.dts > index d956496d5221..6c0c24226b52 100644 > --- a/arch/arm64/boot/dts/rockchip/rk3566-box-demo.dts > +++ b/arch/arm64/boot/dts/rockchip/rk3566-box-demo.dts > @@ -324,8 +324,12 @@ wifi_enable_h: wifi-enable-h { > rockchip,pins = <2 RK_PB1 RK_FUNC_GPIO &pcfg_pull_none>; > }; > > + wifi_host_wake_h: wifi-host-wake-l { > + rockchip,pins = <2 RK_PB2 RK_FUNC_GPIO &pcfg_pull_none>; > + }; > + > wifi_32k: wifi-32k { > - rockchip,pins = <0 RK_PB0 2 &pcfg_pull_none>; > + rockchip,pins = <2 RK_PC6 1 &pcfg_pull_none>; > }; > }; > > @@ -391,16 +395,30 @@ &sdmmc0 { > }; > > &sdmmc1 { > + /* WiFi & BT combo module AMPAK AP6398S */ > + #address-cells = <1>; > + #size-cells = <0>; > bus-width = <4>; > + clock-frequency = <150000000>; > + cap-sdio-irq; > cap-sd-highspeed; > - disable-wp; > + sd-uhs-sdr104; > + keep-power-in-suspend; > mmc-pwrseq = <&sdio_pwrseq>; > non-removable; > pinctrl-names = "default"; > pinctrl-0 = <&sdmmc1_bus4 &sdmmc1_cmd &sdmmc1_clk>; > - vmmc-supply = <&vcc_3v3>; > - vqmmc-supply = <&vcca_1v8>; why are you removing the mmc-supplies here? The supplies needed for mmc to work should very much be present I think, especially to also make sure something else doesn't turn them off. Heiko > status = "okay"; > + > + brcmf: wifi@1 { > + compatible = "brcm,bcm4329-fmac"; > + reg = <1>; > + interrupt-parent = <&gpio2>; > + interrupts = ; > + interrupt-names = "host-wake"; > + pinctrl-names = "default"; > + pinctrl-0 = <&wifi_host_wake_h>; > + }; > }; > > &spdif { >