On Fri, Jan 13, 2023 at 8:41 PM Alexey Khoroshilov <khoroshilov@xxxxxxxxx> wrote: > On 13.01.2023 16:33, Anastasia Belova wrote: > It seems it is better to put the whole validation loop under if (rsp) > check. No. The entire patch is redundant. The code that calls this function is under the control of the same driver, so we know how to avoid shooting in our foot. -- With Best Regards, Andy Shevchenko