On 28-01-22, 11:39, Ben Walker wrote: > Document this function to make clear the expected behavior of the > 'value' parameter. It was intended to match the behavior of POSIX memset > as laid out here: > > https://lore.kernel.org/dmaengine/YejrA5ZWZ3lTRO%2F1@matsya/ Can we add this to Documentation too? Documentation/driver-api/dmaengine/ > > Signed-off-by: Ben Walker <benjamin.walker@xxxxxxxxx> > --- > include/linux/dmaengine.h | 8 ++++++++ > 1 file changed, 8 insertions(+) > > diff --git a/include/linux/dmaengine.h b/include/linux/dmaengine.h > index 842d4f7ca752d..3d3e663e17ac7 100644 > --- a/include/linux/dmaengine.h > +++ b/include/linux/dmaengine.h > @@ -1031,6 +1031,14 @@ static inline struct dma_async_tx_descriptor *dmaengine_prep_interleaved_dma( > return chan->device->device_prep_interleaved_dma(chan, xt, flags); > } > > +/** > + * dmaengine_prep_dma_memset() - Prepare a DMA memset descriptor. > + * @chan: The channel to be used for this descriptor > + * @dest: Address of buffer to be set > + * @value: Treated as a single byte value that fills the destination buffer > + * @len: The total size of dest > + * @flags: DMA engine flags > + */ > static inline struct dma_async_tx_descriptor *dmaengine_prep_dma_memset( > struct dma_chan *chan, dma_addr_t dest, int value, size_t len, > unsigned long flags) > -- > 2.33.1 -- ~Vinod
