Received: by 2002:a05:7412:419a:b0:f3:1519:9f41 with SMTP id i26csp3924861rdh; Tue, 28 Nov 2023 07:21:57 -0800 (PST) X-Google-Smtp-Source: AGHT+IEl0qE+Wimdvg5l8Hbb4aIEwRhXU1l9lDqatRkzvtWEzrbV7EX8e5RzfUBSGTOk2p37/ise X-Received: by 2002:a17:902:c1d5:b0:1cc:4eb0:64cf with SMTP id c21-20020a170902c1d500b001cc4eb064cfmr16954163plc.52.1701184917056; Tue, 28 Nov 2023 07:21:57 -0800 (PST) ARC-Seal: i=2; a=rsa-sha256; t=1701184917; cv=pass; d=google.com; s=arc-20160816; b=WyYOnZ/3I0j2zLsHRBMK885MPmFKHEoGZxxLVrtVCCvE0MNHXxZOPluaXCb6vKp0Qc aSz5BF0OAJBFEf3DxKSK/KBHXaiaW+cT3qADizTt5MDprjJfgbS0dp8xa7CKBgHVWp4F 5btfW0bpgL7xw3e6XOqmy0EMxT2xzAuotDfN0WhO9IfRiw51H5YX4fU59nlbwTVXDMi+ 9HethQGsM+lCodSGWk7msbf7+D3Mc5OTfHZcGuhx34g+Ra7Jmxk+xrtLuDQ3UqizC5Hi PFaaDuuEdw8P+0ldRtDkRcE7kkbQbnUKlreL1YclilsUX4+xXOwQCMWctfDZA3T+FzmN EuFQ== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date:dkim-signature:dkim-signature; bh=njnHQBlxJazLWcAbW1gqa8mgblbBdF2CRCqEZQfBIIw=; fh=uPUez/eP80LG2HsI5uMLA237Inzf8bd9bZ6/GIvRQmA=; b=UgvOPTtJ05uVqZoORXpTT316aWr8kaXTrDfuVwmGEWffW4Pmanv3KiobZo38UOGPCb 51FlPxUAWVkkVXnC9TZBHbkycg5CmlO1HveqNfzYt9hxa0QMBTdVVwoMxEI3poJU+KDv QPM3yCVbnp+XQI1EGuuT5XoHVhjub+O1WXlUhXVcP2cGrCsfP45WHCID1bYl0H2d3vvz X9AccUWZwf/4Z49L+r983xMNFt3tEizAN2NQo1dxTO2MaLffR1b/qbGTYt4WQ8UjC8Z+ AnoZjoy+WF7TqKtQk1xxxF+fY3hga4XQXZDHll9QPN9BOdACUnJGQPVOHmUmi6v3ngda ZLoQ== ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@gerhold.net header.s=strato-dkim-0002 header.b=NGkt5BpA; dkim=neutral (no key) header.i=@gerhold.net header.b="8/1mIY83"; arc=pass (i=1); spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:4 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from howler.vger.email (howler.vger.email. [2620:137:e000::3:4]) by mx.google.com with ESMTPS id k12-20020a170902c40c00b001c6182d7d43si11380397plk.21.2023.11.28.07.21.56 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 28 Nov 2023 07:21:57 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:4 as permitted sender) client-ip=2620:137:e000::3:4; Authentication-Results: mx.google.com; dkim=pass header.i=@gerhold.net header.s=strato-dkim-0002 header.b=NGkt5BpA; dkim=neutral (no key) header.i=@gerhold.net header.b="8/1mIY83"; arc=pass (i=1); spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:4 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: from out1.vger.email (depot.vger.email [IPv6:2620:137:e000::3:0]) by howler.vger.email (Postfix) with ESMTP id 994E280A3080; Tue, 28 Nov 2023 01:02:30 -0800 (PST) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.103.11 at howler.vger.email Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232115AbjK1JCI (ORCPT + 99 others); Tue, 28 Nov 2023 04:02:08 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:44914 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231488AbjK1JCF (ORCPT ); Tue, 28 Nov 2023 04:02:05 -0500 Received: from mo4-p01-ob.smtp.rzone.de (mo4-p01-ob.smtp.rzone.de [81.169.146.165]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id A702DE4; Tue, 28 Nov 2023 01:02:09 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1701162126; cv=none; d=strato.com; s=strato-dkim-0002; b=r6eAu2ggbZSwoSOSC2LB9/TsOtVSDFGz1Yn6exqUJQ0DHNrZcDZFk2KMB6Pf2oxC4i BD9rNHK1hh72OUMXPKLavH9XGEbp8ZBmEli83jQTqYUvLSva2U2lgNj9pIX3HuMVJt+2 qUz6BBCiePvo5XSEigRXFjYsQHZ3rLlZvVPXMb8P5gsWGcddPchoOkSsj1QNmgaWew8G 6c5Ml6V5mmIOeUMjXSAZUPm8rCOv7yZZf2k1BAnlxms0I+mBi9T87BkWQnq6AU/3Zjn5 bCpGLFHdB0BpP9+jgJ/ZYgEWUPJQz3LSIEqXVujdRK4OG77rNrCb8vS78Cn7rkaFkelM VQ8A== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; t=1701162126; s=strato-dkim-0002; d=strato.com; h=In-Reply-To:References:Message-ID:Subject:Cc:To:From:Date:Cc:Date: From:Subject:Sender; bh=njnHQBlxJazLWcAbW1gqa8mgblbBdF2CRCqEZQfBIIw=; b=qzWaaTlBGfcOt2NSli9uC30OIHhTm7Et3bwWlm5HYltZczHIohrUpFsbELgy0mQfkS WLyMz3zcRmaMElchm3RwfdjsZjVb+DeFjRQpFLfanu2K1+Ee6n52PZX/sHVTsBBPg6DY P5KBI2rUPZEwXI181zMZX6FcG0zFldP3BDmfAbL6LD5gWXt6A/XYrNilOalrEWN1yRFr nqPOIVRn30AqUGijCfS29EyfgvhiOP/EmYVHWyrs4QT5OxcZpJv9Y6jmDOwHRCn0negL 1TTkuctz5ZvQSFL1jWE5UVbC2TNr894f2i30jcJhcjDaLQfuF91h5IJQ0/+8Ve/FuZfH 58RA== ARC-Authentication-Results: i=1; strato.com; arc=none; dkim=none X-RZG-CLASS-ID: mo01 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; t=1701162126; s=strato-dkim-0002; d=gerhold.net; h=In-Reply-To:References:Message-ID:Subject:Cc:To:From:Date:Cc:Date: From:Subject:Sender; bh=njnHQBlxJazLWcAbW1gqa8mgblbBdF2CRCqEZQfBIIw=; b=NGkt5BpAHMAwG5ZGf2LpEqrMuOEjWLA6KVKLpltw4TzVMPa9HVzLWEtcKSBfqWrujV qBOb4Dr8yy4PqS7SnkqExRYuFSIIElBJulaur0QNIREh0xy5LNNScHRpV0Ovj65xoe2R MZmoW4OWEBqr1uEslj3AZKcxky/jhP6NL0nhANSgY1EUHpY5zIdu5MsqZ7ghvg2N4jdy fYEqXZ7s2chHCM17/Dx78JTKLrSMGVOScX5rMWRHytabBb4IhYkYLHxhEzbiqRhMJSon MUFdLiI8i8a9xMdgl7S9Jd5vbf79Nk1Dk69X9amD5LK4afivnJjWMjz0ylAWKLMMoOYB UifQ== DKIM-Signature: v=1; a=ed25519-sha256; c=relaxed/relaxed; t=1701162126; s=strato-dkim-0003; d=gerhold.net; h=In-Reply-To:References:Message-ID:Subject:Cc:To:From:Date:Cc:Date: From:Subject:Sender; bh=njnHQBlxJazLWcAbW1gqa8mgblbBdF2CRCqEZQfBIIw=; b=8/1mIY83/yDD+zTLGU3PquPNdvq5bT+5tGhBNu6pRU7uN+7Y0kocxGPDSnyY8/AxHp urFKBT2uGou13TLtIEBw== X-RZG-AUTH: ":P3gBZUipdd93FF5ZZvYFPugejmSTVR2nRPhVOQ/OcYgojyw4j34+u261EJF5OxJD4peA8paF1A==" Received: from gerhold.net by smtp.strato.de (RZmta 49.9.1 DYNA|AUTH) with ESMTPSA id t3efe7zAS925ktT (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256 bits)) (Client did not present a certificate); Tue, 28 Nov 2023 10:02:05 +0100 (CET) Date: Tue, 28 Nov 2023 10:01:47 +0100 From: Stephan Gerhold To: Neil Armstrong Cc: Andy Gross , Bjorn Andersson , Konrad Dybcio , Rob Herring , Krzysztof Kozlowski , Conor Dooley , linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v4 2/8] arm64: dts: qcom: add initial SM8650 dtsi Message-ID: References: <20231124-topic-sm8650-upstream-dt-v4-0-e402e73cc5f0@linaro.org> <20231124-topic-sm8650-upstream-dt-v4-2-e402e73cc5f0@linaro.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20231124-topic-sm8650-upstream-dt-v4-2-e402e73cc5f0@linaro.org> Content-Transfer-Encoding: 7bit X-Spam-Status: No, score=-0.9 required=5.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on howler.vger.email Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (howler.vger.email [0.0.0.0]); Tue, 28 Nov 2023 01:02:30 -0800 (PST) On Fri, Nov 24, 2023 at 10:20:39AM +0100, Neil Armstrong wrote: > Add initial DTSI for the Qualcomm SM8650 platform, > only contains nodes which doesn't depend on interconnect. > > Reviewed-by: Konrad Dybcio > Signed-off-by: Neil Armstrong > --- > arch/arm64/boot/dts/qcom/sm8650.dtsi | 2439 ++++++++++++++++++++++++++++++++++ > 1 file changed, 2439 insertions(+) > > diff --git a/arch/arm64/boot/dts/qcom/sm8650.dtsi b/arch/arm64/boot/dts/qcom/sm8650.dtsi > new file mode 100644 > index 000000000000..b0a9ca53d58e > --- /dev/null > +++ b/arch/arm64/boot/dts/qcom/sm8650.dtsi > @@ -0,0 +1,2439 @@ > +[...] > + timer@17420000 { > + compatible = "arm,armv7-timer-mem"; > + reg = <0 0x17420000 0 0x1000>; > + > + ranges = <0 0 0 0x20000000>; > + #address-cells = <1>; > + #size-cells = <1>; > + > + frame@17421000 { > + reg = <0x17421000 0x1000>, > + <0x17422000 0x1000>; > + > + interrupts = , > + ; > + > + frame-number = <0>; > + }; > + > + frame@17423000 { > + reg = <0x17423000 0x1000>; > + > + interrupts = ; > + > + frame-number = <1>; > + > + status = "disabled"; > + }; > + > + frame@17425000 { > + reg = <0x17425000 0x1000>; > + > + interrupts = ; > + > + frame-number = <2>; > + > + status = "disabled"; > + }; > + > + frame@17427000 { > + reg = <0x17427000 0x1000>; > + > + interrupts = ; > + > + frame-number = <3>; > + > + status = "disabled"; > + }; > + > + frame@17429000 { > + reg = <0x17429000 0x1000>; > + > + interrupts = ; > + > + frame-number = <4>; > + > + status = "disabled"; > + }; > + > + frame@1742b000 { > + reg = <0x1742b000 0x1000>; > + > + interrupts = ; > + > + frame-number = <5>; > + > + status = "disabled"; > + }; > + > + frame@1742d000 { > + reg = <0x1742d000 0x1000>; > + > + interrupts = ; > + > + frame-number = <6>; > + > + status = "disabled"; > + }; > + }; Nitpick: Personally I feel the empty lines between each property here are a bit overly verbose. It would be better readable without them. Might be personal preference though :-) > +[...] > + timer { > + compatible = "arm,armv8-timer"; > + > + interrupts = , > + , > + , > + ; I'm pretty sure GIC_CPU_MASK_SIMPLE() is only valid & used on GICv2. Unlike arm,gic.yaml, arm,gic-v3.yaml doesn't mention "bits[15:8] PPI interrupt cpu mask". Also see e.g. commit 4a92b6d75bab ("arm64: dts: msm8996: Fix wrong use of GIC_CPU_MASK_SIMPLE()"). Would be also good to check if any existing DTs have introduced this incorrectly again since then. Thanks, Stephan