Message-ID: <72c14e53-bfcb-c444-4b5c-98d07fe648e3@nxp.com>
Date:   Wed, 15 Feb 2023 00:21:37 +0200
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101
 Thunderbird/102.7.2
Subject: Re: [RESEND PATCH v4] remoteproc: imx_dsp_rproc: add module parameter
 to ignore ready flag from remote processor
To:     Mathieu Poirier <mathieu.poirier@linaro.org>,
        "Iuliana Prodan (OSS)" <iuliana.prodan@oss.nxp.com>
Cc:     Bjorn Andersson <andersson@kernel.org>,
        Shawn Guo <shawnguo@kernel.org>,
        Sascha Hauer <s.hauer@pengutronix.de>,
        "S.J. Wang" <shengjiu.wang@nxp.com>,
        Fabio Estevam <festevam@gmail.com>,
        Daniel Baluta <daniel.baluta@nxp.com>,
        linux-imx <linux-imx@nxp.com>, linux-remoteproc@vger.kernel.org,
        linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org,
        Pengutronix Kernel Team <kernel@pengutronix.de>
References: <20230214163744.16377-1-iuliana.prodan@oss.nxp.com>
 <20230214202506.GA384054@p14s>
Content-Language: en-US
From:   Iuliana Prodan <iuliana.prodan@nxp.com>
In-Reply-To: <20230214202506.GA384054@p14s>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?cUpHbGpYYVRua0Z1NGpKT3MwR2g1S0FXcWlVQ2hWK3VBdElBcVI2TjNiWkgx?=
 =?utf-8?B?QnVMTVl5eHE0SDVOUUs4Z2hjdTRLVExwNGpDZEN5aFhwRVprRWkwT0RlVHhz?=
 =?utf-8?B?UXBVNmlxdHlKT0pVVy9DWW5VamdBbU02Q01sUHZXRmdrNVk5Tll4T1U0ZCtw?=
 =?utf-8?B?WngreTFVSTlMeVZ4UzhLZWxWcmhGN2ZsTXpEclBHam0xeEovaXFjbGpDNFQ0?=
 =?utf-8?B?ZnZHM0tsajFZU3djN1Jra2NCWkdPVHZiVUVUVkRtRFNMeE9oK3dQZzJ0Zmsv?=
 =?utf-8?B?bTZaSmgvZ3p0ZmtOeGhkbXFVNHgxaVFpRCt1eFJNcWViWWk2T2FIRWFTb2t1?=
 =?utf-8?B?VDc3RzNDaXc3S2REeFBpWmsxNDhWTHdHRUFnTUw2dE54OFpsRDFXTVpnNFRG?=
 =?utf-8?B?bFYvNWgyVEt5bUxBZ2MrNXovcWFyc0I5MWl0aVNvQkg0bkxnVTNQdTB6RVlp?=
 =?utf-8?B?VDVxS01RbHM4UVZPTmNlWEdEa3JYT2Fzd0ZmMzZFYmNiU2lFNExxUEZ3VUNj?=
 =?utf-8?B?cTFtcVdWeU5GdlpvRkFoOUk0bTZpbkorU1RrdU0wTjRqRTNWV3kxVExLcllW?=
 =?utf-8?B?alpQbFlqNUFRRlFCcGVwdVVhY250dUdxRkhWdWMrTlR0ODNyV2dxa1B5ZjdX?=
 =?utf-8?B?d2tSN1ovNkJnT2x6dkFBK0x2QloyNE5wemJ6aHF3QlpQblBPNDQxL0k0OG94?=
 =?utf-8?B?R0JKeWhpTkZVajZkMjlmL2hVS2x0VHgwV015SC93RnZ3ZWQ3aW9YZ0kyZ05V?=
 =?utf-8?B?ZUw0aWJ6KzJGbG41M3pBQjJuRFJPalcveUN0TUtUT1ZZTC9leXlmejVOMjFl?=
 =?utf-8?B?TzZRUjZkOXFsemdsUXZsVXZaQytXSTcwR09mLzBqc1N0RDhPZ2J5dkI4MGh3?=
 =?utf-8?B?RVZpNUhHdC9pTUxrK0hvUHNObXBqLzYxakNOd1pIWnBTTFh2L0d5K3RseXFT?=
 =?utf-8?B?YkFhRDMweWlBSFphZFo3dlVVRUdxQkJMUWpCUTV2OUZnRktIMXovV3Bsbi9W?=
 =?utf-8?B?ZGRNcTlUcDJhME1SeVh0cDNJZXUxUmhrdnk1eGlJTDJVUW84d00yWjlnYmVu?=
 =?utf-8?B?YzNkaGs0VXoyeEZPSEpiRThSc1Nsc21wMEMyWkpkaklJdmVQQngrWGhyTXdY?=
 =?utf-8?B?dmJzQVk3cjZPV2RhRzVrL1RVSURwdk5wd0FzTkhORFl1WEJWV3JKdkUxWElx?=
 =?utf-8?B?RStjM0JLK1FkTlNpZTFQZkdkWW8yTExsMlR3ZzJwcmVwQlRZM05NL3ppRkJM?=
 =?utf-8?B?c3o0ZCs2SHh4ZHBBemx3YUdtVS9OSHVwSHdQSHRaNkxjMTIrSXRqZ2x6cnJx?=
 =?utf-8?B?NzVFbTFxa1lmYk9BMXh3MDIrbllOcGJHWE5JbnNsOWZaZ0d4N09DZGlxdEY5?=
 =?utf-8?B?MTdXVk1nT3l0ZTZEcEhkN3ZMZUdtaUliZ3U1YXZUYm81dnZpejBhVVdFZ0py?=
 =?utf-8?B?TFJQRmZIVE5ESCtEOUphYTB6QWxCT2pHZzM1NUkzSTlvTi8xeGgwNDFmWG5o?=
 =?utf-8?B?VDBraml5cW0yUFlXTVU1ejgrSkVuemNwV1VjekNNdUdRV1JDSGxZUjZvMisy?=
 =?utf-8?B?MnJGVTBoN0ErcnhQQ0k4QmdZbzRDMUxkcUc0RlVXZkVOVUNVUk93Yk9kM05W?=
 =?utf-8?B?WVBVMUo5SHQybk41RVFGQ09rSFpxdnFWV2FyUHBvbGw1VDN0RFI4U3dLK0hG?=
 =?utf-8?B?YUlKUXlMYWNUSG1ud00rS3UxSDFVWkRJVUVUS0hQM0JidTVDL2c2bEdvNVhF?=
 =?utf-8?B?SERNWEp6ekcxRGwwczJDcnhSeVUxcDcwdXMzY0dOZWxxLzJadjJ6ZGsrTExD?=
 =?utf-8?B?Z3BGa2cyNjZNWTJHKzFqZ28xYmhYWFhBRkx6TzdWRTA4RXNEK3dvTnhmbEY1?=
 =?utf-8?B?aG16VWpsL1Q1bGhEUit0elZUdVp2RUlaRjJLaUVRZUIwWHFqeTR0YUwyZjRC?=
 =?utf-8?B?S3Nacm02Q2FzbGRlNm9QQXI2Wk16UWxCdVVQUENHODVhVXpBV01LZkgza25t?=
 =?utf-8?B?bXlDRmRLcmNYd3hja3dlRTVkaXdXOHlqMllDQVJ2UjI2WTRmVUdtbWRKT0xD?=
 =?utf-8?B?aXNMZ0tQUWhXNHlkbFNvV2dkUmlqYnRDNXRtNmlkUElZQktrN2w2eGplMXFM?=
 =?utf-8?B?S3VmSmtiNGlWWVpPN05kdjdQTnM3dGFSd3J2Rzd3TE5yNmtLbE9wVno2QW9H?=
 =?utf-8?B?NEE9PQ==?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 5a86434b-3fca-441b-cefb-08db0ed9d886
