Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Mon, 10 Apr 2023 11:37:56 +0200
From:   Simon Horman <simon.horman@corigine.com>
To:     MD Danish Anwar <danishanwar@ti.com>
Cc:     "Andrew F. Davis" <afd@ti.com>, Suman Anna <s-anna@ti.com>,
        Roger Quadros <rogerq@kernel.org>,
        Vignesh Raghavendra <vigneshr@ti.com>,
        Tero Kristo <kristo@kernel.org>,
        Mathieu Poirier <mathieu.poirier@linaro.org>,
        Bjorn Andersson <andersson@kernel.org>,
        Santosh Shilimkar <ssantosh@kernel.org>,
        Nishanth Menon <nm@ti.com>, linux-remoteproc@vger.kernel.org,
        linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org,
        linux-omap@vger.kernel.org, srk@ti.com, devicetree@vger.kernel.org,
        netdev@vger.kernel.org
Subject: Re: [PATCH v7 0/4] Introduce PRU platform consumer API
Message-ID: <ZDPY9BPr6QhJbnH+@corigine.com>
References: <20230404115336.599430-1-danishanwar@ti.com>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20230404115336.599430-1-danishanwar@ti.com>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?JbuosVnZ34Ksa2knF1x6KoXZgI3k+sCHCDGjeASplVAHngfhP3EWZRTBKs5U?=
 =?us-ascii?Q?PuKmK4yRQqcJ5Fjdr/m+Ffs5cZLvCJE+116GufL11lLS0X0pKQycBsxy5n7D?=
 =?us-ascii?Q?1iXFQO5xvbt1XfiLgHhdJQw1vB9U/8oVpVO/+EEeyQwnWWO9CPHEVAN+KyTw?=
 =?us-ascii?Q?OlmueI4GACT/79rk67czHqtu4QckoeafEB16+lSjSa7iDHcwani1HqUM105S?=
 =?us-ascii?Q?7RHuQutYALdbzzTCtmKN07DpDb0Q2UWw9ZyiIBQwwcBWT+izdzcxvaxZu2cA?=
 =?us-ascii?Q?nBEvQoLy5UwCfIvthtztcr0bcUDPGPRakE/VfOilCXMmpyM1HYpWBKifrEw3?=
 =?us-ascii?Q?Aj5kd+IJKpInEPG6UhS8DqBkms4+pMbwEh/pX+n9gT7yO45LXWo5W9to9wGA?=
 =?us-ascii?Q?qjyhWEzORlXlbiHZQc202oecUsPsgWzGGZJ2i+/KlpHW2MPTbrSXRD3chLfn?=
 =?us-ascii?Q?m9v68w4IKWa2OH5ZvUa0eFKL/Z4UqMLcQuKLgzL684DbCmMYT33tpVON6FKJ?=
 =?us-ascii?Q?SCt7FBsIAkH7nfrkmAfkZMIBigwynx4XdQc2p1KWeaaD8hIUrlb3uQnUhJsl?=
 =?us-ascii?Q?Dx7wPl0oGm6ytmoCBJrlaeHTEqtsWrR64dq58sIkeVH1QXOaBIlg7k8OXYwt?=
 =?us-ascii?Q?MCRvetkayJLt78mZjOiJun3UP34JGX5JWnceaicDMfZWPU3uMrekj/ggAnSr?=
 =?us-ascii?Q?L1TM34EyvK7GE2Z76its0pZln8zJHYfoA705+SSoB2PMG9A0o3HKCgd+7zzB?=
 =?us-ascii?Q?CUNczBQoL2+Ii0j708jfR9LhrqGJd927FPnyt6fAizwDcP7KyhJGhMAzwo+h?=
 =?us-ascii?Q?ljQ6BJmFWX3mzIFKtLPYTJOo/8FGRbn6bfU6bwBJeEymNK+RzA1uJ9aGalBJ?=
 =?us-ascii?Q?OZHCuT6T0OikFHJg8BcKlx1O09ENvCoKtRi5gTrwoO2d3hvqOTUDevI5zXW/?=
 =?us-ascii?Q?SlPADxEVET06v5qeTrT7cI7iZ/UVFQleKEWOkwEn50+Ijdu5QGY6NVb1k7yB?=
 =?us-ascii?Q?a+HylMuYVNLIHhc8+qbSSkFAstrqEIbEUq8NsMtc4UO452duuzLwKxHuHCgm?=
 =?us-ascii?Q?yKmdq1zRav4+IR3VHuNONWXC3Fib/EALhvIbCoQx9UaLsWM9hOWZMVkjadhD?=
 =?us-ascii?Q?I94TrssRkDiIE60MlsKl723OkdDCmLl4fj9C2N7WTz3gOjrUooggCVmKGgB/?=
 =?us-ascii?Q?jgr+dY6FBgag7fHC0P3Y8ezaMml7e3SjgIx2ge1FSLmiI4YqklOOyfwHZk5g?=
 =?us-ascii?Q?eydRf7iAb5LIcff1JaHPsg5Sbf172Hu2OfsIrrnl9wQuvWoxALseq7pRN6Vt?=
 =?us-ascii?Q?mSefOrR3wFtcoTkQMbMSSVh0yu3WG74QdR2KChw4DLLkFvA0ERV+LuICdIbC?=
 =?us-ascii?Q?iofs6iqBKQa6BbEqb5nAWdPHcvQpHTuRi83xMI8YYTDg1ndwe8oAay413LpA?=
 =?us-ascii?Q?kNqhljr3sUt1c1RzAA9z2NW20rgHWUn8Lw6Gt1oMssLgbBU+qb1Zgb5gFcYk?=
 =?us-ascii?Q?gDainq7Gwy54Uiatrxxlum+D99/gWNzOq3xtzUERXgohVA//BrsK5GjkU5fq?=
 =?us-ascii?Q?4aK6AvuRJXm2EB8OP16AM9enlCpTx2+h+uiuB/0MtpKXqx3wk0EcQXak+cCP?=
 =?us-ascii?Q?J4Uhyyjo/cU8Nh+R5X/0HKCnl0wjhIbCv9KuX1SLaQ2mwUau7HaWSubqO0Wh?=
 =?us-ascii?Q?5x2J8Q=3D=3D?=
