Received: by 2002:ad5:474a:0:0:0:0:0 with SMTP id i10csp5909724imu; Mon, 26 Nov 2018 06:48:01 -0800 (PST) X-Google-Smtp-Source: AFSGD/XApLHCPgBsX8JQbJr5qudiqRh584z/GcetJ+DYrcGRV1qYAx+HQO6CFKGlVer5P74weWQ6 X-Received: by 2002:a63:a112:: with SMTP id b18mr25319535pgf.440.1543243681233; Mon, 26 Nov 2018 06:48:01 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1543243681; cv=none; d=google.com; s=arc-20160816; b=nx70Mp/P84OUhS2+uCHaxv2op5D2WWYa1V14N795LqUKZ57ZohSvbXBKIjEjvkxA15 kV6linzcWp9OIAMc0dAViM7DCw5MK+Jk1VtORfIn6Tc0clGXuKECM3ulARV+pqBshLNY FMdxA/WOcW+NaxVjtSeR7u4svK0o+HYysLfJ0BCPDg/wg2ARCiWBaqJCHDMCriViR557 ht1imbWmJLnZVbEQYbJYVCB0SqZ/FdvjltkXkr+gl6fqRF/2HPENybIxpzQLUMA+14tr 0q17jBg2wsC+Q838VZkhm0Bt7ogt7w2eTMWFFRH0TaSSI3WuGl+JxF96ZKpw/VWY0V3b KYjg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date:dkim-signature; bh=hzpFlEQyq7YHi4+ylHi2b0ldi76VJyTuDZGhFWWU47k=; b=agsuE9CMBcpYAPGO8OlnekJPfRJyD9rUwko4/yY2oVRul5K+IMyKqGbL8I8xKfxrk1 ZdjfI966eIQxhlo9qm0lPX/Pcck08/6wCkY9pyHJq2VX4gcoOU/0Qz7q62wFNHducaXq 3lYNil2IQ+Yyr7aE0NoMkctLL7Wb//j4RhG0VG1a3P2E0r93/HgYiehG2vam9l+MH9kp RtdopATFjz2/Iq0PGOJJilQWUdGNhS+J9u/rTaT8wT38H6soz76p6N/SvYYSOEht8NoN PqlXCb6XDpPr7p1SrrIY4KXF/op4Z8X3d5bDsXqqfcV+Pl6GhPaw42qqjRUU6V2gMFaH iBYA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=Ex5d8S6N; 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; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id u5si371601pgr.316.2018.11.26.06.47.34; Mon, 26 Nov 2018 06:48:01 -0800 (PST) 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; dkim=pass header.i=@kernel.org header.s=default header.b=Ex5d8S6N; 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; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726516AbeK0Biv (ORCPT + 99 others); Mon, 26 Nov 2018 20:38:51 -0500 Received: from mail.kernel.org ([198.145.29.99]:44562 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726200AbeK0Biv (ORCPT ); Mon, 26 Nov 2018 20:38:51 -0500 Received: from localhost (unknown [122.178.235.61]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id A74B420817; Mon, 26 Nov 2018 14:44:30 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1543243471; bh=9dQ/l7zGD/mc2KA5oLf6Fke/83EoVDFjqXXzkPhFAvE=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=Ex5d8S6NdEXcP7u1gS7/OBBgXEXMjZIQA+lBhjof+ZaIDksilHTWiGx+JVmIHccqq 4rqd4ADqPme6A1ENpA66A2DAQrywZ4SwxMCHNvB6tvjowoomqJgp+fPBQpYTCOJe40 b1kEvACZdyyl3AQQQ7L9W+e3csY7/J1A6Z9CRd/8= Date: Mon, 26 Nov 2018 20:14:22 +0530 From: Vinod Koul To: Govind Singh Cc: andy.gross@linaro.org, bjorn.andersson@linaro.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-arm-msm@vger.kernel.org Subject: Re: [PATCH] arm64: dts: qcom: qcs404: Add WCN3990 WLAN module device node Message-ID: <20181126144422.GU3175@vkoul-mobl.Dlink> References: <20181126142133.16966-1-govinds@codeaurora.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20181126142133.16966-1-govinds@codeaurora.org> User-Agent: Mutt/1.9.2 (2017-12-15) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Govind, On 26-11-18, 19:51, Govind Singh wrote: > Add device node for the ath10k SNOC platform driver probe > and add resources required for WCN3990 on qcs404 soc. > Optional clock and regulator controls are not yet available in > upstream, hence add them once available. > > Signed-off-by: Govind Singh > --- > arch/arm64/boot/dts/qcom/qcs404.dtsi | 19 +++++++++++++++++++ > 1 file changed, 19 insertions(+) > > diff --git a/arch/arm64/boot/dts/qcom/qcs404.dtsi b/arch/arm64/boot/dts/qcom/qcs404.dtsi > index 9ca4f061ecc5..1a401a32d4a1 100644 > --- a/arch/arm64/boot/dts/qcom/qcs404.dtsi > +++ b/arch/arm64/boot/dts/qcom/qcs404.dtsi > @@ -958,6 +958,25 @@ > status = "disabled"; > }; > }; > + > + wifi: wifi@0A000000 { Please remove leading 0 from node > + compatible = "qcom,wcn3990-wifi"; > + reg = <0x0A000000 0x800000>; > + reg-names = "membase"; > + memory-region = <&wlan_msa_mem>; > + interrupts = , > + , > + , > + , > + , > + , > + , > + , > + , > + , > + , > + ; > + }; > }; > > timer { This file is sorted alphabetically and reg values. So this should be between blsp1_uart2 and intc node, can you please change that Also please compile this with W=12 Lastly, I am not sure, but should the wifi node be always enabled? Should it not be enabled in the board dts file? Bjorn..? Thanks -- ~Vinod