Re: [PATCH v1 3/3] dmaengine: dmatest: Return boolean result directly in filter()

Andy Shevchenko
2020-09-22T11:57:46Z

On Thu, Sep 17, 2020 at 12:39:27PM +0300, Peter Ujfalusi wrote:
> On 16/09/2020 16.34, Andy Shevchenko wrote:
> > There is no need to have a conditional for boolean expression when
> > function returns bool. Drop unnecessary code and return boolean
> > result directly.
> >
> > While at it, drop unneeded casting from void *.
>
> my test scripts are working fine with the three patch applied.
>
> Tested-by: Peter Ujfalusi <peter.ujfalusi@ti.com>
>
> Vinod: for all three patches ^^
>
> Andy, Vladimir: thanks for the fixes!

Thanks for testing! I'll apply your tag to v2 (assuming you are okay with
slight changes Vinod requested).

--
With Best Regards,
Andy Shevchenko