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 3AD2030F55B for ; Sat, 9 May 2026 04:28:15 +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=1778300897; cv=none; b=i8xwKh3BHcVrTCFp4NkCtjhr1CTlkjeSSJ3Jrhcvb8qwwbwokQLvybqP0MNgDydJM2A+c6YddgGs50R63V79vFJS8B+hoaGphzjd5jvBIrdmBm56kSU8Ekf3vWfFBUbvh1w+WAfRlwXN7zApLvWJxtipxUN1uEi2kdj8GPDG8Pc= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778300897; c=relaxed/simple; bh=lH56q4JFas0JYn6KEZjtVCEh+1cdQmJoEz76ASZMq44=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=PfJPtW85AGaXe9V9kOdxh78sNkgdMzAkIrYJ2QZ+i9DJnHfC9c+CFqnT4TFuuSy11YCESym0v+kcI7dZL18ti6mnJBztl3i8IlvAlDwcjnjVA6C8HQM9alzR/1jPXAoBTkK/yksVOIZinhGCSUBTfTXt5yG0dslj3x3/nnnlRZQ= 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=cM5L8nbx; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b=TzfvGZGa; 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="cM5L8nbx"; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b="TzfvGZGa" Received: from pps.filterd (m0279870.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.18.1.11/8.18.1.11) with ESMTP id 6493Po3k2089517 for ; Sat, 9 May 2026 04:28:14 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=qualcomm.com; h= cc:content-type:date:from:in-reply-to:message-id:mime-version :references:subject:to; s=qcppdkim1; bh=oeQVFdzspTdGZ8kTnotfv2Cm F2zsoti19Tmhiiv7Yak=; b=cM5L8nbx53y4q7l3eniWE0tFnv3TmJSjXZA9rRwm O8dBN2D5vd51jTSqV5XJhVOXLaGSBkvQh1IIQR/AzqctrDDxWe2EEU0XoXWJhV4q Jn/Epim/54JuKiQvBvA4lI+3akvzsDXRUH9dXkjLRbgdkS9dn1BE94UzurKQHAO9 BqEeSQm76vfKA+TeRc9whkrSG/6WwL/kxZdu36b+RjRuYJF7MqV1QIPgfzBm3AEL Q3xW5N6OXK6l2Entd2oGLEhGi0pNbjvF09rR21JEW2j43fP/U5L7Sb6jxLAzJe3A ykGHOfMqPZXVoVI6SnMZb5eyv7XkVfL2gEe8RvLRLYTzlw== Received: from mail-dy1-f200.google.com (mail-dy1-f200.google.com [74.125.82.200]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 4e1w0ug3h3-1 (version=TLSv1.3 cipher=TLS_AES_128_GCM_SHA256 bits=128 verify=NOT) for ; Sat, 09 May 2026 04:28:14 +0000 (GMT) Received: by mail-dy1-f200.google.com with SMTP id 5a478bee46e88-2ef62078ee7so3335766eec.0 for ; Fri, 08 May 2026 21:28:14 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oss.qualcomm.com; s=google; t=1778300893; x=1778905693; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=oeQVFdzspTdGZ8kTnotfv2CmF2zsoti19Tmhiiv7Yak=; b=TzfvGZGarj6Z+vB810CDPE8Z+ziBpL0MSvH6KDuWzrufgXBhi/pdy4jzHX8b9z4lD9 PO76u1O5wxkg10OaqW8iRK+SLbys8iTdN9uxtUXcfKw0dvFuXJ/NEV7lw9CCGHzfEvND pQCQJE0UXHncPCjRC4xbHNC2znGwLqsHeUBuByy9yxUkuBnUWBR++uf7uZSchXFxEITv qW2oHRM32BEvKbfG73ULiom/byJ/eydd5zc/lfaOXXFqNb1Zet9Pwdi0DVyE2j/R2ClT ghEIIbovHPV5HK8/ATEiq2LdTW9SCfrYcm2jGEHirfBLYQ2qtkr/P4Had7bo1aWMSg4N Basw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1778300893; x=1778905693; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-gg:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=oeQVFdzspTdGZ8kTnotfv2CmF2zsoti19Tmhiiv7Yak=; b=PGImrMYmtXFI1lMwt8OE0cV/we7nR2fXCK+6dm64mXQF0kTHL+lnJUM97pVnuGfpr+ 48TF1GEnclgeQ6aovEHRF4vlMiYAkaMEkw64JU86XD8Ws9X+MFRNU6k5wmJIjkr0WeL2 SV8BvQnaF+GTl8q+TsPJAWY9b+lMaWYvQjjzuDD2DQbbyeVZb8NXVwSPGv+BQUY4BOkV ok0xS/pyewupSl+d1+EPoB0Ji6VXnpAlVfQIn0m7n+yiRIxlys+jUKMJcvDmofyAabK+ 2UgTZFD3G1Kq1ilGGqJxkVKRu2rj4mticS077qI+A3PNSW8Wvj6SBpRGPbdoGTXryNYT TorQ== X-Forwarded-Encrypted: i=1; AFNElJ8ixyew9O10Bo1rRQrNW6bt2Pp8ncH3fLXuIekyQulNtTk80bZgXxut8f4CtnYib/n7kpqhIZh4Mg==@vger.kernel.org X-Gm-Message-State: AOJu0YzNrB8sDgmcImrtu3gCaxPDzeSJs5obxSa5ZMQ/FyXECaG7Niir GQ+gCvgYpMpe+8zuhcpKrzf+RZDp/Iy0cywHpohpK+cDMg+Emw0henaUbzja0g643INVj6DXn/2 Fz3ulixegpIEmF2tucdOI27RfD/AXQsEnpD6CSVqmKEHFRAwiQgAkhlBn/lkIRg== X-Gm-Gg: Acq92OEyr10vI521vMsYHtWFm4t3ncDXSvXcX01pCdvAzmmVFjMAPjFGRBvZNKIS/W2 yiLAHLMQHDvLbTmMeXFdnZscEJHIfkPnTeeZ23WtXtOLlCHMy4kiIszr5L9NkHVfjpjY0knuory G07jucPEInJ+Tiusk1rF0UvvayyvqfMWbAqGC6GxW3+3//Swy8PmdKqng8dFkhfgBcXwlQCNQ1t bmDplsN0sVaeb67G5NAurP1yVK506dhFYWLBw2wkDtqboavzjdAP7mGyRkFNpI/lBMox/cONWIz MoDSSglPL8T2jlnh1YNcu0GcuTfyd8HALTyFntdwSYb6tJrDd98V+ahCDjDJpSyStsepVOhlrsj ADXXvaxXnDywlIRK7AQdCUoeUffT7DweySXgIy+z1JoYGfFIAI3Ci4zH1f0YtkiP4 X-Received: by 2002:a05:7300:b919:b0:2dd:c066:c02 with SMTP id 5a478bee46e88-2f54a87e845mr8084041eec.22.1778300893042; Fri, 08 May 2026 21:28:13 -0700 (PDT) X-Received: by 2002:a05:7300:b919:b0:2dd:c066:c02 with SMTP id 5a478bee46e88-2f54a87e845mr8084023eec.22.1778300892454; Fri, 08 May 2026 21:28:12 -0700 (PDT) Received: from QCOM-aGQu4IUr3Y (i-global052.qualcomm.com. [199.106.103.52]) by smtp.gmail.com with ESMTPSA id 5a478bee46e88-2f8893441absm6553109eec.31.2026.05.08.21.28.08 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 08 May 2026 21:28:12 -0700 (PDT) Date: Sat, 9 May 2026 12:28:06 +0800 From: Shawn Guo To: Georgi Djakov Cc: Rob Herring , Krzysztof Kozlowski , Conor Dooley , Dmitry Baryshkov , Odelu Kukatla , Konrad Dybcio , Bartosz Golaszewski , Deepti Jaggi , linux-arm-msm@vger.kernel.org, linux-pm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 2/2] interconnect: qcom: Add interconnect provider driver for Nord SoC Message-ID: References: <20260420021351.1239355-1-shengchao.guo@oss.qualcomm.com> <20260420021351.1239355-3-shengchao.guo@oss.qualcomm.com> <27a2c0aa-ca50-4bdf-a735-f4375ffdc8bb@kernel.org> Precedence: bulk X-Mailing-List: linux-pm@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <27a2c0aa-ca50-4bdf-a735-f4375ffdc8bb@kernel.org> X-Proofpoint-GUID: TGjIuwFVMDtmz0470ZwMVMq6vG2APs9l X-Authority-Analysis: v=2.4 cv=asaCzyZV c=1 sm=1 tr=0 ts=69feb7de cx=c_pps a=PfFC4Oe2JQzmKTvty2cRDw==:117 a=b9+bayejhc3NMeqCNyeLQQ==:17 a=kj9zAlcOel0A:10 a=NGcC8JguVDcA:10 a=s4-Qcg_JpJYA:10 a=VkNPw1HP01LnGYTKEx00:22 a=u7WPNUs3qKkmUXheDGA7:22 a=gowsoOTTUOVcmtlkKump:22 a=EUspDBNiAAAA:8 a=_ycruYcDSZZNFFtkL0EA:9 a=CjuIK1q_8ugA:10 a=6Ab_bkdmUrQuMsNx7PHu:22 X-Proofpoint-ORIG-GUID: TGjIuwFVMDtmz0470ZwMVMq6vG2APs9l X-Proofpoint-Spam-Details-Enc: AW1haW4tMjYwNTA5MDA0MiBTYWx0ZWRfX4Y/9m7MQRYDi KpPngJWuXdQ1YflPch1WAp+U/uU+7VfdqR0NsPYWEXhpoZS7m/LH14FQJlautJk0avSp4LUuccy mAGDI3b68cPn8kmBFAJHDCFPsBLjyJ8EpDkLob/1Amg5nXYDor/gsJJwNkzQbsOsGV9hv22Y/gm +iwwC9k/ZGQ5f7V3p5HYMqHYaendeZwBqrNN7V6TWJ8ShM2qyE8RRHo/t5mqqnT1CrYpuRR8oxa tflnZN5V60baqVT4RyhpGGHWmvqjQnkIUFiOr6jxzyylWTfnviUlcsZkqEwOE1jrVfx/A5l2kQx MDUWAnqdmdtqXDRPs+73vtJC98+ETB2Z1ILdTqILNmXQxK79/KKQgyGOJdaMDxLbIytsytjhRUl bwhMQZKIlRIazSaNu5hnB5f91CLOvWj4bYYfEO3mvYtFPBgLsuZzBEEdbJhElqzyHkqapuRddZy KZhUXCreS/byz173s2w== 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-05-09_02,2026-05-08_02,2025-10-01_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 impostorscore=0 phishscore=0 spamscore=0 lowpriorityscore=0 bulkscore=0 adultscore=0 clxscore=1015 suspectscore=0 priorityscore=1501 malwarescore=0 classifier=typeunknown authscore=0 authtc= authcc= route=outbound adjust=0 reason=mlx scancount=1 engine=8.22.0-2604200000 definitions=main-2605090042 On Fri, May 08, 2026 at 11:41:01PM +0300, Georgi Djakov wrote: > On 4/20/26 5:13 AM, Shawn Guo wrote: > > From: Odelu Kukatla > > > > Add driver for the Qualcomm interconnect buses found on Nord SoC. > > The topology consists of several NoCs that are controlled by > > a remote processor that collects the aggregated bandwidth for each > > master-slave pair. > > > > Signed-off-by: Odelu Kukatla > > Signed-off-by: Shawn Guo > > --- > > drivers/interconnect/qcom/Kconfig | 11 + > > drivers/interconnect/qcom/Makefile | 2 + > > drivers/interconnect/qcom/nord.c | 2682 ++++++++++++++++++++++++++++ > > 3 files changed, 2695 insertions(+) > > create mode 100644 drivers/interconnect/qcom/nord.c > > > [..] > > +static struct qcom_icc_node * const aggre1_noc_nodes[] = { > > + [MASTER_QSPI_0] = &qhm_qspi, > > + [MASTER_SAILSS_MD1] = &qnm_sailss_md1, > > + [MASTER_QUP_3] = &qxm_qup02, > > Maybe this should be qxm_qup3 ? Yes, it makes more sense considering QUP_0/1/2 naming. [MASTER_QUP_0] = &qhm_qup0, [MASTER_QUP_1] = &qhm_qup1, [MASTER_QUP_2] = &qhm_qup2, > > > + [SLAVE_A1NOC_SNOC] = &qns_a1noc_snoc, > > +}; > [..] > > +static struct qcom_icc_node * const cnoc_cfg_nodes[] = { > > + [MASTER_CNOC_CFG] = &qsm_cfg, > > + [SLAVE_PS_ETH_0] = &ps_eth_0, > > + [SLAVE_PS_ETH_1] = &ps_eth_1, > > + [SLAVE_SHS_SERVER] = &ps_shs_server, > > + [SLAVE_AHB2PHY_0] = &qhs_ahb2phy0, > > + [SLAVE_AHB2PHY_1] = &qhs_ahb2phy1, > > + [SLAVE_AHB2PHY_2] = &qhs_ahb2phy2, > > + [SLAVE_AHB2PHY_3] = &qhs_ahb2phy3, > > + [SLAVE_AHB2PHY_ETH_0] = &qhs_ahb2phy_eth_0, > > + [SLAVE_AHB2PHY_ETH_1] = &qhs_ahb2phy_eth_1, > > + [SLAVE_CAMERA_CFG] = &qhs_camera_cfg, > > + [SLAVE_CLK_CTL] = &qhs_clk_ctl, > > + [SLAVE_CRYPTO_0_CFG] = &qhs_crypto0_cfg, > > + [SLAVE_CRYPTO_1_CFG] = &qhs_crypto1_cfg, > > + [SLAVE_CRYPTO_2_CFG] = &qhs_crypto2_cfg, > > + [SLAVE_DISPLAY_1_CFG] = &qhs_display_1_cfg, > > + [SLAVE_DISPLAY_CFG] = &qhs_display_cfg, > > + [SLAVE_DPRX0] = &qhs_dprx0, > > + [SLAVE_DPRX1] = &qhs_dprx1, > > + [SLAVE_EVA_CFG] = &qhs_eva_cfg, > > + [SLAVE_GFX3D_CFG] = &qhs_gpuss_0_cfg, > > + [SLAVE_GFX3D_1_CFG] = &qhs_gpuss_1_cfg, > > + [SLAVE_I2C] = &qhs_i2c, > > + [SLAVE_IMEM_CFG] = &qhs_imem_cfg, > > + [SLAVE_MCW_PCIE] = &qhs_mcw_pcie, > > + [SLAVE_MM_RSCC] = &qhs_mm_rscc, > > + [SLAVE_NE_CLK_CTL] = &qhs_ne_clk_ctl, > > + [SLAVE_NSPSS0_CFG] = &qhs_nspss0_cfg, > > + [SLAVE_NSPSS1_CFG] = &qhs_nspss1_cfg, > > + [SLAVE_NSPSS2_CFG] = &qhs_nspss2_cfg, > > + [SLAVE_NSPSS3_CFG] = &qhs_nspss3_cfg, > > + [SLAVE_NW_CLK_CTL] = &qhs_nw_clk_ctl, > > + [SLAVE_PRNG] = &qhs_prng, > > + [SLAVE_QDSS_CFG] = &qhs_qdss_cfg, > > + [SLAVE_QSPI_0] = &qhs_qspi, > > + [SLAVE_QUP_0] = &qhs_qup0, > > + [SLAVE_QUP_3] = &qhs_qup02, > > qhs_qup3 maybe? Indeed! > > + [SLAVE_QUP_1] = &qhs_qup1, > > + [SLAVE_QUP_2] = &qhs_qup2, > > + [SLAVE_SAFEDMA_CFG] = &qhs_safedma_cfg, > > + [SLAVE_SDCC_4] = &qhs_sdc4, > > + [SLAVE_SE_CLK_CTL] = &qhs_se_clk_ctl, > > + [SLAVE_TCSR] = &qhs_tcsr, > > + [SLAVE_TLMM] = &qhs_tlmm, > > + [SLAVE_TSC_CFG] = &qhs_tsc_cfg, > > + [SLAVE_UFS_MEM_CFG] = &qhs_ufs_mem_cfg, > > + [SLAVE_USB2] = &qhs_usb2, > > + [SLAVE_USB3_0] = &qhs_usb3_0, > > + [SLAVE_USB3_1] = &qhs_usb3_1, > > + [SLAVE_VENUS_CFG] = &qhs_venus_cfg, > > + [SLAVE_COMPUTENOC_CFG] = &qss_computenoc_cfg, > > + [SLAVE_PCIE_NOC_CFG] = &qss_pcie_noc_cfg, > > + [SLAVE_QTC_CFG] = &qss_qtc_cfg, > > + [SLAVE_QDSS_STM] = &xs_qdss_stm, > > + [SLAVE_SYS_TCU0_CFG] = &xs_sys_tcu0_cfg, > > + [SLAVE_SYS_TCU1_CFG] = &xs_sys_tcu1_cfg, > > + [SLAVE_SYS_TCU2_CFG] = &xs_sys_tcu2_cfg, > > +}; > [..] > > +static const struct regmap_config nord_hscnoc_regmap_config = { > > + .reg_bits = 32, > > + .reg_stride = 4, > > + .val_bits = 32, > > + .max_register = 0x45080, > > Is this value correct? The qosbox offsets for some nodes go beyond this. The HSCNOC max port_offset is 0xa45000, so the correct max_register should be 0xa45080. I appreciate that you spot this error! > > + .fast_io = true, > > +}; > > + > > The rest looks good! Thank you for the review, Georgi! Shawn