Re: [PATCH 1/5] ARM: ux500: Move DMA40 platform data includes file out to include/
On Mon, 15 Apr 2013, Linus Walleij wrote: > On Mon, Apr 8, 2013 at 5:23 PM, Lee Jones wrote: > > > The pin names for DB8500 based platforms need to be moved out of > > "pin names" ??? > > > ux500 platform data and into the new proper location in include/ > > linux/platform_data/. This way we an reference them from other > > external locations, such as the main DMA50 driver(s). > > > > Signed-off-by: Lee Jones > > (...) > > rename arch/arm/mach-ux500/ste-dma40-db8500.h => > > include/linux/platform_data/dma-ste-dma40-db8500.h (98%) > > Those are the DB8500 DMA channel definitions... > > I will not interfere with the fruitful discussion with Arnd on this > subject though, I suspect he's right. It's okay, I've since removed this patch. -- Lee Jones Linaro ST-Ericsson Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 1/5] ARM: ux500: Move DMA40 platform data includes file out to include/
On Mon, Apr 8, 2013 at 5:23 PM, Lee Jones wrote: > The pin names for DB8500 based platforms need to be moved out of "pin names" ??? > ux500 platform data and into the new proper location in include/ > linux/platform_data/. This way we an reference them from other > external locations, such as the main DMA50 driver(s). > > Signed-off-by: Lee Jones (...) > rename arch/arm/mach-ux500/ste-dma40-db8500.h => > include/linux/platform_data/dma-ste-dma40-db8500.h (98%) Those are the DB8500 DMA channel definitions... I will not interfere with the fruitful discussion with Arnd on this subject though, I suspect he's right. Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 1/5] ARM: ux500: Move DMA40 platform data includes file out to include/
On Mon, Apr 8, 2013 at 5:23 PM, Lee Jones lee.jo...@linaro.org wrote: The pin names for DB8500 based platforms need to be moved out of pin names ??? ux500 platform data and into the new proper location in include/ linux/platform_data/. This way we an reference them from other external locations, such as the main DMA50 driver(s). Signed-off-by: Lee Jones lee.jo...@linaro.org (...) rename arch/arm/mach-ux500/ste-dma40-db8500.h = include/linux/platform_data/dma-ste-dma40-db8500.h (98%) Those are the DB8500 DMA channel definitions... I will not interfere with the fruitful discussion with Arnd on this subject though, I suspect he's right. Yours, Linus Walleij -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 1/5] ARM: ux500: Move DMA40 platform data includes file out to include/
On Mon, 15 Apr 2013, Linus Walleij wrote: On Mon, Apr 8, 2013 at 5:23 PM, Lee Jones lee.jo...@linaro.org wrote: The pin names for DB8500 based platforms need to be moved out of pin names ??? ux500 platform data and into the new proper location in include/ linux/platform_data/. This way we an reference them from other external locations, such as the main DMA50 driver(s). Signed-off-by: Lee Jones lee.jo...@linaro.org (...) rename arch/arm/mach-ux500/ste-dma40-db8500.h = include/linux/platform_data/dma-ste-dma40-db8500.h (98%) Those are the DB8500 DMA channel definitions... I will not interfere with the fruitful discussion with Arnd on this subject though, I suspect he's right. It's okay, I've since removed this patch. -- Lee Jones Linaro ST-Ericsson Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 1/5] ARM: ux500: Move DMA40 platform data includes file out to include/
On Tuesday 09 April 2013, Lee Jones wrote: > On Mon, 08 Apr 2013, Arnd Bergmann wrote: > > > On Monday 08 April 2013, Lee Jones wrote: > > > The pin names for DB8500 based platforms need to be moved out of > > > ux500 platform data and into the new proper location in include/ > > > linux/platform_data/. This way we an reference them from other > > > external locations, such as the main DMA50 driver(s). > > > > > > Signed-off-by: Lee Jones > > > > Hmm, not sure about this one. The slave numbers are not really platform_data > > and I think they should not be exposed to drivers. It makes sense to > > keep the numbers for the memcpy channels in the driver itself as they > > are hardwired in the driver, but the slave channels should stay in the > > platform I think. > > Just looking at this again now. So you're right in that the main > reason for moving these out of platform code and into the system-wide > platform data area is for the memcpy channels. However, I'd prefer to > keep the channel allocation enum fully contiguous rather than > splitting them out. > > The channels are only exposed to the one external driver which > includes them, and that's the D40 driver. > > How against this move are you? I think it's a move in the wrong direction. If you think we should keep the knowledge of which channels are used for memcpy outside of the driver, I would prefer you to drop the the first two patches, which implies that we will eventually pass that information using DT properties in order get rid of the auxdata. Arnd -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 1/5] ARM: ux500: Move DMA40 platform data includes file out to include/
On Mon, 08 Apr 2013, Arnd Bergmann wrote: > On Monday 08 April 2013, Lee Jones wrote: > > The pin names for DB8500 based platforms need to be moved out of > > ux500 platform data and into the new proper location in include/ > > linux/platform_data/. This way we an reference them from other > > external locations, such as the main DMA50 driver(s). > > > > Signed-off-by: Lee Jones > > Hmm, not sure about this one. The slave numbers are not really platform_data > and I think they should not be exposed to drivers. It makes sense to > keep the numbers for the memcpy channels in the driver itself as they > are hardwired in the driver, but the slave channels should stay in the > platform I think. Just looking at this again now. So you're right in that the main reason for moving these out of platform code and into the system-wide platform data area is for the memcpy channels. However, I'd prefer to keep the channel allocation enum fully contiguous rather than splitting them out. The channels are only exposed to the one external driver which includes them, and that's the D40 driver. How against this move are you? -- Lee Jones Linaro ST-Ericsson Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 1/5] ARM: ux500: Move DMA40 platform data includes file out to include/
On Mon, 08 Apr 2013, Arnd Bergmann wrote: On Monday 08 April 2013, Lee Jones wrote: The pin names for DB8500 based platforms need to be moved out of ux500 platform data and into the new proper location in include/ linux/platform_data/. This way we an reference them from other external locations, such as the main DMA50 driver(s). Signed-off-by: Lee Jones lee.jo...@linaro.org Hmm, not sure about this one. The slave numbers are not really platform_data and I think they should not be exposed to drivers. It makes sense to keep the numbers for the memcpy channels in the driver itself as they are hardwired in the driver, but the slave channels should stay in the platform I think. Just looking at this again now. So you're right in that the main reason for moving these out of platform code and into the system-wide platform data area is for the memcpy channels. However, I'd prefer to keep the channel allocation enum fully contiguous rather than splitting them out. The channels are only exposed to the one external driver which includes them, and that's the D40 driver. How against this move are you? -- Lee Jones Linaro ST-Ericsson Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 1/5] ARM: ux500: Move DMA40 platform data includes file out to include/
On Tuesday 09 April 2013, Lee Jones wrote: On Mon, 08 Apr 2013, Arnd Bergmann wrote: On Monday 08 April 2013, Lee Jones wrote: The pin names for DB8500 based platforms need to be moved out of ux500 platform data and into the new proper location in include/ linux/platform_data/. This way we an reference them from other external locations, such as the main DMA50 driver(s). Signed-off-by: Lee Jones lee.jo...@linaro.org Hmm, not sure about this one. The slave numbers are not really platform_data and I think they should not be exposed to drivers. It makes sense to keep the numbers for the memcpy channels in the driver itself as they are hardwired in the driver, but the slave channels should stay in the platform I think. Just looking at this again now. So you're right in that the main reason for moving these out of platform code and into the system-wide platform data area is for the memcpy channels. However, I'd prefer to keep the channel allocation enum fully contiguous rather than splitting them out. The channels are only exposed to the one external driver which includes them, and that's the D40 driver. How against this move are you? I think it's a move in the wrong direction. If you think we should keep the knowledge of which channels are used for memcpy outside of the driver, I would prefer you to drop the the first two patches, which implies that we will eventually pass that information using DT properties in order get rid of the auxdata. Arnd -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 1/5] ARM: ux500: Move DMA40 platform data includes file out to include/
On Monday 08 April 2013, Lee Jones wrote: > The pin names for DB8500 based platforms need to be moved out of > ux500 platform data and into the new proper location in include/ > linux/platform_data/. This way we an reference them from other > external locations, such as the main DMA50 driver(s). > > Signed-off-by: Lee Jones Hmm, not sure about this one. The slave numbers are not really platform_data and I think they should not be exposed to drivers. It makes sense to keep the numbers for the memcpy channels in the driver itself as they are hardwired in the driver, but the slave channels should stay in the platform I think. Arnd -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[PATCH 1/5] ARM: ux500: Move DMA40 platform data includes file out to include/
The pin names for DB8500 based platforms need to be moved out of ux500 platform data and into the new proper location in include/ linux/platform_data/. This way we an reference them from other external locations, such as the main DMA50 driver(s). Signed-off-by: Lee Jones --- arch/arm/mach-ux500/board-mop500-audio.c|2 +- arch/arm/mach-ux500/board-mop500-sdi.c |2 +- arch/arm/mach-ux500/board-mop500.c |2 +- arch/arm/mach-ux500/cpu-db8500.c|2 +- arch/arm/mach-ux500/devices-db8500.c|2 +- drivers/dma/ste_dma40.c |1 + .../linux/platform_data/dma-ste-dma40-db8500.h |3 +-- 7 files changed, 7 insertions(+), 7 deletions(-) rename arch/arm/mach-ux500/ste-dma40-db8500.h => include/linux/platform_data/dma-ste-dma40-db8500.h (98%) diff --git a/arch/arm/mach-ux500/board-mop500-audio.c b/arch/arm/mach-ux500/board-mop500-audio.c index 7209db7..1f8c6e4 100644 --- a/arch/arm/mach-ux500/board-mop500-audio.c +++ b/arch/arm/mach-ux500/board-mop500-audio.c @@ -9,13 +9,13 @@ #include #include #include +#include #include #include #include #include -#include "ste-dma40-db8500.h" #include "board-mop500.h" #include "devices-db8500.h" #include "pins-db8500.h" diff --git a/arch/arm/mach-ux500/board-mop500-sdi.c b/arch/arm/mach-ux500/board-mop500-sdi.c index 7f2cb6c..e4c0a74 100644 --- a/arch/arm/mach-ux500/board-mop500-sdi.c +++ b/arch/arm/mach-ux500/board-mop500-sdi.c @@ -12,6 +12,7 @@ #include #include #include +#include #include #include @@ -19,7 +20,6 @@ #include "devices-db8500.h" #include "board-mop500.h" -#include "ste-dma40-db8500.h" /* * v2 has a new version of this block that need to be forced, the number found diff --git a/arch/arm/mach-ux500/board-mop500.c b/arch/arm/mach-ux500/board-mop500.c index 87d2d7b..dd95399 100644 --- a/arch/arm/mach-ux500/board-mop500.c +++ b/arch/arm/mach-ux500/board-mop500.c @@ -38,6 +38,7 @@ #include #include #include +#include #include #include @@ -48,7 +49,6 @@ #include #include -#include "ste-dma40-db8500.h" #include "devices-db8500.h" #include "board-mop500.h" #include "board-mop500-regulators.h" diff --git a/arch/arm/mach-ux500/cpu-db8500.c b/arch/arm/mach-ux500/cpu-db8500.c index e027865..8553e14 100644 --- a/arch/arm/mach-ux500/cpu-db8500.c +++ b/arch/arm/mach-ux500/cpu-db8500.c @@ -22,6 +22,7 @@ #include #include #include +#include #include #include @@ -35,7 +36,6 @@ #include #include "devices-db8500.h" -#include "ste-dma40-db8500.h" #include "board-mop500.h" #include "id.h" diff --git a/arch/arm/mach-ux500/devices-db8500.c b/arch/arm/mach-ux500/devices-db8500.c index afa5b04..a7c0e521 100644 --- a/arch/arm/mach-ux500/devices-db8500.c +++ b/arch/arm/mach-ux500/devices-db8500.c @@ -13,6 +13,7 @@ #include #include #include +#include #include #include @@ -20,7 +21,6 @@ #include #include "devices-db8500.h" -#include "ste-dma40-db8500.h" static struct resource dma40_resources[] = { [0] = { diff --git a/drivers/dma/ste_dma40.c b/drivers/dma/ste_dma40.c index 1734fee..35f97d2 100644 --- a/drivers/dma/ste_dma40.c +++ b/drivers/dma/ste_dma40.c @@ -20,6 +20,7 @@ #include #include #include +#include #include "dmaengine.h" #include "ste_dma40_ll.h" diff --git a/arch/arm/mach-ux500/ste-dma40-db8500.h b/include/linux/platform_data/dma-ste-dma40-db8500.h similarity index 98% rename from arch/arm/mach-ux500/ste-dma40-db8500.h rename to include/linux/platform_data/dma-ste-dma40-db8500.h index a616419..f538917 100644 --- a/arch/arm/mach-ux500/ste-dma40-db8500.h +++ b/include/linux/platform_data/dma-ste-dma40-db8500.h @@ -1,8 +1,7 @@ /* - * arch/arm/mach-ux500/ste_dma40_db8500.h * DB8500-SoC-specific configuration for DMA40 * - * Copyright (C) ST-Ericsson 2007-2010 + * Copyright (C) ST-Ericsson 2007-2013 * License terms: GNU General Public License (GPL) version 2 * Author: Per Friden * Author: Jonas Aaberg -- 1.7.10.4 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[PATCH 1/5] ARM: ux500: Move DMA40 platform data includes file out to include/
The pin names for DB8500 based platforms need to be moved out of ux500 platform data and into the new proper location in include/ linux/platform_data/. This way we an reference them from other external locations, such as the main DMA50 driver(s). Signed-off-by: Lee Jones lee.jo...@linaro.org --- arch/arm/mach-ux500/board-mop500-audio.c|2 +- arch/arm/mach-ux500/board-mop500-sdi.c |2 +- arch/arm/mach-ux500/board-mop500.c |2 +- arch/arm/mach-ux500/cpu-db8500.c|2 +- arch/arm/mach-ux500/devices-db8500.c|2 +- drivers/dma/ste_dma40.c |1 + .../linux/platform_data/dma-ste-dma40-db8500.h |3 +-- 7 files changed, 7 insertions(+), 7 deletions(-) rename arch/arm/mach-ux500/ste-dma40-db8500.h = include/linux/platform_data/dma-ste-dma40-db8500.h (98%) diff --git a/arch/arm/mach-ux500/board-mop500-audio.c b/arch/arm/mach-ux500/board-mop500-audio.c index 7209db7..1f8c6e4 100644 --- a/arch/arm/mach-ux500/board-mop500-audio.c +++ b/arch/arm/mach-ux500/board-mop500-audio.c @@ -9,13 +9,13 @@ #include linux/gpio.h #include linux/platform_data/pinctrl-nomadik.h #include linux/platform_data/dma-ste-dma40.h +#include linux/platform_data/dma-ste-dma40-db8500.h #include mach/devices.h #include mach/hardware.h #include mach/irqs.h #include mach/msp.h -#include ste-dma40-db8500.h #include board-mop500.h #include devices-db8500.h #include pins-db8500.h diff --git a/arch/arm/mach-ux500/board-mop500-sdi.c b/arch/arm/mach-ux500/board-mop500-sdi.c index 7f2cb6c..e4c0a74 100644 --- a/arch/arm/mach-ux500/board-mop500-sdi.c +++ b/arch/arm/mach-ux500/board-mop500-sdi.c @@ -12,6 +12,7 @@ #include linux/mmc/host.h #include linux/platform_device.h #include linux/platform_data/dma-ste-dma40.h +#include linux/platform_data/dma-ste-dma40-db8500.h #include asm/mach-types.h #include mach/devices.h @@ -19,7 +20,6 @@ #include devices-db8500.h #include board-mop500.h -#include ste-dma40-db8500.h /* * v2 has a new version of this block that need to be forced, the number found diff --git a/arch/arm/mach-ux500/board-mop500.c b/arch/arm/mach-ux500/board-mop500.c index 87d2d7b..dd95399 100644 --- a/arch/arm/mach-ux500/board-mop500.c +++ b/arch/arm/mach-ux500/board-mop500.c @@ -38,6 +38,7 @@ #include linux/pinctrl/consumer.h #include linux/platform_data/pinctrl-nomadik.h #include linux/platform_data/dma-ste-dma40.h +#include linux/platform_data/dma-ste-dma40-db8500.h #include asm/mach-types.h #include asm/mach/arch.h @@ -48,7 +49,6 @@ #include mach/irqs.h #include linux/platform_data/crypto-ux500.h -#include ste-dma40-db8500.h #include devices-db8500.h #include board-mop500.h #include board-mop500-regulators.h diff --git a/arch/arm/mach-ux500/cpu-db8500.c b/arch/arm/mach-ux500/cpu-db8500.c index e027865..8553e14 100644 --- a/arch/arm/mach-ux500/cpu-db8500.c +++ b/arch/arm/mach-ux500/cpu-db8500.c @@ -22,6 +22,7 @@ #include linux/of_platform.h #include linux/regulator/machine.h #include linux/platform_data/pinctrl-nomadik.h +#include linux/platform_data/dma-ste-dma40-db8500.h #include linux/random.h #include asm/pmu.h @@ -35,7 +36,6 @@ #include mach/irqs.h #include devices-db8500.h -#include ste-dma40-db8500.h #include board-mop500.h #include id.h diff --git a/arch/arm/mach-ux500/devices-db8500.c b/arch/arm/mach-ux500/devices-db8500.c index afa5b04..a7c0e521 100644 --- a/arch/arm/mach-ux500/devices-db8500.c +++ b/arch/arm/mach-ux500/devices-db8500.c @@ -13,6 +13,7 @@ #include linux/amba/bus.h #include linux/amba/pl022.h #include linux/platform_data/dma-ste-dma40.h +#include linux/platform_data/dma-ste-dma40-db8500.h #include linux/mfd/dbx500-prcmu.h #include mach/hardware.h @@ -20,7 +21,6 @@ #include mach/irqs.h #include devices-db8500.h -#include ste-dma40-db8500.h static struct resource dma40_resources[] = { [0] = { diff --git a/drivers/dma/ste_dma40.c b/drivers/dma/ste_dma40.c index 1734fee..35f97d2 100644 --- a/drivers/dma/ste_dma40.c +++ b/drivers/dma/ste_dma40.c @@ -20,6 +20,7 @@ #include linux/amba/bus.h #include linux/regulator/consumer.h #include linux/platform_data/dma-ste-dma40.h +#include linux/platform_data/dma-ste-dma40-db8500.h #include dmaengine.h #include ste_dma40_ll.h diff --git a/arch/arm/mach-ux500/ste-dma40-db8500.h b/include/linux/platform_data/dma-ste-dma40-db8500.h similarity index 98% rename from arch/arm/mach-ux500/ste-dma40-db8500.h rename to include/linux/platform_data/dma-ste-dma40-db8500.h index a616419..f538917 100644 --- a/arch/arm/mach-ux500/ste-dma40-db8500.h +++ b/include/linux/platform_data/dma-ste-dma40-db8500.h @@ -1,8 +1,7 @@ /* - * arch/arm/mach-ux500/ste_dma40_db8500.h * DB8500-SoC-specific configuration for DMA40 * - * Copyright (C) ST-Ericsson 2007-2010 + *
Re: [PATCH 1/5] ARM: ux500: Move DMA40 platform data includes file out to include/
On Monday 08 April 2013, Lee Jones wrote: The pin names for DB8500 based platforms need to be moved out of ux500 platform data and into the new proper location in include/ linux/platform_data/. This way we an reference them from other external locations, such as the main DMA50 driver(s). Signed-off-by: Lee Jones lee.jo...@linaro.org Hmm, not sure about this one. The slave numbers are not really platform_data and I think they should not be exposed to drivers. It makes sense to keep the numbers for the memcpy channels in the driver itself as they are hardwired in the driver, but the slave channels should stay in the platform I think. Arnd -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/