X-MS-Exchange-CrossTenant-AuthSource: DB9PR04MB9628.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 14 Feb 2023 22:21:40.7712
 (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: xGV6RJzEgyFnmnYgixqdHOXJHo9rbMgk1nA5WAUtOrdxMFqC76P3f8cmrz9NZ8cTbPw+iOkg/I3TfQBWldwUZg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PAXPR04MB8094
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


On 2/14/2023 10:25 PM, Mathieu Poirier wrote:
> On Tue, Feb 14, 2023 at 06:37:44PM +0200, Iuliana Prodan (OSS) wrote:
>> From: Iuliana Prodan <iuliana.prodan@nxp.com>
>>
>> There are cases when we want to test a simple "hello world"
>> application on the DSP and we don't have IPC between the cores.
>> Therefore, do not wait for a confirmation from the remote processor
>> at start.
>>
>> Added "ignore_dsp_ready" flag while inserting the module to ignore
>> remote processor reply after start.
>> By default, this is off - do not ignore reply from rproc.
>>
>> Signed-off-by: Iuliana Prodan <iuliana.prodan@nxp.com>
>> Reviewed-by: Daniel Baluta <daniel.baluta@nxp.com>
> This patch is on my review list, as indicated in February 8th's patchset review
> order email.  That said, I haven't seen Daniel's RB on the mailing list.

It was given on v4 - please, see here : 
https://patchwork.kernel.org/project/linux-remoteproc/patch/20230119173940.21143-1-iuliana.prodan@oss.nxp.com/#25172596