Message ID | 20211015123447.27560-1-colin.king@canonical.com (mailing list archive) |
---|---|
State | Accepted |
Commit | 1f6a89efbf9981a637bf472a5662e4d3746af530 |
Headers | show |
Series | dmaengine: Remove redundant initialization of variable err | expand |
On 15-10-21, 13:34, Colin King wrote: > From: Colin Ian King <colin.king@canonical.com> > > The variable err is being initialized with a value that is never read, it > is being updated later on. The assignment is redundant and can be removed > and move the declaration into the local scope. Applied, thanks
diff --git a/drivers/dma/dmaengine.c b/drivers/dma/dmaengine.c index af3ee288bc11..d9f7c097cfd6 100644 --- a/drivers/dma/dmaengine.c +++ b/drivers/dma/dmaengine.c @@ -695,13 +695,12 @@ static struct dma_chan *find_candidate(struct dma_device *device, */ struct dma_chan *dma_get_slave_channel(struct dma_chan *chan) { - int err = -EBUSY; - /* lock against __dma_request_channel */ mutex_lock(&dma_list_mutex); if (chan->client_count == 0) { struct dma_device *device = chan->device; + int err; dma_cap_set(DMA_PRIVATE, device->cap_mask); device->privatecnt++;