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 0451820C49F 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=1739365647; cv=none; b=cBVXBRuCo/CN2rrDE7K1GaTJb4dLqk/htQ4SNg8Qozkd6u7ZTwG/MTWpk0EJE+9QD3emDA05I3zK+FHYf1xBOkMBSWxaGB2JHpmpqr9iw+I5LtZBGTb7F5L2QZnWq5vRkPgUOvNEusrg289BVLxSaOk7OOYjr3jR4eH8HWL62rM= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1739365647; c=relaxed/simple; bh=UoAO9GjW2doCYGuRGY8Y8bf14wyHQN29mYKG8IS0QyE=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=d3Fy6vAIrH0E1PEytr0Ts7VH17PvCr5tlSnQOBrtZEtGrdGvlgPR6oB1+pu+DwgeLDsX3qY/EcsB/2vdBbsmWg/3NXruRldiHgRaCySqB015W0DFDMQQ0XGVT0ZQfTSoHZ4w+wNxu4OAACu0DVCzZ2EE4DMN6Cyz+7rzovhKWAA= 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 (m0279868.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.18.1.2/8.18.1.2) with ESMTP id 51C5w6jl006121 for ; Wed, 12 Feb 2025 13:07:25 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 44qe5mykk4-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-6e448963aafso15409856d6.3 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=Md08tozrgv1tEW6/xvQxxruGDwXDCgQBEWkQrMpgzNUv3HhVz0IOi/4bWH0r60HOAn adv9H+ClVnYcNScyY8apGVrNwKLaza74f6A3MUGJDycul6xLGN/wFb2wzTsLX136hSLA f9NHYNuj63BMQHncOVvbvzhGU2c7OWSjyZjeVxY1DuTrR9RQMXaXJ7GFBSCoY2Fe7Typ TAQj7ZfJ53s/RUYf1+tD/nzcrmBPiB8Wtd+dhzgiUdVH8Ih226fAuHp9zAC4/JrKdaDC gEhqHpYEcIArrH94/eBzlgFn9dL5F+C6elOuiKqRx7N4GrNCeZqPl4nNGI56vdVLO/pn YvRQ== X-Forwarded-Encrypted: i=1; AJvYcCXmiHPfRT0Uv8RrcSVCr2GTn8d/zL1h01/dS+n9Cx/xkhQQv/hNRV7ddVlRbTGgfDYrIgCt7hhbYbYE5p4=@vger.kernel.org X-Gm-Message-State: AOJu0YwAtNxH/ISXaeMClOxUNJbBl3KR3O/Mcubre+0UlpouOgX3wtTc S5s7VT/5r/8VFkP0RqM70AANjTuZlr8ekJvatqLFVQzhNLmqSHL2e/MXv9Kt1mZ9kNyWsNZOHIX +CpcKWC1SUFQ2p0Dutv24i0y1cZfyMab0nCzCIJpnrkC3DfLvSFfW+B45gVhnkuI= X-Gm-Gg: ASbGncsxPZ8qu0/p+B4m74ZKXz3yY46VTS/lrOW6UBtv7RR7U8xhhWwgH0OXURPvXek uRzq63wJTTah8T1O5+Ku86swIUpOkWVXv1UxVDbNFb5c6qeLBci49KfMTnoLS1PzrL7a/KC3RSc vC9Slzwaft18o5vunxurqJZOlKjwtdu2ymeBrUjU59hhsVWdDYWic/NhyQHb79YFM1Ad5bDnbzA O05ogRlWnQLsmSmTjgFFsLoIRM/xbrYGN3XZo7ReHPfYOaFKm5IDI2V+g3/dn7Hk33iuCwnt5av 8ALHArl9hVD4pKvpHX4KfDEJxYTmpeCaxlq1azsyR1V/khWpqYrxwVCdwDc= X-Received: by 2002:ac8:5e12:0:b0:461:6599:b9a with SMTP id d75a77b69052e-471afed742emr16227461cf.11.1739365643720; 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-kernel@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-GUID: CjAx8pgUDk87tzn0lC0QtjFtjrTc2YMr X-Proofpoint-ORIG-GUID: CjAx8pgUDk87tzn0lC0QtjFtjrTc2YMr 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 priorityscore=1501 clxscore=1015 suspectscore=0 mlxscore=0 lowpriorityscore=0 phishscore=0 impostorscore=0 malwarescore=0 spamscore=0 mlxlogscore=999 adultscore=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