From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id ECA49EB64D9 for ; Fri, 7 Jul 2023 19:55:09 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231745AbjGGTzI (ORCPT ); Fri, 7 Jul 2023 15:55:08 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:36920 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229600AbjGGTzE (ORCPT ); Fri, 7 Jul 2023 15:55:04 -0400 Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 978671FEE for ; Fri, 7 Jul 2023 12:55:02 -0700 (PDT) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 36E7961A33 for ; Fri, 7 Jul 2023 19:55:02 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id E02AFC433C7; Fri, 7 Jul 2023 19:55:00 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1688759701; bh=RsJA1J7DRjoIYo78M1XUEsFCF0c2xLVidbDZoutsGgc=; h=Date:Subject:To:Cc:References:From:In-Reply-To:From; b=jLoea8gSW34amoR1P+rYB9Zkeyoy8A6WiK/PJuaBaa544/JL7RXe3lt/MMfGf4Jrk V7fUz988bzZbghWlkyYpZ0ashvOkNFxw9h8tIK+8ZL+mjIbSD7wyANqlZcS7Gy2p+Z vmRSHFPMU1lskxyWzX6X2hHrjUIPEM8OGw/SWhm4d/5wVwA1kDEjxU9zpBXB/bnvCg y+6AwzrXLaewXf1GgL4G+87C0R73yCUHXkXI3qsFtqSNFbOtV6COx8yVm7DEvX0Qck u0IEKD1Fkl7nAWlmnB+QleEMpvq9ZjkoOcMkznih/cfmwAtvJ3tz7vSnOez8JQAmd9 C+56DBSlCg05g== Message-ID: <6c7ec9f8-6dae-8db6-c5ba-e88b641b34e7@kernel.org> Date: Fri, 7 Jul 2023 14:54:59 -0500 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.11.0 Subject: Re: [PATCH 2/2 RESEND] firmware: stratix10-rsu: query spt addresses Content-Language: en-US To: kah.jing.lee@intel.com Cc: linux-kernel@vger.kernel.org, radu.bacrau@intel.com, tien.sung.ang@intel.com References: <20230707080112.1722827-1-kah.jing.lee@intel.com> <20230707080349.1723083-1-kah.jing.lee@intel.com> From: Dinh Nguyen In-Reply-To: <20230707080349.1723083-1-kah.jing.lee@intel.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 7/7/23 03:03, kah.jing.lee@intel.com wrote: > From: Radu Bacrau > > Extend Intel Remote System Update (RSU) driver to get SPT > (Sub-Partition Table) addresses. The query SPT address can be used > to determine if the RSU QSPI layout is 32kB or 64kB aligned. > The alignment can be determined by minus the upper with the lower of > the SPT addresses. > > This patch depends on patch: > firmware: stratix10-svc: Generic Mailbox Command > > Signed-off-by: Radu Bacrau > Signed-off-by: Kah Jing Lee > --- > drivers/firmware/stratix10-rsu.c | 100 ++++++++++++++++++++++++++++++- > 1 file changed, 99 insertions(+), 1 deletion(-) > > diff --git a/drivers/firmware/stratix10-rsu.c b/drivers/firmware/stratix10-rsu.c > index e51c95f8d445..9f82d5147890 100644 > --- a/drivers/firmware/stratix10-rsu.c > +++ b/drivers/firmware/stratix10-rsu.c > @@ -34,6 +34,10 @@ > #define INVALID_RETRY_COUNTER 0xFF > #define INVALID_DCMF_VERSION 0xFF > #define INVALID_DCMF_STATUS 0xFFFFFFFF > +#define INVALID_SPT_ADDRESS 0x0 > + > +#define RSU_GET_SPT_CMD 0x5A > +#define RSU_GET_SPT_RESP_LEN (4 * sizeof(unsigned int)) > > typedef void (*rsu_callback)(struct stratix10_svc_client *client, > struct stratix10_svc_cb_data *data); > @@ -59,6 +63,9 @@ typedef void (*rsu_callback)(struct stratix10_svc_client *client, > * @dcmf_status.dcmf3: dcmf3 status > * @retry_counter: the current image's retry counter > * @max_retry: the preset max retry value > + * @spt0_address: address of spt0 > + * @spt1_address: address of spt1 > + * @get_spt_response_buf: response from sdm for get_spt command > */ > struct stratix10_rsu_priv { > struct stratix10_svc_chan *chan; > @@ -90,6 +97,11 @@ struct stratix10_rsu_priv { > > unsigned int retry_counter; > unsigned int max_retry; > + > + unsigned long spt0_address; > + unsigned long spt1_address; > + > + unsigned int *get_spt_response_buf; > }; > > /** > @@ -259,6 +271,36 @@ static void rsu_dcmf_status_callback(struct stratix10_svc_client *client, > complete(&priv->completion); > } > > +static void rsu_get_spt_callback(struct stratix10_svc_client *client, > + struct stratix10_svc_cb_data *data) > +{ > + struct stratix10_rsu_priv *priv = client->priv; > + unsigned long *mbox_err = (unsigned long *)data->kaddr1; > + unsigned long *resp_len = (unsigned long *)data->kaddr2; > + > + if ((data->status != BIT(SVC_STATUS_OK)) || (*mbox_err) || > + (*resp_len != RSU_GET_SPT_RESP_LEN)) > + goto error; > + > + priv->spt0_address = priv->get_spt_response_buf[0]; > + priv->spt0_address <<= 32; > + priv->spt0_address |= priv->get_spt_response_buf[1]; > + > + priv->spt1_address = priv->get_spt_response_buf[2]; > + priv->spt1_address <<= 32; > + priv->spt1_address |= priv->get_spt_response_buf[3]; > + > + goto complete; > + > +error: > + dev_err(client->dev, "failed to get SPTs\n"); > + > +complete: > + stratix10_svc_free_memory(priv->chan, priv->get_spt_response_buf); > + priv->get_spt_response_buf = NULL; > + complete(&priv->completion); > +} > + > /** > * rsu_send_msg() - send a message to Intel service layer > * @priv: pointer to rsu private data > @@ -288,6 +330,14 @@ static int rsu_send_msg(struct stratix10_rsu_priv *priv, > if (arg) > msg.arg[0] = arg; > > + if (command == COMMAND_MBOX_SEND_CMD) { > + msg.arg[1] = 0; > + msg.payload = NULL; > + msg.payload_length = 0; > + msg.payload_output = priv->get_spt_response_buf; > + msg.payload_length_output = RSU_GET_SPT_RESP_LEN; > + } > + > ret = stratix10_svc_send(priv->chan, &msg); > if (ret < 0) > goto status_done; > @@ -572,6 +622,34 @@ static ssize_t notify_store(struct device *dev, > return count; > } > > +static ssize_t spt0_address_show(struct device *dev, > + struct device_attribute *attr, char *buf) > +{ > + struct stratix10_rsu_priv *priv = dev_get_drvdata(dev); > + > + if (!priv) > + return -ENODEV; > + > + if (priv->spt0_address == INVALID_SPT_ADDRESS) > + return -EIO; > + > + return scnprintf(buf, PAGE_SIZE, "0x%08lx\n", priv->spt0_address); > +} > + > +static ssize_t spt1_address_show(struct device *dev, > + struct device_attribute *attr, char *buf) > +{ > + struct stratix10_rsu_priv *priv = dev_get_drvdata(dev); > + > + if (!priv) > + return -ENODEV; > + > + if (priv->spt1_address == INVALID_SPT_ADDRESS) > + return -EIO; > + > + return scnprintf(buf, PAGE_SIZE, "0x%08lx\n", priv->spt1_address); > +} > + > static DEVICE_ATTR_RO(current_image); > static DEVICE_ATTR_RO(fail_image); > static DEVICE_ATTR_RO(state); > @@ -590,6 +668,8 @@ static DEVICE_ATTR_RO(dcmf2_status); > static DEVICE_ATTR_RO(dcmf3_status); > static DEVICE_ATTR_WO(reboot_image); > static DEVICE_ATTR_WO(notify); > +static DEVICE_ATTR_RO(spt0_address); > +static DEVICE_ATTR_RO(spt1_address); > > static struct attribute *rsu_attrs[] = { > &dev_attr_current_image.attr, > @@ -610,6 +690,8 @@ static struct attribute *rsu_attrs[] = { > &dev_attr_dcmf3_status.attr, > &dev_attr_reboot_image.attr, > &dev_attr_notify.attr, > + &dev_attr_spt0_address.attr, > + &dev_attr_spt1_address.attr, > NULL > }; > > @@ -639,11 +721,13 @@ static int stratix10_rsu_probe(struct platform_device *pdev) > priv->dcmf_version.dcmf1 = INVALID_DCMF_VERSION; > priv->dcmf_version.dcmf2 = INVALID_DCMF_VERSION; > priv->dcmf_version.dcmf3 = INVALID_DCMF_VERSION; > - priv->max_retry = INVALID_RETRY_COUNTER; > priv->dcmf_status.dcmf0 = INVALID_DCMF_STATUS; > priv->dcmf_status.dcmf1 = INVALID_DCMF_STATUS; > priv->dcmf_status.dcmf2 = INVALID_DCMF_STATUS; > priv->dcmf_status.dcmf3 = INVALID_DCMF_STATUS; > + priv->max_retry = INVALID_RETRY_COUNTER; > + priv->spt0_address = INVALID_SPT_ADDRESS; > + priv->spt1_address = INVALID_SPT_ADDRESS; > > mutex_init(&priv->lock); > priv->chan = stratix10_svc_request_channel_byname(&priv->client, > @@ -693,6 +777,20 @@ static int stratix10_rsu_probe(struct platform_device *pdev) > stratix10_svc_free_channel(priv->chan); > } > > + priv->get_spt_response_buf = > + stratix10_svc_allocate_memory(priv->chan, RSU_GET_SPT_RESP_LEN); > + > + if (!priv->get_spt_response_buf) { This is wrong, stratix10_svc_allocate_memory does not return NULL, it returns an ERR_PTR() > + dev_err(dev, "failed to allocate get spt buffer\n"); > + } else { > + int ret_val = rsu_send_msg(priv, COMMAND_MBOX_SEND_CMD, > + RSU_GET_SPT_CMD, rsu_get_spt_callback); Reuse the already defined ret; > + if (ret_val) { > + dev_err(dev, "Error, getting SPT table %i\n", ret_val); > + stratix10_svc_free_channel(priv->chan); > + } > + } > + > return ret; > } >