X-OriginatorOrg: corigine.com
X-MS-Exchange-CrossTenant-Network-Message-Id: da84f250-856b-4065-d1ee-08db39a74839
X-MS-Exchange-CrossTenant-AuthSource: PH0PR13MB4842.namprd13.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 10 Apr 2023 09:38:03.8984
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: fe128f2c-073b-4c20-818e-7246a585940c
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: RTEO/8C9D33sWxFfjxj2O3bdZskk4QxDgEJDCleNvyQ/HrJlmtSPzTXyKBEJ6lQu8fgjICmVOBI6dr+M51W48alXlw9q4IlH3az3qqBdTu0=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH0PR13MB5900
X-Spam-Status: No, score=-0.0 required=5.0 tests=DKIM_SIGNED,DKIM_VALID,
        RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS
        autolearn=unavailable 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

On Tue, Apr 04, 2023 at 05:23:32PM +0530, MD Danish Anwar wrote:
> Hi All,
> The Programmable Real-Time Unit and Industrial Communication Subsystem (PRU-ICSS
> or simply PRUSS) on various TI SoCs consists of dual 32-bit RISC cores
> (Programmable Real-Time Units, or PRUs) for program execution.

...

I've reviewed this series from a code style, static analysis, etc.. pov.

I'm happy that issues that I reported in earlier revisions have been
addressed. And have no outstanding issues with the patchset.

So from that position, FWIIW,

Reviewed-by: Simon Horman <simon.horman@corigine.com>