| Message ID | 20180517095916.11362-1-srinivas.kandagatla@linaro.org |
|---|---|
| State | Accepted |
| Commit | 9c3655cba0a84bb95c68b0570a060c2bee6ccc2c |
| Headers | show |
| Series | dmaengine: qcom: bam_dma: fix some doc warnings. | expand |
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 -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
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 -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
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 -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
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 * This function resets and initializes the BAM channel */ static void bam_chan_init_hw(struct bam_chan *bchan, @@ -687,7 +687,7 @@ static struct dma_async_tx_descriptor *bam_prep_slave_sg(struct dma_chan *chan, /** * bam_dma_terminate_all - terminate all transactions on a channel - * @bchan: bam dma channel + * @chan: bam dma channel * * Dequeues and frees all transactions * No callbacks are done @@ -958,7 +958,7 @@ static void bam_apply_new_config(struct bam_chan *bchan, /** * bam_start_dma - start next transaction - * @bchan - bam dma channel + * @bchan: bam dma channel */ static void bam_start_dma(struct bam_chan *bchan) {
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!. Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org> --- drivers/dma/qcom/bam_dma.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) -- 2.16.2 -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html