Messages in this thread | | | Date | Thu, 18 Apr 2024 22:10:04 +0300 | From | Serge Semin <> | Subject | Re: [PATCH 4/4] dmaengine: dw: Simplify max-burst calculation procedure |
| |
On Thu, Apr 18, 2024 at 02:49:26PM +0300, Andy Shevchenko wrote: > 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.
Ok. Let's have it returning zero then.
-Serge(y)
> > -- > With Best Regards, > Andy Shevchenko > >
| |