[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <1334916592.31825.35.camel@vkoul-udesk3>
Date: Fri, 20 Apr 2012 15:39:51 +0530
From: Vinod Koul <vinod.koul@...ux.intel.com>
To: Mika Westerberg <mika.westerberg@....fi>
Cc: H Hartley Sweeten <hartleys@...ionengravers.com>,
Linux Kernel <linux-kernel@...r.kernel.org>,
spi-devel-general@...ts.sourceforge.net, grant.likely@...retlab.ca
Subject: Re: [PATCH] spi/spi-ep93xx.c: use dma_transfer_direction instead
of dma_data_direction
On Thu, 2012-04-19 at 09:10 +0300, Mika Westerberg wrote:
> On Tue, Apr 17, 2012 at 06:46:36PM -0700, H Hartley Sweeten wrote:
> > A new enum indicating the dma channel direction was introduced by:
> >
> > commit 49920bc66984a512f4bcc7735a61642cd0e4d6f2
> > dmaengine: add new enum dma_transfer_direction
> >
> > The following commit changed spi-ep93xx to use the new enum:
> >
> > commit a485df4b4404379786c4bdd258bc528b2617449d
> > spi, serial: move to dma_transfer_direction
> >
> > In doing so a sparse warning was introduced:
> >
> > warning: mixing different enum types
> > int enum dma_data_direction versus
> > int enum dma_transfer_direction
> >
> > This is produced because the 'dir' passed in ep93xx_spi_dma_prepare
> > is an enum dma_data_direction and is being used to set the
> > dma_slave_config 'direction' which is now an enum dma_transfer_direction.
> >
> > Fix this by converting spi-ep93xx to use the new enum type in all
> > places.
> >
> > Signed-off-by: H Hartley Sweeten <hsweeten@...ionengravers.com>
> > Cc: Grant Likely <grant.likely@...retlab.ca>
> > Cc: Vinod Koul <vinod.koul@...ux.intel.com>
>
> Acked-by: Mika Westerberg <mika.westerberg@....fi>
Acked-by: Vinod Koul <vinod.koul@...el.com>
--
~Vinod
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists