linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: lee.jones@linaro.org (Lee Jones)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 06/32] ARM: ux500: Strip out duplicate USB DMA configuration
Date: Thu, 18 Apr 2013 11:11:48 +0100	[thread overview]
Message-ID: <1366279934-30761-7-git-send-email-lee.jones@linaro.org> (raw)
In-Reply-To: <1366279934-30761-1-git-send-email-lee.jones@linaro.org>

For the moment at least, the TX and RX channels for DB8500 USB are
identical, so this patch generalises them into a single structure
and passes it twice. Once as the TX and again for the RX configuration.
We're keeping the infrastructure the same i.e. passing the TX and RX
separately in case they start to differ on latter incarnations of the
platform.

Signed-off-by: Lee Jones <lee.jones@linaro.org>
---
 arch/arm/mach-ux500/cpu-db8500.c |   17 +++--------------
 1 file changed, 3 insertions(+), 14 deletions(-)

diff --git a/arch/arm/mach-ux500/cpu-db8500.c b/arch/arm/mach-ux500/cpu-db8500.c
index f72cc9b..a15678d 100644
--- a/arch/arm/mach-ux500/cpu-db8500.c
+++ b/arch/arm/mach-ux500/cpu-db8500.c
@@ -166,18 +166,7 @@ static void __init db8500_add_gpios(struct device *parent)
 	dbx500_add_pinctrl(parent, "pinctrl-db8500", U8500_PRCMU_BASE);
 }
 
