From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-ed1-f50.google.com (mail-ed1-f50.google.com [209.85.208.50]) (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 7394634AB09 for ; Thu, 13 Nov 2025 12:12:24 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.208.50 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763035947; cv=none; b=tvZHlwk5LQWeqj/PhoktJhjshmUmHnVBfXdfDd38MCLYLmGcCNZx2MNLDKrj2Cz9fttT3ApqqRopIUK51DxC0zb6Ur7l93bJo6AJ4DFKb7Sw3lLQfnqpUsueW+px6eoQpZLyX5u3J1Z4TFQSiX3gNCNCjDXvzJS8pGWx6c+keeQ= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763035947; c=relaxed/simple; bh=tIRyBtq6ZgmwZ/RdMREj1gR29I0tL8lowm1gb91/zSU=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=rAfVzUMPPu9zuF68oVTx6VR0i07AUCPdTPMiAiFq6EHq9FS/iJ+2+eiI3hiCFMemOUn3Yf+HQkwkgHBKbT6P6Ag+EKQXBzWIzviR6uvJ7xw7tjnqlbyw9nngk6J1pQRSWcjiGwbjmwun0sXhN0KJx3ga5rpfPO4uHwtVLQAe5lM= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=CYNF0ml1; arc=none smtp.client-ip=209.85.208.50 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="CYNF0ml1" Received: by mail-ed1-f50.google.com with SMTP id 4fb4d7f45d1cf-640860f97b5so1185939a12.2 for ; Thu, 13 Nov 2025 04:12:24 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1763035943; x=1763640743; 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=ZmL32TLheuHbN2HDaUa3UG1QnGtlOG9N/Q5N775z6d0=; b=CYNF0ml1DpleNUfz+G+2lL9hhJrYQUWslZOJEJWeGWEs6q9d8EhrKwnukmTypyCk5e i4oJiVHSZF5EQTq5CLRYHRHr0JctLV6pRJTInSCEO2WrjGW0vTwzBMBooviHxzvuzxRa thld1GQu+rf+00Xjt4dEjBuU55mjjMlPj3Ty482vLZ1oUT64CxGKf0t7hCG0h11YOHSC COGZYFk5DJMpaepmWN1NjFBEvvCv1BdLXaJJKoWL8cI7AF45m+dgi4z0BlXeEPAw1crt YUIZa+v6Wqtt02fy+NoA/fIBp4sR0iyE3ZLdH8kMmKpHzcDcFDOCpL1yuC/fTXyAOqOP tJxw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1763035943; x=1763640743; 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=ZmL32TLheuHbN2HDaUa3UG1QnGtlOG9N/Q5N775z6d0=; b=BoSpl4L82wJJes9zoGBTVzfVoZeXpLK7CND6hF/51TB4+x6na3aaDdOY9mw5iZkHld TyxYAVN47yLO+jv9qaxu8sE3JxS0hLsZjOLt6KIbXxzm359fINDmZzX6vlT4yzyK0gG6 qSO2vkiIUHG3DjBBGv8Q9h8Ow3rY1u/xNnivQtGJQ4CmWmIYqtaWYBEiFjOZ+KDfuIkl /0JggszTKZxtlR7Tv3EzmZhBuqiYheX5PoCrwFjrrE8cPiWnNCKNz2A6DCb8tPc1ABx/ /y6VUAGzg9WitgD5uykmyH90o00Fo3A6j8L/a7zR/JBOcjakJHtEFjJqbcFMRrLwMVCs W83Q== X-Forwarded-Encrypted: i=1; AJvYcCXK+C/SZmG3O+ev2C1V6adz8FsAn8ftGF47NDS/OmexQCs6p6v5vFpQ3WHLdczOMoEYFzSVtIJSonA5@vger.kernel.org X-Gm-Message-State: AOJu0YyxVheu7e5pwzaIsy+7aoy9xC43S8vXO3t00RMriPcd+8c7WQMP 7OznJapWQ5f7uTFvJENQiiU3mMvsscVHvBiQJ9jyxMU8XtTGIWEpGev5mDIhuwqSUus= X-Gm-Gg: ASbGncsWY/8ascZgVU5x2w9f+wZZUCDIUiWCB2SY5KI3gXdcfsc/v08WH16JQPLavWv I/XRs9oB74Sb31Lfh48goQQUVNjllGZ5tXtVFIiZb75NEctHGbCDShTmU97GBNDkmzb93+kK4Pi 5+fNWc1mAZLHz22aYzXqt7eRM9xiD2IMaLX0OWkdg0ugyK07pz+t6HLThLYywIAGVW/qttvZDcR nO8ItDAr3eUeDhP6Qn6Xg3gRj4RR9CDk+2kiX4+nv64plZLMQ8AD8RWbKDlgKVqvailgZRxxVIZ HnTEmQ1Fx7QPMbDMx57XRiTmpBad/VojUlfhrLWssz0Oc/6z6TRMxklcq7BmPaVAatUi3DyLHrb K+3R2JyrUolGPdlK/zfTVYJX4bmgFmTVdOqcXeoQCKzWzDMZ0GkGF3FDDlVMjU+yCD2KvG+2gi8 /CoyovKZvBfPpNPrITNNMGMME= X-Google-Smtp-Source: AGHT+IES54zSRW8sTH8TIe84JLMl9O+9k+g7kEG/EMcHoTIpOAkv6hlmsk6iKOP7BgiRYf1ZJ102EA== X-Received: by 2002:a17:907:6ea6:b0:b72:c103:88db with SMTP id a640c23a62f3a-b7331960ea3mr649017866b.9.1763035942693; Thu, 13 Nov 2025 04:12:22 -0800 (PST) Received: from linaro.org ([2a02:2454:ff23:4430:e68d:9e37:1627:2b9b]) by smtp.gmail.com with ESMTPSA id a640c23a62f3a-b734fd80d27sm151837166b.40.2025.11.13.04.12.21 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 13 Nov 2025 04:12:22 -0800 (PST) Date: Thu, 13 Nov 2025 13:12:20 +0100 From: Stephan Gerhold To: Mukesh Ojha Cc: Bjorn Andersson , Mathieu Poirier , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Manivannan Sadhasivam , Konrad Dybcio , linux-arm-msm@vger.kernel.org, linux-remoteproc@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Bryan O'Donoghue , Konrad Dybcio Subject: Re: [PATCH v7 09/14] firmware: qcom_scm: Simplify qcom_scm_pas_init_image() Message-ID: References: <20251113-kvm-rproc-v7-v7-0-df4910b7c20a@oss.qualcomm.com> <20251113-kvm-rproc-v7-v7-9-df4910b7c20a@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: <20251113-kvm-rproc-v7-v7-9-df4910b7c20a@oss.qualcomm.com> On Thu, Nov 13, 2025 at 04:05:59PM +0530, Mukesh Ojha wrote: > Simplify qcom_scm_pas_init_image() by making the memory allocation, > copy and free operations done in a separate function than the actual > SMC call. Nitpick: This feels more like "refactoring in preparation of the following patch" rather than a real simplification. > > Reviewed-by: Bryan O'Donoghue > Reviewed-by: Konrad Dybcio > Signed-off-by: Mukesh Ojha > --- > drivers/firmware/qcom/qcom_scm.c | 58 +++++++++++++++++++++++----------------- > 1 file changed, 33 insertions(+), 25 deletions(-) > > diff --git a/drivers/firmware/qcom/qcom_scm.c b/drivers/firmware/qcom/qcom_scm.c > index 550707f2b46d..1d8a2c537c44 100644 > --- a/drivers/firmware/qcom/qcom_scm.c > +++ b/drivers/firmware/qcom/qcom_scm.c > @@ -590,6 +590,37 @@ void *devm_qcom_scm_pas_context_init(struct device *dev, u32 pas_id, phys_addr_t > } > EXPORT_SYMBOL_GPL(devm_qcom_scm_pas_context_init); > > +static int __qcom_scm_pas_init_image(u32 pas_id, dma_addr_t mdata_phys, void *metadata, > + size_t size, struct qcom_scm_res *res) The metadata and size parameters do not seem to be used inside this function (also not after your follow-up patches), do we need them here? > +{ > + struct qcom_scm_desc desc = { > + .svc = QCOM_SCM_SVC_PIL, > + .cmd = QCOM_SCM_PIL_PAS_INIT_IMAGE, > + .arginfo = QCOM_SCM_ARGS(2, QCOM_SCM_VAL, QCOM_SCM_RW), > + .args[0] = pas_id, > + .owner = ARM_SMCCC_OWNER_SIP, > + }; > + int ret; > + > + ret = qcom_scm_clk_enable(); > + if (ret) > + return ret; > + > + ret = qcom_scm_bw_enable(); > + if (ret) > + goto disable_clk; > + > + desc.args[1] = mdata_phys; > + > + ret = qcom_scm_call(__scm->dev, &desc, res); > + qcom_scm_bw_disable(); > + > +disable_clk: > + qcom_scm_clk_disable(); > + > + return ret; > +} Thanks, Stephan