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 416E222156C for ; Tue, 14 Apr 2026 05:08:27 +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=1776143308; cv=none; b=lk7Ue/coZntpyqnGJe8F0UX1X+BrJgu2my6wQHIBPjOsJiSy43xdoqn+cLEZAJpP4qsJjwWfmHvd/xRD0o4Wkqc4IuPm0jpGsbDRMdPspU0rUMejpJlamroSDSA0xCXkrEIUF/RIRm/FYk00qxH4HkLq3etLEzOqvZpSFuwkR8Y= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1776143308; c=relaxed/simple; bh=lI71pR7b1UtY+HJX/qJKan+BvKisGJTcDsTu9l2I034=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=aU6TZY3NZWuw28unwtjQ6MCKXA6ZKFNCri8y2dpacfUvz1EZ8c982PysR31gfMwDmaxuiJ0FkxDfPAFQvK5hGv+ADtaE9X4w2DOaH/gxzSPrsxd5X4HZiuhGLUlb0Xl9jNTB5aYpKtUXsy5mjQoPa4Tbg1j8cZUND+0WIzchUjM= 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=ctu95rg2; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b=YLxS7+U+; 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="ctu95rg2"; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b="YLxS7+U+" Received: from pps.filterd (m0279873.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.18.1.11/8.18.1.11) with ESMTP id 63DLB2Ql2510017 for ; Tue, 14 Apr 2026 05:08:26 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= Fend+ODuKiVJMX0bQ/3KqhrC40puAzg7wd7+IABXEz4=; b=ctu95rg2oTUEsUgL Ydvj+yBEuZBteguOdTX1WmL9U/ZkXJaXdvKGocIODuY+XIDgkHkthkgllg1aOpQO OXv35QY/+GOGGAcd0lPaHHnsbHIlQ2XPrNtUwxK7fPJnPpq4Bbjp/mnAq5LgqNpm Yq8UZ8s/5yk6DWRr7wIgEtfD38FGfhhvbmnEEOIESI+rsDD1egOCzRj+Z3A5w3we ZdxyD6XbteYim90U95MQ1bnaxPt+6Bo9uuxFJqUZfzZ2Z5MckgJnNA41zgDyv6bV //63wVZTCm4osJyNWc5A4F53SBxkE4YrfIQkkOhB7MH6p1HJnnDHUIqkYHPzoNFw fYldZg== Received: from mail-pj1-f72.google.com (mail-pj1-f72.google.com [209.85.216.72]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 4dh867s2t5-1 (version=TLSv1.3 cipher=TLS_AES_128_GCM_SHA256 bits=128 verify=NOT) for ; Tue, 14 Apr 2026 05:08:26 +0000 (GMT) Received: by mail-pj1-f72.google.com with SMTP id 98e67ed59e1d1-35e59791605so2203676a91.1 for ; Mon, 13 Apr 2026 22:08:26 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oss.qualcomm.com; s=google; t=1776143305; x=1776748105; 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=Fend+ODuKiVJMX0bQ/3KqhrC40puAzg7wd7+IABXEz4=; b=YLxS7+U+RuB6n7izTDyjiNj5q24so/F2HT3YmzY49d39bkr7WQZNEm0Rt46juHgHwu EXKlw5iu0UG9OQPiTbV2RQCL8zuiIqJu/fHWAzN+UPSa7DF/m4vMqyq2I/RkhvTV7Jng d36/feD9Ih+nn4ITe/CDDevTEtjmYvEvXFoUdKuhPdkcaRn4kN1om43oVx2IPuWyw7/1 tq66KR+r03359TSgdGGTz6bIp92TjTJjKmjMUV0d6Pxm6hVNYETqd7qhwqCPKNJBkTJc xJQ953A56mKtnH5DUTJ4244IZusW8lYU8t0ABN4ubkJ2CKo3DehXWFJlSLiuK6jL5eXn NRjw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1776143305; x=1776748105; 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=Fend+ODuKiVJMX0bQ/3KqhrC40puAzg7wd7+IABXEz4=; b=FP7d7xClHxg2WpK4B6ZMI3SToQ0JdA49Q1fRD7PYntCeExF5nQbiWZbZxX+mh+PPaP gktKBcTdkX3InIm16CltX+9o7LkoCXPO1WRUUZd1rT6omdh85gI1Ke81DSyYTJkrWLBn 7oTeRUphGAG3euBMgc/AiGenPi5RTYIA1sZFLweOovy9tbGMOPk6dirfi3/doxBFuXD2 tLQ9XBUnwiPmuTgIoYBmIOKPckgnr5NiPOyu+by3gHwhbuBlsk6yCW0M9D7UvIQds5gD csMK+GHyq1Xr4tLGnhuR7I1U60fN2cJBzQ8TESKsxj+OJnJ4AaefGfc9IUKgSuVExA/b pqeA== X-Forwarded-Encrypted: i=1; AFNElJ+DNuRT1XYuCilyOrxbTMKx6IbnznF6gnhLeDdgog9g/kCV9XJO3UL3EmZSBmYL0Ehl3UC2X1S1GqngF9Y=@vger.kernel.org X-Gm-Message-State: AOJu0YzZJA3BVG+8x2zMNxso9X4Lmow/rVqgfPlARQEBm2RS0xLyrTbV vrCblYaBsaIs6Isg9dIReO1eYnbBri5JevidYK9/AYP+PAtV2tacBJaZwBnA1p8MJ5+O8ghHOAn 1lCcn7sRjrFLnKJtOAcTMKP7RhD7sxwLA7znDBYM+chgVRFkViDZp+9ofUOtqsOmjYNM= X-Gm-Gg: AeBDietfwPrtvT4K2PquYj2+XCKaWg+gpsByEYNG4H3g/R1ulBb41fuFgJS7BlCcpsh bIeRdVlN6+I/Dzj1l/sAQfc4IZBherGxAbIzh9ZyiZovatJA5JHuXNKPA5lf8zTJ8doUDSPoqb+ DFryEc5qK48BBjTtQ3KhquRDOtiwujeDROgMLmv5IQ1a9WU+Y2DDuUSJQku3qmcGsRxbvlcdCP/ 66/Jqn9S5Zf/OCVAURSpo2W2nHz6oCQpxeND5N9MRKPzEYocrhFT1yNHyYdZF/b2qWiXBPlUhMZ +UCyOgvPgRVyfDV3HeaaSQCXXIlXzthSYaRBrGwCRTd3fxBa7dlvVVQGQ268fZv5KDnLD7eT/UZ TA1oTmjIO7e3zHfVsPEXAzrr2reC2r1OQ66kp+dnEfqR/YgAHSyUpjSBuwyGJfT7RsK1bnPRnZ8 MS2mZiVzNsmANmZS4= X-Received: by 2002:a05:6a00:808c:b0:829:6f7d:3093 with SMTP id d2e1a72fcca58-82f0c27862bmr14821299b3a.48.1776143305369; Mon, 13 Apr 2026 22:08:25 -0700 (PDT) X-Received: by 2002:a05:6a00:808c:b0:829:6f7d:3093 with SMTP id d2e1a72fcca58-82f0c27862bmr14821264b3a.48.1776143304792; Mon, 13 Apr 2026 22:08:24 -0700 (PDT) Received: from [10.133.33.94] (tpe-colo-wan-fw-bordernet.qualcomm.com. [103.229.16.4]) by smtp.gmail.com with ESMTPSA id d2e1a72fcca58-82f0c371825sm12754570b3a.25.2026.04.13.22.08.20 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 13 Apr 2026 22:08:24 -0700 (PDT) Message-ID: <4aa94982-bcae-4335-b70d-4b0b4cdd697b@oss.qualcomm.com> Date: Tue, 14 Apr 2026 13:08:19 +0800 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 v5 4/5] remoteproc: qcom: pas: Add late attach support for subsystems To: Jie Gan , Bjorn Andersson , Mathieu Poirier , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Manivannan Sadhasivam , Luca Weiss , Bartosz Golaszewski , Konrad Dybcio Cc: aiqun.yu@oss.qualcomm.com, tingwei.zhang@oss.qualcomm.com, trilok.soni@oss.qualcomm.com, yijie.yang@oss.qualcomm.com, linux-arm-msm@vger.kernel.org, linux-remoteproc@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Gokul Krishna Krishnakumar References: <20260409-knp-soccp-v5-0-805a492124da@oss.qualcomm.com> <20260409-knp-soccp-v5-4-805a492124da@oss.qualcomm.com> Content-Language: en-US From: Jingyi Wang In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit X-Proofpoint-ORIG-GUID: yKdUGEPpdRhIiOC0W49qHRlCL1rp24Se X-Proofpoint-GUID: yKdUGEPpdRhIiOC0W49qHRlCL1rp24Se X-Proofpoint-Spam-Details-Enc: AW1haW4tMjYwNDE0MDA0NiBTYWx0ZWRfX99h9+/m6v8T6 O6bkV2wp6hxLmQee6lPy7ETNprUaIbNYJ9kfYsf+8szrw03HFYWUI0SzEMkDKZayJjNcWiH48KI rq2XtBT19bpefTz6QsbzL3FzzvYHZQVO5A0goo6JvS7Xg/9iYHE7XPYSslQKkV97t5fW33uAchb /VKj4LL/botQFF8ii4hXLJennPbJhNS5V9UeEzIIbRJvu5HcyvKAqbPHQ4XYBuWAqaezYsfzQ33 z65fjfPwRy0UUDMPz+qMpzWYEvUpqEnuJCjlIMkKYenj6diXRC69xy2Q6AqzIxzjOGW59EtljRP LcowkJL8w9JZBbKwh0jEoqUHqnSX/UZWfskGyrw5F4PIC0DP4md7tCZr6usXya8LPjkebIacmkK wRmUDiDkw4ydag3JmqOy5YZ4L6nBV6yySMc0T+1g4CZC0ERuWjY2bDuFsoHzfCB3+XlYIcqtn/Y PNHRMXzKm++rymc+4dg== X-Authority-Analysis: v=2.4 cv=etzvCIpX c=1 sm=1 tr=0 ts=69ddcbca cx=c_pps a=RP+M6JBNLl+fLTcSJhASfg==:117 a=nuhDOHQX5FNHPW3J6Bj6AA==:17 a=IkcTkHD0fZMA:10 a=A5OVakUREuEA:10 a=s4-Qcg_JpJYA:10 a=VkNPw1HP01LnGYTKEx00:22 a=u7WPNUs3qKkmUXheDGA7:22 a=rJkE3RaqiGZ5pbrm-msn:22 a=OYuZKRwE9jNIGw8p0IgA:9 a=3ZKOabzyN94A:10 a=QEXdDO2ut3YA:10 a=iS9zxrgQBfv6-_F4QbHw:22 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-04-14_01,2026-04-13_04,2025-10-01_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 malwarescore=0 phishscore=0 lowpriorityscore=0 adultscore=0 impostorscore=0 priorityscore=1501 bulkscore=0 suspectscore=0 spamscore=0 clxscore=1015 classifier=typeunknown authscore=0 authtc= authcc= route=outbound adjust=0 reason=mlx scancount=1 engine=8.22.0-2604070000 definitions=main-2604140046 On 4/11/2026 2:04 PM, Jie Gan wrote: > > > On 4/9/2026 4:52 PM, Jingyi Wang wrote: <...> >>   static const struct rproc_ops qcom_pas_ops = { >>       .unprepare = qcom_pas_unprepare, >>       .start = qcom_pas_start, >> @@ -518,6 +582,7 @@ static const struct rproc_ops qcom_pas_ops = { >>       .parse_fw = qcom_pas_parse_firmware, >>       .load = qcom_pas_load, >>       .panic = qcom_pas_panic, >> +    .attach = qcom_pas_attach, > > Possible issue in the future here. The kaanapali_soccp_resource does not set minidump_id, so this is not triggered today, but it is a latent bug for any future device that sets both early_boot and minidump_id. > > qcom_pas_attach is added to qcom_pas_ops but not to qcom_pas_minidump_ops. When a device with minidump_id set uses the minidump ops table, the .attach pointer is NULL. rproc_attach_device() checks if (rproc->ops->attach) before calling it, so the attach callback is silently skipped. For a device with early_boot = true and minidump_id != 0, the state is set to RPROC_DETACHED in probe, but the attach logic (crash check, ready check, ping) is never executed, leaving the subsystem in an inconsistent state. > > Thanks, > Jie > ACK, we can add qcom_pas_attach to minidumps_ops in next version. Thanks, Jingyi >>   }; >>   static const struct rproc_ops qcom_pas_minidump_ops = { >> @@ -855,6 +920,15 @@ static int qcom_pas_probe(struct platform_device *pdev) >>       pas->pas_ctx->use_tzmem = rproc->has_iommu; >>       pas->dtb_pas_ctx->use_tzmem = rproc->has_iommu; >> + >> +    if (desc->early_boot) { >> +        ret = qcom_q6v5_ping_subsystem_init(&pas->q6v5, pdev); >> +        if (ret) >> +            dev_warn(&pdev->dev, "Falling back to firmware load\n"); >> +        else >> +            pas->rproc->state = RPROC_DETACHED; >> +    } >> + >>       ret = rproc_add(rproc); >>       if (ret) >>           goto remove_ssr_sysmon; >> >