* [PATCH v10 0/2] ARM: davinci: move EDMA API to arm/common
@ 2013-06-17 10:06 ` Sekhar Nori
0 siblings, 0 replies; 16+ messages in thread
From: Sekhar Nori @ 2013-06-17 10:06 UTC (permalink / raw)
To: Russell King, Vinod Koul, Chris Ball, Liam Girdwood, Mark Brown
Cc: Joel Fernandes A,
davinci-linux-open-source-VycZQUHpC/PFrsHnngEfi1aTQe2KTcn/,
Arnd Bergmann, Kevin Hilman, linux-mmc-u79uwXL29TY76Z2rM5mHXA,
alsa-devel-K7yf7f+aM1XWsZ/bQMPhNw, Olof Johansson,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
This is a repost of first two patches of Matt Porter's EDMA
series below:
http://www.mail-archive.com/davinci-linux-open-source-VycZQUHpC/PFrsHnngEfi1aTQe2KTcn/@public.gmane.org/msg25317.html
The only change that has been made is to drop changes
related to DaVinci SFFSDR machine support for sound
since that code is now queued to be removed.
Thanks,
Sekhar
Matt Porter (2):
ARM: davinci: move private EDMA API to arm/common
ARM: edma: remove unused transfer controller handlers
arch/arm/Kconfig | 1 +
arch/arm/common/Kconfig | 3 +
arch/arm/common/Makefile | 1 +
arch/arm/{mach-davinci/dma.c => common/edma.c} | 39 +--------
arch/arm/mach-davinci/Makefile | 2 +-
arch/arm/mach-davinci/board-tnetv107x-evm.c | 2 +-
arch/arm/mach-davinci/davinci.h | 2 +-
arch/arm/mach-davinci/devices-tnetv107x.c | 2 +-
arch/arm/mach-davinci/devices.c | 6 +-
arch/arm/mach-davinci/dm355.c | 2 +-
arch/arm/mach-davinci/dm365.c | 2 +-
arch/arm/mach-davinci/dm644x.c | 2 +-
arch/arm/mach-davinci/dm646x.c | 2 +-
arch/arm/mach-davinci/include/mach/da8xx.h | 2 +-
drivers/dma/edma.c | 2 +-
drivers/mmc/host/davinci_mmc.c | 1 +
include/linux/mfd/davinci_voicecodec.h | 3 +-
.../mach => include/linux/platform_data}/edma.h | 89 +-------------------
include/linux/platform_data/spi-davinci.h | 2 +-
sound/soc/davinci/davinci-evm.c | 1 +
sound/soc/davinci/davinci-pcm.c | 1 +
sound/soc/davinci/davinci-pcm.h | 2 +-
22 files changed, 29 insertions(+), 140 deletions(-)
rename arch/arm/{mach-davinci/dma.c => common/edma.c} (97%)
rename {arch/arm/mach-davinci/include/mach => include/linux/platform_data}/edma.h (59%)
--
1.7.10.1
^ permalink raw reply [flat|nested] 16+ messages in thread* [PATCH v10 0/2] ARM: davinci: move EDMA API to arm/common @ 2013-06-17 10:06 ` Sekhar Nori 0 siblings, 0 replies; 16+ messages in thread From: Sekhar Nori @ 2013-06-17 10:06 UTC (permalink / raw) To: linux-arm-kernel This is a repost of first two patches of Matt Porter's EDMA series below: http://www.mail-archive.com/davinci-linux-open-source at linux.davincidsp.com/msg25317.html The only change that has been made is to drop changes related to DaVinci SFFSDR machine support for sound since that code is now queued to be removed. Thanks, Sekhar Matt Porter (2): ARM: davinci: move private EDMA API to arm/common ARM: edma: remove unused transfer controller handlers arch/arm/Kconfig | 1 + arch/arm/common/Kconfig | 3 + arch/arm/common/Makefile | 1 + arch/arm/{mach-davinci/dma.c => common/edma.c} | 39 +-------- arch/arm/mach-davinci/Makefile | 2 +- arch/arm/mach-davinci/board-tnetv107x-evm.c | 2 +- arch/arm/mach-davinci/davinci.h | 2 +- arch/arm/mach-davinci/devices-tnetv107x.c | 2 +- arch/arm/mach-davinci/devices.c | 6 +- arch/arm/mach-davinci/dm355.c | 2 +- arch/arm/mach-davinci/dm365.c | 2 +- arch/arm/mach-davinci/dm644x.c | 2 +- arch/arm/mach-davinci/dm646x.c | 2 +- arch/arm/mach-davinci/include/mach/da8xx.h | 2 +- drivers/dma/edma.c | 2 +- drivers/mmc/host/davinci_mmc.c | 1 + include/linux/mfd/davinci_voicecodec.h | 3 +- .../mach => include/linux/platform_data}/edma.h | 89 +------------------- include/linux/platform_data/spi-davinci.h | 2 +- sound/soc/davinci/davinci-evm.c | 1 + sound/soc/davinci/davinci-pcm.c | 1 + sound/soc/davinci/davinci-pcm.h | 2 +- 22 files changed, 29 insertions(+), 140 deletions(-) rename arch/arm/{mach-davinci/dma.c => common/edma.c} (97%) rename {arch/arm/mach-davinci/include/mach => include/linux/platform_data}/edma.h (59%) -- 1.7.10.1 ^ permalink raw reply [flat|nested] 16+ messages in thread
[parent not found: <1371463571-2578-1-git-send-email-nsekhar-l0cyMroinI0@public.gmane.org>]
* [PATCH v10 1/2] ARM: davinci: move private EDMA API to arm/common 2013-06-17 10:06 ` Sekhar Nori @ 2013-06-17 10:06 ` Sekhar Nori -1 siblings, 0 replies; 16+ messages in thread From: Sekhar Nori @ 2013-06-17 10:06 UTC (permalink / raw) To: Russell King, Vinod Koul, Chris Ball, Liam Girdwood, Mark Brown Cc: Joel Fernandes A, davinci-linux-open-source-VycZQUHpC/PFrsHnngEfi1aTQe2KTcn/, Arnd Bergmann, Kevin Hilman, linux-mmc-u79uwXL29TY76Z2rM5mHXA, alsa-devel-K7yf7f+aM1XWsZ/bQMPhNw, Olof Johansson, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r From: Matt Porter <mporter-l0cyMroinI0@public.gmane.org> Move mach-davinci/dma.c to common/edma.c so it can be used by OMAP (specifically AM33xx) as well. Signed-off-by: Matt Porter <mporter-l0cyMroinI0@public.gmane.org> Acked-by: Chris Ball <cjb-2X9k7bc8m7Mdnm+yROfE0A@public.gmane.org> # davinci_mmc.c [nsekhar-l0cyMroinI0@public.gmane.org: dropped davinci sffsdr changes] Signed-off-by: Sekhar Nori <nsekhar-l0cyMroinI0@public.gmane.org> --- arch/arm/Kconfig | 1 + arch/arm/common/Kconfig | 3 + arch/arm/common/Makefile | 1 + arch/arm/{mach-davinci/dma.c => common/edma.c} | 2 +- arch/arm/mach-davinci/Makefile | 2 +- arch/arm/mach-davinci/board-tnetv107x-evm.c | 2 +- arch/arm/mach-davinci/davinci.h | 2 +- arch/arm/mach-davinci/devices-tnetv107x.c | 2 +- arch/arm/mach-davinci/devices.c | 6 +- arch/arm/mach-davinci/dm355.c | 2 +- arch/arm/mach-davinci/dm365.c | 2 +- arch/arm/mach-davinci/dm644x.c | 2 +- arch/arm/mach-davinci/dm646x.c | 2 +- arch/arm/mach-davinci/include/mach/da8xx.h | 2 +- drivers/dma/edma.c | 2 +- drivers/mmc/host/davinci_mmc.c | 1 + include/linux/mfd/davinci_voicecodec.h | 3 +- .../mach => include/linux/platform_data}/edma.h | 89 +------------------- include/linux/platform_data/spi-davinci.h | 2 +- sound/soc/davinci/davinci-evm.c | 1 + sound/soc/davinci/davinci-pcm.c | 1 + sound/soc/davinci/davinci-pcm.h | 2 +- 22 files changed, 29 insertions(+), 103 deletions(-) rename arch/arm/{mach-davinci/dma.c => common/edma.c} (99%) rename {arch/arm/mach-davinci/include/mach => include/linux/platform_data}/edma.h (59%) diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig index 49d993c..b1c66a4 100644 --- a/arch/arm/Kconfig +++ b/arch/arm/Kconfig @@ -840,6 +840,7 @@ config ARCH_DAVINCI select GENERIC_IRQ_CHIP select HAVE_IDE select NEED_MACH_GPIO_H + select TI_PRIV_EDMA select USE_OF select ZONE_DMA help diff --git a/arch/arm/common/Kconfig b/arch/arm/common/Kconfig index 9353184..c3a4e9c 100644 --- a/arch/arm/common/Kconfig +++ b/arch/arm/common/Kconfig @@ -17,3 +17,6 @@ config SHARP_PARAM config SHARP_SCOOP bool + +config TI_PRIV_EDMA + bool diff --git a/arch/arm/common/Makefile b/arch/arm/common/Makefile index 48434cb..8c60f47 100644 --- a/arch/arm/common/Makefile +++ b/arch/arm/common/Makefile @@ -16,3 +16,4 @@ obj-$(CONFIG_ARM_TIMER_SP804) += timer-sp.o obj-$(CONFIG_MCPM) += mcpm_head.o mcpm_entry.o mcpm_platsmp.o vlock.o AFLAGS_mcpm_head.o := -march=armv7-a AFLAGS_vlock.o := -march=armv7-a +obj-$(CONFIG_TI_PRIV_EDMA) += edma.o diff --git a/arch/arm/mach-davinci/dma.c b/arch/arm/common/edma.c similarity index 99% rename from arch/arm/mach-davinci/dma.c rename to arch/arm/common/edma.c index 45b7c71..dcaeb8e 100644 --- a/arch/arm/mach-davinci/dma.c +++ b/arch/arm/common/edma.c @@ -25,7 +25,7 @@ #include <linux/io.h> #include <linux/slab.h> -#include <mach/edma.h> +#include <linux/platform_data/edma.h> /* Offsets matching "struct edmacc_param" */ #define PARM_OPT 0x00 diff --git a/arch/arm/mach-davinci/Makefile b/arch/arm/mach-davinci/Makefile index dd1ffcc..63997a1 100644 --- a/arch/arm/mach-davinci/Makefile +++ b/arch/arm/mach-davinci/Makefile @@ -5,7 +5,7 @@ # Common objects obj-y := time.o clock.o serial.o psc.o \ - dma.o usb.o common.o sram.o aemif.o + usb.o common.o sram.o aemif.o obj-$(CONFIG_DAVINCI_MUX) += mux.o diff --git a/arch/arm/mach-davinci/board-tnetv107x-evm.c b/arch/arm/mach-davinci/board-tnetv107x-evm.c index ba79837..78ea395 100644 --- a/arch/arm/mach-davinci/board-tnetv107x-evm.c +++ b/arch/arm/mach-davinci/board-tnetv107x-evm.c @@ -26,12 +26,12 @@ #include <linux/input.h> #include <linux/input/matrix_keypad.h> #include <linux/spi/spi.h> +#include <linux/platform_data/edma.h> #include <asm/mach/arch.h> #include <asm/mach-types.h> #include <mach/irqs.h> -#include <mach/edma.h> #include <mach/mux.h> #include <mach/cp_intc.h> #include <mach/tnetv107x.h> diff --git a/arch/arm/mach-davinci/davinci.h b/arch/arm/mach-davinci/davinci.h index b4b7451..a883043 100644 --- a/arch/arm/mach-davinci/davinci.h +++ b/arch/arm/mach-davinci/davinci.h @@ -23,9 +23,9 @@ #include <linux/platform_device.h> #include <linux/spi/spi.h> #include <linux/platform_data/davinci_asp.h> +#include <linux/platform_data/edma.h> #include <linux/platform_data/keyscan-davinci.h> #include <mach/hardware.h> -#include <mach/edma.h> #include <media/davinci/vpfe_capture.h> #include <media/davinci/vpif_types.h> diff --git a/arch/arm/mach-davinci/devices-tnetv107x.c b/arch/arm/mach-davinci/devices-tnetv107x.c index cfb194d..612a085 100644 --- a/arch/arm/mach-davinci/devices-tnetv107x.c +++ b/arch/arm/mach-davinci/devices-tnetv107x.c @@ -18,10 +18,10 @@ #include <linux/dma-mapping.h> #include <linux/clk.h> #include <linux/slab.h> +#include <linux/platform_data/edma.h> #include <mach/common.h> #include <mach/irqs.h> -#include <mach/edma.h> #include <mach/tnetv107x.h> #include "clock.h" diff --git a/arch/arm/mach-davinci/devices.c b/arch/arm/mach-davinci/devices.c index a7068a3..90b83d0 100644 --- a/arch/arm/mach-davinci/devices.c +++ b/arch/arm/mach-davinci/devices.c @@ -19,9 +19,10 @@ #include <mach/irqs.h> #include <mach/cputype.h> #include <mach/mux.h> -#include <mach/edma.h> #include <linux/platform_data/mmc-davinci.h> #include <mach/time.h> +#include <linux/platform_data/edma.h> + #include "davinci.h" #include "clock.h" @@ -34,6 +35,9 @@ #define DM365_MMCSD0_BASE 0x01D11000 #define DM365_MMCSD1_BASE 0x01D00000 +#define DAVINCI_DMA_MMCRXEVT 26 +#define DAVINCI_DMA_MMCTXEVT 27 + void __iomem *davinci_sysmod_base; void davinci_map_sysmod(void) diff --git a/arch/arm/mach-davinci/dm355.c b/arch/arm/mach-davinci/dm355.c index a11034a..526cf7d 100644 --- a/arch/arm/mach-davinci/dm355.c +++ b/arch/arm/mach-davinci/dm355.c @@ -19,7 +19,6 @@ #include <asm/mach/map.h> #include <mach/cputype.h> -#include <mach/edma.h> #include <mach/psc.h> #include <mach/mux.h> #include <mach/irqs.h> @@ -28,6 +27,7 @@ #include <mach/common.h> #include <linux/platform_data/spi-davinci.h> #include <mach/gpio-davinci.h> +#include <linux/platform_data/edma.h> #include "davinci.h" #include "clock.h" diff --git a/arch/arm/mach-davinci/dm365.c b/arch/arm/mach-davinci/dm365.c index 40fa4fe..c4b7411 100644 --- a/arch/arm/mach-davinci/dm365.c +++ b/arch/arm/mach-davinci/dm365.c @@ -18,11 +18,11 @@ #include <linux/platform_device.h> #include <linux/dma-mapping.h> #include <linux/spi/spi.h> +#include <linux/platform_data/edma.h> #include <asm/mach/map.h> #include <mach/cputype.h> -#include <mach/edma.h> #include <mach/psc.h> #include <mach/mux.h> #include <mach/irqs.h> diff --git a/arch/arm/mach-davinci/dm644x.c b/arch/arm/mach-davinci/dm644x.c index 4d37d3e..dd156d5 100644 --- a/arch/arm/mach-davinci/dm644x.c +++ b/arch/arm/mach-davinci/dm644x.c @@ -12,11 +12,11 @@ #include <linux/clk.h> #include <linux/serial_8250.h> #include <linux/platform_device.h> +#include <linux/platform_data/edma.h> #include <asm/mach/map.h> #include <mach/cputype.h> -#include <mach/edma.h> #include <mach/irqs.h> #include <mach/psc.h> #include <mach/mux.h> diff --git a/arch/arm/mach-davinci/dm646x.c b/arch/arm/mach-davinci/dm646x.c index ac7b431..6d52a32 100644 --- a/arch/arm/mach-davinci/dm646x.c +++ b/arch/arm/mach-davinci/dm646x.c @@ -13,11 +13,11 @@ #include <linux/clk.h> #include <linux/serial_8250.h> #include <linux/platform_device.h> +#include <linux/platform_data/edma.h> #include <asm/mach/map.h> #include <mach/cputype.h> -#include <mach/edma.h> #include <mach/irqs.h> #include <mach/psc.h> #include <mach/mux.h> diff --git a/arch/arm/mach-davinci/include/mach/da8xx.h b/arch/arm/mach-davinci/include/mach/da8xx.h index faec783..3c797e2 100644 --- a/arch/arm/mach-davinci/include/mach/da8xx.h +++ b/arch/arm/mach-davinci/include/mach/da8xx.h @@ -20,8 +20,8 @@ #include <linux/videodev2.h> #include <mach/serial.h> -#include <mach/edma.h> #include <mach/pm.h> +#include <linux/platform_data/edma.h> #include <linux/platform_data/i2c-davinci.h> #include <linux/platform_data/mmc-davinci.h> #include <linux/platform_data/usb-davinci.h> diff --git a/drivers/dma/edma.c b/drivers/dma/edma.c index cd7e328..5f3e532 100644 --- a/drivers/dma/edma.c +++ b/drivers/dma/edma.c @@ -24,7 +24,7 @@ #include <linux/slab.h> #include <linux/spinlock.h> -#include <mach/edma.h> +#include <linux/platform_data/edma.h> #include "dmaengine.h" #include "virt-dma.h" diff --git a/drivers/mmc/host/davinci_mmc.c b/drivers/mmc/host/davinci_mmc.c index 3946a0e..5dfb70c 100644 --- a/drivers/mmc/host/davinci_mmc.c +++ b/drivers/mmc/host/davinci_mmc.c @@ -37,6 +37,7 @@ #include <linux/of.h> #include <linux/of_device.h> +#include <linux/platform_data/edma.h> #include <linux/platform_data/mmc-davinci.h> /* diff --git a/include/linux/mfd/davinci_voicecodec.h b/include/linux/mfd/davinci_voicecodec.h index 0ab6132..7dd6524 100644 --- a/include/linux/mfd/davinci_voicecodec.h +++ b/include/linux/mfd/davinci_voicecodec.h @@ -26,8 +26,7 @@ #include <linux/kernel.h> #include <linux/platform_device.h> #include <linux/mfd/core.h> - -#include <mach/edma.h> +#include <linux/platform_data/edma.h> /* * Register values. diff --git a/arch/arm/mach-davinci/include/mach/edma.h b/include/linux/platform_data/edma.h similarity index 59% rename from arch/arm/mach-davinci/include/mach/edma.h rename to include/linux/platform_data/edma.h index 7e84c90..2344ea2 100644 --- a/arch/arm/mach-davinci/include/mach/edma.h +++ b/include/linux/platform_data/edma.h @@ -1,28 +1,12 @@ /* - * TI DAVINCI dma definitions + * TI EDMA definitions * - * Copyright (C) 2006-2009 Texas Instruments. + * Copyright (C) 2006-2013 Texas Instruments. * * This program is free software; you can redistribute it and/or modify it * under the terms of the GNU General Public License as published by the * Free Software Foundation; either version 2 of the License, or (at your * option) any later version. - * - * THIS SOFTWARE IS PROVIDED ``AS IS'' AND ANY EXPRESS OR IMPLIED - * WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF - * MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN - * NO EVENT SHALL THE AUTHOR BE LIABLE FOR ANY DIRECT, INDIRECT, - * INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT - * NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF - * USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON - * ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT - * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF - * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. - * - * You should have received a copy of the GNU General Public License along - * with this program; if not, write to the Free Software Foundation, Inc., - * 675 Mass Ave, Cambridge, MA 02139, USA. - * */ /* @@ -69,11 +53,6 @@ struct edmacc_param { unsigned int ccnt; }; -#define CCINT0_INTERRUPT 16 -#define CCERRINT_INTERRUPT 17 -#define TCERRINT0_INTERRUPT 18 -#define TCERRINT1_INTERRUPT 19 - /* fields in edmacc_param.opt */ #define SAM BIT(0) #define DAM BIT(1) @@ -87,70 +66,6 @@ struct edmacc_param { #define TCCHEN BIT(22) #define ITCCHEN BIT(23) -#define TRWORD (0x7<<2) -#define PAENTRY (0x1ff<<5) - -/* Drivers should avoid using these symbolic names for dm644x - * channels, and use platform_device IORESOURCE_DMA resources - * instead. (Other DaVinci chips have different peripherals - * and thus have different DMA channel mappings.) - */ -#define DAVINCI_DMA_MCBSP_TX 2 -#define DAVINCI_DMA_MCBSP_RX 3 -#define DAVINCI_DMA_VPSS_HIST 4 -#define DAVINCI_DMA_VPSS_H3A 5 -#define DAVINCI_DMA_VPSS_PRVU 6 -#define DAVINCI_DMA_VPSS_RSZ 7 -#define DAVINCI_DMA_IMCOP_IMXINT 8 -#define DAVINCI_DMA_IMCOP_VLCDINT 9 -#define DAVINCI_DMA_IMCO_PASQINT 10 -#define DAVINCI_DMA_IMCOP_DSQINT 11 -#define DAVINCI_DMA_SPI_SPIX 16 -#define DAVINCI_DMA_SPI_SPIR 17 -#define DAVINCI_DMA_UART0_URXEVT0 18 -#define DAVINCI_DMA_UART0_UTXEVT0 19 -#define DAVINCI_DMA_UART1_URXEVT1 20 -#define DAVINCI_DMA_UART1_UTXEVT1 21 -#define DAVINCI_DMA_UART2_URXEVT2 22 -#define DAVINCI_DMA_UART2_UTXEVT2 23 -#define DAVINCI_DMA_MEMSTK_MSEVT 24 -#define DAVINCI_DMA_MMCRXEVT 26 -#define DAVINCI_DMA_MMCTXEVT 27 -#define DAVINCI_DMA_I2C_ICREVT 28 -#define DAVINCI_DMA_I2C_ICXEVT 29 -#define DAVINCI_DMA_GPIO_GPINT0 32 -#define DAVINCI_DMA_GPIO_GPINT1 33 -#define DAVINCI_DMA_GPIO_GPINT2 34 -#define DAVINCI_DMA_GPIO_GPINT3 35 -#define DAVINCI_DMA_GPIO_GPINT4 36 -#define DAVINCI_DMA_GPIO_GPINT5 37 -#define DAVINCI_DMA_GPIO_GPINT6 38 -#define DAVINCI_DMA_GPIO_GPINT7 39 -#define DAVINCI_DMA_GPIO_GPBNKINT0 40 -#define DAVINCI_DMA_GPIO_GPBNKINT1 41 -#define DAVINCI_DMA_GPIO_GPBNKINT2 42 -#define DAVINCI_DMA_GPIO_GPBNKINT3 43 -#define DAVINCI_DMA_GPIO_GPBNKINT4 44 -#define DAVINCI_DMA_TIMER0_TINT0 48 -#define DAVINCI_DMA_TIMER1_TINT1 49 -#define DAVINCI_DMA_TIMER2_TINT2 50 -#define DAVINCI_DMA_TIMER3_TINT3 51 -#define DAVINCI_DMA_PWM0 52 -#define DAVINCI_DMA_PWM1 53 -#define DAVINCI_DMA_PWM2 54 - -/* DA830 specific EDMA3 information */ -#define EDMA_DA830_NUM_DMACH 32 -#define EDMA_DA830_NUM_TCC 32 -#define EDMA_DA830_NUM_PARAMENTRY 128 -#define EDMA_DA830_NUM_EVQUE 2 -#define EDMA_DA830_NUM_TC 2 -#define EDMA_DA830_CHMAP_EXIST 0 -#define EDMA_DA830_NUM_REGIONS 4 -#define DA830_DMACH2EVENT_MAP0 0x000FC03Fu -#define DA830_DMACH2EVENT_MAP1 0x00000000u -#define DA830_EDMA_ARM_OWN 0x30FFCCFFu - /*ch_status paramater of callback function possible values*/ #define DMA_COMPLETE 1 #define DMA_CC_ERROR 2 diff --git a/include/linux/platform_data/spi-davinci.h b/include/linux/platform_data/spi-davinci.h index 7af305b..8dc2fa47 100644 --- a/include/linux/platform_data/spi-davinci.h +++ b/include/linux/platform_data/spi-davinci.h @@ -19,7 +19,7 @@ #ifndef __ARCH_ARM_DAVINCI_SPI_H #define __ARCH_ARM_DAVINCI_SPI_H -#include <mach/edma.h> +#include <linux/platform_data/edma.h> #define SPI_INTERN_CS 0xFF diff --git a/sound/soc/davinci/davinci-evm.c b/sound/soc/davinci/davinci-evm.c index 484b22c..fd7c45b 100644 --- a/sound/soc/davinci/davinci-evm.c +++ b/sound/soc/davinci/davinci-evm.c @@ -14,6 +14,7 @@ #include <linux/timer.h> #include <linux/interrupt.h> #include <linux/platform_device.h> +#include <linux/platform_data/edma.h> #include <linux/i2c.h> #include <sound/core.h> #include <sound/pcm.h> diff --git a/sound/soc/davinci/davinci-pcm.c b/sound/soc/davinci/davinci-pcm.c index b2f27c2..8460edc 100644 --- a/sound/soc/davinci/davinci-pcm.c +++ b/sound/soc/davinci/davinci-pcm.c @@ -17,6 +17,7 @@ #include <linux/dma-mapping.h> #include <linux/kernel.h> #include <linux/genalloc.h> +#include <linux/platform_data/edma.h> #include <sound/core.h> #include <sound/pcm.h> diff --git a/sound/soc/davinci/davinci-pcm.h b/sound/soc/davinci/davinci-pcm.h index b6ef703..fbb710c 100644 --- a/sound/soc/davinci/davinci-pcm.h +++ b/sound/soc/davinci/davinci-pcm.h @@ -14,7 +14,7 @@ #include <linux/genalloc.h> #include <linux/platform_data/davinci_asp.h> -#include <mach/edma.h> +#include <linux/platform_data/edma.h> struct davinci_pcm_dma_params { int channel; /* sync dma channel ID */ -- 1.7.10.1 ^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH v10 1/2] ARM: davinci: move private EDMA API to arm/common @ 2013-06-17 10:06 ` Sekhar Nori 0 siblings, 0 replies; 16+ messages in thread From: Sekhar Nori @ 2013-06-17 10:06 UTC (permalink / raw) To: linux-arm-kernel From: Matt Porter <mporter@ti.com> Move mach-davinci/dma.c to common/edma.c so it can be used by OMAP (specifically AM33xx) as well. Signed-off-by: Matt Porter <mporter@ti.com> Acked-by: Chris Ball <cjb@laptop.org> # davinci_mmc.c [nsekhar at ti.com: dropped davinci sffsdr changes] Signed-off-by: Sekhar Nori <nsekhar@ti.com> --- arch/arm/Kconfig | 1 + arch/arm/common/Kconfig | 3 + arch/arm/common/Makefile | 1 + arch/arm/{mach-davinci/dma.c => common/edma.c} | 2 +- arch/arm/mach-davinci/Makefile | 2 +- arch/arm/mach-davinci/board-tnetv107x-evm.c | 2 +- arch/arm/mach-davinci/davinci.h | 2 +- arch/arm/mach-davinci/devices-tnetv107x.c | 2 +- arch/arm/mach-davinci/devices.c | 6 +- arch/arm/mach-davinci/dm355.c | 2 +- arch/arm/mach-davinci/dm365.c | 2 +- arch/arm/mach-davinci/dm644x.c | 2 +- arch/arm/mach-davinci/dm646x.c | 2 +- arch/arm/mach-davinci/include/mach/da8xx.h | 2 +- drivers/dma/edma.c | 2 +- drivers/mmc/host/davinci_mmc.c | 1 + include/linux/mfd/davinci_voicecodec.h | 3 +- .../mach => include/linux/platform_data}/edma.h | 89 +------------------- include/linux/platform_data/spi-davinci.h | 2 +- sound/soc/davinci/davinci-evm.c | 1 + sound/soc/davinci/davinci-pcm.c | 1 + sound/soc/davinci/davinci-pcm.h | 2 +- 22 files changed, 29 insertions(+), 103 deletions(-) rename arch/arm/{mach-davinci/dma.c => common/edma.c} (99%) rename {arch/arm/mach-davinci/include/mach => include/linux/platform_data}/edma.h (59%) diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig index 49d993c..b1c66a4 100644 --- a/arch/arm/Kconfig +++ b/arch/arm/Kconfig @@ -840,6 +840,7 @@ config ARCH_DAVINCI select GENERIC_IRQ_CHIP select HAVE_IDE select NEED_MACH_GPIO_H + select TI_PRIV_EDMA select USE_OF select ZONE_DMA help diff --git a/arch/arm/common/Kconfig b/arch/arm/common/Kconfig index 9353184..c3a4e9c 100644 --- a/arch/arm/common/Kconfig +++ b/arch/arm/common/Kconfig @@ -17,3 +17,6 @@ config SHARP_PARAM config SHARP_SCOOP bool + +config TI_PRIV_EDMA + bool diff --git a/arch/arm/common/Makefile b/arch/arm/common/Makefile index 48434cb..8c60f47 100644 --- a/arch/arm/common/Makefile +++ b/arch/arm/common/Makefile @@ -16,3 +16,4 @@ obj-$(CONFIG_ARM_TIMER_SP804) += timer-sp.o obj-$(CONFIG_MCPM) += mcpm_head.o mcpm_entry.o mcpm_platsmp.o vlock.o AFLAGS_mcpm_head.o := -march=armv7-a AFLAGS_vlock.o := -march=armv7-a +obj-$(CONFIG_TI_PRIV_EDMA) += edma.o diff --git a/arch/arm/mach-davinci/dma.c b/arch/arm/common/edma.c similarity index 99% rename from arch/arm/mach-davinci/dma.c rename to arch/arm/common/edma.c index 45b7c71..dcaeb8e 100644 --- a/arch/arm/mach-davinci/dma.c +++ b/arch/arm/common/edma.c @@ -25,7 +25,7 @@ #include <linux/io.h> #include <linux/slab.h> -#include <mach/edma.h> +#include <linux/platform_data/edma.h> /* Offsets matching "struct edmacc_param" */ #define PARM_OPT 0x00 diff --git a/arch/arm/mach-davinci/Makefile b/arch/arm/mach-davinci/Makefile index dd1ffcc..63997a1 100644 --- a/arch/arm/mach-davinci/Makefile +++ b/arch/arm/mach-davinci/Makefile @@ -5,7 +5,7 @@ # Common objects obj-y := time.o clock.o serial.o psc.o \ - dma.o usb.o common.o sram.o aemif.o + usb.o common.o sram.o aemif.o obj-$(CONFIG_DAVINCI_MUX) += mux.o diff --git a/arch/arm/mach-davinci/board-tnetv107x-evm.c b/arch/arm/mach-davinci/board-tnetv107x-evm.c index ba79837..78ea395 100644 --- a/arch/arm/mach-davinci/board-tnetv107x-evm.c +++ b/arch/arm/mach-davinci/board-tnetv107x-evm.c @@ -26,12 +26,12 @@ #include <linux/input.h> #include <linux/input/matrix_keypad.h> #include <linux/spi/spi.h> +#include <linux/platform_data/edma.h> #include <asm/mach/arch.h> #include <asm/mach-types.h> #include <mach/irqs.h> -#include <mach/edma.h> #include <mach/mux.h> #include <mach/cp_intc.h> #include <mach/tnetv107x.h> diff --git a/arch/arm/mach-davinci/davinci.h b/arch/arm/mach-davinci/davinci.h index b4b7451..a883043 100644 --- a/arch/arm/mach-davinci/davinci.h +++ b/arch/arm/mach-davinci/davinci.h @@ -23,9 +23,9 @@ #include <linux/platform_device.h> #include <linux/spi/spi.h> #include <linux/platform_data/davinci_asp.h> +#include <linux/platform_data/edma.h> #include <linux/platform_data/keyscan-davinci.h> #include <mach/hardware.h> -#include <mach/edma.h> #include <media/davinci/vpfe_capture.h> #include <media/davinci/vpif_types.h> diff --git a/arch/arm/mach-davinci/devices-tnetv107x.c b/arch/arm/mach-davinci/devices-tnetv107x.c index cfb194d..612a085 100644 --- a/arch/arm/mach-davinci/devices-tnetv107x.c +++ b/arch/arm/mach-davinci/devices-tnetv107x.c @@ -18,10 +18,10 @@ #include <linux/dma-mapping.h> #include <linux/clk.h> #include <linux/slab.h> +#include <linux/platform_data/edma.h> #include <mach/common.h> #include <mach/irqs.h> -#include <mach/edma.h> #include <mach/tnetv107x.h> #include "clock.h" diff --git a/arch/arm/mach-davinci/devices.c b/arch/arm/mach-davinci/devices.c index a7068a3..90b83d0 100644 --- a/arch/arm/mach-davinci/devices.c +++ b/arch/arm/mach-davinci/devices.c @@ -19,9 +19,10 @@ #include <mach/irqs.h> #include <mach/cputype.h> #include <mach/mux.h> -#include <mach/edma.h> #include <linux/platform_data/mmc-davinci.h> #include <mach/time.h> +#include <linux/platform_data/edma.h> + #include "davinci.h" #include "clock.h" @@ -34,6 +35,9 @@ #define DM365_MMCSD0_BASE 0x01D11000 #define DM365_MMCSD1_BASE 0x01D00000 +#define DAVINCI_DMA_MMCRXEVT 26 +#define DAVINCI_DMA_MMCTXEVT 27 + void __iomem *davinci_sysmod_base; void davinci_map_sysmod(void) diff --git a/arch/arm/mach-davinci/dm355.c b/arch/arm/mach-davinci/dm355.c index a11034a..526cf7d 100644 --- a/arch/arm/mach-davinci/dm355.c +++ b/arch/arm/mach-davinci/dm355.c @@ -19,7 +19,6 @@ #include <asm/mach/map.h> #include <mach/cputype.h> -#include <mach/edma.h> #include <mach/psc.h> #include <mach/mux.h> #include <mach/irqs.h> @@ -28,6 +27,7 @@ #include <mach/common.h> #include <linux/platform_data/spi-davinci.h> #include <mach/gpio-davinci.h> +#include <linux/platform_data/edma.h> #include "davinci.h" #include "clock.h" diff --git a/arch/arm/mach-davinci/dm365.c b/arch/arm/mach-davinci/dm365.c index 40fa4fe..c4b7411 100644 --- a/arch/arm/mach-davinci/dm365.c +++ b/arch/arm/mach-davinci/dm365.c @@ -18,11 +18,11 @@ #include <linux/platform_device.h> #include <linux/dma-mapping.h> #include <linux/spi/spi.h> +#include <linux/platform_data/edma.h> #include <asm/mach/map.h> #include <mach/cputype.h> -#include <mach/edma.h> #include <mach/psc.h> #include <mach/mux.h> #include <mach/irqs.h> diff --git a/arch/arm/mach-davinci/dm644x.c b/arch/arm/mach-davinci/dm644x.c index 4d37d3e..dd156d5 100644 --- a/arch/arm/mach-davinci/dm644x.c +++ b/arch/arm/mach-davinci/dm644x.c @@ -12,11 +12,11 @@ #include <linux/clk.h> #include <linux/serial_8250.h> #include <linux/platform_device.h> +#include <linux/platform_data/edma.h> #include <asm/mach/map.h> #include <mach/cputype.h> -#include <mach/edma.h> #include <mach/irqs.h> #include <mach/psc.h> #include <mach/mux.h> diff --git a/arch/arm/mach-davinci/dm646x.c b/arch/arm/mach-davinci/dm646x.c index ac7b431..6d52a32 100644 --- a/arch/arm/mach-davinci/dm646x.c +++ b/arch/arm/mach-davinci/dm646x.c @@ -13,11 +13,11 @@ #include <linux/clk.h> #include <linux/serial_8250.h> #include <linux/platform_device.h> +#include <linux/platform_data/edma.h> #include <asm/mach/map.h> #include <mach/cputype.h> -#include <mach/edma.h> #include <mach/irqs.h> #include <mach/psc.h> #include <mach/mux.h> diff --git a/arch/arm/mach-davinci/include/mach/da8xx.h b/arch/arm/mach-davinci/include/mach/da8xx.h index faec783..3c797e2 100644 --- a/arch/arm/mach-davinci/include/mach/da8xx.h +++ b/arch/arm/mach-davinci/include/mach/da8xx.h @@ -20,8 +20,8 @@ #include <linux/videodev2.h> #include <mach/serial.h> -#include <mach/edma.h> #include <mach/pm.h> +#include <linux/platform_data/edma.h> #include <linux/platform_data/i2c-davinci.h> #include <linux/platform_data/mmc-davinci.h> #include <linux/platform_data/usb-davinci.h> diff --git a/drivers/dma/edma.c b/drivers/dma/edma.c index cd7e328..5f3e532 100644 --- a/drivers/dma/edma.c +++ b/drivers/dma/edma.c @@ -24,7 +24,7 @@ #include <linux/slab.h> #include <linux/spinlock.h> -#include <mach/edma.h> +#include <linux/platform_data/edma.h> #include "dmaengine.h" #include "virt-dma.h" diff --git a/drivers/mmc/host/davinci_mmc.c b/drivers/mmc/host/davinci_mmc.c index 3946a0e..5dfb70c 100644 --- a/drivers/mmc/host/davinci_mmc.c +++ b/drivers/mmc/host/davinci_mmc.c @@ -37,6 +37,7 @@ #include <linux/of.h> #include <linux/of_device.h> +#include <linux/platform_data/edma.h> #include <linux/platform_data/mmc-davinci.h> /* diff --git a/include/linux/mfd/davinci_voicecodec.h b/include/linux/mfd/davinci_voicecodec.h index 0ab6132..7dd6524 100644 --- a/include/linux/mfd/davinci_voicecodec.h +++ b/include/linux/mfd/davinci_voicecodec.h @@ -26,8 +26,7 @@ #include <linux/kernel.h> #include <linux/platform_device.h> #include <linux/mfd/core.h> - -#include <mach/edma.h> +#include <linux/platform_data/edma.h> /* * Register values. diff --git a/arch/arm/mach-davinci/include/mach/edma.h b/include/linux/platform_data/edma.h similarity index 59% rename from arch/arm/mach-davinci/include/mach/edma.h rename to include/linux/platform_data/edma.h index 7e84c90..2344ea2 100644 --- a/arch/arm/mach-davinci/include/mach/edma.h +++ b/include/linux/platform_data/edma.h @@ -1,28 +1,12 @@ /* - * TI DAVINCI dma definitions + * TI EDMA definitions * - * Copyright (C) 2006-2009 Texas Instruments. + * Copyright (C) 2006-2013 Texas Instruments. * * This program is free software; you can redistribute it and/or modify it * under the terms of the GNU General Public License as published by the * Free Software Foundation; either version 2 of the License, or (at your * option) any later version. - * - * THIS SOFTWARE IS PROVIDED ``AS IS'' AND ANY EXPRESS OR IMPLIED - * WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF - * MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN - * NO EVENT SHALL THE AUTHOR BE LIABLE FOR ANY DIRECT, INDIRECT, - * INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT - * NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF - * USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON - * ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT - * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF - * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. - * - * You should have received a copy of the GNU General Public License along - * with this program; if not, write to the Free Software Foundation, Inc., - * 675 Mass Ave, Cambridge, MA 02139, USA. - * */ /* @@ -69,11 +53,6 @@ struct edmacc_param { unsigned int ccnt; }; -#define CCINT0_INTERRUPT 16 -#define CCERRINT_INTERRUPT 17 -#define TCERRINT0_INTERRUPT 18 -#define TCERRINT1_INTERRUPT 19 - /* fields in edmacc_param.opt */ #define SAM BIT(0) #define DAM BIT(1) @@ -87,70 +66,6 @@ struct edmacc_param { #define TCCHEN BIT(22) #define ITCCHEN BIT(23) -#define TRWORD (0x7<<2) -#define PAENTRY (0x1ff<<5) - -/* Drivers should avoid using these symbolic names for dm644x - * channels, and use platform_device IORESOURCE_DMA resources - * instead. (Other DaVinci chips have different peripherals - * and thus have different DMA channel mappings.) - */ -#define DAVINCI_DMA_MCBSP_TX 2 -#define DAVINCI_DMA_MCBSP_RX 3 -#define DAVINCI_DMA_VPSS_HIST 4 -#define DAVINCI_DMA_VPSS_H3A 5 -#define DAVINCI_DMA_VPSS_PRVU 6 -#define DAVINCI_DMA_VPSS_RSZ 7 -#define DAVINCI_DMA_IMCOP_IMXINT 8 -#define DAVINCI_DMA_IMCOP_VLCDINT 9 -#define DAVINCI_DMA_IMCO_PASQINT 10 -#define DAVINCI_DMA_IMCOP_DSQINT 11 -#define DAVINCI_DMA_SPI_SPIX 16 -#define DAVINCI_DMA_SPI_SPIR 17 -#define DAVINCI_DMA_UART0_URXEVT0 18 -#define DAVINCI_DMA_UART0_UTXEVT0 19 -#define DAVINCI_DMA_UART1_URXEVT1 20 -#define DAVINCI_DMA_UART1_UTXEVT1 21 -#define DAVINCI_DMA_UART2_URXEVT2 22 -#define DAVINCI_DMA_UART2_UTXEVT2 23 -#define DAVINCI_DMA_MEMSTK_MSEVT 24 -#define DAVINCI_DMA_MMCRXEVT 26 -#define DAVINCI_DMA_MMCTXEVT 27 -#define DAVINCI_DMA_I2C_ICREVT 28 -#define DAVINCI_DMA_I2C_ICXEVT 29 -#define DAVINCI_DMA_GPIO_GPINT0 32 -#define DAVINCI_DMA_GPIO_GPINT1 33 -#define DAVINCI_DMA_GPIO_GPINT2 34 -#define DAVINCI_DMA_GPIO_GPINT3 35 -#define DAVINCI_DMA_GPIO_GPINT4 36 -#define DAVINCI_DMA_GPIO_GPINT5 37 -#define DAVINCI_DMA_GPIO_GPINT6 38 -#define DAVINCI_DMA_GPIO_GPINT7 39 -#define DAVINCI_DMA_GPIO_GPBNKINT0 40 -#define DAVINCI_DMA_GPIO_GPBNKINT1 41 -#define DAVINCI_DMA_GPIO_GPBNKINT2 42 -#define DAVINCI_DMA_GPIO_GPBNKINT3 43 -#define DAVINCI_DMA_GPIO_GPBNKINT4 44 -#define DAVINCI_DMA_TIMER0_TINT0 48 -#define DAVINCI_DMA_TIMER1_TINT1 49 -#define DAVINCI_DMA_TIMER2_TINT2 50 -#define DAVINCI_DMA_TIMER3_TINT3 51 -#define DAVINCI_DMA_PWM0 52 -#define DAVINCI_DMA_PWM1 53 -#define DAVINCI_DMA_PWM2 54 - -/* DA830 specific EDMA3 information */ -#define EDMA_DA830_NUM_DMACH 32 -#define EDMA_DA830_NUM_TCC 32 -#define EDMA_DA830_NUM_PARAMENTRY 128 -#define EDMA_DA830_NUM_EVQUE 2 -#define EDMA_DA830_NUM_TC 2 -#define EDMA_DA830_CHMAP_EXIST 0 -#define EDMA_DA830_NUM_REGIONS 4 -#define DA830_DMACH2EVENT_MAP0 0x000FC03Fu -#define DA830_DMACH2EVENT_MAP1 0x00000000u -#define DA830_EDMA_ARM_OWN 0x30FFCCFFu - /*ch_status paramater of callback function possible values*/ #define DMA_COMPLETE 1 #define DMA_CC_ERROR 2 diff --git a/include/linux/platform_data/spi-davinci.h b/include/linux/platform_data/spi-davinci.h index 7af305b..8dc2fa47 100644 --- a/include/linux/platform_data/spi-davinci.h +++ b/include/linux/platform_data/spi-davinci.h @@ -19,7 +19,7 @@ #ifndef __ARCH_ARM_DAVINCI_SPI_H #define __ARCH_ARM_DAVINCI_SPI_H -#include <mach/edma.h> +#include <linux/platform_data/edma.h> #define SPI_INTERN_CS 0xFF diff --git a/sound/soc/davinci/davinci-evm.c b/sound/soc/davinci/davinci-evm.c index 484b22c..fd7c45b 100644 --- a/sound/soc/davinci/davinci-evm.c +++ b/sound/soc/davinci/davinci-evm.c @@ -14,6 +14,7 @@ #include <linux/timer.h> #include <linux/interrupt.h> #include <linux/platform_device.h> +#include <linux/platform_data/edma.h> #include <linux/i2c.h> #include <sound/core.h> #include <sound/pcm.h> diff --git a/sound/soc/davinci/davinci-pcm.c b/sound/soc/davinci/davinci-pcm.c index b2f27c2..8460edc 100644 --- a/sound/soc/davinci/davinci-pcm.c +++ b/sound/soc/davinci/davinci-pcm.c @@ -17,6 +17,7 @@ #include <linux/dma-mapping.h> #include <linux/kernel.h> #include <linux/genalloc.h> +#include <linux/platform_data/edma.h> #include <sound/core.h> #include <sound/pcm.h> diff --git a/sound/soc/davinci/davinci-pcm.h b/sound/soc/davinci/davinci-pcm.h index b6ef703..fbb710c 100644 --- a/sound/soc/davinci/davinci-pcm.h +++ b/sound/soc/davinci/davinci-pcm.h @@ -14,7 +14,7 @@ #include <linux/genalloc.h> #include <linux/platform_data/davinci_asp.h> -#include <mach/edma.h> +#include <linux/platform_data/edma.h> struct davinci_pcm_dma_params { int channel; /* sync dma channel ID */ -- 1.7.10.1 ^ permalink raw reply related [flat|nested] 16+ messages in thread
* Re: [PATCH v10 1/2] ARM: davinci: move private EDMA API to arm/common 2013-06-17 10:06 ` Sekhar Nori @ 2013-06-17 10:43 ` Mark Brown -1 siblings, 0 replies; 16+ messages in thread From: Mark Brown @ 2013-06-17 10:43 UTC (permalink / raw) To: Sekhar Nori Cc: Russell King, Vinod Koul, Chris Ball, Liam Girdwood, Kevin Hilman, Matt Porter, Joel Fernandes A, linux-arm-kernel, davinci-linux-open-source, linux-mmc, alsa-devel, Arnd Bergmann, Olof Johansson [-- Attachment #1: Type: text/plain, Size: 460 bytes --] On Mon, Jun 17, 2013 at 03:36:10PM +0530, Sekhar Nori wrote: > From: Matt Porter <mporter@ti.com> > > Move mach-davinci/dma.c to common/edma.c so it can be used > by OMAP (specifically AM33xx) as well. > > Signed-off-by: Matt Porter <mporter@ti.com> > Acked-by: Chris Ball <cjb@laptop.org> # davinci_mmc.c > [nsekhar@ti.com: dropped davinci sffsdr changes] > Signed-off-by: Sekhar Nori <nsekhar@ti.com> Acked-by: Mark Brown <broonie@linaro.org> [-- Attachment #2: Digital signature --] [-- Type: application/pgp-signature, Size: 836 bytes --] ^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH v10 1/2] ARM: davinci: move private EDMA API to arm/common @ 2013-06-17 10:43 ` Mark Brown 0 siblings, 0 replies; 16+ messages in thread From: Mark Brown @ 2013-06-17 10:43 UTC (permalink / raw) To: linux-arm-kernel On Mon, Jun 17, 2013 at 03:36:10PM +0530, Sekhar Nori wrote: > From: Matt Porter <mporter@ti.com> > > Move mach-davinci/dma.c to common/edma.c so it can be used > by OMAP (specifically AM33xx) as well. > > Signed-off-by: Matt Porter <mporter@ti.com> > Acked-by: Chris Ball <cjb@laptop.org> # davinci_mmc.c > [nsekhar at ti.com: dropped davinci sffsdr changes] > Signed-off-by: Sekhar Nori <nsekhar@ti.com> Acked-by: Mark Brown <broonie@linaro.org> -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20130617/5ea59388/attachment.sig> ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH v10 1/2] ARM: davinci: move private EDMA API to arm/common 2013-06-17 10:06 ` Sekhar Nori @ 2013-06-17 17:03 ` Olof Johansson -1 siblings, 0 replies; 16+ messages in thread From: Olof Johansson @ 2013-06-17 17:03 UTC (permalink / raw) To: Sekhar Nori Cc: Russell King, Vinod Koul, Chris Ball, Liam Girdwood, Mark Brown, Kevin Hilman, Matt Porter, Joel Fernandes A, linux-arm-kernel, davinci-linux-open-source, linux-mmc, alsa-devel, Arnd Bergmann On Mon, Jun 17, 2013 at 03:36:10PM +0530, Sekhar Nori wrote: > From: Matt Porter <mporter@ti.com> > > Move mach-davinci/dma.c to common/edma.c so it can be used > by OMAP (specifically AM33xx) as well. > > Signed-off-by: Matt Porter <mporter@ti.com> > Acked-by: Chris Ball <cjb@laptop.org> # davinci_mmc.c > [nsekhar@ti.com: dropped davinci sffsdr changes] > Signed-off-by: Sekhar Nori <nsekhar@ti.com> Acked-by: Olof Johansson <olof@lixom.net> Tiny nit below, no need to respin just for that. I suppose we should take this through arm-soc. > --- > arch/arm/Kconfig | 1 + > arch/arm/common/Kconfig | 3 + > arch/arm/common/Makefile | 1 + > arch/arm/{mach-davinci/dma.c => common/edma.c} | 2 +- > arch/arm/mach-davinci/Makefile | 2 +- > arch/arm/mach-davinci/board-tnetv107x-evm.c | 2 +- > arch/arm/mach-davinci/davinci.h | 2 +- > arch/arm/mach-davinci/devices-tnetv107x.c | 2 +- > arch/arm/mach-davinci/devices.c | 6 +- > arch/arm/mach-davinci/dm355.c | 2 +- > arch/arm/mach-davinci/dm365.c | 2 +- > arch/arm/mach-davinci/dm644x.c | 2 +- > arch/arm/mach-davinci/dm646x.c | 2 +- > arch/arm/mach-davinci/include/mach/da8xx.h | 2 +- > drivers/dma/edma.c | 2 +- > drivers/mmc/host/davinci_mmc.c | 1 + > include/linux/mfd/davinci_voicecodec.h | 3 +- > .../mach => include/linux/platform_data}/edma.h | 89 +------------------- > include/linux/platform_data/spi-davinci.h | 2 +- > sound/soc/davinci/davinci-evm.c | 1 + > sound/soc/davinci/davinci-pcm.c | 1 + > sound/soc/davinci/davinci-pcm.h | 2 +- > 22 files changed, 29 insertions(+), 103 deletions(-) > rename arch/arm/{mach-davinci/dma.c => common/edma.c} (99%) > rename {arch/arm/mach-davinci/include/mach => include/linux/platform_data}/edma.h (59%) > > diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig > index 49d993c..b1c66a4 100644 > --- a/arch/arm/Kconfig > +++ b/arch/arm/Kconfig > @@ -840,6 +840,7 @@ config ARCH_DAVINCI > select GENERIC_IRQ_CHIP > select HAVE_IDE > select NEED_MACH_GPIO_H > + select TI_PRIV_EDMA What does PRIV here mean? Privileged? Private? I think it can be shortened to TI_EDMA. > select USE_OF > select ZONE_DMA > help > diff --git a/arch/arm/common/Kconfig b/arch/arm/common/Kconfig > index 9353184..c3a4e9c 100644 > --- a/arch/arm/common/Kconfig > +++ b/arch/arm/common/Kconfig > @@ -17,3 +17,6 @@ config SHARP_PARAM > > config SHARP_SCOOP > bool > + > +config TI_PRIV_EDMA > + bool > diff --git a/arch/arm/common/Makefile b/arch/arm/common/Makefile > index 48434cb..8c60f47 100644 > --- a/arch/arm/common/Makefile > +++ b/arch/arm/common/Makefile > @@ -16,3 +16,4 @@ obj-$(CONFIG_ARM_TIMER_SP804) += timer-sp.o > obj-$(CONFIG_MCPM) += mcpm_head.o mcpm_entry.o mcpm_platsmp.o vlock.o > AFLAGS_mcpm_head.o := -march=armv7-a > AFLAGS_vlock.o := -march=armv7-a > +obj-$(CONFIG_TI_PRIV_EDMA) += edma.o ^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH v10 1/2] ARM: davinci: move private EDMA API to arm/common @ 2013-06-17 17:03 ` Olof Johansson 0 siblings, 0 replies; 16+ messages in thread From: Olof Johansson @ 2013-06-17 17:03 UTC (permalink / raw) To: linux-arm-kernel On Mon, Jun 17, 2013 at 03:36:10PM +0530, Sekhar Nori wrote: > From: Matt Porter <mporter@ti.com> > > Move mach-davinci/dma.c to common/edma.c so it can be used > by OMAP (specifically AM33xx) as well. > > Signed-off-by: Matt Porter <mporter@ti.com> > Acked-by: Chris Ball <cjb@laptop.org> # davinci_mmc.c > [nsekhar at ti.com: dropped davinci sffsdr changes] > Signed-off-by: Sekhar Nori <nsekhar@ti.com> Acked-by: Olof Johansson <olof@lixom.net> Tiny nit below, no need to respin just for that. I suppose we should take this through arm-soc. > --- > arch/arm/Kconfig | 1 + > arch/arm/common/Kconfig | 3 + > arch/arm/common/Makefile | 1 + > arch/arm/{mach-davinci/dma.c => common/edma.c} | 2 +- > arch/arm/mach-davinci/Makefile | 2 +- > arch/arm/mach-davinci/board-tnetv107x-evm.c | 2 +- > arch/arm/mach-davinci/davinci.h | 2 +- > arch/arm/mach-davinci/devices-tnetv107x.c | 2 +- > arch/arm/mach-davinci/devices.c | 6 +- > arch/arm/mach-davinci/dm355.c | 2 +- > arch/arm/mach-davinci/dm365.c | 2 +- > arch/arm/mach-davinci/dm644x.c | 2 +- > arch/arm/mach-davinci/dm646x.c | 2 +- > arch/arm/mach-davinci/include/mach/da8xx.h | 2 +- > drivers/dma/edma.c | 2 +- > drivers/mmc/host/davinci_mmc.c | 1 + > include/linux/mfd/davinci_voicecodec.h | 3 +- > .../mach => include/linux/platform_data}/edma.h | 89 +------------------- > include/linux/platform_data/spi-davinci.h | 2 +- > sound/soc/davinci/davinci-evm.c | 1 + > sound/soc/davinci/davinci-pcm.c | 1 + > sound/soc/davinci/davinci-pcm.h | 2 +- > 22 files changed, 29 insertions(+), 103 deletions(-) > rename arch/arm/{mach-davinci/dma.c => common/edma.c} (99%) > rename {arch/arm/mach-davinci/include/mach => include/linux/platform_data}/edma.h (59%) > > diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig > index 49d993c..b1c66a4 100644 > --- a/arch/arm/Kconfig > +++ b/arch/arm/Kconfig > @@ -840,6 +840,7 @@ config ARCH_DAVINCI > select GENERIC_IRQ_CHIP > select HAVE_IDE > select NEED_MACH_GPIO_H > + select TI_PRIV_EDMA What does PRIV here mean? Privileged? Private? I think it can be shortened to TI_EDMA. > select USE_OF > select ZONE_DMA > help > diff --git a/arch/arm/common/Kconfig b/arch/arm/common/Kconfig > index 9353184..c3a4e9c 100644 > --- a/arch/arm/common/Kconfig > +++ b/arch/arm/common/Kconfig > @@ -17,3 +17,6 @@ config SHARP_PARAM > > config SHARP_SCOOP > bool > + > +config TI_PRIV_EDMA > + bool > diff --git a/arch/arm/common/Makefile b/arch/arm/common/Makefile > index 48434cb..8c60f47 100644 > --- a/arch/arm/common/Makefile > +++ b/arch/arm/common/Makefile > @@ -16,3 +16,4 @@ obj-$(CONFIG_ARM_TIMER_SP804) += timer-sp.o > obj-$(CONFIG_MCPM) += mcpm_head.o mcpm_entry.o mcpm_platsmp.o vlock.o > AFLAGS_mcpm_head.o := -march=armv7-a > AFLAGS_vlock.o := -march=armv7-a > +obj-$(CONFIG_TI_PRIV_EDMA) += edma.o ^ permalink raw reply [flat|nested] 16+ messages in thread
[parent not found: <20130617170326.GA28497-O5ziIzlqnXUVNXGz7ipsyg@public.gmane.org>]
* RE: [PATCH v10 1/2] ARM: davinci: move private EDMA API to arm/common 2013-06-17 17:03 ` Olof Johansson @ 2013-06-17 17:55 ` Fernandes, Joel A -1 siblings, 0 replies; 16+ messages in thread From: Fernandes, Joel A @ 2013-06-17 17:55 UTC (permalink / raw) To: Olof Johansson, Nori, Sekhar Cc: davinci-linux-open-source-VycZQUHpC/PFrsHnngEfi1aTQe2KTcn/@public.gmane.org, Russell King, Arnd Bergmann, alsa-devel-K7yf7f+aM1XWsZ/bQMPhNw@public.gmane.org, Vinod Koul, linux-mmc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Liam Girdwood, Kevin Hilman, Mark Brown, Chris Ball, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org Hi Olof, > -----Original Message----- > From: Olof Johansson [mailto:olof-nZhT3qVonbNeoWH0uzbU5w@public.gmane.org] > Sent: Monday, June 17, 2013 12:03 PM > To: Nori, Sekhar > Cc: Russell King; Vinod Koul; Chris Ball; Liam Girdwood; Mark Brown; Kevin > Hilman; Matt Porter; Fernandes, Joel A; linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org; > davinci-linux-open-source-VycZQUHpC/PFrsHnngEfi1aTQe2KTcn/@public.gmane.org; linux-mmc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org; > alsa-devel-K7yf7f+aM1XWsZ/bQMPhNw@public.gmane.org; Arnd Bergmann > Subject: Re: [PATCH v10 1/2] ARM: davinci: move private EDMA API to > arm/common > > On Mon, Jun 17, 2013 at 03:36:10PM +0530, Sekhar Nori wrote: > > From: Matt Porter <mporter-l0cyMroinI0@public.gmane.org> > > > > Move mach-davinci/dma.c to common/edma.c so it can be used by OMAP > > (specifically AM33xx) as well. > > > > Signed-off-by: Matt Porter <mporter-l0cyMroinI0@public.gmane.org> > > Acked-by: Chris Ball <cjb-2X9k7bc8m7Mdnm+yROfE0A@public.gmane.org> # davinci_mmc.c > > [nsekhar-l0cyMroinI0@public.gmane.org: dropped davinci sffsdr changes] > > Signed-off-by: Sekhar Nori <nsekhar-l0cyMroinI0@public.gmane.org> > > Acked-by: Olof Johansson <olof-nZhT3qVonbNeoWH0uzbU5w@public.gmane.org> > > Tiny nit below, no need to respin just for that. > > I suppose we should take this through arm-soc. > > > --- > > arch/arm/Kconfig | 1 + > > arch/arm/common/Kconfig | 3 + > > arch/arm/common/Makefile | 1 + > > arch/arm/{mach-davinci/dma.c => common/edma.c} | 2 +- > > arch/arm/mach-davinci/Makefile | 2 +- > > arch/arm/mach-davinci/board-tnetv107x-evm.c | 2 +- > > arch/arm/mach-davinci/davinci.h | 2 +- > > arch/arm/mach-davinci/devices-tnetv107x.c | 2 +- > > arch/arm/mach-davinci/devices.c | 6 +- > > arch/arm/mach-davinci/dm355.c | 2 +- > > arch/arm/mach-davinci/dm365.c | 2 +- > > arch/arm/mach-davinci/dm644x.c | 2 +- > > arch/arm/mach-davinci/dm646x.c | 2 +- > > arch/arm/mach-davinci/include/mach/da8xx.h | 2 +- > > drivers/dma/edma.c | 2 +- > > drivers/mmc/host/davinci_mmc.c | 1 + > > include/linux/mfd/davinci_voicecodec.h | 3 +- > > .../mach => include/linux/platform_data}/edma.h | 89 +------------------- > > include/linux/platform_data/spi-davinci.h | 2 +- > > sound/soc/davinci/davinci-evm.c | 1 + > > sound/soc/davinci/davinci-pcm.c | 1 + > > sound/soc/davinci/davinci-pcm.h | 2 +- > > 22 files changed, 29 insertions(+), 103 deletions(-) rename > > arch/arm/{mach-davinci/dma.c => common/edma.c} (99%) rename > > {arch/arm/mach-davinci/include/mach => > > include/linux/platform_data}/edma.h (59%) > > > > diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig index > > 49d993c..b1c66a4 100644 > > --- a/arch/arm/Kconfig > > +++ b/arch/arm/Kconfig > > @@ -840,6 +840,7 @@ config ARCH_DAVINCI > > select GENERIC_IRQ_CHIP > > select HAVE_IDE > > select NEED_MACH_GPIO_H > > + select TI_PRIV_EDMA > > What does PRIV here mean? Privileged? Private? I think it can be shortened to > TI_EDMA. > [Joel] TI_EDMA already exists. It is used to build drivers/dma/edma. I have a patch that reuses the TI_EDMA option instead of the PRIV one . I will include that In my v11 series for EDMA (v10 was just posted) . If its ok, let us merge Sekhar's patch as-is, and then mine will get rid of the PRIV option (among other things). Thanks, Joel ^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH v10 1/2] ARM: davinci: move private EDMA API to arm/common @ 2013-06-17 17:55 ` Fernandes, Joel A 0 siblings, 0 replies; 16+ messages in thread From: Fernandes, Joel A @ 2013-06-17 17:55 UTC (permalink / raw) To: linux-arm-kernel Hi Olof, > -----Original Message----- > From: Olof Johansson [mailto:olof at lixom.net] > Sent: Monday, June 17, 2013 12:03 PM > To: Nori, Sekhar > Cc: Russell King; Vinod Koul; Chris Ball; Liam Girdwood; Mark Brown; Kevin > Hilman; Matt Porter; Fernandes, Joel A; linux-arm-kernel at lists.infradead.org; > davinci-linux-open-source at linux.davincidsp.com; linux-mmc at vger.kernel.org; > alsa-devel at alsa-project.org; Arnd Bergmann > Subject: Re: [PATCH v10 1/2] ARM: davinci: move private EDMA API to > arm/common > > On Mon, Jun 17, 2013 at 03:36:10PM +0530, Sekhar Nori wrote: > > From: Matt Porter <mporter@ti.com> > > > > Move mach-davinci/dma.c to common/edma.c so it can be used by OMAP > > (specifically AM33xx) as well. > > > > Signed-off-by: Matt Porter <mporter@ti.com> > > Acked-by: Chris Ball <cjb@laptop.org> # davinci_mmc.c > > [nsekhar at ti.com: dropped davinci sffsdr changes] > > Signed-off-by: Sekhar Nori <nsekhar@ti.com> > > Acked-by: Olof Johansson <olof@lixom.net> > > Tiny nit below, no need to respin just for that. > > I suppose we should take this through arm-soc. > > > --- > > arch/arm/Kconfig | 1 + > > arch/arm/common/Kconfig | 3 + > > arch/arm/common/Makefile | 1 + > > arch/arm/{mach-davinci/dma.c => common/edma.c} | 2 +- > > arch/arm/mach-davinci/Makefile | 2 +- > > arch/arm/mach-davinci/board-tnetv107x-evm.c | 2 +- > > arch/arm/mach-davinci/davinci.h | 2 +- > > arch/arm/mach-davinci/devices-tnetv107x.c | 2 +- > > arch/arm/mach-davinci/devices.c | 6 +- > > arch/arm/mach-davinci/dm355.c | 2 +- > > arch/arm/mach-davinci/dm365.c | 2 +- > > arch/arm/mach-davinci/dm644x.c | 2 +- > > arch/arm/mach-davinci/dm646x.c | 2 +- > > arch/arm/mach-davinci/include/mach/da8xx.h | 2 +- > > drivers/dma/edma.c | 2 +- > > drivers/mmc/host/davinci_mmc.c | 1 + > > include/linux/mfd/davinci_voicecodec.h | 3 +- > > .../mach => include/linux/platform_data}/edma.h | 89 +------------------- > > include/linux/platform_data/spi-davinci.h | 2 +- > > sound/soc/davinci/davinci-evm.c | 1 + > > sound/soc/davinci/davinci-pcm.c | 1 + > > sound/soc/davinci/davinci-pcm.h | 2 +- > > 22 files changed, 29 insertions(+), 103 deletions(-) rename > > arch/arm/{mach-davinci/dma.c => common/edma.c} (99%) rename > > {arch/arm/mach-davinci/include/mach => > > include/linux/platform_data}/edma.h (59%) > > > > diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig index > > 49d993c..b1c66a4 100644 > > --- a/arch/arm/Kconfig > > +++ b/arch/arm/Kconfig > > @@ -840,6 +840,7 @@ config ARCH_DAVINCI > > select GENERIC_IRQ_CHIP > > select HAVE_IDE > > select NEED_MACH_GPIO_H > > + select TI_PRIV_EDMA > > What does PRIV here mean? Privileged? Private? I think it can be shortened to > TI_EDMA. > [Joel] TI_EDMA already exists. It is used to build drivers/dma/edma. I have a patch that reuses the TI_EDMA option instead of the PRIV one . I will include that In my v11 series for EDMA (v10 was just posted) . If its ok, let us merge Sekhar's patch as-is, and then mine will get rid of the PRIV option (among other things). Thanks, Joel ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH v10 1/2] ARM: davinci: move private EDMA API to arm/common 2013-06-17 17:55 ` Fernandes, Joel A @ 2013-06-17 17:57 ` Olof Johansson -1 siblings, 0 replies; 16+ messages in thread From: Olof Johansson @ 2013-06-17 17:57 UTC (permalink / raw) To: Fernandes, Joel A Cc: Nori, Sekhar, Russell King, Vinod Koul, Chris Ball, Liam Girdwood, Mark Brown, Kevin Hilman, Matt Porter, linux-arm-kernel@lists.infradead.org, davinci-linux-open-source@linux.davincidsp.com, linux-mmc@vger.kernel.org, alsa-devel@alsa-project.org, Arnd Bergmann On Mon, Jun 17, 2013 at 10:55 AM, Fernandes, Joel A <joelagnel@ti.com> wrote: > Hi Olof, > >> -----Original Message----- >> From: Olof Johansson [mailto:olof@lixom.net] >> Sent: Monday, June 17, 2013 12:03 PM >> To: Nori, Sekhar >> Cc: Russell King; Vinod Koul; Chris Ball; Liam Girdwood; Mark Brown; Kevin >> Hilman; Matt Porter; Fernandes, Joel A; linux-arm-kernel@lists.infradead.org; >> davinci-linux-open-source@linux.davincidsp.com; linux-mmc@vger.kernel.org; >> alsa-devel@alsa-project.org; Arnd Bergmann >> Subject: Re: [PATCH v10 1/2] ARM: davinci: move private EDMA API to >> arm/common >> >> On Mon, Jun 17, 2013 at 03:36:10PM +0530, Sekhar Nori wrote: >> > From: Matt Porter <mporter@ti.com> >> > >> > Move mach-davinci/dma.c to common/edma.c so it can be used by OMAP >> > (specifically AM33xx) as well. >> > >> > Signed-off-by: Matt Porter <mporter@ti.com> >> > Acked-by: Chris Ball <cjb@laptop.org> # davinci_mmc.c >> > [nsekhar@ti.com: dropped davinci sffsdr changes] >> > Signed-off-by: Sekhar Nori <nsekhar@ti.com> >> >> Acked-by: Olof Johansson <olof@lixom.net> >> >> Tiny nit below, no need to respin just for that. >> >> I suppose we should take this through arm-soc. >> >> > --- >> > arch/arm/Kconfig | 1 + >> > arch/arm/common/Kconfig | 3 + >> > arch/arm/common/Makefile | 1 + >> > arch/arm/{mach-davinci/dma.c => common/edma.c} | 2 +- >> > arch/arm/mach-davinci/Makefile | 2 +- >> > arch/arm/mach-davinci/board-tnetv107x-evm.c | 2 +- >> > arch/arm/mach-davinci/davinci.h | 2 +- >> > arch/arm/mach-davinci/devices-tnetv107x.c | 2 +- >> > arch/arm/mach-davinci/devices.c | 6 +- >> > arch/arm/mach-davinci/dm355.c | 2 +- >> > arch/arm/mach-davinci/dm365.c | 2 +- >> > arch/arm/mach-davinci/dm644x.c | 2 +- >> > arch/arm/mach-davinci/dm646x.c | 2 +- >> > arch/arm/mach-davinci/include/mach/da8xx.h | 2 +- >> > drivers/dma/edma.c | 2 +- >> > drivers/mmc/host/davinci_mmc.c | 1 + >> > include/linux/mfd/davinci_voicecodec.h | 3 +- >> > .../mach => include/linux/platform_data}/edma.h | 89 +------------------- >> > include/linux/platform_data/spi-davinci.h | 2 +- >> > sound/soc/davinci/davinci-evm.c | 1 + >> > sound/soc/davinci/davinci-pcm.c | 1 + >> > sound/soc/davinci/davinci-pcm.h | 2 +- >> > 22 files changed, 29 insertions(+), 103 deletions(-) rename >> > arch/arm/{mach-davinci/dma.c => common/edma.c} (99%) rename >> > {arch/arm/mach-davinci/include/mach => >> > include/linux/platform_data}/edma.h (59%) >> > >> > diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig index >> > 49d993c..b1c66a4 100644 >> > --- a/arch/arm/Kconfig >> > +++ b/arch/arm/Kconfig >> > @@ -840,6 +840,7 @@ config ARCH_DAVINCI >> > select GENERIC_IRQ_CHIP >> > select HAVE_IDE >> > select NEED_MACH_GPIO_H >> > + select TI_PRIV_EDMA >> >> What does PRIV here mean? Privileged? Private? I think it can be shortened to >> TI_EDMA. >> > [Joel] TI_EDMA already exists. It is used to build drivers/dma/edma. > I have a patch that reuses the TI_EDMA option instead of the PRIV one . I will > include that In my v11 series for EDMA (v10 was just posted) . If its ok, let us > merge Sekhar's patch as-is, and then mine will get rid of the PRIV option > (among other things). Ok, fair enough. As I said, small nit. -Olof ^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH v10 1/2] ARM: davinci: move private EDMA API to arm/common @ 2013-06-17 17:57 ` Olof Johansson 0 siblings, 0 replies; 16+ messages in thread From: Olof Johansson @ 2013-06-17 17:57 UTC (permalink / raw) To: linux-arm-kernel On Mon, Jun 17, 2013 at 10:55 AM, Fernandes, Joel A <joelagnel@ti.com> wrote: > Hi Olof, > >> -----Original Message----- >> From: Olof Johansson [mailto:olof at lixom.net] >> Sent: Monday, June 17, 2013 12:03 PM >> To: Nori, Sekhar >> Cc: Russell King; Vinod Koul; Chris Ball; Liam Girdwood; Mark Brown; Kevin >> Hilman; Matt Porter; Fernandes, Joel A; linux-arm-kernel at lists.infradead.org; >> davinci-linux-open-source at linux.davincidsp.com; linux-mmc at vger.kernel.org; >> alsa-devel at alsa-project.org; Arnd Bergmann >> Subject: Re: [PATCH v10 1/2] ARM: davinci: move private EDMA API to >> arm/common >> >> On Mon, Jun 17, 2013 at 03:36:10PM +0530, Sekhar Nori wrote: >> > From: Matt Porter <mporter@ti.com> >> > >> > Move mach-davinci/dma.c to common/edma.c so it can be used by OMAP >> > (specifically AM33xx) as well. >> > >> > Signed-off-by: Matt Porter <mporter@ti.com> >> > Acked-by: Chris Ball <cjb@laptop.org> # davinci_mmc.c >> > [nsekhar at ti.com: dropped davinci sffsdr changes] >> > Signed-off-by: Sekhar Nori <nsekhar@ti.com> >> >> Acked-by: Olof Johansson <olof@lixom.net> >> >> Tiny nit below, no need to respin just for that. >> >> I suppose we should take this through arm-soc. >> >> > --- >> > arch/arm/Kconfig | 1 + >> > arch/arm/common/Kconfig | 3 + >> > arch/arm/common/Makefile | 1 + >> > arch/arm/{mach-davinci/dma.c => common/edma.c} | 2 +- >> > arch/arm/mach-davinci/Makefile | 2 +- >> > arch/arm/mach-davinci/board-tnetv107x-evm.c | 2 +- >> > arch/arm/mach-davinci/davinci.h | 2 +- >> > arch/arm/mach-davinci/devices-tnetv107x.c | 2 +- >> > arch/arm/mach-davinci/devices.c | 6 +- >> > arch/arm/mach-davinci/dm355.c | 2 +- >> > arch/arm/mach-davinci/dm365.c | 2 +- >> > arch/arm/mach-davinci/dm644x.c | 2 +- >> > arch/arm/mach-davinci/dm646x.c | 2 +- >> > arch/arm/mach-davinci/include/mach/da8xx.h | 2 +- >> > drivers/dma/edma.c | 2 +- >> > drivers/mmc/host/davinci_mmc.c | 1 + >> > include/linux/mfd/davinci_voicecodec.h | 3 +- >> > .../mach => include/linux/platform_data}/edma.h | 89 +------------------- >> > include/linux/platform_data/spi-davinci.h | 2 +- >> > sound/soc/davinci/davinci-evm.c | 1 + >> > sound/soc/davinci/davinci-pcm.c | 1 + >> > sound/soc/davinci/davinci-pcm.h | 2 +- >> > 22 files changed, 29 insertions(+), 103 deletions(-) rename >> > arch/arm/{mach-davinci/dma.c => common/edma.c} (99%) rename >> > {arch/arm/mach-davinci/include/mach => >> > include/linux/platform_data}/edma.h (59%) >> > >> > diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig index >> > 49d993c..b1c66a4 100644 >> > --- a/arch/arm/Kconfig >> > +++ b/arch/arm/Kconfig >> > @@ -840,6 +840,7 @@ config ARCH_DAVINCI >> > select GENERIC_IRQ_CHIP >> > select HAVE_IDE >> > select NEED_MACH_GPIO_H >> > + select TI_PRIV_EDMA >> >> What does PRIV here mean? Privileged? Private? I think it can be shortened to >> TI_EDMA. >> > [Joel] TI_EDMA already exists. It is used to build drivers/dma/edma. > I have a patch that reuses the TI_EDMA option instead of the PRIV one . I will > include that In my v11 series for EDMA (v10 was just posted) . If its ok, let us > merge Sekhar's patch as-is, and then mine will get rid of the PRIV option > (among other things). Ok, fair enough. As I said, small nit. -Olof ^ permalink raw reply [flat|nested] 16+ messages in thread
[parent not found: <083BC63EECB6FD41B8E81CF7FD87CC0F2E4E95C2-YmePFLaaepqIQmiDNMet8wC/G2K4zDHf@public.gmane.org>]
* Re: [PATCH v10 1/2] ARM: davinci: move private EDMA API to arm/common 2013-06-17 17:55 ` Fernandes, Joel A @ 2013-06-18 4:34 ` Sekhar Nori -1 siblings, 0 replies; 16+ messages in thread From: Sekhar Nori @ 2013-06-18 4:34 UTC (permalink / raw) To: Fernandes, Joel A Cc: davinci-linux-open-source-VycZQUHpC/PFrsHnngEfi1aTQe2KTcn/@public.gmane.org, Russell King, Arnd Bergmann, alsa-devel-K7yf7f+aM1XWsZ/bQMPhNw@public.gmane.org, Vinod Koul, linux-mmc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Liam Girdwood, Kevin Hilman, Mark Brown, Olof Johansson, Chris Ball, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org On 6/17/2013 11:25 PM, Fernandes, Joel A wrote: > Hi Olof, > >> -----Original Message----- >> From: Olof Johansson [mailto:olof-nZhT3qVonbNeoWH0uzbU5w@public.gmane.org] >> Sent: Monday, June 17, 2013 12:03 PM >> To: Nori, Sekhar >> Cc: Russell King; Vinod Koul; Chris Ball; Liam Girdwood; Mark Brown; Kevin >> Hilman; Matt Porter; Fernandes, Joel A; linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org; >> davinci-linux-open-source-VycZQUHpC/PFrsHnngEfi1aTQe2KTcn/@public.gmane.org; linux-mmc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org; >> alsa-devel-K7yf7f+aM1XWsZ/bQMPhNw@public.gmane.org; Arnd Bergmann >> Subject: Re: [PATCH v10 1/2] ARM: davinci: move private EDMA API to >> arm/common >> >> On Mon, Jun 17, 2013 at 03:36:10PM +0530, Sekhar Nori wrote: >>> From: Matt Porter <mporter-l0cyMroinI0@public.gmane.org> >>> >>> Move mach-davinci/dma.c to common/edma.c so it can be used by OMAP >>> (specifically AM33xx) as well. >>> >>> Signed-off-by: Matt Porter <mporter-l0cyMroinI0@public.gmane.org> >>> Acked-by: Chris Ball <cjb-2X9k7bc8m7Mdnm+yROfE0A@public.gmane.org> # davinci_mmc.c >>> [nsekhar-l0cyMroinI0@public.gmane.org: dropped davinci sffsdr changes] >>> Signed-off-by: Sekhar Nori <nsekhar-l0cyMroinI0@public.gmane.org> >> >> Acked-by: Olof Johansson <olof-nZhT3qVonbNeoWH0uzbU5w@public.gmane.org> >> >> Tiny nit below, no need to respin just for that. >> >> I suppose we should take this through arm-soc. >> >>> --- >>> arch/arm/Kconfig | 1 + >>> arch/arm/common/Kconfig | 3 + >>> arch/arm/common/Makefile | 1 + >>> arch/arm/{mach-davinci/dma.c => common/edma.c} | 2 +- >>> arch/arm/mach-davinci/Makefile | 2 +- >>> arch/arm/mach-davinci/board-tnetv107x-evm.c | 2 +- >>> arch/arm/mach-davinci/davinci.h | 2 +- >>> arch/arm/mach-davinci/devices-tnetv107x.c | 2 +- >>> arch/arm/mach-davinci/devices.c | 6 +- >>> arch/arm/mach-davinci/dm355.c | 2 +- >>> arch/arm/mach-davinci/dm365.c | 2 +- >>> arch/arm/mach-davinci/dm644x.c | 2 +- >>> arch/arm/mach-davinci/dm646x.c | 2 +- >>> arch/arm/mach-davinci/include/mach/da8xx.h | 2 +- >>> drivers/dma/edma.c | 2 +- >>> drivers/mmc/host/davinci_mmc.c | 1 + >>> include/linux/mfd/davinci_voicecodec.h | 3 +- >>> .../mach => include/linux/platform_data}/edma.h | 89 +------------------- >>> include/linux/platform_data/spi-davinci.h | 2 +- >>> sound/soc/davinci/davinci-evm.c | 1 + >>> sound/soc/davinci/davinci-pcm.c | 1 + >>> sound/soc/davinci/davinci-pcm.h | 2 +- >>> 22 files changed, 29 insertions(+), 103 deletions(-) rename >>> arch/arm/{mach-davinci/dma.c => common/edma.c} (99%) rename >>> {arch/arm/mach-davinci/include/mach => >>> include/linux/platform_data}/edma.h (59%) >>> >>> diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig index >>> 49d993c..b1c66a4 100644 >>> --- a/arch/arm/Kconfig >>> +++ b/arch/arm/Kconfig >>> @@ -840,6 +840,7 @@ config ARCH_DAVINCI >>> select GENERIC_IRQ_CHIP >>> select HAVE_IDE >>> select NEED_MACH_GPIO_H >>> + select TI_PRIV_EDMA >> >> What does PRIV here mean? Privileged? Private? I think it can be shortened to >> TI_EDMA. >> > [Joel] TI_EDMA already exists. It is used to build drivers/dma/edma. > I have a patch that reuses the TI_EDMA option instead of the PRIV one . I will > include that In my v11 series for EDMA (v10 was just posted) . If its ok, let us > merge Sekhar's patch as-is, and then mine will get rid of the PRIV option > (among other things). PRIV is short for "private". We will get rid of TI_PRIV_EDMA along with all users of private DMA API (including audio). TI_EDMA and TI_PRIV_EDMA denote different things and we cannot reuse one for the other. Thanks, Sekhar ^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH v10 1/2] ARM: davinci: move private EDMA API to arm/common @ 2013-06-18 4:34 ` Sekhar Nori 0 siblings, 0 replies; 16+ messages in thread From: Sekhar Nori @ 2013-06-18 4:34 UTC (permalink / raw) To: linux-arm-kernel On 6/17/2013 11:25 PM, Fernandes, Joel A wrote: > Hi Olof, > >> -----Original Message----- >> From: Olof Johansson [mailto:olof at lixom.net] >> Sent: Monday, June 17, 2013 12:03 PM >> To: Nori, Sekhar >> Cc: Russell King; Vinod Koul; Chris Ball; Liam Girdwood; Mark Brown; Kevin >> Hilman; Matt Porter; Fernandes, Joel A; linux-arm-kernel at lists.infradead.org; >> davinci-linux-open-source at linux.davincidsp.com; linux-mmc at vger.kernel.org; >> alsa-devel at alsa-project.org; Arnd Bergmann >> Subject: Re: [PATCH v10 1/2] ARM: davinci: move private EDMA API to >> arm/common >> >> On Mon, Jun 17, 2013 at 03:36:10PM +0530, Sekhar Nori wrote: >>> From: Matt Porter <mporter@ti.com> >>> >>> Move mach-davinci/dma.c to common/edma.c so it can be used by OMAP >>> (specifically AM33xx) as well. >>> >>> Signed-off-by: Matt Porter <mporter@ti.com> >>> Acked-by: Chris Ball <cjb@laptop.org> # davinci_mmc.c >>> [nsekhar at ti.com: dropped davinci sffsdr changes] >>> Signed-off-by: Sekhar Nori <nsekhar@ti.com> >> >> Acked-by: Olof Johansson <olof@lixom.net> >> >> Tiny nit below, no need to respin just for that. >> >> I suppose we should take this through arm-soc. >> >>> --- >>> arch/arm/Kconfig | 1 + >>> arch/arm/common/Kconfig | 3 + >>> arch/arm/common/Makefile | 1 + >>> arch/arm/{mach-davinci/dma.c => common/edma.c} | 2 +- >>> arch/arm/mach-davinci/Makefile | 2 +- >>> arch/arm/mach-davinci/board-tnetv107x-evm.c | 2 +- >>> arch/arm/mach-davinci/davinci.h | 2 +- >>> arch/arm/mach-davinci/devices-tnetv107x.c | 2 +- >>> arch/arm/mach-davinci/devices.c | 6 +- >>> arch/arm/mach-davinci/dm355.c | 2 +- >>> arch/arm/mach-davinci/dm365.c | 2 +- >>> arch/arm/mach-davinci/dm644x.c | 2 +- >>> arch/arm/mach-davinci/dm646x.c | 2 +- >>> arch/arm/mach-davinci/include/mach/da8xx.h | 2 +- >>> drivers/dma/edma.c | 2 +- >>> drivers/mmc/host/davinci_mmc.c | 1 + >>> include/linux/mfd/davinci_voicecodec.h | 3 +- >>> .../mach => include/linux/platform_data}/edma.h | 89 +------------------- >>> include/linux/platform_data/spi-davinci.h | 2 +- >>> sound/soc/davinci/davinci-evm.c | 1 + >>> sound/soc/davinci/davinci-pcm.c | 1 + >>> sound/soc/davinci/davinci-pcm.h | 2 +- >>> 22 files changed, 29 insertions(+), 103 deletions(-) rename >>> arch/arm/{mach-davinci/dma.c => common/edma.c} (99%) rename >>> {arch/arm/mach-davinci/include/mach => >>> include/linux/platform_data}/edma.h (59%) >>> >>> diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig index >>> 49d993c..b1c66a4 100644 >>> --- a/arch/arm/Kconfig >>> +++ b/arch/arm/Kconfig >>> @@ -840,6 +840,7 @@ config ARCH_DAVINCI >>> select GENERIC_IRQ_CHIP >>> select HAVE_IDE >>> select NEED_MACH_GPIO_H >>> + select TI_PRIV_EDMA >> >> What does PRIV here mean? Privileged? Private? I think it can be shortened to >> TI_EDMA. >> > [Joel] TI_EDMA already exists. It is used to build drivers/dma/edma. > I have a patch that reuses the TI_EDMA option instead of the PRIV one . I will > include that In my v11 series for EDMA (v10 was just posted) . If its ok, let us > merge Sekhar's patch as-is, and then mine will get rid of the PRIV option > (among other things). PRIV is short for "private". We will get rid of TI_PRIV_EDMA along with all users of private DMA API (including audio). TI_EDMA and TI_PRIV_EDMA denote different things and we cannot reuse one for the other. Thanks, Sekhar ^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH v10 2/2] ARM: edma: remove unused transfer controller handlers 2013-06-17 10:06 ` Sekhar Nori @ 2013-06-17 10:06 ` Sekhar Nori -1 siblings, 0 replies; 16+ messages in thread From: Sekhar Nori @ 2013-06-17 10:06 UTC (permalink / raw) To: Russell King, Vinod Koul, Chris Ball, Liam Girdwood, Mark Brown Cc: Joel Fernandes A, davinci-linux-open-source-VycZQUHpC/PFrsHnngEfi1aTQe2KTcn/, Arnd Bergmann, Kevin Hilman, linux-mmc-u79uwXL29TY76Z2rM5mHXA, alsa-devel-K7yf7f+aM1XWsZ/bQMPhNw, Olof Johansson, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r From: Matt Porter <mporter-l0cyMroinI0@public.gmane.org> Fix build on OMAP, the irqs are undefined on AM33xx. These error interrupt handlers were hardcoded as disabled so since they are unused code, simply remove them. Signed-off-by: Matt Porter <mporter-l0cyMroinI0@public.gmane.org> Signed-off-by: Sekhar Nori <nsekhar-l0cyMroinI0@public.gmane.org> --- arch/arm/common/edma.c | 37 ------------------------------------- 1 file changed, 37 deletions(-) diff --git a/arch/arm/common/edma.c b/arch/arm/common/edma.c index dcaeb8e..a1db6cd 100644 --- a/arch/arm/common/edma.c +++ b/arch/arm/common/edma.c @@ -494,26 +494,6 @@ static irqreturn_t dma_ccerr_handler(int irq, void *data) return IRQ_HANDLED; } -/****************************************************************************** - * - * Transfer controller error interrupt handlers - * - *****************************************************************************/ - -#define tc_errs_handled false /* disabled as long as they're NOPs */ - -static irqreturn_t dma_tc0err_handler(int irq, void *data) -{ - dev_dbg(data, "dma_tc0err_handler\n"); - return IRQ_HANDLED; -} - -static irqreturn_t dma_tc1err_handler(int irq, void *data) -{ - dev_dbg(data, "dma_tc1err_handler\n"); - return IRQ_HANDLED; -} - static int reserve_contiguous_slots(int ctlr, unsigned int id, unsigned int num_slots, unsigned int start_slot) @@ -1541,23 +1521,6 @@ static int __init edma_probe(struct platform_device *pdev) arch_num_cc++; } - if (tc_errs_handled) { - status = request_irq(IRQ_TCERRINT0, dma_tc0err_handler, 0, - "edma_tc0", &pdev->dev); - if (status < 0) { - dev_dbg(&pdev->dev, "request_irq %d failed --> %d\n", - IRQ_TCERRINT0, status); - return status; - } - status = request_irq(IRQ_TCERRINT, dma_tc1err_handler, 0, - "edma_tc1", &pdev->dev); - if (status < 0) { - dev_dbg(&pdev->dev, "request_irq %d --> %d\n", - IRQ_TCERRINT, status); - return status; - } - } - return 0; fail: -- 1.7.10.1 ^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH v10 2/2] ARM: edma: remove unused transfer controller handlers @ 2013-06-17 10:06 ` Sekhar Nori 0 siblings, 0 replies; 16+ messages in thread From: Sekhar Nori @ 2013-06-17 10:06 UTC (permalink / raw) To: linux-arm-kernel From: Matt Porter <mporter@ti.com> Fix build on OMAP, the irqs are undefined on AM33xx. These error interrupt handlers were hardcoded as disabled so since they are unused code, simply remove them. Signed-off-by: Matt Porter <mporter@ti.com> Signed-off-by: Sekhar Nori <nsekhar@ti.com> --- arch/arm/common/edma.c | 37 ------------------------------------- 1 file changed, 37 deletions(-) diff --git a/arch/arm/common/edma.c b/arch/arm/common/edma.c index dcaeb8e..a1db6cd 100644 --- a/arch/arm/common/edma.c +++ b/arch/arm/common/edma.c @@ -494,26 +494,6 @@ static irqreturn_t dma_ccerr_handler(int irq, void *data) return IRQ_HANDLED; } -/****************************************************************************** - * - * Transfer controller error interrupt handlers - * - *****************************************************************************/ - -#define tc_errs_handled false /* disabled as long as they're NOPs */ - -static irqreturn_t dma_tc0err_handler(int irq, void *data) -{ - dev_dbg(data, "dma_tc0err_handler\n"); - return IRQ_HANDLED; -} - -static irqreturn_t dma_tc1err_handler(int irq, void *data) -{ - dev_dbg(data, "dma_tc1err_handler\n"); - return IRQ_HANDLED; -} - static int reserve_contiguous_slots(int ctlr, unsigned int id, unsigned int num_slots, unsigned int start_slot) @@ -1541,23 +1521,6 @@ static int __init edma_probe(struct platform_device *pdev) arch_num_cc++; } - if (tc_errs_handled) { - status = request_irq(IRQ_TCERRINT0, dma_tc0err_handler, 0, - "edma_tc0", &pdev->dev); - if (status < 0) { - dev_dbg(&pdev->dev, "request_irq %d failed --> %d\n", - IRQ_TCERRINT0, status); - return status; - } - status = request_irq(IRQ_TCERRINT, dma_tc1err_handler, 0, - "edma_tc1", &pdev->dev); - if (status < 0) { - dev_dbg(&pdev->dev, "request_irq %d --> %d\n", - IRQ_TCERRINT, status); - return status; - } - } - return 0; fail: -- 1.7.10.1 ^ permalink raw reply related [flat|nested] 16+ messages in thread
end of thread, other threads:[~2013-06-18 4:34 UTC | newest]
Thread overview: 16+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-06-17 10:06 [PATCH v10 0/2] ARM: davinci: move EDMA API to arm/common Sekhar Nori
2013-06-17 10:06 ` Sekhar Nori
[not found] ` <1371463571-2578-1-git-send-email-nsekhar-l0cyMroinI0@public.gmane.org>
2013-06-17 10:06 ` [PATCH v10 1/2] ARM: davinci: move private " Sekhar Nori
2013-06-17 10:06 ` Sekhar Nori
2013-06-17 10:43 ` Mark Brown
2013-06-17 10:43 ` Mark Brown
2013-06-17 17:03 ` Olof Johansson
2013-06-17 17:03 ` Olof Johansson
[not found] ` <20130617170326.GA28497-O5ziIzlqnXUVNXGz7ipsyg@public.gmane.org>
2013-06-17 17:55 ` Fernandes, Joel A
2013-06-17 17:55 ` Fernandes, Joel A
2013-06-17 17:57 ` Olof Johansson
2013-06-17 17:57 ` Olof Johansson
[not found] ` <083BC63EECB6FD41B8E81CF7FD87CC0F2E4E95C2-YmePFLaaepqIQmiDNMet8wC/G2K4zDHf@public.gmane.org>
2013-06-18 4:34 ` Sekhar Nori
2013-06-18 4:34 ` Sekhar Nori
2013-06-17 10:06 ` [PATCH v10 2/2] ARM: edma: remove unused transfer controller handlers Sekhar Nori
2013-06-17 10:06 ` Sekhar Nori
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.