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 08940382F13 for ; Mon, 16 Mar 2026 09:03:51 +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=1773651833; cv=none; b=E673f6XB6+MIOQV6YeAMXpUDSH8ANrBI3XsN9H1O3OI/w66OJIA5uqGbbjWtLlfZKKtMYtush808z6kEu2x0RbL6fH+u/i7ZT5owiyWbCmkm2QzvO0hsC27jC/vPbn2ADPU76CdZQ3xHPoNZNA/kdrcYT5TAujN34GjVwzZuKto= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773651833; c=relaxed/simple; bh=NTcvsuGdFFCNDl1LmYf5IJqXK3/yowkel0MRzCFjEA8=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=rit3vwBUwbpLgB7634sxMv7yo9tePLU8YhaAqYfWs7hXourIPHMufA42mahsC8ViKRqQqRCugkorZHXL3nslHdS+3YMpyDEPOWyw27G0yGQXvpe2xCib6Vvj060yRfy6OSN/l/hT0AcUSx4/NkwNA9fx7ycQ5UW1vlhBDI9vOeo= 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=XzN5Wves; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b=bWoeMhsa; 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="XzN5Wves"; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b="bWoeMhsa" Received: from pps.filterd (m0279871.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.18.1.11/8.18.1.11) with ESMTP id 62G64kDq744179 for ; Mon, 16 Mar 2026 09:03:51 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= uxIt5bFyOn21UFobDspTNyIJUK1VFpOCpDcq0kigCmw=; b=XzN5WvesKeTofC7H 71emGR6akr7BAIw18eRTDtKaDaPpG4SL6Ck8aNqNq7Y2ZflWiNhCemEsvK1w51iL Xa9Ku6lFxoPG/NnY5uwxJ9Md7InPysX4iR9VriRB3qy71IdwNN0GC9EDurBQJrVe lYNSmNpk0PpxQoaUoyqba0fkh98NeAAaN2Z+WqSFu1FpYHNldZTIZDBagaUSA/ez sP9OgmnPTocO8fAHA+fIz4AT4Ec/vvXzDtoFgcvwnLH5Wl015FYC6DNHBKNbKBl3 DygEddJowzNJj6wGGGO7sYZI84rsvmX3PYMfDcMOZ2B1kOGInE59FU2eFCTqvI/s 4hLFzw== Received: from mail-pg1-f197.google.com (mail-pg1-f197.google.com [209.85.215.197]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 4cw00bd0ys-1 (version=TLSv1.3 cipher=TLS_AES_128_GCM_SHA256 bits=128 verify=NOT) for ; Mon, 16 Mar 2026 09:03:50 +0000 (GMT) Received: by mail-pg1-f197.google.com with SMTP id 41be03b00d2f7-c6e7f45e2ddso23136869a12.1 for ; Mon, 16 Mar 2026 02:03:50 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oss.qualcomm.com; s=google; t=1773651830; x=1774256630; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=uxIt5bFyOn21UFobDspTNyIJUK1VFpOCpDcq0kigCmw=; b=bWoeMhsa9riDiUmH8NWPJRvIEorscO8cwUAocO9VGmZJnwNT6t/tvt50HImUP0ov80 IH7IoYvspWlHJOgqLEy6oYEsvy36NTyY0kKewzVawkNT2lRmsBNhRd96XTg68BfuATUG k/VNfRHaaDFF4HsAEJwCUF5geqkZr5oky32d03o4Oa5Ncg275XgfWKFVI9fYjtUaVR43 dmVONu2zDl9GO/U4AGeOs86b99Npp2jF2TWKuYxm1NTKuBuLnfe+SzRODFniz6vSqxmG WoPc48nJ2efba7hwF4oeAvZpCQkNEtpPYmMqCto3bXLNW3DSUiw3MXmFMY7w59VMIPpN oDPg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1773651830; x=1774256630; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-gg:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=uxIt5bFyOn21UFobDspTNyIJUK1VFpOCpDcq0kigCmw=; b=ZCe6r0uIAIYNT6QT5InbwVGzC4Y07I8+Wo3va0lyN++Cwe43x61Em6R13Z+R9TDBDh AdcvrS/KG0c12bnLA9PM6mFTt6PAti4FaoqCXLk8gXxROUjNYr+2+aPj56d2aSsnzK3i rDiYpOSALt4uUPCwQhqpSu5/77aKHmO80QDc/PI3ru7HgBSwGglA7ln3DaPejxEhTWck 4BlhEzUe+b4nuPj9UQ4FJ1KaWjaRsI+zPoP5438g+YeBTlycIxxugoHrI9Umc4t91pTq XRY14jBsNmcHXURO8rry66l3Cf7gXSNslqFjrkUv00xwe1n3chfYoLRj7XWYHGtpuAwV Z+kQ== X-Forwarded-Encrypted: i=1; AJvYcCXPTh7Pgsd4SQ6TeeyFWjhskyAlGstzeCpQZOr7QiuZskfiAMWE6bCsBXpW6Y7T/P53W1reBrbidpy+@vger.kernel.org X-Gm-Message-State: AOJu0YwHGBuw/k7OBYxalCeivscHlE17nIUgMTSOyNbQE1tWQWh1qnbT tQ8mwaeJHksnmTh3tKJCLfd6mj0SQOKWdNuzEmThtq+5CECgKsY853MivL/WYS6KxM1tr3gt2Rc z+ZmffB/7Ngw58TinDVqnedqwb9sfzUlo/A25nk+WW6YvPBgOYVkVY88HB0cyQyLu X-Gm-Gg: ATEYQzwf2Ab/CV4/nXKOJTW52ZtSozxVhIGBC8hZtI9eTzsk06rpBXDhWShoyk9LybF c0os+eQ6h/zEKrLgNuHKMkV8mHSq0QeKFWIpxqLXLtsAkqinWvybc/5UJAfE16SVZgaFyvibbVL ZJ5XoHzciQMrhtp8F8+mISvBUgX8b7KzGD8qlTiZdKmXnoSABfkO7B0P7RAAie9EGwDXN6wziEk 4EEYfUUfaWu64J4AiHOtqNuAFaFfKUr05f5wiy6N+nboFryxuKucpVhbAA/8dFzltpuyzY3LWzd nEL5ccw4ltw3CcpwVVB8Kx2Xva1B7RG91d6JwsvgHJEBFy5vu2nKn3swie9pV/58OTqnh8SsyOX pW5mgzv7LMoScHJHkHIJSstFELnZsg62WJWxhxR/nEUY6KHXtI+hvH5Rl/KM+sjzZ0g== X-Received: by 2002:a05:6a00:761b:b0:829:8e63:fca1 with SMTP id d2e1a72fcca58-82a198d4ccamr8653984b3a.33.1773651829982; Mon, 16 Mar 2026 02:03:49 -0700 (PDT) X-Received: by 2002:a05:6a00:761b:b0:829:8e63:fca1 with SMTP id d2e1a72fcca58-82a198d4ccamr8653948b3a.33.1773651829361; Mon, 16 Mar 2026 02:03:49 -0700 (PDT) Received: from [10.151.37.217] ([202.46.23.19]) by smtp.gmail.com with ESMTPSA id d2e1a72fcca58-82a07244071sm13207421b3a.6.2026.03.16.02.03.43 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 16 Mar 2026 02:03:48 -0700 (PDT) Message-ID: <84f1407d-62c6-4ec2-8976-eabe66c74728@oss.qualcomm.com> Date: Mon, 16 Mar 2026 14:33:41 +0530 Precedence: bulk X-Mailing-List: linux-gpio@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH 8/9] arm64: dts: qcom: add IPQ5210 SoC and rdp504 board support To: Konrad Dybcio , Bjorn Andersson , Michael Turquette , Stephen Boyd , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Philipp Zabel , Linus Walleij , Konrad Dybcio , Ulf Hansson , Robert Marko , Guru Das Srinagesh 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, linux-mmc@vger.kernel.org References: <20260311-ipq5210_boot_to_shell-v1-0-fe857d68d698@oss.qualcomm.com> <20260311-ipq5210_boot_to_shell-v1-8-fe857d68d698@oss.qualcomm.com> <6e33daf1-bb90-47d8-955c-12ccdd30ebce@oss.qualcomm.com> Content-Language: en-US From: Kathiravan Thirumoorthy In-Reply-To: <6e33daf1-bb90-47d8-955c-12ccdd30ebce@oss.qualcomm.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-Authority-Analysis: v=2.4 cv=IsATsb/g c=1 sm=1 tr=0 ts=69b7c777 cx=c_pps a=rz3CxIlbcmazkYymdCej/Q==:117 a=j4ogTh8yFefVWWEFDRgCtg==:17 a=IkcTkHD0fZMA:10 a=Yq5XynenixoA:10 a=s4-Qcg_JpJYA:10 a=VkNPw1HP01LnGYTKEx00:22 a=u7WPNUs3qKkmUXheDGA7:22 a=3WHJM1ZQz_JShphwDgj5:22 a=EUspDBNiAAAA:8 a=dbLGCUDwb5mCwCoKLG4A:9 a=QEXdDO2ut3YA:10 a=bFCP_H2QrGi7Okbo017w:22 X-Proofpoint-Spam-Details-Enc: AW1haW4tMjYwMzE2MDA2OCBTYWx0ZWRfX9d6yKc4URUNm WN38AnXx0ylqXG70ZIzDFbVZ9VNXLwmO4nqzOECqj6Nc5iT95w/da6QjrXJftQ1C6rLDm3QqzwY LPhs/PnysZTepYVrS848fbb2Ic1kzgfDIvektcRG9JctylTzUbZx1Zf86xKuXTTPvv2besL80ma prJloUuLMeFB0+VKzFeA4LeCWPWy/50zIYxLuZveS25lwk4TnjCq8t4tvzo8sKIwEBakmdJowFX AaTh/NjAD+ALNej9Xq7J9a0oFfgRKRvEXlqVG1z+7+4KwH+Gt647rt4t60YEZ8K1+I3tGMvZNi0 jfb8K90Jwsl37mq5Zh7xyJL9pR63pBV94iEbbeA3SU2BE31vxYqjXr9K66D/a7Xw5IiwkDSIfTt EaCO9BtkdkSaTLUF3kQ0ObxftM8pJZ3VVOXYCooVqQmmgKG6lve+8H9Eyj3aH1izxJQu9BgbTfv Hx83lfkuJyLeR8xPucg== X-Proofpoint-GUID: wnEmCxIwfxQQ1TOHFKJKagpkvwEhHuF1 X-Proofpoint-ORIG-GUID: wnEmCxIwfxQQ1TOHFKJKagpkvwEhHuF1 X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1143,Hydra:6.1.51,FMLib:17.12.100.49 definitions=2026-03-16_03,2026-03-13_01,2025-10-01_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 spamscore=0 adultscore=0 priorityscore=1501 bulkscore=0 malwarescore=0 lowpriorityscore=0 impostorscore=0 phishscore=0 suspectscore=0 clxscore=1015 classifier=typeunknown authscore=0 authtc= authcc= route=outbound adjust=0 reason=mlx scancount=1 engine=8.22.0-2603050001 definitions=main-2603160068 On 3/13/2026 6:18 PM, Konrad Dybcio wrote: > On 3/11/26 10:45 AM, Kathiravan Thirumoorthy wrote: >> Add initial device tree support for the Qualcomm IPQ5210 SoC and >> rdp504 board. >> >> Signed-off-by: Kathiravan Thirumoorthy >> --- >> arch/arm64/boot/dts/qcom/Makefile | 1 + >> arch/arm64/boot/dts/qcom/ipq5210-rdp504.dts | 93 +++++++++ >> arch/arm64/boot/dts/qcom/ipq5210.dtsi | 304 ++++++++++++++++++++++++++++ >> 3 files changed, 398 insertions(+) >> >> diff --git a/arch/arm64/boot/dts/qcom/Makefile b/arch/arm64/boot/dts/qcom/Makefile >> index 02921a495b2cbabcbacc74fbbb99eafe1f6478ac..e7748af640cccffa5c83ec82c37aa441444c2b13 100644 >> --- a/arch/arm64/boot/dts/qcom/Makefile >> +++ b/arch/arm64/boot/dts/qcom/Makefile >> @@ -22,6 +22,7 @@ hamoa-iot-evk-el2-dtbs := hamoa-iot-evk.dtb x1-el2.dtbo >> dtb-$(CONFIG_ARCH_QCOM) += hamoa-iot-evk-el2.dtb >> dtb-$(CONFIG_ARCH_QCOM) += ipq5018-rdp432-c2.dtb >> dtb-$(CONFIG_ARCH_QCOM) += ipq5018-tplink-archer-ax55-v1.dtb >> +dtb-$(CONFIG_ARCH_QCOM) += ipq5210-rdp504.dtb >> dtb-$(CONFIG_ARCH_QCOM) += ipq5332-rdp441.dtb >> dtb-$(CONFIG_ARCH_QCOM) += ipq5332-rdp442.dtb >> dtb-$(CONFIG_ARCH_QCOM) += ipq5332-rdp468.dtb >> diff --git a/arch/arm64/boot/dts/qcom/ipq5210-rdp504.dts b/arch/arm64/boot/dts/qcom/ipq5210-rdp504.dts >> new file mode 100644 >> index 0000000000000000000000000000000000000000..3d95939785bc5bd4f510e2f992f0a1e80848c8de >> --- /dev/null >> +++ b/arch/arm64/boot/dts/qcom/ipq5210-rdp504.dts >> @@ -0,0 +1,93 @@ >> +// SPDX-License-Identifier: BSD-3-Clause >> +/* >> + * Copyright (c) Qualcomm Technologies, Inc. and/or its subsidiaries. >> + */ >> + >> +/dts-v1/; >> + >> +#include "ipq5210.dtsi" >> + >> +/ { >> + model = "Qualcomm Technologies, Inc. IPQ5210 RDP504"; >> + compatible = "qcom,ipq5210-rdp504", "qcom,ipq5210"; >> + >> + aliases { >> + serial0 = &uart1; >> + }; >> + >> + chosen { >> + stdout-path = "serial0"; >> + }; >> + >> + soc@0 { >> + qupv3: geniqup@1ac0000 { >> + status = "okay"; >> + >> + uart1: serial@1a84000 { > Please define this in the SoC DTSI and enable it via a &uart1 {} reference > from the board DT Ack. > > [...] > >> + xo: xo { > "xo_board" is the convention in other DTs Ack. > > [...] > >> + L2_0: l2-cache { > labels must apparently be lowercase nowadays Ack. > >> + compatible = "cache"; >> + cache-level = <0x2>; > = <2> Ack. > > >> + cache-unified; >> + }; >> + }; >> + >> + firmware { >> + optee { >> + compatible = "linaro,optee-tz"; >> + method = "smc"; >> + }; >> + >> + scm { >> + compatible = "qcom,scm-ipq5210", "qcom,scm"; >> + }; > OP-TEE *and* SCM? What does the former do? For OP-TEE, we have use cases like getting the random numbers, remoteproc PAS and few others. For SCM, setting the download mode is one typical example where OP-TEE doesn't come into picture. Please let me know if this helps. > > [...] > >> + soc@0 { >> + compatible = "simple-bus"; >> + #address-cells = <2>; >> + #size-cells = <2>; >> + ranges = <0 0 0 0 0x10 0>; > should dma-ranges also be 36-bit? Ack. > > [...] > >> + qupv3: geniqup@1ac0000 { >> + compatible = "qcom,geni-se-qup"; >> + reg = <0x0 0x01ac0000 0x0 0x2000>; >> + clocks = <&gcc GCC_QUPV3_AHB_MST_CLK>, >> + <&gcc GCC_QUPV3_AHB_SLV_CLK>; >> + clock-names = "m-ahb", "s-ahb"; >> + ranges; >> + #address-cells = <2>; >> + #size-cells = <2>; >> + >> + status = "disabled"; > I see there's only a single QUP wrapper - therefore I'm going to assume > there will never be a SoC where this needs to be disabled Ack. > > [...] > >> + >> + uart1: serial@1a84000 { >> + compatible = "qcom,geni-debug-uart"; >> + reg = <0x0 0x01a84000 0x0 0x4000>; >> + clocks = <&gcc GCC_QUPV3_WRAP_SE1_CLK>; >> + clock-names = "se"; >> + interrupts = ; >> + status = "disabled"; > Please uniformly keep a \n before status, file-wide Ack. > > Konrad