Messages in this thread | | | Date | Tue, 26 Oct 2021 21:50:57 +0200 | Subject | Re: [PATCH V5 10/20] rtla/osnoise: Add osnoise top mode | From | Daniel Bristot de Oliveira <> |
| |
On 10/26/21 18:32, Tao Zhou wrote: >> + if (params->stop_us) >> + osnoise_set_stop_us(tool->context, params->stop_us); > osnoise_set_stop_us() also need to check return value feels by me. > >> + >> + if (params->stop_total_us) >> + osnoise_set_stop_total_us(tool->context, params->stop_total_us); > The same here. Also need to check the return value. > > PATCH 10 also has these two places(call the two functions) that not check > the return value. > > Or I am wrong.
You are right, reviewing that.
Thanks! -- Daniel
> > > Thanks, > Tao >
| |