From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-oi1-f169.google.com (mail-oi1-f169.google.com [209.85.167.169]) (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 58F8C1419A9 for ; Wed, 24 Dec 2025 17:36:18 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.167.169 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1766597779; cv=none; b=bnhrjAV29c6BfgTaGrDFpPcgR+Dnxgukd4FRfTX4by6vLmTJUObrGSwySVXCrD6uWOzKcls+zwjZpPb6KU55YvZsvprBgYSeuFSkRYxo8C4fqFFxrMeD1rXqd27Oo8WjCj0NGBVBA1x5LM1OgPCPwZwYuy19t/xtBfq2oprZl18= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1766597779; c=relaxed/simple; bh=PClKipvoSN/NnumpcyN+/KuxzqPi200kimFq8t05q80=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=Hro5A+oxNyW1AqzIXSFTCB1mAlHhbh6FMdhB2BNtpn5kHCWsIDl/03cWjwQaYYP0E7D7HeD052zRyEHceN/0C40LMwBND+SU28I6xEMXgFG1GY0MkFPLV8XpmjZda/e8y7gPdjYhfDsHbLau/YXI9r7yGh0pkyyKEFp/uVi4gQM= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=RY0BaL3V; arc=none smtp.client-ip=209.85.167.169 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="RY0BaL3V" Received: by mail-oi1-f169.google.com with SMTP id 5614622812f47-4511744b411so3277537b6e.3 for ; Wed, 24 Dec 2025 09:36:18 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1766597777; x=1767202577; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=MpMnqFtDTmoVp19QSLvEcIFu/SJ/IsYwE4qFGJyAisg=; b=RY0BaL3VQxJqm3nUyGuB9UkykFhgSsccUewP6vtaZVXcVUKEBbJlo6Zi5jp67LgYdV 1hgRqip5/B2CbV895GzvcHOz0fC+flxZh84ATLflP+XlwGMCAj1AaZDh+puTKKzA/hY5 T4wpYxDgTzZcm5PM92GGy70+pmnl4fpY1BNp28gg6Z3kGvp3vgTkdgnCXu4TcXIcMerC yEsJr7VVLk0hQoazDsDUkJxV9D/bD3SLmpeS3SpDjwpyQFdlAqNSSI6bS+GOjxcJ///s CtJJJZVErS+brQK8P95D6Rl8nIryZ6lqCqy4udAH5IeJTb2j685nZQrTbF0gUAwJeB1Y dviQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1766597777; x=1767202577; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to; bh=MpMnqFtDTmoVp19QSLvEcIFu/SJ/IsYwE4qFGJyAisg=; b=W+K8ABmn2/9TBC6ILg7g8Q4XacaurZvTaeuBTWhLjXFps7aS5T4pqddTmEz4HyKA15 uKJ2A5J9kGlWj8bpGuapzpNybJSffD0rWa1YGi0nJHe+bFNXGDa5HDw8UEQAejg1EilF GnGEzsOAveCRBT0eILtpTdwSDpuMU0GoKDxVfCuQpe8sBrQFozBHENrgX1frXmlb/3E1 +NsfsPhPSZHeEfhFJ+UM4nfdiy7hjKxBvxyPOkCXyaJNFkpVH6EhPGfp16MsNwhacBXd dweBG8jMt5jIMEORjNSpf25Eb3fSLa6B+YALPZ5DVVuBZdMET344YijE/POA6pTqxi/3 NXxg== X-Forwarded-Encrypted: i=1; AJvYcCVMxfvzhiuCnpIefOXcH8s3wIHZbxXLVtbulLogiKQn068WdnA/O70XEEgyzFoPj/3GYO79aFuOa2J9/IE=@vger.kernel.org X-Gm-Message-State: AOJu0Yyynn3/72QuY85AqdSg22n36n4EpGgAkakc1GwcFH9sZUqEgKSo NKAKfyNpvcawwQ6iv1gC3tPLmMbbno70JPNfWUVWbgK4SRabjjB9Mg8fbnPXHg== X-Gm-Gg: AY/fxX4yo74/vQXzqXURt4uVOOAKP1+g82JiLz4lUBct6CVBOa6zNd33dmHnOqFSoiu qrdv9Kovv1FpnskHQnmxSai0Ybv8/Zn/COW+zsf3GujOxBbEWbR0xFeQPQvysNXBJ2afPMZO61g lUM5QF0GXnrtDxh71g89poByq2xtjUPkpSb2w0VV9aj1pgwloc2BDl89W4RfGX6QY4WGcOLqGRY DiedMfCu/Zdi9dkCYtZEZjfTDpFe5JbqeQNslHHuO+PIXBLACTGFpyvVZ9kC2M7TcBfjoG5Ir9h LhJkX36BmOebddLBG6MbewYO9pQ3xpOU+1fOt2MswQrCo13a91te9OxXdpJHQPGNCFwyxKZ1C4m bm97jqEtVSDpAEvMDhwzXVjePSK2xjrQpS2QpUAmmM/KBE1HnDIOE8xqt2aMesU7DnVlyrZO+AK qC5Dqwpz81yWpm80IosiayYTyZoBeXBGsSpDAN5tSv6lEjLCwjkitkW5JmLdGwLMLNq6VlxpxAp ipzvbG2oycz8zlto9UOPp04ud7YvZY= X-Google-Smtp-Source: AGHT+IEwhNLGB/ObbcXq24oPZkJatmLwE/LdvbRV3KFdoUMFsBjOOpCavIyiLN9TSPdAOX5X01lMsQ== X-Received: by 2002:a05:6808:1202:b0:451:4c6d:a638 with SMTP id 5614622812f47-457b20140eamr8904780b6e.7.1766597777141; Wed, 24 Dec 2025 09:36:17 -0800 (PST) Received: from nukework.gtech (c-98-57-15-22.hsd1.tx.comcast.net. [98.57.15.22]) by smtp.gmail.com with ESMTPSA id 586e51a60fabf-3fdaab620b8sm10695688fac.14.2025.12.24.09.36.15 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 24 Dec 2025 09:36:16 -0800 (PST) From: "Alex G." To: andersson@kernel.org, mathieu.poirier@linaro.org, Konrad Dybcio , linux-kernel@vger.kernel.org, Vignesh Viswanathan Cc: krzk+dt@kernel.org, Philipp Zabel , linux-arm-msm@vger.kernel.org, linux-remoteproc@vger.kernel.org Subject: Re: [PATCH 6/9] remoteproc: qcom_q6v5_wcss: support IPQ9574 Date: Wed, 24 Dec 2025 11:36:15 -0600 Message-ID: <7564740.MhkbZ0Pkbq@nukework.gtech> In-Reply-To: <41616f5e-68a6-4354-8520-4321e747efc9@oss.qualcomm.com> References: <20251219043425.888585-1-mr.nuke.me@gmail.com> <12223416.nUPlyArG6x@nukework.gtech> <41616f5e-68a6-4354-8520-4321e747efc9@oss.qualcomm.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" On Wednesday, December 24, 2025 3:44:56 AM CST Vignesh Viswanathan wrote: > On 12/24/2025 1:51 AM, Alex G. wrote: > > On Friday, December 19, 2025 7:20:04 AM CST Konrad Dybcio wrote: > >> On 12/19/25 5:34 AM, Alexandru Gagniuc wrote: > >>> Q6 based firmware loading is also present on IPQ9574, when coupled > >>> with a wifi-6 device, such as QCN5024. Populate driver data for > >>> IPQ9574 with values from the downstream 5.4 kerrnel. > >>> > >>> Add the new sequences for the WCSS reset and stop. The downstream > >>> 5.4 kernel calls these "Q6V7", so keep the name. This is still worth > >>> using with the "q6v5" driver because all other parts of the driver > >>> can be seamlessly reused. > >>> > >>> The IPQ9574 uses two sets of clocks. the first, dubbed "q6_clocks" > >>> must be enabled before the Q6 is started by writing the Q6SS_RST_EVB > >>> register. The second set of clocks, "clks" should only be enabled > >>> after the Q6 is placed out of reset. Otherwise, the host CPU core that > >>> tries to start the remoteproc will hang. > >>> > >>> The downstream kernel had a funny comment, "Pray god and wait for > >>> reset to complete", which I decided to keep for entertainment value. > >>> > >>> Signed-off-by: Alexandru Gagniuc > >>> --- > >> > >> [...] > >> > >>> @@ -128,6 +137,12 @@ struct q6v5_wcss { > >>> > >>> struct clk *qdsp6ss_xo_cbcr; > >>> struct clk *qdsp6ss_core_gfmux; > >>> struct clk *lcc_bcr_sleep; > >>> > >>> + struct clk_bulk_data *clks; > >>> + /* clocks that must be started before the Q6 is booted */ > >>> + struct clk_bulk_data *q6_clks; > >> > >> "pre_boot_clks" or something along those lines? > > > > I like "pre_boot_clocks". > > > >> In general i'm not super stoked to see another platform where manual and > >> through-TZ bringup of remoteprocs is supposed to be supported in > >> parallel.. > >> > >> Are you sure your firmware doesn't allow you to just do a simple > >> qcom_scm_pas_auth_and_reset() like in the multipd series? > > > > I am approaching this from the perspective of an aftermarket OS, like > > OpenWRT. I don't know if the firmware will do the right thing. I can > > mitigate this for OS-loaded firmware, like ath11k 16/m3 firmware, because > > I can test the driver and firmware together. I can't do that for > > bootloader-loaded firmware, so I try to depend on it as little as > > possible. I hope that native remoterproc loading for IPQ9574 will be > > allowed. Hi Vignesh, > Does this rproc start sequence work on IPQ9574 without using the > qcom_scm_pas_auth_and_reset ? Yes, it works as presented in this series, without qcom_scm_pas_auth_and_reset(). Alex