[lkml]   [2011]   [Oct]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH v4] dmaengine: add CSR SiRFprimaII DMAC driver
    On Mon, 2011-10-17 at 20:08 -0700, Barry Song wrote:
    > From: Rongjun Ying <>
    > Cc: Jassi Brar <>
    > Cc: Arnd Bergmann <>
    > Cc: Linus Walleij <>
    > Signed-off-by: Rongjun Ying <>
    > Signed-off-by: Barry Song <>
    > ---
    > -v4:
    > add error handler in probe;
    > free resource in remove;
    > add necessary comments in source codes;
    > fix the mismatch of license between file head and MODULE_LINCENSE;
    > define MACRO to replace magic number for xlen/dma_width boundary.

    You should _always_ run checkpatch before sending patches. It should
    have 0 errors.

    ERROR: trailing whitespace
    #773: FILE: drivers/dma/sirf-dma.c:649:

    Also there are bunch of 80char warning, see if you can reduce/eliminate
    them without sacrificing the code readability, and looking at them it
    seems possible

    Also, Please ensure you send patches which compile with ZERO warnings:
    drivers/dma/sirf-dma.c: In function 'sirfsoc_dma_probe':
    drivers/dma/sirf-dma.c:616: warning: assignment from incompatible pointer type
    drivers/dma/sirf-dma.c:617: warning: assignment from incompatible pointer type
    which comes from the fact that your prep_interleaved_dma does not have
    last argument of flags.
    On the second one you have not rebased to patch from Jassi which I
    applied after converting direction stuff.


     \ /
      Last update: 2011-10-27 10:25    [W:0.046 / U:30.952 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site