Re: [PATCH] dmaengine: qcom: bam_dma: fix some doc warnings.
On 17-05-18, 10:59, Srinivas Kandagatla wrote: > Building kernel with W=1 throws up below warnings: > bam_dma.c:459: warning: Function parameter or member 'dir' > not described in 'bam_chan_init_hw' > bam_dma.c:697: warning: Function parameter or member 'chan' > not described in 'bam_dma_terminate_all' > bam_dma.c:697: warning: Excess function parameter 'bchan' > description in 'bam_dma_terminate_all' > bam_dma.c:964: warning: Function parameter or member 'bchan' > not described in 'bam_start_dma' > > Fix these!. Applied both, thanks -- ~Vinod
Re: [PATCH] dmaengine: qcom: bam_dma: fix some doc warnings.
On 17/05/18 11:55, Vinod wrote: let me know if you still want an empty line, I can respin. Yeah that's why asked, I can edit while applying so no issues Thanks, that will work! --srini
Re: [PATCH] dmaengine: qcom: bam_dma: fix some doc warnings.
On 17-05-18, 11:53, Srinivas Kandagatla wrote: > > > On 17/05/18 11:50, Vinod Koul wrote: > > On 17-05-18, 10:59, Srinivas Kandagatla wrote: > > > diff --git a/drivers/dma/qcom/bam_dma.c b/drivers/dma/qcom/bam_dma.c > > > index 4a828c18099a..a05fff8a1447 100644 > > > --- a/drivers/dma/qcom/bam_dma.c > > > +++ b/drivers/dma/qcom/bam_dma.c > > > @@ -451,7 +451,7 @@ static void bam_reset_channel(struct bam_chan *bchan) > > > /** > > >* bam_chan_init_hw - Initialize channel hardware > > >* @bchan: bam channel > > > - * > > > + * @dir: DMA transfer direction > > > > Do you want to add an empty line here? > > > TBH, Am not 100% sure it it adds any value. > > Most of the function docs in the file have this same issue.. :-) > > let me know if you still want an empty line, I can respin. Yeah that's why asked, I can edit while applying so no issues -- ~Vinod
Re: [PATCH] dmaengine: qcom: bam_dma: fix some doc warnings.
On 17/05/18 11:50, Vinod Koul wrote: On 17-05-18, 10:59, Srinivas Kandagatla wrote: diff --git a/drivers/dma/qcom/bam_dma.c b/drivers/dma/qcom/bam_dma.c index 4a828c18099a..a05fff8a1447 100644 --- a/drivers/dma/qcom/bam_dma.c +++ b/drivers/dma/qcom/bam_dma.c @@ -451,7 +451,7 @@ static void bam_reset_channel(struct bam_chan *bchan) /** * bam_chan_init_hw - Initialize channel hardware * @bchan: bam channel - * + * @dir: DMA transfer direction Do you want to add an empty line here? TBH, Am not 100% sure it it adds any value. Most of the function docs in the file have this same issue.. :-) let me know if you still want an empty line, I can respin. --srini * This function resets and initializes the BAM channel
Re: [PATCH] dmaengine: qcom: bam_dma: fix some doc warnings.
On 17-05-18, 10:59, Srinivas Kandagatla wrote: > diff --git a/drivers/dma/qcom/bam_dma.c b/drivers/dma/qcom/bam_dma.c > index 4a828c18099a..a05fff8a1447 100644 > --- a/drivers/dma/qcom/bam_dma.c > +++ b/drivers/dma/qcom/bam_dma.c > @@ -451,7 +451,7 @@ static void bam_reset_channel(struct bam_chan *bchan) > /** > * bam_chan_init_hw - Initialize channel hardware > * @bchan: bam channel > - * > + * @dir: DMA transfer direction Do you want to add an empty line here? > * This function resets and initializes the BAM channel -- ~Vinod