On Wed, Apr 17, 2024 at 11:35:39PM +0300, Serge Semin wrote: > On Tue, Apr 16, 2024 at 10:11:58PM +0300, Andy Shevchenko wrote: > > On Tue, Apr 16, 2024 at 07:28:58PM +0300, Serge Semin wrote: ... > > > +static void dwc_verify_maxburst(struct dma_chan *chan) > > > It's inconsistent to the rest of _verify methods. It doesn't verify as it > > doesn't return anything. Make it int or rename the function. > > Making it int won't make much sense since currently the method doesn't > imply returning an error status. IMO using "verify" was ok, but since > you don't see it suitable please suggest a better alternative. mend, > fix, align? My suggestion is (and was) to have it return 0 for now. -- With Best Regards, Andy Shevchenko
