Messages in this thread | | | Subject | Re: [PATCH] dmaengine: squash lines for immediate return | From | Jon Hunter <> | Date | Tue, 13 Sep 2016 09:45:18 +0100 |
| |
On 12/09/16 20:20, Masahiro Yamada wrote: > Remove unneeded variables and assignments. > > Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com> > --- > > drivers/dma/cppi41.c | 30 +++++------------------------- > drivers/dma/imx-sdma.c | 15 ++++----------- > drivers/dma/ppc4xx/adma.c | 15 ++++++--------- > drivers/dma/tegra20-apb-dma.c | 4 +--- > 4 files changed, 16 insertions(+), 48 deletions(-)
...
> diff --git a/drivers/dma/tegra20-apb-dma.c b/drivers/dma/tegra20-apb-dma.c > index 6ab9eb9..89795a2 100644 > --- a/drivers/dma/tegra20-apb-dma.c > +++ b/drivers/dma/tegra20-apb-dma.c > @@ -337,9 +337,7 @@ static struct tegra_dma_sg_req *tegra_dma_sg_req_get( > } > spin_unlock_irqrestore(&tdc->lock, flags); > > - sg_req = kzalloc(sizeof(struct tegra_dma_sg_req), GFP_NOWAIT); > - > - return sg_req; > + return kzalloc(sizeof(struct tegra_dma_sg_req), GFP_NOWAIT); > } > > static int tegra_dma_slave_config(struct dma_chan *dc,
For the Tegra bit ...
Acked-by: Jon Hunter <jonathanh@nvidia.com>
Cheers Jon
-- nvpublic
| |