On Tue, Jul 15, 2025 at 11:39:54AM +0100, Jonathan Cameron wrote:

> > +/* Update batch->ncmds to the number of execute cmds */
> 
> Not obvious what the return value here means. Maybe a comment?

I think it would be clear once we fix the typo in the current one:
s/execute/executed

That being said, if something else is still preferred, we can add:
Return is true if all cmds are issued correctly. False otherwise.

Thanks
Nicolin

Reply via email to