From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755825Ab3BDTno (ORCPT ); Mon, 4 Feb 2013 14:43:44 -0500 Received: from mail-ie0-f174.google.com ([209.85.223.174]:50903 "EHLO mail-ie0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755368Ab3BDTnn (ORCPT ); Mon, 4 Feb 2013 14:43:43 -0500 MIME-Version: 1.0 In-Reply-To: <1358850393-23111-6-git-send-email-mika.westerberg@linux.intel.com> References: <1358850393-23111-1-git-send-email-mika.westerberg@linux.intel.com> <1358850393-23111-6-git-send-email-mika.westerberg@linux.intel.com> Date: Mon, 4 Feb 2013 20:43:42 +0100 Message-ID: Subject: Re: [PATCH v3 05/10] spi/pxa2xx: break out the private DMA API usage into a separate file From: Linus Walleij To: Mika Westerberg Cc: linux-kernel@vger.kernel.org, Grant Likely , Eric Miao , Russell King , Haojian Zhuang , Mark Brown , "Rafael J. Wysocki" , chao.bi@intel.com, Andy Shevchenko , cxie4@marvell.com, ytang5@marvell.com Content-Type: text/plain; charset=ISO-8859-1 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Jan 22, 2013 at 11:26 AM, Mika Westerberg wrote: > The PXA SPI driver uses PXA platform specific private DMA implementation > which does not work on non-PXA platforms. In order to use this driver on > other platforms we break out the private DMA implementation into a separate > file that gets compiled only when CONFIG_SPI_PXA2XX_PXADMA is set. The DMA > functions are stubbed out if there is no DMA implementation selected (i.e > we are building on non-PXA platform). > > While we are there we can kill the dummy DMA bits in pxa2xx_spi.h as they > are not needed anymore for CE4100. > > Once this is done we can add the generic DMA engine support to the driver > that allows usage of any DMA controller that implements DMA engine API. > > Signed-off-by: Mika Westerberg > --- > This patch was already acked by Linus W but since I changed this by > breaking out the PXA private implementation into a separate file, I'm > hoping to get new ack from him ;-) Sorry for the long delay. This is a nice and clean cut. Acked-by: Linus Walleij Yours, Linus Walleij