-static int usb_db8500_rx_dma_cfg[] = {
-	DB8500_DMA_DEV38_USB_OTG_IEP_AND_OEP_1_9,
-	DB8500_DMA_DEV37_USB_OTG_IEP_AND_OEP_2_10,
-	DB8500_DMA_DEV36_USB_OTG_IEP_AND_OEP_3_11,
-	DB8500_DMA_DEV19_USB_OTG_IEP_AND_OEP_4_12,
-	DB8500_DMA_DEV18_USB_OTG_IEP_AND_OEP_5_13,
-	DB8500_DMA_DEV17_USB_OTG_IEP_AND_OEP_6_14,
-	DB8500_DMA_DEV16_USB_OTG_IEP_AND_OEP_7_15,
-	DB8500_DMA_DEV39_USB_OTG_IEP_AND_OEP_8
-};
-
-static int usb_db8500_tx_dma_cfg[] = {
+static int usb_db8500_dma_cfg[] = {
 	DB8500_DMA_DEV38_USB_OTG_IEP_AND_OEP_1_9,
 	DB8500_DMA_DEV37_USB_OTG_IEP_AND_OEP_2_10,
 	DB8500_DMA_DEV36_USB_OTG_IEP_AND_OEP_3_11,
@@ -219,7 +208,7 @@ struct device * __init u8500_init_devices(struct ab8500_platform_data *ab8500)
 
 	db8500_add_rtc(parent);
 	db8500_add_gpios(parent);
-	db8500_add_usb(parent, usb_db8500_rx_dma_cfg, usb_db8500_tx_dma_cfg);
+	db8500_add_usb(parent, usb_db8500_dma_cfg, usb_db8500_dma_cfg);
 
 	for (i = 0; i < ARRAY_SIZE(platform_devs); i++)
 		platform_devs[i]->dev.parent = parent;
@@ -238,7 +227,7 @@ static struct device * __init u8500_of_init_devices(void)
 {
 	struct device *parent = db8500_soc_device_init();
 
-	db8500_add_usb(parent, usb_db8500_rx_dma_cfg, usb_db8500_tx_dma_cfg);
+	db8500_add_usb(parent, usb_db8500_dma_cfg, usb_db8500_dma_cfg);
 
 	u8500_dma40_device.dev.parent = parent;
 
-- 
1.7.10.4

  parent reply	other threads:[~2013-04-18 10:11 UTC|newest]

Thread overview: 168+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-04-18 10:11 [PATCH 00/32] dmaengine: Refactor the DMA40 driver Lee Jones
2013-04-18 10:11 ` [PATCH 01/32] dmaengine: ste_dma40: Assign memcpy channels in the driver Lee Jones
2013-04-22  9:22   ` Vinod Koul
2013-04-25  9:20   ` Linus Walleij
2013-04-18 10:11 ` [PATCH 02/32] dmaengine: ste_dma40: Move default memcpy configs into " Lee Jones
2013-04-22  9:21   ` Vinod Koul
2013-04-24 14:44   ` Linus Walleij
2013-04-18 10:11 ` [PATCH 03/32] dmaengine: ste_dma40: Use the BIT macro to replace ugly '(1 << x)'s Lee Jones
2013-04-18 10:43   ` Russell King - ARM Linux
2013-04-18 11:00     ` Lee Jones
2013-04-18 12:16   ` [PATCH 03/32 v2] " Lee Jones
2013-04-22 10:13     ` Vinod Koul
2013-04-24 14:54     ` Linus Walleij
2013-04-24 15:08       ` Lee Jones
2013-04-24 15:09   ` [PATCH 03/32] " Lee Jones
2013-04-24 15:11   ` [PATCH 03/32 v3] " Lee Jones
2013-04-24 19:24     ` Rabin Vincent
2013-04-25  8:13       ` Lee Jones
2013-04-26 11:39   ` [PATCH 03/32 v4] " Lee Jones
2013-04-26 15:04     ` Linus Walleij
2013-04-18 10:11 ` [PATCH 04/32] dmaengine: ste_dma40: Amalgamate DMA source and destination channel numbers Lee Jones
2013-04-25  8:06   ` Linus Walleij
2013-04-25  8:36     ` Arnd Bergmann
2013-04-25  8:55       ` Linus Walleij
2013-04-25  9:06     ` Lee Jones
2013-04-25 12:43       ` Linus Walleij
2013-04-25 13:09         ` Russell King - ARM Linux
2013-04-25 13:21           ` Linus Walleij
2013-04-25 13:20         ` Lee Jones
2013-04-25 13:24           ` Linus Walleij
2013-04-26 14:28             ` Lee Jones
2013-04-18 10:11 ` [PATCH 05/32] dmaengine: ste_dma40: Supply macros to resolve 'src' and 'dst' directions Lee Jones
2013-04-18 10:50   ` Arnd Bergmann
2013-04-22  9:42   ` Vinod Koul
2013-04-22 10:27     ` Lee Jones
2013-04-22 10:19       ` Vinod Koul
2013-04-24  8:53       ` Lee Jones
2013-04-25  8:22   ` Linus Walleij
2013-04-25  9:19     ` Lee Jones
2013-04-18 10:11 ` Lee Jones [this message]
2013-04-18 10:51   ` [PATCH 06/32] ARM: ux500: Strip out duplicate USB DMA configuration Arnd Bergmann
2013-04-25  8:24   ` Linus Walleij
2013-04-18 10:11 ` [PATCH 07/32] ARM: ux500: Supply address location names for the DMA40 DMA controller Lee Jones
2013-04-18 10:48   ` Arnd Bergmann
2013-04-18 11:01     ` Lee Jones
2013-04-18 11:09     ` Lee Jones
2013-04-25  8:26   ` Linus Walleij
2013-04-25  9:17     ` Lee Jones
2013-04-25 12:45       ` Linus Walleij
2013-04-18 10:11 ` [PATCH 08/32] dmaengine: ste_dma40: Optimise local MAX() macro Lee Jones
2013-04-18 10:46   ` Arnd Bergmann
2013-04-18 11:00     ` Russell King - ARM Linux
2013-04-18 11:19       ` Arnd Bergmann
2013-04-22 10:10   ` Vinod Koul
2013-04-22 10:53     ` Lee Jones
2013-04-24  8:49   ` [PATCH 08/32 v2] dmaengine: ste_dma40: Remove home-brew " Lee Jones
2013-04-25 12:48     ` Linus Walleij
2013-05-01 14:28       ` Lee Jones
2013-04-25  8:36   ` [PATCH 08/32] dmaengine: ste_dma40: Optimise local " Linus Walleij
2013-04-25  9:15     ` Lee Jones
2013-04-18 10:11 ` [PATCH 09/32] ARM: ux500: Remove unused 'data_width' attributes from SDI DMA configs Lee Jones
2013-04-25  8:44   ` Linus Walleij
2013-04-25  9:14     ` Lee Jones
2013-04-25 12:49       ` Linus Walleij
2013-04-25 13:13         ` Lee Jones
2013-04-18 10:11 ` [PATCH 10/32] ARM: ux500: Remove unused 'data_width' attributes from SSP " Lee Jones
2013-04-25  8:45   ` Linus Walleij
2013-04-18 10:11 ` [PATCH 11/32] ARM: ux500: Remove unused 'data_width' attributes from UART " Lee Jones
2013-04-25  8:45   ` Linus Walleij
2013-04-18 10:11 ` [PATCH 12/32] ARM: ux500: Remove superfluous 'psize' attribute from Audio platform data Lee Jones
2013-04-25  9:00   ` Linus Walleij
2013-05-01 14:42   ` Lee Jones
2013-05-02  8:38     ` Lee Jones
2013-05-03 13:57       ` Linus Walleij
2013-06-10  9:04         ` Lee Jones
2013-06-10  9:12           ` Lee Jones
2013-04-18 10:11 ` [PATCH 13/32] dmaengine: ste_dma40: Calculate number of logical channels from physical ones Lee Jones
2013-04-22  9:23   ` Vinod Koul
2013-04-22 10:11     ` Lee Jones
2013-04-25  9:13   ` Linus Walleij
2013-04-25  9:29     ` Lee Jones
2013-04-25 12:51       ` Linus Walleij
2013-04-18 10:11 ` [PATCH 14/32] dmaengine: ste_dma40: Remove 'always true' checking Lee Jones
2013-04-18 10:53   ` Arnd Bergmann
2013-04-22  9:44   ` Vinod Koul
2013-04-25  9:17   ` Linus Walleij
2013-04-25  9:24     ` Lee Jones
2013-04-18 10:11 ` [PATCH 15/32] dmaengine: ste_dma40: Separate Logical Global Interrupt Mask (GIM) unmasking Lee Jones
2013-04-18 10:45   ` Russell King - ARM Linux
2013-04-18 10:54   ` Arnd Bergmann
2013-04-22  9:51   ` Vinod Koul
2013-04-22 10:40     ` Lee Jones
2013-04-24  8:51       ` Lee Jones
2013-04-25 11:00   ` Linus Walleij
2013-04-18 10:11 ` [PATCH 16/32] dmaengine: ste_dma40: Remove unnecessary call to d40_phy_cfg() Lee Jones
2013-04-18 10:55   ` Arnd Bergmann
2013-04-22  9:52   ` Vinod Koul
2013-04-25 11:09   ` Linus Walleij
2013-04-18 10:11 ` [PATCH 17/32] dmaengine: ste_dma40: Remove redundant argument from d40_phy_cfg() Lee Jones
2013-04-22  9:34   ` Vinod Koul
2013-04-22 10:18     ` Lee Jones
2013-04-25 11:14       ` Linus Walleij
2013-04-24  8:55     ` Lee Jones
2013-04-25 11:12   ` Linus Walleij
2013-04-18 10:12 ` [PATCH 18/32] dmaengine: ste_dma40: Don't configure runtime configurable setup during allocate Lee Jones
2013-04-25 11:15   ` Linus Walleij
2013-04-18 10:12 ` [PATCH 19/32] dmaengine: ste_dma40: Move more setup into the configuration routines Lee Jones
2013-04-22  9:37   ` Vinod Koul
2013-04-25 11:17   ` Linus Walleij
2013-04-18 10:12 ` [PATCH 20/32] dmaengine: ste_dma40: Move rev error-check up to revision acquisition Lee Jones
2013-04-22  9:41   ` Vinod Koul
2013-04-25 11:17   ` Linus Walleij
2013-04-18 10:12 ` [PATCH 21/32] dmaengine: ste_dma40: Also report the number of logical channels Lee Jones
2013-04-22  9:36   ` Vinod Koul
2013-04-22 10:14     ` Lee Jones
2013-04-22  9:48       ` Vinod Koul
2013-04-22 10:37         ` Lee Jones
2013-04-22 10:23           ` Vinod Koul
2013-04-22 10:52           ` Russell King - ARM Linux
2013-04-24  8:35             ` Lee Jones
2013-04-24  8:39   ` [PATCH 21/32 v2] " Lee Jones
2013-04-25 11:20     ` Linus Walleij
2013-04-18 10:12 ` [PATCH 22/32] dmaengine: ste_dma40: Allocate plat_data on declaration Lee Jones
2013-04-22  9:38   ` Vinod Koul
2013-04-22  9:40   ` Vinod Koul
2013-04-25 11:22   ` Linus Walleij
2013-04-18 10:12 ` [PATCH 23/32] dmaengine: ste_dma40: Allow driver to be probe()able when DT is enabled Lee Jones
2013-04-18 10:58   ` Arnd Bergmann
2013-04-22 10:02   ` Vinod Koul
2013-04-25 11:24   ` Linus Walleij
2013-04-18 10:12 ` [PATCH 24/32] dmaengine: ste_dma40: Supply full Device Tree parsing support Lee Jones
2013-04-18 11:06   ` Arnd Bergmann
2013-04-18 11:31     ` Lee Jones
2013-04-18 11:37       ` Arnd Bergmann
2013-04-18 11:47         ` Lee Jones
2013-04-18 12:23           ` Arnd Bergmann
2013-04-18 11:07   ` Arnd Bergmann
2013-04-18 12:12   ` [PATCH 24/32 v2] " Lee Jones
2013-04-18 12:32     ` Arnd Bergmann
2013-04-18 13:43       ` Lee Jones
2013-04-18 14:17     ` [PATCH 24/32 v3] " Lee Jones
2013-04-18 21:53       ` Arnd Bergmann
2013-04-22 10:18       ` Vinod Koul
2013-04-25 11:33       ` Linus Walleij
2013-04-22 10:17     ` [PATCH 24/32 v2] " Vinod Koul
2013-04-22 10:16   ` [PATCH 24/32] " Vinod Koul
2013-04-18 10:12 ` [PATCH 25/32] ARM: ux500: Setup the DMA40 driver's DT node using the new DMA API Lee Jones
2013-04-18 11:08   ` Arnd Bergmann
2013-04-25 11:35   ` Linus Walleij
2013-04-18 10:12 ` [PATCH 26/32] ARM: ux500: Supply UART's DMA configuration via Device Tree Lee Jones
2013-04-25 11:36   ` Linus Walleij
2013-04-18 10:12 ` [PATCH 27/32] ARM: ux500: Stop registering DMA40 from platform data when DT is enabled Lee Jones
2013-04-25 11:37   ` Linus Walleij
2013-04-18 10:12 ` [PATCH 28/32] ARM: ux500: Pass remnant platform data though to DMA40 driver Lee Jones
2013-04-18 11:11   ` Arnd Bergmann
2013-04-25 11:39   ` Linus Walleij
2013-04-18 10:12 ` [PATCH 29/32] ARM: ux500: Stop passing UART's platform data for Device Tree boots Lee Jones
2013-04-25 11:41   ` Linus Walleij
2013-04-18 10:12 ` [PATCH 30/32] ARM: ux500: Supply MMC DMA configuration via Device Tree Lee Jones
2013-04-25 11:43   ` Linus Walleij
2013-04-25 11:49     ` Lee Jones
2013-04-25 12:56       ` Linus Walleij
2013-04-25 13:11         ` Lee Jones
2013-04-18 10:12 ` [PATCH 31/32] ARM: ux500: Stop passing MMC's platform data for Device Tree boots Lee Jones
2013-04-25 11:44   ` Linus Walleij
2013-04-18 10:12 ` [PATCH 32/32] ARM: ux500: Move SDI (MMC) and UART devices under more descriptive heading Lee Jones
2013-04-25 11:46   ` Linus Walleij
2013-04-26 13:42     ` Lee Jones

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1366279934-30761-7-git-send-email-lee.jones@linaro.org \
    --to=lee.jones@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).