Re: [PATCH v4 01/10] dmaengine: Move enum dma_slave_buswidth to a new header

[Nuno Sá]

On Tue, Sep 15, 2026 at 09:37:44AM -0500, Frank Li wrote:
> On Tue, Sep 15, 2026 at 10:29:03AM +0300, Andy Shevchenko wrote:
> > On Mon, Sep 14, 2026 at 09:18:42AM -0500, Frank Li wrote:
> > > On Fri, Sep 11, 2026 at 06:25:36PM +0100, Nuno Sá wrote:
> > > > In preparation for the bitmap based bus width capabilities, move
> > > > enum dma_slave_buswidth out of linux/dmaengine.h into a new
> > > > include/linux/dma/engine/types.h.
> > > >
> > > > linux/dmaengine.h is included nearly everywhere, so growing it with the
> > > > new interface would make every one of its users pay for it. Keeping the
> > > > basic types in a separate header lets the new interface live next to it
> > > > without that cost. linux/dmaengine.h includes the new header, so its
> > > > users are unaffected.
> > > >
> > > > While at it, add the missing kernel-doc for the enum members. The block
> > > > already opened with /**, so kernel-doc expected every member to be
> > > > described and warned about each one of them. Describe the UNDEFINED case
> > > > in particular, as its meaning depends on where the value is used.
> >
> > ...
> >
> > > >  include/linux/dmaengine.h        | 20 +++-----------------
> > >
> > > I think it is good to split dmengine consumer and provider header files to
> > > hidden dmaengine's detail.
> >
> > It's indeed desirable change. but isn't it a bit out of scope of the series?
> > I suggest to do one step at a time. Would it work for you?
> 
> Yes, but I hope the file name is finial expected file name. it is okay
> just move enum dma_slave_buswidth to there now.

Hi Frank, if I understood you, you mean to have an engine.h as kind of a
consumer and have enum dma_slave_buswidth on it?

If so, note that enum is used both by consumers and providers so having
them in types kind of header makes sense to me. Kind of the same as with
the new widthmask header. It's used by both.

- Nuno Sá

> 
> Frank
> 
> >
> > > suggest name
> > > 	dma/engine.h
> > >
> > > so we can start move more API to this files.
> > >
> > > after all user include this consumer header file,  old
> > > include/linux/dmaengine.h rename to include/linux/dmaengine-provider.h,
> > > which only include by dmaengine drivers.
> >
> > Traditional naming would be
> > dma/engine/provider.h
> > dma/engine/consumer.h
> >
> > --
> > With Best Regards,
> > Andy Shevchenko
> >
> >