From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pf1-f172.google.com (mail-pf1-f172.google.com [209.85.210.172]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id D006724DCE2 for ; Thu, 11 Sep 2025 19:00:11 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.210.172 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1757617213; cv=none; b=DzFDiGahF/ob47+oKwC2i7WZGTCAswxU4RpHRMI8p1dXq0veO33KdXglRCWg5XrxxsZMO/rw3YiLaG77gvPkSHPgyRuphvBSy8wqVJW0VmdId9ZFjLUtOvOXKWJN3EB4pIgYKwsyr7ofztrJOevBBEh7nAOx2RqLoRP55j+8v9A= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1757617213; c=relaxed/simple; bh=jEuIH5ZZU/EeovpVRYUTQgyVitJw0q9Q0W9biAJS/lc=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=VIerG9V9KaTCPqaIFH9lNTt+CNwVgwMlxGSHKslkgd6Pri+mhjLt78MKmwt2sRi0D6yIE3YpRTOWXCAWgulTh6jo4tPJULX20FTvM+GixsEuJZmnJTeYimHI5oebndXxGutlgQi0U3aM8YN9Ph2IF/2QJ1FLPnLSP5Z8nhQXG4s= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=chromium.org; spf=pass smtp.mailfrom=chromium.org; dkim=pass (1024-bit key) header.d=chromium.org header.i=@chromium.org header.b=a4CJlwA6; arc=none smtp.client-ip=209.85.210.172 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=chromium.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=chromium.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=chromium.org header.i=@chromium.org header.b="a4CJlwA6" Received: by mail-pf1-f172.google.com with SMTP id d2e1a72fcca58-772679eb358so996112b3a.1 for ; Thu, 11 Sep 2025 12:00:11 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org; s=google; t=1757617211; x=1758222011; 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=03MaJ+6ytyPItqY2qyW/Ajcv1dYvzwe/SXOVragEWgY=; b=a4CJlwA6VVy3uyYy7GblD9FTXrIHb3du8k7MPiIQOeNw7U7Du8t/SxkfXUsQal8lOQ MMB5JeEk+CO+6xCxtVm/wtcQuyoyUWnF1TvGrcMcT5gHd25nitXVwmOMbdl1jhIoXYMl tnwu3idsErMW9kh4uGukwd94ZIVr6mpIjvm9Q= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1757617211; x=1758222011; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=03MaJ+6ytyPItqY2qyW/Ajcv1dYvzwe/SXOVragEWgY=; b=BTnOK5/4CeqV0c7rx4Fh2pOX9KIhXK6OLXs0HL571mEAPNL7aEhQJVjqbPyjMsAGXZ nf/O7Q/Wr2D7rZGIHbDREOgiChhV2PbtY/5sqJ/qQ2wiThzgwAB5zvALx1oDj0WjFIpB lY2nKElxaqAvU8PZNII9Sdbxeo+M7OsX6xWt4ABm8CU6I4nDRkoPJyntp7OEv0DHaobO wDAkGiWSPChf7Fn5vA//Tdset97YriFe3QdIXME+ySUoULJAWAQ332MpoNgEcdmrMVCd +hWlQv9zkHg1hdaqr+xDZJoId7gH0SN2QIH8tgaBCLm7wQCVG2VHREpiQH92QcYF2Olo WAgQ== X-Forwarded-Encrypted: i=1; AJvYcCX53XyP1mnxVZDgUQIqXg6yr2pgLwvocAK70Z05LAdDnYppgIEuHYLG0WzJV45bXsy2XvexVSIrrWvn@vger.kernel.org X-Gm-Message-State: AOJu0YwPXAjI92myz1SIxwfWzJRkes9yXEZCD67J3vI8oR482tMhG4va iuikKi6SnOlBMfAgyYGyVSizK91nSudSK4yDIJyiFNkwEFgzEirEqwgXDjxdYaLe5w== X-Gm-Gg: ASbGncs4xew5HG5hAPsqYzH+EmcC9R+ZscV+sVBuxMoHZi/CU48FbH/ZAu/7hW04HkF 2ivoOlyoOO4QM2q7NGQrOHk4An1e4gGo/1an3om2v8f609u6lG3R9JCUb7kCRx60xNgk3mtbgOl xrmBHu3OmmfxA8+YZY4VNB+qQGLm8R3UvCho+edyGOPUq+KROUOfhR5ALt5S1Z29Vzk3aUQeVtq O4wqOxl3vI5suThNux/6Mg1PzhPn3/A10WI2kw7Gn/+lIhUPULV2+JCGL0gzmORMPHZXZS1xLwC w4kBen89KKwYWLRWprZddBPBUJH5OEDFQce4QORK82xvkyXe8J9Ivdae0oFCXcWQ3zOxcbcIvLz WwWOXuxZCely0oroS1a1VtDmFRixIB/YbbrJewzy9EVtPVTolwG+i19ST6+q0ACUUPKbFqmXQU9 46sX8z X-Google-Smtp-Source: AGHT+IF+E6MG4K4ru1HMLDWmmyjU/+acTljGFHni8hj8gJV4DBQdRTFv1WxeplPA7OnCTPEFvHrPHQ== X-Received: by 2002:a17:902:d592:b0:246:441f:f144 with SMTP id d9443c01a7336-25d27c2236fmr3853315ad.56.1757617209513; Thu, 11 Sep 2025 12:00:09 -0700 (PDT) Received: from localhost ([2a00:79e0:2e14:7:6690:568:13de:b368]) by smtp.gmail.com with UTF8SMTPSA id d9443c01a7336-25c3b305427sm25200305ad.138.2025.09.11.12.00.07 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 11 Sep 2025 12:00:08 -0700 (PDT) Date: Thu, 11 Sep 2025 12:00:06 -0700 From: Brian Norris To: Konrad Dybcio Cc: Bjorn Andersson , Konrad Dybcio , Georgi Djakov , Odelu Kukatla , cros-qcom-dts-watchers@chromium.org, Conor Dooley , linux-kernel@vger.kernel.org, linux-arm-msm@vger.kernel.org, Krzysztof Kozlowski , Rob Herring , Douglas Anderson , devicetree@vger.kernel.org Subject: Re: [PATCH v2 2/2] arm64: dts: qcom: sc7280: Drop aggre{1,2}_noc QOS clocks on Herobrine Message-ID: References: <20250825155557.v2.1.I018984907c1e6322cf4710bd1ce805580ed33261@changeid> <20250825155557.v2.2.Idebf1d8bd8ff507462fef9dc1ff47e84c01e9b60@changeid> <90b13660-1844-4701-8e63-7fde2f093db0@oss.qualcomm.com> Precedence: bulk X-Mailing-List: devicetree@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: <90b13660-1844-4701-8e63-7fde2f093db0@oss.qualcomm.com> Hi Konrad, On Tue, Sep 02, 2025 at 02:02:15PM +0200, Konrad Dybcio wrote: > On 8/26/25 12:55 AM, Brian Norris wrote: > > Ever since these two commits > > > > fbd908bb8bc0 ("interconnect: qcom: sc7280: enable QoS configuration") > > 2b5004956aff ("arm64: dts: qcom: sc7280: Add clocks for QOS configuration") > > > > Herobrine systems fail to boot due to crashes like the following: > > > > [ 0.243171] Kernel panic - not syncing: Asynchronous SError Interrupt > > [ 0.243173] CPU: 7 UID: 0 PID: 1 Comm: swapper/0 Not tainted 6.11.0 #1 c5464041cff584ced692726af2c4400fa2bde1db > > [ 0.243178] Hardware name: Qualcomm Technologies, Inc. sc7280 CRD platform (rev5+) (DT) > > [ 0.243180] Call trace: > > [ 0.243182] dump_backtrace+0x104/0x128 > > [ 0.243194] show_stack+0x24/0x38 > > [ 0.243202] __dump_stack+0x28/0x38 > > [ 0.243208] dump_stack_lvl+0x28/0xb8 > > [ 0.243211] dump_stack+0x18/0x30 > > [ 0.243215] panic+0x134/0x340 > > [ 0.243219] nmi_panic+0x48/0x98 > > [ 0.243227] arm64_serror_panic+0x6c/0x80 > > [ 0.243245] arm64_is_fatal_ras_serror+0xd8/0xe0 > > [ 0.243261] do_serror+0x5c/0xa8 > > [ 0.243265] el1h_64_error_handler+0x34/0x48 > > [ 0.243272] el1h_64_error+0x7c/0x80 > > [ 0.243285] regmap_mmio_read+0x5c/0xc0 > > [ 0.243289] _regmap_bus_reg_read+0x78/0xf8 > > [ 0.243296] regmap_update_bits_base+0xec/0x3a8 > > [ 0.243300] qcom_icc_rpmh_probe+0x2d4/0x490 > > [ 0.243308] platform_probe+0xb4/0xe0 > > [...] > > > > Specifically, they fail in qcom_icc_set_qos() when trying to write the > > QoS settings for qhm_qup1. Several of the previous nodes (qhm_qspi, > > qhm_qup0, ...) seem to configure without crashing. > > > > We suspect that the TZ firmware on these devices does not expose QoS > > regions to Linux. The right solution here might involve deleting both > > 'clocks' and 'reg', but 'reg' would cause more problems. Linux is > > already OK with a missing 'clocks', since pre-2b5004956aff DTBs need to > > be supported, so we go with an easier solution. > > Just to make sure I'm reading this right - the clocks enable just fine, > but it's the writes to the QoS settings that trigger the hang? Yes. > Any chance skipping qhm_qup1 specifically makes things better? Yes, it seems so. Or specifically, this diff: --- a/drivers/interconnect/qcom/sc7280.c +++ b/drivers/interconnect/qcom/sc7280.c @@ -52,12 +52,6 @@ static struct qcom_icc_node qhm_qup1 = { .id = SC7280_MASTER_QUP_1, .channels = 1, .buswidth = 4, - .qosbox = &(const struct qcom_icc_qosbox) { - .num_ports = 1, - .port_offsets = { 0x8000 }, - .prio = 2, - .urg_fwd = 0, - }, .num_links = 1, .links = { SC7280_SLAVE_A1NOC_SNOC }, }; > Could you please share your exact software version (which I assume is really > just the version of TF-A in this case) so I can try and reproduce it? I'm not much of an expert on the makeup of QCOM firmware, but reading my firmware logs, that'd be: coreboot-v1.9308_26_0.0.22-32067-g641732a20a and BL31: v2.8(debug):v2.8-776-g0223d1576 IIUC, the latter points to TF-A hash: 0223d15764ed Merge "feat(docs): allow verbose build" into integration Brian