From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mx0b-0031df01.pphosted.com (mx0b-0031df01.pphosted.com [205.220.180.131]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id A6AA8209F4B for ; Wed, 12 Feb 2025 13:07:25 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=205.220.180.131 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1739365648; cv=none; b=ICZ70uQ7QO+FQ/eUOl9R57rpd7hKn/6ljanS3XBz/DbJMQuceklRtJNISve58CjN8cOl9i9ICuORF4BX/WAVRT5dK1WJl0vE2aBoH8aB65OtviT4rJhz5LnAsiMFPzKQPfH/In0+ho1PAuMITO20C/bcfsbTZ8QjNozmgZYEVI8= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1739365648; c=relaxed/simple; bh=UoAO9GjW2doCYGuRGY8Y8bf14wyHQN29mYKG8IS0QyE=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=biR66insP3soId0ilQGG4EsRFj1YZF1wr6ZnVwaMIMlPMoTv4/k+2CbgIrliD+LL2Unb7y4/4jXLiKKLJaEBtE1E6onE8dEw6HzuDWQrcYwV32c3zlF91CWpONhXCYuub7Qm3VMzMLzi/JCCBX76CUyV1pntF910w6sfc2O+VwE= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=oss.qualcomm.com; spf=pass smtp.mailfrom=oss.qualcomm.com; dkim=pass (2048-bit key) header.d=qualcomm.com header.i=@qualcomm.com header.b=dcDhl7r8; arc=none smtp.client-ip=205.220.180.131 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=oss.qualcomm.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=oss.qualcomm.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=qualcomm.com header.i=@qualcomm.com header.b="dcDhl7r8" Received: from pps.filterd (m0279870.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.18.1.2/8.18.1.2) with ESMTP id 51CB6Ssa030235 for ; Wed, 12 Feb 2025 13:07:24 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=qualcomm.com; h= cc:content-transfer-encoding:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to; s=qcppdkim1; bh= gRuPU5MLz6UvJIh9FeNXTscDgCKFwAX6i3T1Y2CbOg8=; b=dcDhl7r8v560eZxp Cym9zu8gp6OvQcSAE6DpHGAmiWEnn3ssZR2Tpm7WdYLQ0TqYdr2JPGvB6tDpcNJt EmvYx8dtrL40UhKEG0Nka9M6X9YDcByIOHFiF8KAQP9qFEvp3GMEHslE4QzBLfks g/2Ps4oxO6mREh3VRiirHF60qu7uohzY1dhBgh5TUE3L9GGquAqtpToIOcGKUpkl 6Lra6incRkkcMbK1chA0SJ25akNXkGOqWk6hcI81leZrKOY/o3f7JjAO7T0LpQ4Y wT4S7MDOqbpba1xk5ssH3JpOAuNUsrITCqg3/VsjrI1XUbTu5GE/YyCUEeguLbJf 0dvOZA== Received: from mail-qv1-f70.google.com (mail-qv1-f70.google.com [209.85.219.70]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 44qxg9myww-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128 verify=NOT) for ; Wed, 12 Feb 2025 13:07:24 +0000 (GMT) Received: by mail-qv1-f70.google.com with SMTP id 6a1803df08f44-6e65862f4e0so798096d6.2 for ; Wed, 12 Feb 2025 05:07:24 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1739365644; x=1739970444; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=gRuPU5MLz6UvJIh9FeNXTscDgCKFwAX6i3T1Y2CbOg8=; b=sdkIrwA0aS9ijuiFjj6aNLTFvABjhN4bHW4t5T7VVIBq6oW6wJRr8R5n6aUXSvWeth A5U/NvrFx7IcOjr7TTL6sbBMFVXvkDLVXAiGGDCW82CzK9tx/ONjA89EyZD6v19n54uz xx9UkEoLD9rsuPMc9E2+I30nQiBhp/Lr35L9VZ+7K6AyiDNrVdPFDswsp23adr6uPd3/ 6dc0hKsJT6ztjJ/qMtscCs5aGa0uZ6iA3eedY+r7F2zNgZ6v/zGnZ+cnDXpT0bbBCC4s z7e1xXaEecXaXXVbuD//QG4RTaAPSkNF9CqUBCdiCqUO5K2anb9PwfG41fBzaCul2wVZ krAQ== X-Forwarded-Encrypted: i=1; AJvYcCUZCmR9O5S5HjABvIGmEG70ix/WUcnPtZikrlrVyTUUvqENcYB1QuQxyu6Dgx6AGtn/It87CRMU4Cs=@vger.kernel.org X-Gm-Message-State: AOJu0Yzm9gXcj3RwxUKQUaOF9bEaWPAY9bH64Wroc5n3hBC23WaNfp+9 nE756IKQQFG4wVGQ20SGJbU3NqJwJQ7ODa64CnJWDcwekKJL7BGTTWFI33lyNLiJnfCDDg7cHct pxYqtqIx25GIw5Ppab0sGSt18fE5H9L9Zh3tigbEa7Z2QiF0EFcq+awyGgvo= X-Gm-Gg: ASbGncvlsgd3prkgCCPTH+16XtBESzQVntnZpsb3Ie1rbMR3I02zjh6D4fofYmRUMti ZypBve/KkPp+AtSZeEGL9IVbQRmoxjB5a3igUEAX313ItzHVGscMHVbakConFJ0a2S6tEresn+S XH+XqRWHJjZz3+OQr/kMmHc6QqHNnLmE4pXjcFvGUvzefVeDIFnSqaGyrl57Kog4AL34tWslRbz cKt0g0ltCx/n1aDdg69Bd9QwAWUmhvMNu+IXnjKFIIB6ab4D9slxlI7vhdImY2bI6wGU5YXnxgb KJSZSFdWmCtD5/FQ+dcLp4JZFUG/h7BMpOqOSnCj6n75f1kLMcpRf5HOigs= X-Received: by 2002:ac8:5e12:0:b0:461:6599:b9a with SMTP id d75a77b69052e-471afed742emr16227271cf.11.1739365643675; Wed, 12 Feb 2025 05:07:23 -0800 (PST) X-Google-Smtp-Source: AGHT+IGCA7giG/gMWBnfeZO8qNqAYL1V7WqLfjH9to4onOJOTmpMIsvYAtGlEa5pxRnxYvlkUj5Q3Q== X-Received: by 2002:ac8:5e12:0:b0:461:6599:b9a with SMTP id d75a77b69052e-471afed742emr16227051cf.11.1739365642987; Wed, 12 Feb 2025 05:07:22 -0800 (PST) Received: from [192.168.65.90] (078088045245.garwolin.vectranet.pl. [78.88.45.245]) by smtp.gmail.com with ESMTPSA id a640c23a62f3a-ab7dca728fcsm350960766b.166.2025.02.12.05.07.19 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 12 Feb 2025 05:07:22 -0800 (PST) Message-ID: <7664b71c-ed47-4765-9ac4-5dbe3ec80d3c@oss.qualcomm.com> Date: Wed, 12 Feb 2025 14:07:19 +0100 Precedence: bulk X-Mailing-List: linux-clk@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH 08/10] arm64: dts: qcom: Add initial support for MSM8937 To: =?UTF-8?B?QmFybmFiw6FzIEN6w6ltw6Fu?= , Bjorn Andersson , Michael Turquette , Stephen Boyd , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Stephan Gerhold , =?UTF-8?Q?Otto_Pfl=C3=BCger?= , Linus Walleij , Lee Jones , Srinivas Kandagatla , Joerg Roedel , Will Deacon , Robin Murphy , Konrad Dybcio Cc: linux-arm-msm@vger.kernel.org, linux-clk@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-gpio@vger.kernel.org, iommu@lists.linux.dev, Dang Huynh References: <20250211-msm8937-v1-0-7d27ed67f708@mainlining.org> <20250211-msm8937-v1-8-7d27ed67f708@mainlining.org> Content-Language: en-US From: Konrad Dybcio In-Reply-To: <20250211-msm8937-v1-8-7d27ed67f708@mainlining.org> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-Proofpoint-ORIG-GUID: uIGQhSr7RQUVkauYnakcUlhPjE_YuQyP X-Proofpoint-GUID: uIGQhSr7RQUVkauYnakcUlhPjE_YuQyP X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1057,Hydra:6.0.680,FMLib:17.12.68.34 definitions=2025-02-12_04,2025-02-11_01,2024-11-22_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 bulkscore=0 adultscore=0 mlxscore=0 priorityscore=1501 malwarescore=0 spamscore=0 impostorscore=0 mlxlogscore=999 lowpriorityscore=0 phishscore=0 clxscore=1015 suspectscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.19.0-2501170000 definitions=main-2502120101 On 11.02.2025 11:37 PM, Barnabás Czémán wrote: > From: Dang Huynh > > Add initial support for MSM8937 SoC. > > Signed-off-by: Dang Huynh > Co-developed-by: Barnabás Czémán > Signed-off-by: Barnabás Czémán > --- So the computer tells me 8917 and 8937 are *very* similar. Have you tried assessing how making 8937.dtsi an overlay atop 8917.dtsi would work out? > arch/arm64/boot/dts/qcom/msm8937.dtsi | 2145 +++++++++++++++++++++++++++++++++ > 1 file changed, 2145 insertions(+) > > diff --git a/arch/arm64/boot/dts/qcom/msm8937.dtsi b/arch/arm64/boot/dts/qcom/msm8937.dtsi > new file mode 100644 > index 0000000000000000000000000000000000000000..ef633c1694ad98165e58130cbeb186d2f0e2dcaa > --- /dev/null > +++ b/arch/arm64/boot/dts/qcom/msm8937.dtsi > @@ -0,0 +1,2145 @@ > +// SPDX-License-Identifier: BSD-3-Clause > +/* > + * Copyright (c) 2023, Dang Huynh > + */ > + > +#include > +#include > +#include > +#include > +#include > + > +/ { > + interrupt-parent = <&intc>; > + > + #address-cells = <2>; > + #size-cells = <2>; > + > + clocks { > + xo_board: xo-board { > + compatible = "fixed-clock"; > + #clock-cells = <0>; > + }; > + > + sleep_clk: sleep-clk { > + compatible = "fixed-clock"; > + #clock-cells = <0>; > + }; > + }; > + > + cpus { > + #address-cells = <1>; > + #size-cells = <0>; > + > + cpu4: cpu@0 { I'm pretty sure a CPU with a MPIDR of 0 should be called CPU0 > + compatible = "arm,cortex-a53"; > + reg = <0x0>; > + device_type = "cpu"; > + enable-method = "psci"; > + operating-points-v2 = <&cpu_opp_table_c0>; > + next-level-cache = <&l2_0>; > + #cooling-cells = <2>; > + l2_0: l2-cache { Please add a newline between the last property and the subnode > + compatible = "cache"; > + cache-level = <2>; cache-size = <0x80000>; [...] > + cpu0: cpu@100 { > + compatible = "arm,cortex-a53"; > + reg = <0x100>; > + device_type = "cpu"; > + next-level-cache = <&l2_1>; > + enable-method = "psci"; > + operating-points-v2 = <&cpu_opp_table_c1>; > + #cooling-cells = <2>; > + power-domains = <&cpu_pd0>; > + power-domain-names = "psci"; > + l2_1: l2-cache { > + compatible = "cache"; > + cache-level = <2>; cache-size = <0x100000>; I'll do further review if you decide it makes sense to keep this separate from 8917 Konrad