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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 00F63C7EE23 for ; Wed, 7 Jun 2023 08:01:53 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Transfer-Encoding: Content-Type:In-Reply-To:From:References:Cc:To:Subject:MIME-Version:Date: Message-ID:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=aXlaebkIIuXQ0e+ILa/iloMj3qy1Iz7mdy9cDpaae1Q=; b=DZPgbkFd5fd+JZo8ubq66mgZJg J4WV+tCivY7Gk81ok5fRVXdqUzqhgAvxAOTnggmVlfB8WdscOrYCyyPUmgSMZnvdb9Vd5UnAIx1zo bIWYsJk7Ynq1iMSbSyjp+GeOkfWo5Purrom9LFqmVWMV1RZJlMUEP5GecUVZdNeF/drV3dhb8j/bc xQiBmTxXDe3t2D/AwB7A+DBaSJWOxxgPM7u+K2GKyz5S8vfgITNe9cSpixZlzs3DVy92voILDAZr2 awQGzP2vatRJGPPG4s9Dgwvcy0pVNvjVgB5ydzI4FNdh7QriN+/WgrlHjAPuUlPsQLTtb6Mqdu3jl phfqFHMw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1q6o72-004qaN-2l; Wed, 07 Jun 2023 08:01:48 +0000 Received: from madras.collabora.co.uk ([46.235.227.172]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1q6o6z-004qYv-2F; Wed, 07 Jun 2023 08:01:47 +0000 Received: from [IPV6:2001:b07:2ed:14ed:a962:cd4d:a84:1eab] (unknown [IPv6:2001:b07:2ed:14ed:a962:cd4d:a84:1eab]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits)) (No client certificate requested) (Authenticated sender: kholk11) by madras.collabora.co.uk (Postfix) with ESMTPSA id 85BE46606EE7; Wed, 7 Jun 2023 09:01:43 +0100 (BST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=collabora.com; s=mail; t=1686124904; bh=7/zTInyb0fTEFIGhobmXUcrdn5DzghzCHPw6C0UuWec=; h=Date:Subject:To:Cc:References:From:In-Reply-To:From; b=YjIVS2xVlYnQjrQAiuIY81ZtB6q7gAfyWuV4UcbWh30M0SgFiQqHhftI5121ESmQ4 B/gdRqAV0swiR9QCHPVcu7d84tLdMLid+Idhq5+eqWLpcAPnusbi46f3G+WXif5zGL odSAkMB0HvUsHSX6beyNhNtpkLMrm3UWk2hTnbn4yLIX28dTXDrWKjl+WcrQjfF9up E2vIVZOD6HDL0tkVHCtWkyad7EWTS2jpGi8pxK4+H7e0K/z+anHlZfGT1mw6izbHj2 BYrCHNIDrsp64uTw14xn6sLp/YHhrq8SYGRijOFJJm2h0W2UowfLIp4Zf5iQE+tAPI 6uHi4XXLTIYzQ== Message-ID: <60809262-26ca-50f1-9ee4-705d46723517@collabora.com> Date: Wed, 7 Jun 2023 10:01:40 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.11.2 Subject: Re: [PATCH v13 05/11] remoteproc: mediatek: Introduce cluster on single-core SCP Content-Language: en-US To: =?UTF-8?B?VGluZ0hhbiBTaGVuICjmsojlu7fnv7Ap?= , "mathieu.poirier@linaro.org" , "conor+dt@kernel.org" , "matthias.bgg@gmail.com" , "andersson@kernel.org" , "robh+dt@kernel.org" , "krzysztof.kozlowski+dt@linaro.org" Cc: "linux-remoteproc@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "linux-mediatek@lists.infradead.org" , "linux-arm-kernel@lists.infradead.org" , "devicetree@vger.kernel.org" , Project_Global_Chrome_Upstream_Group References: <20230607072222.8628-1-tinghan.shen@mediatek.com> <20230607072222.8628-6-tinghan.shen@mediatek.com> <9c710fca38ba53da3ba1aaf32a2221676eb584be.camel@mediatek.com> From: AngeloGioacchino Del Regno In-Reply-To: <9c710fca38ba53da3ba1aaf32a2221676eb584be.camel@mediatek.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230607_010145_986646_0AFF7EA7 X-CRM114-Status: GOOD ( 21.33 ) X-BeenThere: linux-mediatek@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "Linux-mediatek" Errors-To: linux-mediatek-bounces+linux-mediatek=archiver.kernel.org@lists.infradead.org Il 07/06/23 10:00, TingHan Shen (沈廷翰) ha scritto: > Hi Angelo, > > On Wed, 2023-06-07 at 09:43 +0200, AngeloGioacchino Del Regno wrote: >> Il 07/06/23 09:22, Tinghan Shen ha scritto: >>> This is the preliminary step for probing multi-core SCP. >>> The initialization procedure for remoteproc is similar for both >>> single-core and multi-core architectures and is reusing to avoid >>> redundant code. >>> >>> Rewrite the probing flow of single-core SCP to adapt with the 'cluster' >>> concept needed by probing the multi-core SCP. The main differences >>> are, >>> - the SCP core object(s) is maintained at the cluster list instead of at >>> the platofmr device driver data property. >> >> s/platofmr/platform/g >> >>> - save the cluster information at the platofmr device driver data property. >>> - In order to keep the compatibility of exported SCP APIs which getting >>> the SCP core object by SCP node phandle, move the SCP core object >>> pointers to the platform device platform data property. >>> >>> The registers of config and l1tcm are shared for multi-core >>> SCP. Reuse the mapped addresses for all cores. >>> >>> Signed-off-by: Tinghan Shen >>> --- >>> drivers/remoteproc/mtk_common.h | 2 + >>> drivers/remoteproc/mtk_scp.c | 151 +++++++++++++++++++++++--------- >>> 2 files changed, 112 insertions(+), 41 deletions(-) >>> >>> diff --git a/drivers/remoteproc/mtk_common.h b/drivers/remoteproc/mtk_common.h >>> index c0905aec3b4b..56395e8664cb 100644 >>> --- a/drivers/remoteproc/mtk_common.h >>> +++ b/drivers/remoteproc/mtk_common.h >>> @@ -128,6 +128,8 @@ struct mtk_scp { >>> size_t dram_size; >>> >>> struct rproc_subdev *rpmsg_subdev; >>> + >>> + struct list_head elem; >>> }; >>> >>> /** >>> diff --git a/drivers/remoteproc/mtk_scp.c b/drivers/remoteproc/mtk_scp.c >>> index d66822dad943..c8fc6b46f82b 100644 >>> --- a/drivers/remoteproc/mtk_scp.c >>> +++ b/drivers/remoteproc/mtk_scp.c >>> @@ -23,6 +23,14 @@ >>> #define MAX_CODE_SIZE 0x500000 >>> #define SECTION_NAME_IPI_BUFFER ".ipi_buffer" >>> >>> +struct mtk_scp_of_cluster { >>> + void __iomem *reg_base; >>> + void __iomem *l1tcm_base; >>> + size_t l1tcm_size; >>> + phys_addr_t l1tcm_phys; >>> + struct list_head mtk_scp_list; >>> +}; >>> + >>> /** >>> * scp_get() - get a reference to SCP. >>> * >>> @@ -51,7 +59,7 @@ struct mtk_scp *scp_get(struct platform_device *pdev) >>> return NULL; >>> } >>> >>> - return platform_get_drvdata(scp_pdev); >>> + return *(struct mtk_scp **)dev_get_platdata(&scp_pdev->dev); >>> } >>> EXPORT_SYMBOL_GPL(scp_get); >>> >>> @@ -810,14 +818,14 @@ static void scp_unmap_memory_region(struct mtk_scp *scp) >>> static int scp_register_ipi(struct platform_device *pdev, u32 id, >>> ipi_handler_t handler, void *priv) >>> { >>> - struct mtk_scp *scp = platform_get_drvdata(pdev); >>> + struct mtk_scp *scp = *(struct mtk_scp **)dev_get_platdata(&pdev->dev); >>> >>> return scp_ipi_register(scp, id, handler, priv); >>> } >>> >>> static void scp_unregister_ipi(struct platform_device *pdev, u32 id) >>> { >>> - struct mtk_scp *scp = platform_get_drvdata(pdev); >>> + struct mtk_scp *scp = *(struct mtk_scp **)dev_get_platdata(&pdev->dev); >>> >>> scp_ipi_unregister(scp, id); >>> } >>> @@ -825,7 +833,7 @@ static void scp_unregister_ipi(struct platform_device *pdev, u32 id) >>> static int scp_send_ipi(struct platform_device *pdev, u32 id, void *buf, >>> unsigned int len, unsigned int wait) >>> { >>> - struct mtk_scp *scp = platform_get_drvdata(pdev); >>> + struct mtk_scp *scp = *(struct mtk_scp **)dev_get_platdata(&pdev->dev); >>> >>> return scp_ipi_send(scp, id, buf, len, wait); >>> } >>> @@ -855,7 +863,8 @@ static void scp_remove_rpmsg_subdev(struct mtk_scp *scp) >>> } >>> } >>> >>> -static int scp_probe(struct platform_device *pdev) >>> +static struct mtk_scp *scp_rproc_init(struct platform_device *pdev, >>> + struct mtk_scp_of_cluster *scp_cluster) >>> { >>> struct device *dev = &pdev->dev; >>> struct device_node *np = dev->of_node; >>> @@ -867,52 +876,42 @@ static int scp_probe(struct platform_device *pdev) >>> >>> ret = rproc_of_parse_firmware(dev, 0, &fw_name); >>> if (ret < 0 && ret != -EINVAL) >>> - return ret; >>> + return ERR_PTR(ret); >>> >>> rproc = devm_rproc_alloc(dev, np->name, &scp_ops, fw_name, sizeof(*scp)); >>> - if (!rproc) >>> - return dev_err_probe(dev, -ENOMEM, "unable to allocate remoteproc\n"); >>> + if (!rproc) { >>> + dev_err(dev, "unable to allocate remoteproc\n"); >>> + return ERR_PTR(-ENOMEM); >> >> Why are you converting those dev_err_probe to dev_err->return?! >> >> Regards, >> Angelo > > It's because the return type of this function changed from "int" to "struct mtk_scp *". > Maybe I should change them to "ERR_PTR(dev_err_probe(...))" ? > Oh, sorry - you're right. No, it's fine as you did it. Cheers