lkml.org 
[lkml]   [2016]   [May]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    SubjectRE: [RESEND PATCH v4 1/3] dmaengine: vdma: Add config structure to differentiate dmas
    Date
    Hi Vinod,

    > On Thu, May 12, 2016 at 03:13:46PM +0530, Kedareswara rao Appana wrote:
    > > This patch adds config structure in the driver to differentiate AXI
    > > DMA's and to add more features(clock support etc..) to these DMA's.
    > >
    > > Signed-off-by: Kedareswara rao Appana <appanad@xilinx.com>
    > > ---
    > > drivers/dma/xilinx/xilinx_vdma.c | 83
    > > ++++++++++++++++++++++++----------------
    > > 1 file changed, 51 insertions(+), 32 deletions(-)
    > >
    > > diff --git a/drivers/dma/xilinx/xilinx_vdma.c
    > > b/drivers/dma/xilinx/xilinx_vdma.c
    > > index 626939b..1d50e05 100644
    > > --- a/drivers/dma/xilinx/xilinx_vdma.c
    > > +++ b/drivers/dma/xilinx/xilinx_vdma.c
    > > @@ -342,6 +342,10 @@ struct xilinx_dma_chan {
    > > void (*start_transfer)(struct xilinx_dma_chan *chan); };
    > >
    > > +struct dma_config {
    >
    > xilinx_dma_config ?

    Ok will fix in the next version...

    >
    > This makes reader believe that thismight be coething related to dmaengine core
    >
    > > static const struct of_device_id xilinx_dma_of_ids[] = {
    > > - { .compatible = "xlnx,axi-dma-1.00.a",
    > > - .data = (void *)XDMA_TYPE_AXIDMA },
    > > - { .compatible = "xlnx,axi-cdma-1.00.a",
    > > - .data = (void *)XDMA_TYPE_CDMA },
    > > - { .compatible = "xlnx,axi-vdma-1.00.a",
    > > - .data = (void *)XDMA_TYPE_VDMA },
    > > + { .compatible = "xlnx,axi-dma-1.00.a", .data = &axidma_config },
    > > + { .compatible = "xlnx,axi-cdma-1.00.a", .data = &axicdma_config },
    > > + { .compatible = "xlnx,axi-vdma-1.00.a", .data = &axivdma_config },
    >
    > why do we want a struct for this and not pass XDMA_TYPE_xxx as .data

    In the next patch of this series added clock configuration in the config structure.
    As the number clocks for each of the IP varies and the clocks are configurable
    That's why added a separate config struct instead of XDMA_TYPE...

    Regards,
    Kedar.

    >
    > --
    > ~Vinod

    \
     
     \ /
      Last update: 2016-05-12 18:21    [W:3.381 / U:0.228 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site