> -----Original Message----- > From: Shubhrajyoti Datta [mailto:omaplinuxker...@gmail.com] > Sent: Monday, June 15, 2015 8:35 PM > To: Punnaiah Choudary Kalluri > Cc: robh...@kernel.org; pawel.m...@arm.com; mark.rutl...@arm.com; > ijc+devicet...@hellion.org.uk; Kumar Gala; Michal Simek; Soren Brinkmann; > vinod.k...@intel.com; dan.j.willi...@intel.com; > devicetree@vger.kernel.org; Linux Kernel Mailing List; kpc...@gmail.com; > kalluripunnaiahchoud...@gmail.com; dmaeng...@vger.kernel.org; > Punnaiah Choudary Kalluri; linux-arm-ker...@lists.infradead.org > Subject: Re: [PATCH v2 2/2] dma: Add Xilinx zynqmp dma engine driver > support > > On Mon, Jun 15, 2015 at 8:06 PM, Punnaiah Choudary Kalluri > <punnaiah.choudary.kall...@xilinx.com> wrote: > > Added the basic driver for zynqmp dma engine used in Zynq > > UltraScale+ MPSoC. The initial release of this driver supports > > only memory to memory transfers. > > > > Signed-off-by: Punnaiah Choudary Kalluri <punn...@xilinx.com> > > --- > <snip> > > > +/** > > + * zynqmp_dma_chan_is_idle - Provides the channel idle status > > + * @chan: ZynqMP DMA DMA channel pointer > > + * > > + * Return: '1' if the channel is idle otherwise '0' > > + */ > > +static int zynqmp_dma_chan_is_idle(struct zynqmp_dma_chan *chan) > > maybe this could return bool.
Ok. I will modify the return type. Regards, Punnaiah > > > +{ > > + u32 regval; > > + > > + regval = readl(chan->regs + STATUS); > > + if (regval & STATUS_BUSY) > > + return 0; > > + > > + return 1; > > +} > > +