From mboxrd@z Thu Jan 1 00:00:00 1970 From: mika.westerberg@iki.fi (Mika Westerberg) Date: Thu, 9 Jun 2011 23:41:13 +0300 Subject: [PATCH v2 1/5] ep93xx: set DMA masks for the ep93xx_eth In-Reply-To: References: Message-ID: <20110609204113.GE2857@acer> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Thu, Jun 09, 2011 at 03:27:11PM -0500, H Hartley Sweeten wrote: > On Thursday, June 02, 2011 12:00 PM, Mika Westerberg wrote: > > > > As the driver is now passing platform device to the DMA mapping functions, > > we should give it valid DMA masks. > > > > Signed-off-by: Mika Westerberg > > Acked-by: Russell King > > Since you made this patch the first in the series the commit message is not > quite right. Other than that: > > Acked-by: H Hartley Sweeten Ah, right. I forgot to change the commit message. How about following? From: Mika Westerberg Subject: [PATCH 1/5] ep93xx: set DMA masks for the ep93xx_eth Since the driver uses the DMA API, we should pass it valid DMA masks. Signed-off-by: Mika Westerberg Acked-by: Russell King Acked-by: H Hartley Sweeten --- arch/arm/mach-ep93xx/core.c | 6 +++++- 1 files changed, 5 insertions(+), 1 deletions(-) diff --git a/arch/arm/mach-ep93xx/core.c b/arch/arm/mach-ep93xx/core.c index 8207954..1d4b65f 100644 --- a/arch/arm/mach-ep93xx/core.c +++ b/arch/arm/mach-ep93xx/core.c @@ -402,11 +402,15 @@ static struct resource ep93xx_eth_resource[] = { } }; +static u64 ep93xx_eth_dma_mask = DMA_BIT_MASK(32); + static struct platform_device ep93xx_eth_device = { .name = "ep93xx-eth", .id = -1, .dev = { - .platform_data = &ep93xx_eth_data, + .platform_data = &ep93xx_eth_data, + .coherent_dma_mask = DMA_BIT_MASK(32), + .dma_mask = &ep93xx_eth_dma_mask, }, .num_resources = ARRAY_SIZE(ep93xx_eth_resource), .resource = ep93xx_eth_resource, -- 1.7.4.4