Received-SPF: softfail (google.com: domain of transitioning linux-kernel-owner@vger.kernel.org does not designate 23.128.96.19 as permitted sender) client-ip=23.128.96.19;
From:   Peng Fan <peng.fan@nxp.com>
To:     Mathieu Poirier <mathieu.poirier@linaro.org>,
        "Peng Fan (OSS)" <peng.fan@oss.nxp.com>
CC:     "bjorn.andersson@linaro.org" <bjorn.andersson@linaro.org>,
        "shawnguo@kernel.org" <shawnguo@kernel.org>,
        "s.hauer@pengutronix.de" <s.hauer@pengutronix.de>,
        "kernel@pengutronix.de" <kernel@pengutronix.de>,
        "festevam@gmail.com" <festevam@gmail.com>,
        dl-linux-imx <linux-imx@nxp.com>,
        "linux-remoteproc@vger.kernel.org" <linux-remoteproc@vger.kernel.org>,
        "linux-arm-kernel@lists.infradead.org" 
        <linux-arm-kernel@lists.infradead.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: RE: [PATCH V2] remoteproc: imx_rproc: Ignore create mem entry for
 resource table
Thread-Topic: [PATCH V2] remoteproc: imx_rproc: Ignore create mem entry for
 resource table
Thread-Index: AQHYMrQ3WJ3UeNc+okuz81vxgHqt4KzOvXsAgAC1LqA=
Date:   Fri, 25 Mar 2022 02:02:10 +0000
Message-ID: <DU0PR04MB9417991BDB9564321FC30EE7881A9@DU0PR04MB9417.eurprd04.prod.outlook.com>
References: <20220308065754.3355-1-peng.fan@oss.nxp.com>
 <20220324151132.GA3514576@p14s>
In-Reply-To: <20220324151132.GA3514576@p14s>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?Bvnf8NxPArB10SlPRW5ZbD+rfW4LzaM01QeghmgrZb9IXGD9jQF0QFa9NNlw?=
 =?us-ascii?Q?6P/Qag71pPuczcvllIYt3hKmKjYbkdSqnxFVGSLy4FAryOAwsNP+A2RWoL6N?=
 =?us-ascii?Q?KCLd/1SAzl8JKPm4jeGBTCzQhHzynICkU1aa4kVuKZmkvENplqgxYIk5na0N?=
 =?us-ascii?Q?4KAgXzhkSw5mA8itBmxUZLYv8+zCOhySuhlUml2GCn2LRPBT7/8mrdbasF6P?=
 =?us-ascii?Q?SHAKDnX67t7lrLkeqK09GYfr5cOnMb/8dOQT2yN5y+VuJ+L4txMDqNSgk+X7?=
 =?us-ascii?Q?807n09Ku7rwaDXsmSPKCP9Ctj/BGdCsCpBjmqQY2Q2Om3Bpg0jJe9WcJhj1b?=
 =?us-ascii?Q?uGGc8qwSwLKOYsuo0qC6J2TZZHzriYxKGbintcGaTU/cHwrxg6vmkNgYYJvu?=
 =?us-ascii?Q?AbHFe9vNJqssqE2HXzLanSyNLhiwfA22nGR25Zs7F877NnEyu5n0+tfEHydZ?=
 =?us-ascii?Q?9c9DockcpRIBXfdqf1chyKa1GP1NRP1SD7vgAhsAx5zfGkPVYyLuaA7IIU2c?=
 =?us-ascii?Q?xrLM1CaR16qSjGe5ryq/RpDAGYUNuyRgUwZ8j9ZD96TQkuSlROQGA9xSZ4x6?=
 =?us-ascii?Q?f+UVbp2KRQAshAde06yI3jo2qrLBgqpsmD/gkI9fFI725QjEulP84hprVbNY?=
 =?us-ascii?Q?VDWR1EZHyG8NqvPNS+tX6tXVmxcjaLxnmRg7xtS4XjKtWfZ4B5dJlam5gDI3?=
 =?us-ascii?Q?0dcjuNydUYM5iuxDGw8pL4VXgrBzOIM0WSXUVDJDdUyHjYYwKbLpPIwcTlBK?=
 =?us-ascii?Q?ri5WBeM4kG5ZYxqsAxb+4Ylfz06EnJElsF0AfMjjE3RSPisqqDfvt2G96Ohe?=
 =?us-ascii?Q?l2gxiQq+9GgG0sdTHBbBro6Ui432wRYy/vK6JW0xeGmD8H6gSFN2CLk2SQjj?=
 =?us-ascii?Q?Cis1c0QS77zsTMZrEqIG11ZOdIXRfdKaFYwbUwjeHWxc1GXcdFCzr2jcJHEo?=
 =?us-ascii?Q?TNh6voRaARZJTe2lQ7LlJ/FjPBscXbFXiz6IFKupelHruYoV1s4wCzE0JTyK?=
 =?us-ascii?Q?F2t3kjdwvmzP8NJMF0AD8DMzrSqGM4yUEbJ+SJ/kUz13zrIHa2Xg36qJkKGb?=
 =?us-ascii?Q?mS3skJq31uaI9FkRAlswF7bvwbuBecsi12EiJZvlDAPZ9DtLqheerrS1FJ73?=
 =?us-ascii?Q?e1j+9Krx90Z4MaKclClM8RKQu8Wh26IpY/e7NqovcchRpAtBRsUtLfyMIWPG?=
 =?us-ascii?Q?cO61jOZiMtRhSbNX02AY/7DW5x+TclOEToLru9fSy30+7mWeCjw+xTxa+Fsm?=
 =?us-ascii?Q?/Em7nh/DqtU36BTjyydCb67wwsk2DJ919mXOmLqKRSuffsSDkrSs4FWyRmiL?=
 =?us-ascii?Q?o5vALzZ9BwoX0iN9vG66KVWiAKGgXo7uxBiTAhCB1+RnJfbm3UqBWS8J2mW4?=
 =?us-ascii?Q?TjtCTLhpKs4s0SdHxYJOv7pGpA8Max/HH/+u4v9tYkZorx0hYD3Igw1mjtQV?=
 =?us-ascii?Q?2VwzDwqGFO4IGrEhwhHiof1nSd+wpo+Vmh3fzCzQy5s1cDpFg6u2xmkjVceA?=
 =?us-ascii?Q?JioQOIfXliE9k+XCUCSNAH92m+hr7PUVuDrBcGXilr0ffBMJfncIagucmLAR?=
 =?us-ascii?Q?/AAUpLTuHE32VFaS6xLQ02O1/EIcgRfdyz5JexvTA2y40zXTpSorwhsVbJx2?=
 =?us-ascii?Q?q9zk6RHH/jrkOVGg7zX3GLPdhtZI6d6YoKicJBzB56ICfEO1S9mQyJv8FQVz?=
 =?us-ascii?Q?j/nsZbHrEJvs0IflF5omnWlheacYegK+HYcz/7Ptx+DRY4WiJUXg4I3Wj1N5?=
 =?us-ascii?Q?rPIeKJ/bEQ=3D=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: DU0PR04MB9417.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 8ce1e2f7-ade0-4ef0-ca1d-08da0e0378d9
X-MS-Exchange-CrossTenant-originalarrivaltime: 25 Mar 2022 02:02:10.0638
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: y3Kaq5TBei9CxheLeDREHxHRRFDbkIokdGHZjwD2rsdturcwv9a6xJ/QzUVPfCs9wx6cZ96YCMegYd5XAZW7bA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM0PR04MB6516
X-Spam-Status: No, score=-2.0 required=5.0 tests=BAYES_00,DKIM_SIGNED,
	DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,
	MAILING_LIST_MULTI,RDNS_NONE,SPF_HELO_NONE,T_SCC_BODY_TEXT_LINE
	autolearn=no autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
	lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi Mathieu,

> Subject: Re: [PATCH V2] remoteproc: imx_rproc: Ignore create mem entry fo=
r
> resource table
>=20
> On Tue, Mar 08, 2022 at 02:57:54PM +0800, Peng Fan (OSS) wrote:
> > From: Peng Fan <peng.fan@nxp.com>
> >
> > Resource table will not be used for memory allocation, no need to
> > create rproc mem entry.
> >
> > Fixes: b29b4249f8f0c ("remoteproc: imx_rproc: add i.MX specific parse
> > fw hook")
>=20
>=20
> https://eur01.safelinks.protection.outlook.com/?url=3Dhttps%3A%2F%2Felixi=
r.b
> ootlin.com%2Flinux%2Fv5.17%2Fsource%2FDocumentation%2Fprocess%2Fsu
> bmitting-patches.rst&amp;data=3D04%7C01%7Cpeng.fan%40nxp.com%7Cae62
> e633010048f972db08da0da89836%7C686ea1d3bc2b4c6fa92cd99c5c301635
> %7C0%7C1%7C637837315009092474%7CUnknown%7CTWFpbGZsb3d8eyJW
> IjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C
> 3000&amp;sdata=3DXod3YxSRW8h2WXdnpu1WX4DzRpCTnYE9gVlt9HXMlnU%3
> D&amp;reserved=3D0

Sorry, I could not get your point of which part conflicts with the patch ru=
les.
Please clarify and I'll improve.

Thanks,
Peng.


>=20
>=20
> > Signed-off-by: Peng Fan <peng.fan@nxp.com>
> > ---
> >
> > V2:
> >  Add Fixes tag
> >  Separate the patch from
> > https://eur01.safelinks.protection.outlook.com/?url=3Dhttps%3A%2F%2Fpat=
c
> >
> hwork.kernel.org%2Fproject%2Flinux-remoteproc%2Fpatch%2F20220111033
> 333
> > .403448-7-peng.fan%40oss.nxp.com%2F&amp;data=3D04%7C01%7Cpeng.fan
> %40nxp.
> >
> com%7Cae62e633010048f972db08da0da89836%7C686ea1d3bc2b4c6fa92cd
> 99c5c301
> >
> 635%7C0%7C1%7C637837315009092474%7CUnknown%7CTWFpbGZsb3d8e
> yJWIjoiMC4wL
> >
> jAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&am
> p;sdata
> >
> =3DCYL9grwXHpUud4qpzFAEijOWv%2BehvefwyZ4aU02wJnw%3D&amp;reserv
> ed=3D0
> >  Address typo
> >
> >  drivers/remoteproc/imx_rproc.c | 3 +++
> >  1 file changed, 3 insertions(+)
> >
> > diff --git a/drivers/remoteproc/imx_rproc.c
> > b/drivers/remoteproc/imx_rproc.c index 7a096f1891e6..f2bfc9077c19
> > 100644
> > --- a/drivers/remoteproc/imx_rproc.c
> > +++ b/drivers/remoteproc/imx_rproc.c
> > @@ -423,6 +423,9 @@ static int imx_rproc_prepare(struct rproc *rproc)
> >  		if (!strcmp(it.node->name, "vdev0buffer"))
> >  			continue;
> >
> > +		if (!strncmp(it.node->name, "rsc-table", strlen("rsc-table")))
> > +			continue;
> > +
> >  		rmem =3D of_reserved_mem_lookup(it.node);
> >  		if (!rmem) {
> >  			dev_err(priv->dev, "unable to acquire memory-region\n");
> > --
> > 2.30.0
> >