* [PATCH 0/4] ARM: Exynos: Fixes for Exynos5 device tree support
@ 2012-03-27 9:26 Thomas Abraham
2012-03-27 9:26 ` [PATCH 1/4] ARM: Exynos: Use CHIP_ID to detect cpu type during decompression setup Thomas Abraham
` (3 more replies)
0 siblings, 4 replies; 10+ messages in thread
From: Thomas Abraham @ 2012-03-27 9:26 UTC (permalink / raw)
To: linux-arm-kernel
This series includes general fixes for Exynos5 device tree support. This
is based on the following tree. Both Exynos4 and Exynos5 device tree
support have been tested when this series.
http://git.kernel.org/pub/scm/linux/kernel/git/kgene/linux-samsung.git [for-next]
Thomas Abraham (4):
ARM: Exynos: Use CHIP_ID to detect cpu type during decompression setup
ARM: Exynos: Remove a new bus_type instance for Exynos5
ARM: Exynos5: Fix incorrect initialization of GIC
ARM: Exynos5: Add PDMA and MDMA physical base address defines
arch/arm/mach-exynos/common.c | 21 +++------------------
arch/arm/mach-exynos/include/mach/map.h | 5 +++++
arch/arm/mach-exynos/include/mach/uncompress.h | 6 +++++-
arch/arm/mach-exynos/mach-exynos5-dt.c | 3 ++-
4 files changed, 15 insertions(+), 20 deletions(-)
^ permalink raw reply [flat|nested] 10+ messages in thread* [PATCH 1/4] ARM: Exynos: Use CHIP_ID to detect cpu type during decompression setup 2012-03-27 9:26 [PATCH 0/4] ARM: Exynos: Fixes for Exynos5 device tree support Thomas Abraham @ 2012-03-27 9:26 ` Thomas Abraham 2012-03-29 1:34 ` Kukjin Kim 2012-03-27 9:26 ` [PATCH 2/4] ARM: Exynos: Remove a new bus_type instance for Exynos5 Thomas Abraham ` (2 subsequent siblings) 3 siblings, 1 reply; 10+ messages in thread From: Thomas Abraham @ 2012-03-27 9:26 UTC (permalink / raw) To: linux-arm-kernel The config option MACH_SMDK5250 is not available for Exynos5 since only a device tree based machine is supported and selected by MACH_EXYNOS5_DT. Hence, the detection of the cpu type based on machine_is_smdk5250 always fails and boot hangs due to incorrect uart base being used. Fix this by reading the chip id to determine the cpu type and setup the uart_base accordingly. Signed-off-by: Thomas Abraham <thomas.ab@samsung.com> --- arch/arm/mach-exynos/include/mach/uncompress.h | 6 +++++- 1 files changed, 5 insertions(+), 1 deletions(-) diff --git a/arch/arm/mach-exynos/include/mach/uncompress.h b/arch/arm/mach-exynos/include/mach/uncompress.h index 493f4f3..603ea9b 100644 --- a/arch/arm/mach-exynos/include/mach/uncompress.h +++ b/arch/arm/mach-exynos/include/mach/uncompress.h @@ -14,15 +14,19 @@ #include <asm/mach-types.h> +#include <plat/cpu.h> #include <mach/map.h> volatile u8 *uart_base; #include <plat/uncompress.h> +#define __raw_readl(addr) \ + (*((volatile unsigned int __force *)(addr))) + static void arch_detect_cpu(void) { - if (machine_is_smdk5250()) + if ((__raw_readl(EXYNOS_PA_CHIPID) & EXYNOS5_SOC_MASK) == EXYNOS5250_SOC_ID) uart_base = (volatile u8 *)EXYNOS5_PA_UART + (S3C_UART_OFFSET * CONFIG_S3C_LOWLEVEL_UART_PORT); else uart_base = (volatile u8 *)EXYNOS4_PA_UART + (S3C_UART_OFFSET * CONFIG_S3C_LOWLEVEL_UART_PORT); -- 1.6.6.rc2 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 1/4] ARM: Exynos: Use CHIP_ID to detect cpu type during decompression setup 2012-03-27 9:26 ` [PATCH 1/4] ARM: Exynos: Use CHIP_ID to detect cpu type during decompression setup Thomas Abraham @ 2012-03-29 1:34 ` Kukjin Kim 0 siblings, 0 replies; 10+ messages in thread From: Kukjin Kim @ 2012-03-29 1:34 UTC (permalink / raw) To: linux-arm-kernel Thomas Abraham wrote: > The config option MACH_SMDK5250 is not available for Exynos5 since only > a device tree based machine is supported and selected by MACH_EXYNOS5_DT. > Hence, the detection of the cpu type based on machine_is_smdk5250 always > fails and boot hangs due to incorrect uart base being used. Fix this by > reading the chip id to determine the cpu type and setup the uart_base > accordingly. > > Signed-off-by: Thomas Abraham<thomas.ab@samsung.com> > --- > arch/arm/mach-exynos/include/mach/uncompress.h | 6 +++++- > 1 files changed, 5 insertions(+), 1 deletions(-) > > diff --git a/arch/arm/mach-exynos/include/mach/uncompress.h b/arch/arm/mach-exynos/include/mach/uncompress.h > index 493f4f3..603ea9b 100644 > --- a/arch/arm/mach-exynos/include/mach/uncompress.h > +++ b/arch/arm/mach-exynos/include/mach/uncompress.h > @@ -14,15 +14,19 @@ > > #include<asm/mach-types.h> > > +#include<plat/cpu.h> > #include<mach/map.h> > > volatile u8 *uart_base; > > #include<plat/uncompress.h> > > +#define __raw_readl(addr) \ > + (*((volatile unsigned int __force *)(addr))) > + > static void arch_detect_cpu(void) > { > - if (machine_is_smdk5250()) > + if ((__raw_readl(EXYNOS_PA_CHIPID)& EXYNOS5_SOC_MASK) == EXYNOS5250_SOC_ID) > uart_base = (volatile u8 *)EXYNOS5_PA_UART + (S3C_UART_OFFSET * CONFIG_S3C_LOWLEVEL_UART_PORT); > else > uart_base = (volatile u8 *)EXYNOS4_PA_UART + (S3C_UART_OFFSET * CONFIG_S3C_LOWLEVEL_UART_PORT); Hi Thomas, Thanks for fix. But I have following patch and it is not required to add inclusion of <plat/cpu.h>. How do you think? If you're ok, I would preferred to use follwoing. diff --git a/arch/arm/mach-exynos/include/mach/uncompress.h b/arch/arm/mach-exynos/include/mach/uncompress.h index 493f4f3..2979995 100644 --- a/arch/arm/mach-exynos/include/mach/uncompress.h +++ b/arch/arm/mach-exynos/include/mach/uncompress.h @@ -20,9 +20,24 @@ volatile u8 *uart_base; #include <plat/uncompress.h> +static unsigned int __raw_readl(unsigned int ptr) +{ + return *((volatile unsigned int *)ptr); +} + static void arch_detect_cpu(void) { - if (machine_is_smdk5250()) + u32 chip_id = __raw_readl(EXYNOS_PA_CHIPID); + + /* + * product_id is bits 31:12 + * bits 23:20 describe the exynosX family + * + */ + chip_id >>= 20; + chip_id &= 0xf; + + if (chip_id == 0x5) uart_base = (volatile u8 *)EXYNOS5_PA_UART + (S3C_UART_OFFSET * CONFIG_S3C_LOWLEVEL_UART_PORT); else uart_base = (volatile u8 *)EXYNOS4_PA_UART + (S3C_UART_OFFSET * CONFIG_S3C_LOWLEVEL_UART_PORT); Thanks. Best regards, Kgene. -- Kukjin Kim <kgene.kim@samsung.com>, Senior Engineer, SW Solution Development Team, Samsung Electronics Co., Ltd. ^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 2/4] ARM: Exynos: Remove a new bus_type instance for Exynos5 2012-03-27 9:26 [PATCH 0/4] ARM: Exynos: Fixes for Exynos5 device tree support Thomas Abraham 2012-03-27 9:26 ` [PATCH 1/4] ARM: Exynos: Use CHIP_ID to detect cpu type during decompression setup Thomas Abraham @ 2012-03-27 9:26 ` Thomas Abraham 2012-03-27 9:26 ` [PATCH 3/4] ARM: Exynos5: Fix incorrect initialization of GIC Thomas Abraham 2012-03-27 9:26 ` [PATCH 4/4] ARM: Exynos5: Add PDMA and MDMA physical base address defines Thomas Abraham 3 siblings, 0 replies; 10+ messages in thread From: Thomas Abraham @ 2012-03-27 9:26 UTC (permalink / raw) To: linux-arm-kernel A seperate bus_type instance is not required for Exynos5. The existing bus_type instance used with Exynos4 is sufficient for both Exynos4 and Exynos5. This also solves issue of uninitialized usage of exynos4_subsys in Exynos4 power management code that is reused for Exynos5 also. Signed-off-by: Thomas Abraham <thomas.ab@samsung.com> --- arch/arm/mach-exynos/common.c | 19 ++----------------- 1 files changed, 2 insertions(+), 17 deletions(-) diff --git a/arch/arm/mach-exynos/common.c b/arch/arm/mach-exynos/common.c index a750a5d..f265dc4 100644 --- a/arch/arm/mach-exynos/common.c +++ b/arch/arm/mach-exynos/common.c @@ -563,25 +563,13 @@ struct bus_type exynos4_subsys = { .dev_name = "exynos4-core", }; -struct bus_type exynos5_subsys = { - .name = "exynos5-core", - .dev_name = "exynos5-core", -}; - static struct device exynos4_dev = { .bus = &exynos4_subsys, }; -static struct device exynos5_dev = { - .bus = &exynos5_subsys, -}; - static int __init exynos_core_init(void) { - if (soc_is_exynos5250()) - return subsys_system_register(&exynos5_subsys, NULL); - else - return subsys_system_register(&exynos4_subsys, NULL); + return subsys_system_register(&exynos4_subsys, NULL); } core_initcall(exynos_core_init); @@ -667,10 +655,7 @@ static int __init exynos_init(void) { printk(KERN_INFO "EXYNOS: Initializing architecture\n"); - if (soc_is_exynos5250()) - return device_register(&exynos5_dev); - else - return device_register(&exynos4_dev); + return device_register(&exynos4_dev); } /* uart registration process */ -- 1.6.6.rc2 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 3/4] ARM: Exynos5: Fix incorrect initialization of GIC 2012-03-27 9:26 [PATCH 0/4] ARM: Exynos: Fixes for Exynos5 device tree support Thomas Abraham 2012-03-27 9:26 ` [PATCH 1/4] ARM: Exynos: Use CHIP_ID to detect cpu type during decompression setup Thomas Abraham 2012-03-27 9:26 ` [PATCH 2/4] ARM: Exynos: Remove a new bus_type instance for Exynos5 Thomas Abraham @ 2012-03-27 9:26 ` Thomas Abraham 2012-03-29 1:46 ` Kukjin Kim 2012-03-27 9:26 ` [PATCH 4/4] ARM: Exynos5: Add PDMA and MDMA physical base address defines Thomas Abraham 3 siblings, 1 reply; 10+ messages in thread From: Thomas Abraham @ 2012-03-27 9:26 UTC (permalink / raw) To: linux-arm-kernel Use the of_irq_init() call to setup the gic which also properly registers the gic device node pointer with gic irq domain, without which all interrupt specifier translations for gic fail. Signed-off-by: Thomas Abraham <thomas.ab@samsung.com> --- arch/arm/mach-exynos/common.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/arch/arm/mach-exynos/common.c b/arch/arm/mach-exynos/common.c index f265dc4..3bf39a8 100644 --- a/arch/arm/mach-exynos/common.c +++ b/arch/arm/mach-exynos/common.c @@ -542,7 +542,7 @@ void __init exynos5_init_irq(void) { int irq; - gic_init(0, IRQ_PPI(0), S5P_VA_GIC_DIST, S5P_VA_GIC_CPU); + of_irq_init(exynos4_dt_irq_match); for (irq = 0; irq < EXYNOS5_MAX_COMBINER_NR; irq++) { combiner_init(irq, (void __iomem *)S5P_VA_COMBINER(irq), -- 1.6.6.rc2 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 3/4] ARM: Exynos5: Fix incorrect initialization of GIC 2012-03-27 9:26 ` [PATCH 3/4] ARM: Exynos5: Fix incorrect initialization of GIC Thomas Abraham @ 2012-03-29 1:46 ` Kukjin Kim 0 siblings, 0 replies; 10+ messages in thread From: Kukjin Kim @ 2012-03-29 1:46 UTC (permalink / raw) To: linux-arm-kernel Thomas Abraham wrote: > Use the of_irq_init() call to setup the gic which also properly registers > the gic device node pointer with gic irq domain, without which all interrupt > specifier translations for gic fail. > > Signed-off-by: Thomas Abraham<thomas.ab@samsung.com> > --- > arch/arm/mach-exynos/common.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/arch/arm/mach-exynos/common.c b/arch/arm/mach-exynos/common.c > index f265dc4..3bf39a8 100644 > --- a/arch/arm/mach-exynos/common.c > +++ b/arch/arm/mach-exynos/common.c > @@ -542,7 +542,7 @@ void __init exynos5_init_irq(void) > { > int irq; > > - gic_init(0, IRQ_PPI(0), S5P_VA_GIC_DIST, S5P_VA_GIC_CPU); > + of_irq_init(exynos4_dt_irq_match); > > for (irq = 0; irq< EXYNOS5_MAX_COMBINER_NR; irq++) { > combiner_init(irq, (void __iomem *)S5P_VA_COMBINER(irq), Yes, right. The of_irq_init() shold be used instead. But I need to check about the usage of SPI and if any updates, let you know :) Thanks. Best regards, Kgene. -- Kukjin Kim <kgene.kim@samsung.com>, Senior Engineer, SW Solution Development Team, Samsung Electronics Co., Ltd. ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH 4/4] ARM: Exynos5: Add PDMA and MDMA physical base address defines 2012-03-27 9:26 [PATCH 0/4] ARM: Exynos: Fixes for Exynos5 device tree support Thomas Abraham ` (2 preceding siblings ...) 2012-03-27 9:26 ` [PATCH 3/4] ARM: Exynos5: Fix incorrect initialization of GIC Thomas Abraham @ 2012-03-27 9:26 ` Thomas Abraham 2012-03-29 1:54 ` Kukjin Kim 3 siblings, 1 reply; 10+ messages in thread From: Thomas Abraham @ 2012-03-27 9:26 UTC (permalink / raw) To: linux-arm-kernel Add PDMA and MDMA physical base address macros which is require for Exynos5 of_dev_auxdata setup. Signed-off-by: Thomas Abraham <thomas.ab@samsung.com> --- arch/arm/mach-exynos/include/mach/map.h | 5 +++++ arch/arm/mach-exynos/mach-exynos5-dt.c | 3 ++- 2 files changed, 7 insertions(+), 1 deletions(-) diff --git a/arch/arm/mach-exynos/include/mach/map.h b/arch/arm/mach-exynos/include/mach/map.h index 024d38f..37d93fd 100644 --- a/arch/arm/mach-exynos/include/mach/map.h +++ b/arch/arm/mach-exynos/include/mach/map.h @@ -90,6 +90,11 @@ #define EXYNOS4_PA_PDMA0 0x12680000 #define EXYNOS4_PA_PDMA1 0x12690000 +#define EXYNOS5_PA_PDMA0 0x121A0000 +#define EXYNOS5_PA_PDMA1 0x121B0000 +#define EXYNOS5_PA_MDMA0 0x10800000 +#define EXYNOS5_PA_MDMA1 0x11C10000 + #define EXYNOS4_PA_SYSMMU_MDMA 0x10A40000 #define EXYNOS4_PA_SYSMMU_SSS 0x10A50000 #define EXYNOS4_PA_SYSMMU_FIMC0 0x11A20000 diff --git a/arch/arm/mach-exynos/mach-exynos5-dt.c b/arch/arm/mach-exynos/mach-exynos5-dt.c index 0d26f50..cdc3a26 100644 --- a/arch/arm/mach-exynos/mach-exynos5-dt.c +++ b/arch/arm/mach-exynos/mach-exynos5-dt.c @@ -45,7 +45,8 @@ static const struct of_dev_auxdata exynos5250_auxdata_lookup[] __initconst = { "exynos4210-uart.3", NULL), OF_DEV_AUXDATA("arm,pl330", EXYNOS5_PA_PDMA0, "dma-pl330.0", NULL), OF_DEV_AUXDATA("arm,pl330", EXYNOS5_PA_PDMA1, "dma-pl330.1", NULL), - OF_DEV_AUXDATA("arm,pl330", EXYNOS5_PA_PDMA1, "dma-pl330.2", NULL), + OF_DEV_AUXDATA("arm,pl330", EXYNOS5_PA_MDMA0, "dma-pl330.2", NULL), + OF_DEV_AUXDATA("arm,pl330", EXYNOS5_PA_MDMA0, "dma-pl330.3", NULL), {}, }; -- 1.6.6.rc2 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 4/4] ARM: Exynos5: Add PDMA and MDMA physical base address defines 2012-03-27 9:26 ` [PATCH 4/4] ARM: Exynos5: Add PDMA and MDMA physical base address defines Thomas Abraham @ 2012-03-29 1:54 ` Kukjin Kim 2012-04-09 3:48 ` Olof Johansson 0 siblings, 1 reply; 10+ messages in thread From: Kukjin Kim @ 2012-03-29 1:54 UTC (permalink / raw) To: linux-arm-kernel Thomas Abraham wrote: > Add PDMA and MDMA physical base address macros which is require for Exynos5 > of_dev_auxdata setup. > > Signed-off-by: Thomas Abraham<thomas.ab@samsung.com> > --- > arch/arm/mach-exynos/include/mach/map.h | 5 +++++ > arch/arm/mach-exynos/mach-exynos5-dt.c | 3 ++- > 2 files changed, 7 insertions(+), 1 deletions(-) > > diff --git a/arch/arm/mach-exynos/include/mach/map.h b/arch/arm/mach-exynos/include/mach/map.h > index 024d38f..37d93fd 100644 > --- a/arch/arm/mach-exynos/include/mach/map.h > +++ b/arch/arm/mach-exynos/include/mach/map.h > @@ -90,6 +90,11 @@ > #define EXYNOS4_PA_PDMA0 0x12680000 > #define EXYNOS4_PA_PDMA1 0x12690000 > > +#define EXYNOS5_PA_PDMA0 0x121A0000 > +#define EXYNOS5_PA_PDMA1 0x121B0000 > +#define EXYNOS5_PA_MDMA0 0x10800000 > +#define EXYNOS5_PA_MDMA1 0x11C10000 > + See, "ARM: EXYNOS: Support DMA for Exynos5250 SoC" http://lists.infradead.org/pipermail/linux-arm-kernel/2012-February/085434.html But this merge window is not including it :( > #define EXYNOS4_PA_SYSMMU_MDMA 0x10A40000 > #define EXYNOS4_PA_SYSMMU_SSS 0x10A50000 > #define EXYNOS4_PA_SYSMMU_FIMC0 0x11A20000 > diff --git a/arch/arm/mach-exynos/mach-exynos5-dt.c b/arch/arm/mach-exynos/mach-exynos5-dt.c > index 0d26f50..cdc3a26 100644 > --- a/arch/arm/mach-exynos/mach-exynos5-dt.c > +++ b/arch/arm/mach-exynos/mach-exynos5-dt.c > @@ -45,7 +45,8 @@ static const struct of_dev_auxdata exynos5250_auxdata_lookup[] __initconst = { > "exynos4210-uart.3", NULL), > OF_DEV_AUXDATA("arm,pl330", EXYNOS5_PA_PDMA0, "dma-pl330.0", NULL), > OF_DEV_AUXDATA("arm,pl330", EXYNOS5_PA_PDMA1, "dma-pl330.1", NULL), > - OF_DEV_AUXDATA("arm,pl330", EXYNOS5_PA_PDMA1, "dma-pl330.2", NULL), > + OF_DEV_AUXDATA("arm,pl330", EXYNOS5_PA_MDMA0, "dma-pl330.2", NULL), OK. > + OF_DEV_AUXDATA("arm,pl330", EXYNOS5_PA_MDMA0, "dma-pl330.3", NULL), Well...there is a MDMA1 on EXYNOS5250 SoC, so if we can use MDMA1, should be MDMA1 for dma-pl330.3 but...I think, I need to check :) > {}, > }; > Thanks. Best regards, Kgene. -- Kukjin Kim <kgene.kim@samsung.com>, Senior Engineer, SW Solution Development Team, Samsung Electronics Co., Ltd. ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH 4/4] ARM: Exynos5: Add PDMA and MDMA physical base address defines 2012-03-29 1:54 ` Kukjin Kim @ 2012-04-09 3:48 ` Olof Johansson 2012-04-10 16:56 ` Kukjin Kim 0 siblings, 1 reply; 10+ messages in thread From: Olof Johansson @ 2012-04-09 3:48 UTC (permalink / raw) To: linux-arm-kernel Hi, 2012/3/28 Kukjin Kim <kgene.kim@samsung.com>: > Thomas Abraham wrote: >> Add PDMA and MDMA physical base address macros which is require for Exynos5 >> of_dev_auxdata setup. >> >> Signed-off-by: Thomas Abraham<thomas.ab@samsung.com> >> --- >> ? arch/arm/mach-exynos/include/mach/map.h | ? ?5 +++++ >> ? arch/arm/mach-exynos/mach-exynos5-dt.c ?| ? ?3 ++- >> ? 2 files changed, 7 insertions(+), 1 deletions(-) >> >> diff --git a/arch/arm/mach-exynos/include/mach/map.h b/arch/arm/mach-exynos/include/mach/map.h >> index 024d38f..37d93fd 100644 >> --- a/arch/arm/mach-exynos/include/mach/map.h >> +++ b/arch/arm/mach-exynos/include/mach/map.h >> @@ -90,6 +90,11 @@ >> ? #define EXYNOS4_PA_PDMA0 ? ? ? ? ? ?0x12680000 >> ? #define EXYNOS4_PA_PDMA1 ? ? ? ? ? ?0x12690000 >> >> +#define EXYNOS5_PA_PDMA0 ? ? ? ? ? ? 0x121A0000 >> +#define EXYNOS5_PA_PDMA1 ? ? ? ? ? ? 0x121B0000 >> +#define EXYNOS5_PA_MDMA0 ? ? ? ? ? ? 0x10800000 >> +#define EXYNOS5_PA_MDMA1 ? ? ? ? ? ? 0x11C10000 >> + > > See, "ARM: EXYNOS: Support DMA for Exynos5250 SoC" > http://lists.infradead.org/pipermail/linux-arm-kernel/2012-February/085434.html > > But this merge window is not including it :( Compilation of CONFIG_MACH_EXYNOS5_DT is broken without the above PDMA defines. Either add the defines, or remove the AUXDATA as part of the fixes for this release, please. -Olof ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH 4/4] ARM: Exynos5: Add PDMA and MDMA physical base address defines 2012-04-09 3:48 ` Olof Johansson @ 2012-04-10 16:56 ` Kukjin Kim 0 siblings, 0 replies; 10+ messages in thread From: Kukjin Kim @ 2012-04-10 16:56 UTC (permalink / raw) To: linux-arm-kernel Olof Johansson wrote: > Hi, > > 2012/3/28 Kukjin Kim<kgene.kim@samsung.com>: >> Thomas Abraham wrote: >>> Add PDMA and MDMA physical base address macros which is require for Exynos5 >>> of_dev_auxdata setup. >>> >>> Signed-off-by: Thomas Abraham<thomas.ab@samsung.com> >>> --- >>> arch/arm/mach-exynos/include/mach/map.h | 5 +++++ >>> arch/arm/mach-exynos/mach-exynos5-dt.c | 3 ++- >>> 2 files changed, 7 insertions(+), 1 deletions(-) >>> >>> diff --git a/arch/arm/mach-exynos/include/mach/map.h b/arch/arm/mach-exynos/include/mach/map.h >>> index 024d38f..37d93fd 100644 >>> --- a/arch/arm/mach-exynos/include/mach/map.h >>> +++ b/arch/arm/mach-exynos/include/mach/map.h >>> @@ -90,6 +90,11 @@ >>> #define EXYNOS4_PA_PDMA0 0x12680000 >>> #define EXYNOS4_PA_PDMA1 0x12690000 >>> >>> +#define EXYNOS5_PA_PDMA0 0x121A0000 >>> +#define EXYNOS5_PA_PDMA1 0x121B0000 >>> +#define EXYNOS5_PA_MDMA0 0x10800000 >>> +#define EXYNOS5_PA_MDMA1 0x11C10000 >>> + >> >> See, "ARM: EXYNOS: Support DMA for Exynos5250 SoC" >> http://lists.infradead.org/pipermail/linux-arm-kernel/2012-February/085434.html >> >> But this merge window is not including it :( > > Compilation of CONFIG_MACH_EXYNOS5_DT is broken without the above PDMA > defines. Either add the defines, or remove the AUXDATA as part of the > fixes for this release, please. > Yeah, right. I fixed it with this but changed dma channel for mdma1. @@ -45,7 +45,7 @@ static const struct of_dev_auxdata exynos5250_auxdata_lookup[] "exynos4210-uart.3", NULL), OF_DEV_AUXDATA("arm,pl330", EXYNOS5_PA_PDMA0, "dma-pl330.0", NULL), OF_DEV_AUXDATA("arm,pl330", EXYNOS5_PA_PDMA1, "dma-pl330.1", NULL), - OF_DEV_AUXDATA("arm,pl330", EXYNOS5_PA_PDMA1, "dma-pl330.2", NULL), + OF_DEV_AUXDATA("arm,pl330", EXYNOS5_PA_MDMA1, "dma-pl330.2", NULL), {}, }; Thanks. Best regards, Kgene. -- Kukjin Kim <kgene.kim@samsung.com>, Senior Engineer, SW Solution Development Team, Samsung Electronics Co., Ltd. ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2012-04-10 16:56 UTC | newest] Thread overview: 10+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2012-03-27 9:26 [PATCH 0/4] ARM: Exynos: Fixes for Exynos5 device tree support Thomas Abraham 2012-03-27 9:26 ` [PATCH 1/4] ARM: Exynos: Use CHIP_ID to detect cpu type during decompression setup Thomas Abraham 2012-03-29 1:34 ` Kukjin Kim 2012-03-27 9:26 ` [PATCH 2/4] ARM: Exynos: Remove a new bus_type instance for Exynos5 Thomas Abraham 2012-03-27 9:26 ` [PATCH 3/4] ARM: Exynos5: Fix incorrect initialization of GIC Thomas Abraham 2012-03-29 1:46 ` Kukjin Kim 2012-03-27 9:26 ` [PATCH 4/4] ARM: Exynos5: Add PDMA and MDMA physical base address defines Thomas Abraham 2012-03-29 1:54 ` Kukjin Kim 2012-04-09 3:48 ` Olof Johansson 2012-04-10 16:56 ` Kukjin Kim
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).