Message ID | 20230918120238.13132-1-kaiwei.liu@unisoc.com (mailing list archive) |
---|---|
State | Accepted |
Commit | 8038557622beaa6145a6b17be948490b835691ac |
Headers | show |
Series | [V2] dmaengine: sprd: delect redundant parameter for dma driver function | expand |
On Mon, 18 Sept 2023 at 20:03, Kaiwei Liu <kaiwei.liu@unisoc.com> wrote: > > The parameter *sdesc in function sprd_dma_check_trans_done is not > used, so here delect redundant parameter. There's a typo here, should be "delete" > > Signed-off-by: Kaiwei Liu <kaiwei.liu@unisoc.com> > Reviewed-by: Baolin Wang <baolin.wang@linux.alibaba.com> > --- > Change in V2: > -Change subject line. > --- > drivers/dma/sprd-dma.c | 6 ++---- > 1 file changed, 2 insertions(+), 4 deletions(-) > > diff --git a/drivers/dma/sprd-dma.c b/drivers/dma/sprd-dma.c > index 2b639adb48ba..20c3cb1ef2f5 100644 > --- a/drivers/dma/sprd-dma.c > +++ b/drivers/dma/sprd-dma.c > @@ -572,8 +572,7 @@ static void sprd_dma_stop(struct sprd_dma_chn *schan) > schan->cur_desc = NULL; > } > > -static bool sprd_dma_check_trans_done(struct sprd_dma_desc *sdesc, > - enum sprd_dma_int_type int_type, > +static bool sprd_dma_check_trans_done(enum sprd_dma_int_type int_type, > enum sprd_dma_req_mode req_mode) > { > if (int_type == SPRD_DMA_NO_INT) > @@ -619,8 +618,7 @@ static irqreturn_t dma_irq_handle(int irq, void *dev_id) > vchan_cyclic_callback(&sdesc->vd); > } else { > /* Check if the dma request descriptor is done. */ > - trans_done = sprd_dma_check_trans_done(sdesc, int_type, > - req_type); > + trans_done = sprd_dma_check_trans_done(int_type, req_type); > if (trans_done == true) { > vchan_cookie_complete(&sdesc->vd); > schan->cur_desc = NULL; > -- > 2.17.1 >
diff --git a/drivers/dma/sprd-dma.c b/drivers/dma/sprd-dma.c index 2b639adb48ba..20c3cb1ef2f5 100644 --- a/drivers/dma/sprd-dma.c +++ b/drivers/dma/sprd-dma.c @@ -572,8 +572,7 @@ static void sprd_dma_stop(struct sprd_dma_chn *schan) schan->cur_desc = NULL; } -static bool sprd_dma_check_trans_done(struct sprd_dma_desc *sdesc, - enum sprd_dma_int_type int_type, +static bool sprd_dma_check_trans_done(enum sprd_dma_int_type int_type, enum sprd_dma_req_mode req_mode) { if (int_type == SPRD_DMA_NO_INT) @@ -619,8 +618,7 @@ static irqreturn_t dma_irq_handle(int irq, void *dev_id) vchan_cyclic_callback(&sdesc->vd); } else { /* Check if the dma request descriptor is done. */ - trans_done = sprd_dma_check_trans_done(sdesc, int_type, - req_type); + trans_done = sprd_dma_check_trans_done(int_type, req_type); if (trans_done == true) { vchan_cookie_complete(&sdesc->vd); schan->cur_desc = NULL;