Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753419Ab2JOB20 (ORCPT ); Sun, 14 Oct 2012 21:28:26 -0400 Received: from bosmailout16.eigbox.net ([66.96.185.16]:39507 "EHLO bosmailout16.eigbox.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752555Ab2JOB2Z (ORCPT ); Sun, 14 Oct 2012 21:28:25 -0400 X-Greylist: delayed 2875 seconds by postgrey-1.27 at vger.kernel.org; Sun, 14 Oct 2012 21:28:25 EDT X-Authority-Analysis: v=2.0 cv=LIXRtuq9 c=1 sm=1 a=xCnBQGcVMpKAnt6pssLkvQ==:17 a=bc2JKO6qiGsA:10 a=jAXA38P_poEA:10 a=nlncmwdmcoAA:10 a=IkcTkHD0fZMA:10 a=cRAXhCL3cXIA:10 a=bJ0fqD8TFZgqkSadqForXVIPBlU=:19 a=5z8PQ-8DrPULJUTkwQsA:9 a=QEXdDO2ut3YA:10 a=x8qw8EAkfcRkIpZA8Q87Bg==:117 X-EN-OrigOutIP: 10.20.18.7 X-EN-IMPSID: BCgU1k001099BUA01CgUkV Message-ID: <507B5B3E.60102@yahoo.es> Date: Mon, 15 Oct 2012 08:39:26 +0800 From: Hein Tibosch User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:16.0) Gecko/20121010 Thunderbird/16.0.1 MIME-Version: 1.0 To: Andy Shevchenko CC: Andrew Morton , Andy Shevchenko , viresh kumar , spear-devel , Hans-Christian Egtvedt , Arnd Bergmann , Linux Kernel Mailing List , "ludovic.desroches" , Havard Skinnemoen , Nicolas Ferre Subject: Re: [PATCH REGRESSION FIX] dw_dmac: make driver's endianness configurable References: <507A6FA5.1000800@yahoo.es> In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-EN-UserInfo: 3946c951b80c12a8be5482963a0b1232:e0ae43bc192b431f8b69f09a37527cbc X-EN-AuthUser: hein@htibosch.net X-EN-OrigIP: 114.79.56.37 X-EN-OrigHost: unknown Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1072 Lines: 26 Hi Andy, On 10/15/2012 4:08 AM, Andy Shevchenko wrote: > On Sun, Oct 14, 2012 at 10:54 AM, Hein Tibosch wrote: >> From: Hein Tibosch >> >> The dw_dmac was originally developed for avr32 to be used with the Synopsys >> DesignWare AHB DMA controller. Starting from 2.6.38, access to the device's i/o >> >> #define dma_readl(dw, name) \ >> - readl(&(__dw_regs(dw)->name)) >> + dma_readl_native(&(__dw_regs(dw)->name)) >> #define dma_writel(dw, name, val) \ >> - writel((val), &(__dw_regs(dw)->name)) >> + dma_writel_native((val), &(__dw_regs(dw)->name)) >> >> #define channel_set_bit(dw, reg, mask) \ >> dma_writel(dw, reg, ((mask) << 8) | (mask)) > Why did you not change this one? Because "dma_writel" already calls "dma_writel_native" -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/