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 X-Spam-Level: X-Spam-Status: No, score=-9.0 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 98CEAC433E0 for ; Thu, 11 Mar 2021 18:08:11 +0000 (UTC) Received: from desiato.infradead.org (desiato.infradead.org [90.155.92.199]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id E205B64FD4 for ; Thu, 11 Mar 2021 18:08:10 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org E205B64FD4 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=linaro.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=desiato.20200630; h=Sender:Content-Transfer-Encoding :Content-Type:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References:Message-ID: Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=hvyLFhQR5B1rcqO5lZf2HYEKO6Orzq9uCGhDQRM4geM=; b=W8L23CaQ7emxD/wQ1wAgUWOCb XmuIIP1Xb21m7j3LLnLNVuOBKks1Nz/3e9/3tujM2cqG8Ic81zYcGcoRRyEP8lkAdICh41DJfOE0X xnNPNWDqD6M09gMvD8p1pGftuoycCYu+9upH0atlVpAwV/dXz5x4MBm8KoZzxcSSntyC04vu9QE4q LeIvkXGKRJHCc1zXcWU8IRrnkXmlKeVpLwvOjO+IDDYzxeawhoLGJEyUuIMyKwA7oTvBSvk7PnhNE xtc1MUABIDE4i+6wbt/xnGl2vG5Jdi9GgaJSNB/b1skJkwE17TztYF56h3wZn+oWOpsPE5zN6RqOY PZzY0Ar7Q==; Received: from localhost ([::1] helo=desiato.infradead.org) by desiato.infradead.org with esmtp (Exim 4.94 #2 (Red Hat Linux)) id 1lKPhr-009nlY-DT; Thu, 11 Mar 2021 18:06:43 +0000 Received: from mail-ot1-x332.google.com ([2607:f8b0:4864:20::332]) by desiato.infradead.org with esmtps (Exim 4.94 #2 (Red Hat Linux)) id 1lKPhj-009njt-L3 for linux-arm-kernel@lists.infradead.org; Thu, 11 Mar 2021 18:06:38 +0000 Received: by mail-ot1-x332.google.com with SMTP id e45so2261550ote.9 for ; Thu, 11 Mar 2021 10:06:34 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to; bh=BAkVYaDZXDvqzd+93q6mQyCc1QCzmHmhvWDJslwlLVQ=; b=ZpwaJSeDKiTDoplyS5R3U3C6LU/TzxwkihHTdD24J5F3Gy7cHA4cHcaBGn76kCWLIK TFlFe/Jntcph5ybgEWjuxnSa3IdxAsUB7bh3FqO/C3nXDyDhapRBXtBdASbpLFSBWxT5 G0/+aodlizUHZehcYgPJDXIc/sDXnIhxzFBaRsK2dCYnSrcXYEr4kgDZd3qIyOBTM32h bYdj7+z26r6eLv+xcbYFOqLc6vsg/ZT+Kt9qagkR5QfNvsr8WPb5PhkHrhUZ7DwdfhiT MwCr36Tk2tfVTuN1Yz4H0HZQYLpCzEm9OXaZXdaci7gUzlcekzX+cY9c0+QZTZcRDFqv HKXw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to; bh=BAkVYaDZXDvqzd+93q6mQyCc1QCzmHmhvWDJslwlLVQ=; b=kaGRx8i4rh788RxmBsq3loP4Gb14mDcX+KiPXuUn9o7vu4sRqZ/4+MdYw4Gsc/NWpT qp2lLCx9nXwjxiLnZqWK79qv10XAOe5ZgAl5VfhxpnOpbhgJFalep1/zSvmsqUNJY6h5 EoGTCtPEZyoV8QWQCjn6nUniENDe77KLfd3aJq5rhXk2KQj4eTrwkKHaBIdIeHi4UBUe HYdvjhf74nlTTpwbFIhz4hr4F4mEyfgBUgvvOkWUAcNwG2SeY2ZL439U2CFtKxeZ3NBR 2cMPik2inaFmlzW3h03R+ObIvdKo2iQsOOul91z65AYX3Y/Ut9kIzaGZHfqOIdQcw376 1E0w== X-Gm-Message-State: AOAM533EdoM0TFal5/xe7EqePviifiS0dUotU/UCnQvp3Nn8ksch1w15 1Pp82mYRGYT5wvHZ9I8OcscWUQ== X-Google-Smtp-Source: ABdhPJyVctZCnnNhBZKJiX69sS4mTpIVTOhqV1IBzdi6XWJosQqPD25nL134rQEZMjq4nsgeTq9Z6Q== X-Received: by 2002:a9d:86c:: with SMTP id 99mr91856oty.71.1615485993381; Thu, 11 Mar 2021 10:06:33 -0800 (PST) Received: from builder.lan (104-57-184-186.lightspeed.austtx.sbcglobal.net. [104.57.184.186]) by smtp.gmail.com with ESMTPSA id f8sm774478otp.71.2021.03.11.10.06.32 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 11 Mar 2021 10:06:32 -0800 (PST) Date: Thu, 11 Mar 2021 12:06:30 -0600 From: Bjorn Andersson To: Mathieu Poirier Cc: "Peng Fan (OSS)" , "ohad@wizery.com" , "o.rempel@pengutronix.de" , "robh+dt@kernel.org" , "devicetree@vger.kernel.org" , "shawnguo@kernel.org" , "s.hauer@pengutronix.de" , "kernel@pengutronix.de" , "festevam@gmail.com" , "linux-remoteproc@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "linux-kernel@vger.kernel.org" , "paul@crapouillou.net" , "matthias.bgg@gmail.com" , "agross@kernel.org" , "patrice.chotard@st.com" Subject: Re: [PATCH V13 00/10] remoteproc: imx_rproc: support iMX8MQ/M Message-ID: References: <1615029865-23312-1-git-send-email-peng.fan@oss.nxp.com> <20210308160815.GB3977653@xps15> <20210311173818.GB1113417@xps15> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20210311173818.GB1113417@xps15> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210311_180635_913384_2B00A630 X-CRM114-Status: GOOD ( 40.18 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Thu 11 Mar 11:38 CST 2021, Mathieu Poirier wrote: > On Thu, Mar 11, 2021 at 01:26:25AM +0000, Peng Fan (OSS) wrote: > > Mathieu, > > > > > Subject: Re: [PATCH V13 00/10] remoteproc: imx_rproc: support iMX8MQ/M > > > > > > On Sat, Mar 06, 2021 at 07:24:15PM +0800, peng.fan@oss.nxp.com wrote: > > > > From: Peng Fan > > > > > > > > V13: > > > > Add R-b tag from Rob for patch 1. > > > > Drop the reserved memory node from patch 2 per Rob's comment. > > > > Mathieu, Bjorn > > > > Only patch 2 not have R-b/A-b tag, but since Rob's only has a minor > > > comment, and > > > > addressed in this version, is it ok for you take into remoteproc next > > > branch? > > > > Thanks. > > > > > > As much as I want to, there is no way to move forward without an > > > acknowledgement from Rob. > > > > Rob has gave his R-b tag for patch 2/13, please help pick up this patchset. > > I really wanna this feature in which has been for a long time. Then > > I will move on to add new stuff. > > > > This set looks good to me - Bjorn will likely pick it up shortly. > Certainly, I'm just waiting for tuxbuild to get back with some build results before I'm pushing the new branch(es). Regards, Bjorn > > Thanks, > > Peng. > > > > > > > > > > > > > > > V12: > > > > Add maxItems to avoid dt_bindings_check fail Rebased on top of > > > > linux-next > > > > > > > > V11: > > > > Per Rob's comments, fix memory-region in patch 1/10 Rebased on top > > > > of Linux-next > > > > > > > > V10: > > > > Per Rob's comments, fix patch 1/10 > > > > > > > > V9: > > > > Per Mathieu's comments, > > > > update the tile of yaml in patch 2/10 > > > > update the Kconfig and MODULE_DESCRIPTION, I merge this change in > > > patch 8/10, > > > > since this is a minor change, I still keep Mathieu's R-b tag. If any > > > objection, I could remove. > > > > Add R-b tag in Patch 10/10 > > > > > > > > Rob, please help review patch 1/10 and 2/10 > > > > > > > > V8: > > > > Address sparse warning in patch 4/10 reported by kernel test robot > > > > > > > > V7: > > > > Add R-b tag from Mathieu > > > > vdevbuffer->vdev0buffer in patch 1/10, 7/10 correct err msg and > > > > shutdown seq per Mathieu's comments in patch 10/10 Hope this version > > > > is ok to be merged. > > > > > > > > V6: > > > > Add R-b tag from Mathieu > > > > Convert imx-rproc.txt to yaml and add dt-bindings support for > > > > i.MX8MQ/M, patch 1/10 2/10 No other changes. > > > > > > > > V5: > > > > Apply on Linux next > > > > Add V5 subject prefix > > > > Add R-b tag from Bjorn for 1/8, 2/8, 3/8 > > > > > > > > https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpatc > > > > > > > hwork.kernel.org%2Fproject%2Flinux-remoteproc%2Fcover%2F20201229033 > > > 019 > > > > .25899-1-peng.fan%40nxp.com%2F&data=04%7C01%7Cpeng.fan%40n > > > xp.com%7 > > > > > > > C18c2709725e14cbab54408d8e24c6492%7C686ea1d3bc2b4c6fa92cd99c5c3 > > > 01635%7 > > > > > > > C0%7C0%7C637508165016988280%7CUnknown%7CTWFpbGZsb3d8eyJWIjoi > > > MC4wLjAwMD > > > > > > > AiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata > > > =bwlT > > > > uTk6iRuZke6SMhuyB2gxniStfxnX%2BKlVgm7MZuk%3D&reserved=0 > > > > > > > > V4: > > > > According to Bjorn's comments, add is_iomem for da to va usage 1/8, > > > > 2/8 is new patch 3/8, follow Bjorn's comments to correct/update the > > > > err msg. > > > > 6/8, new patch > > > > 8/8, use dev_err_probe to simplify code, use queue_work instead > > > > schedule_delayed_work > > > > > > > > V3: > > > > Since I was quite busy in the past days, V3 is late Rebased on > > > > Linux-next Add R-b tags > > > > 1/7: Add R-b tag of Mathieu, add comments > > > > 4/7: Typo fix > > > > 5/7: Add R-b tag of Mathieu, drop index Per Mathieu's comments > > > > 6/7: Add R-b tag of Mathieu > > > > 7/7: Add comment for vqid << 16, drop unneeded timeout settings of > > > mailbox > > > > Use queue_work instead of schedule_delayed_work > > > > free mbox channels when remove > > > > > > > > https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Flkml > > > > .org%2Flkml%2F2020%2F12%2F4%2F82&data=04%7C01%7Cpeng.fan > > > %40nxp.com > > > > %7C18c2709725e14cbab54408d8e24c6492%7C686ea1d3bc2b4c6fa92cd9 > > > 9c5c301635 > > > > %7C0%7C0%7C637508165016988280%7CUnknown%7CTWFpbGZsb3d8eyJ > > > WIjoiMC4wLjAw > > > > > > > MDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sd > > > ata=xQ > > > > > > > ReFFdUT2ZLhWyT2Vt2v0frG0xKq2psP1ExnLx%2BLXw%3D&reserved=0 > > > > > > > > V2: > > > > Rebased on linux-next > > > > Dropped early boot feature to make patchset simple. > > > > Drop rsc-da > > > > > > > > https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpatc > > > > > > > hwork.kernel.org%2Fproject%2Flinux-remoteproc%2Fcover%2F20200927064 > > > 131 > > > > .24101-1-peng.fan%40nxp.com%2F&data=04%7C01%7Cpeng.fan%40n > > > xp.com%7 > > > > > > > C18c2709725e14cbab54408d8e24c6492%7C686ea1d3bc2b4c6fa92cd99c5c3 > > > 01635%7 > > > > > > > C0%7C0%7C637508165016988280%7CUnknown%7CTWFpbGZsb3d8eyJWIjoi > > > MC4wLjAwMD > > > > > > > AiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata > > > =BWGP > > > > > > > gbVtOjOa%2BriGryGp9sh2CSY%2BhMESdGD%2F7LvPJ6w%3D&reserved > > > =0 > > > > > > > > V1: > > > > > > > > https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpatc > > > > > > > hwork.kernel.org%2Fcover%2F11682461%2F&data=04%7C01%7Cpeng.f > > > an%40n > > > > > > > xp.com%7C18c2709725e14cbab54408d8e24c6492%7C686ea1d3bc2b4c6fa9 > > > 2cd99c5c > > > > > > > 301635%7C0%7C0%7C637508165016988280%7CUnknown%7CTWFpbGZsb3 > > > d8eyJWIjoiMC > > > > > > > 4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000& > > > amp;sd > > > > > > > ata=XkGYIxBgrHLwV7R3bro87N6fL7777Wa5rOAZdVjCoZs%3D&reserved > > > =0 > > > > > > > > This patchset is to support i.MX8MQ/M coproc. > > > > The early boot feature was dropped to make the patchset small in V2. > > > > > > > > Since i.MX specific TCM memory requirement, add elf platform hook. > > > > Several patches have got reviewed by Oleksij and Mathieu in v1. > > > > > > > > > > > > Peng Fan (10): > > > > dt-bindings: remoteproc: convert imx rproc bindings to json-schema > > > > dt-bindings: remoteproc: imx_rproc: add i.MX8MQ/M support > > > > remoteproc: introduce is_iomem to rproc_mem_entry > > > > remoteproc: add is_iomem to da_to_va > > > > remoteproc: imx_rproc: correct err message > > > > remoteproc: imx_rproc: use devm_ioremap > > > > remoteproc: imx_rproc: add i.MX specific parse fw hook > > > > remoteproc: imx_rproc: support i.MX8MQ/M > > > > remoteproc: imx_rproc: ignore mapping vdev regions > > > > remoteproc: imx_proc: enable virtio/mailbox > > > > > > > > .../bindings/remoteproc/fsl,imx-rproc.yaml | 90 ++++++ > > > > .../bindings/remoteproc/imx-rproc.txt | 33 --- > > > > drivers/remoteproc/Kconfig | 6 +- > > > > drivers/remoteproc/imx_rproc.c | 262 > > > +++++++++++++++++- > > > > drivers/remoteproc/ingenic_rproc.c | 2 +- > > > > drivers/remoteproc/keystone_remoteproc.c | 2 +- > > > > drivers/remoteproc/mtk_scp.c | 6 +- > > > > drivers/remoteproc/omap_remoteproc.c | 2 +- > > > > drivers/remoteproc/pru_rproc.c | 2 +- > > > > drivers/remoteproc/qcom_q6v5_adsp.c | 2 +- > > > > drivers/remoteproc/qcom_q6v5_pas.c | 2 +- > > > > drivers/remoteproc/qcom_q6v5_wcss.c | 2 +- > > > > drivers/remoteproc/qcom_wcnss.c | 2 +- > > > > drivers/remoteproc/remoteproc_core.c | 7 +- > > > > drivers/remoteproc/remoteproc_coredump.c | 8 +- > > > > drivers/remoteproc/remoteproc_debugfs.c | 2 +- > > > > drivers/remoteproc/remoteproc_elf_loader.c | 21 +- > > > > drivers/remoteproc/remoteproc_internal.h | 2 +- > > > > drivers/remoteproc/st_slim_rproc.c | 2 +- > > > > drivers/remoteproc/ti_k3_dsp_remoteproc.c | 2 +- > > > > drivers/remoteproc/ti_k3_r5_remoteproc.c | 2 +- > > > > drivers/remoteproc/wkup_m3_rproc.c | 2 +- > > > > include/linux/remoteproc.h | 4 +- > > > > 23 files changed, 393 insertions(+), 72 deletions(-) create mode > > > > 100644 Documentation/devicetree/bindings/remoteproc/fsl,imx-rproc.yaml > > > > delete mode 100644 > > > > Documentation/devicetree/bindings/remoteproc/imx-rproc.txt > > > > > > > > -- > > > > 2.30.0 > > > > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel