* [GIT PULL] qcom SoC changes for v3.16
@ 2014-05-23 18:42 Kumar Gala
2014-05-23 20:10 ` Arnd Bergmann
2014-05-26 12:49 ` Paul Bolle
0 siblings, 2 replies; 13+ messages in thread
From: Kumar Gala @ 2014-05-23 18:42 UTC (permalink / raw)
To: linux-arm-kernel
The following changes since commit c9eaa447e77efe77b7fa4c953bd62de8297fd6c5:
Linux 3.15-rc1 (2014-04-13 14:18:35 -0700)
are available in the git repository at:
git://git.kernel.org/pub/scm/linux/kernel/git/galak/linux-qcom.git tags/qcom-soc-for-3.16
for you to fetch changes up to 87cfb00d666cca3abf36f755279cde1b27b9c377:
ARM: qcom: Select PINCTRL by default for ARCH_QCOM (2014-05-22 11:50:42 -0500)
----------------------------------------------------------------
Qualcomm ARM Based SoC Updates for v3.16
* Enabling building pinctrl and AMBA bus support
* Clean up debug UART selection
----------------------------------------------------------------
Andy Gross (1):
ARM: qcom: Select PINCTRL by default for ARCH_QCOM
Ivan T. Ivanov (1):
ARM: debug: qcom: make UART address selection configuration option
Srinivas Kandagatla (1):
ARM: qcom: Enable ARM_AMBA option for Qualcomm SOCs.
arch/arm/Kconfig.debug | 81 ++++++++++++++++++--------------------------
arch/arm/include/debug/msm.S | 46 +++----------------------
arch/arm/mach-msm/Kconfig | 3 --
arch/arm/mach-qcom/Kconfig | 2 ++
4 files changed, 40 insertions(+), 92 deletions(-)
--
Employee of Qualcomm Innovation Center, Inc.
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation
^ permalink raw reply [flat|nested] 13+ messages in thread* [GIT PULL] qcom SoC changes for v3.16 2014-05-23 18:42 [GIT PULL] qcom SoC changes for v3.16 Kumar Gala @ 2014-05-23 20:10 ` Arnd Bergmann 2014-05-26 12:49 ` Paul Bolle 1 sibling, 0 replies; 13+ messages in thread From: Arnd Bergmann @ 2014-05-23 20:10 UTC (permalink / raw) To: linux-arm-kernel On Friday 23 May 2014, Kumar Gala wrote: > The following changes since commit c9eaa447e77efe77b7fa4c953bd62de8297fd6c5: > > Linux 3.15-rc1 (2014-04-13 14:18:35 -0700) > > are available in the git repository at: > > git://git.kernel.org/pub/scm/linux/kernel/git/galak/linux-qcom.git tags/qcom-soc-for-3.16 > > for you to fetch changes up to 87cfb00d666cca3abf36f755279cde1b27b9c377: > > ARM: qcom: Select PINCTRL by default for ARCH_QCOM (2014-05-22 11:50:42 -0500) > > ---------------------------------------------------------------- > Qualcomm ARM Based SoC Updates for v3.16 > > * Enabling building pinctrl and AMBA bus support > * Clean up debug UART selection Pulled into next/soc, thanks! Arnd ^ permalink raw reply [flat|nested] 13+ messages in thread
* [GIT PULL] qcom SoC changes for v3.16 2014-05-23 18:42 [GIT PULL] qcom SoC changes for v3.16 Kumar Gala 2014-05-23 20:10 ` Arnd Bergmann @ 2014-05-26 12:49 ` Paul Bolle 2014-05-26 13:47 ` Georgi Djakov ` (2 more replies) 1 sibling, 3 replies; 13+ messages in thread From: Paul Bolle @ 2014-05-26 12:49 UTC (permalink / raw) To: linux-arm-kernel On Fri, 2014-05-23 at 13:42 -0500, Kumar Gala wrote: > Ivan T. Ivanov (1): > ARM: debug: qcom: make UART address selection configuration option This one just landed in linux-next (next-20140526). It removed the Kconfig options DEBUG_MSM_UART1, DEBUG_MSM_UART3, and DEBUG_MSM_UART3. It did not touch these references to their macros: arch/arm/mach-msm/io.c:60:#if defined(CONFIG_DEBUG_MSM_UART1) || defined(CONFIG_DEBUG_MSM_UART2) || \ arch/arm/mach-msm/io.c:61: defined(CONFIG_DEBUG_MSM_UART3) arch/arm/mach-msm/io.c:79:#if defined(CONFIG_DEBUG_MSM_UART1) || defined(CONFIG_DEBUG_MSM_UART2) || \ arch/arm/mach-msm/io.c:80: defined(CONFIG_DEBUG_MSM_UART3) Is a patch to fix this queued somewhere? Paul Bolle ^ permalink raw reply [flat|nested] 13+ messages in thread
* [GIT PULL] qcom SoC changes for v3.16 2014-05-26 12:49 ` Paul Bolle @ 2014-05-26 13:47 ` Georgi Djakov 2014-05-27 6:12 ` Ivan T. Ivanov 2014-05-26 13:48 ` Ivan T. Ivanov 2014-05-26 13:58 ` [PATCH] ARM: msm: Update the references to DEBUG_MSM_UARTx Georgi Djakov 2 siblings, 1 reply; 13+ messages in thread From: Georgi Djakov @ 2014-05-26 13:47 UTC (permalink / raw) To: linux-arm-kernel On 05/26/2014 03:49 PM, Paul Bolle wrote: > On Fri, 2014-05-23 at 13:42 -0500, Kumar Gala wrote: >> Ivan T. Ivanov (1): >> ARM: debug: qcom: make UART address selection configuration option > > This one just landed in linux-next (next-20140526). > > It removed the Kconfig options DEBUG_MSM_UART1, DEBUG_MSM_UART3, and > DEBUG_MSM_UART3. It did not touch these references to their macros: > arch/arm/mach-msm/io.c:60:#if defined(CONFIG_DEBUG_MSM_UART1) || defined(CONFIG_DEBUG_MSM_UART2) || \ > arch/arm/mach-msm/io.c:61: defined(CONFIG_DEBUG_MSM_UART3) > arch/arm/mach-msm/io.c:79:#if defined(CONFIG_DEBUG_MSM_UART1) || defined(CONFIG_DEBUG_MSM_UART2) || \ > arch/arm/mach-msm/io.c:80: defined(CONFIG_DEBUG_MSM_UART3) > > Is a patch to fix this queued somewhere? > Not yet, but I'll send one in a minute. Thanks, Georgi ^ permalink raw reply [flat|nested] 13+ messages in thread
* [GIT PULL] qcom SoC changes for v3.16 2014-05-26 13:47 ` Georgi Djakov @ 2014-05-27 6:12 ` Ivan T. Ivanov 0 siblings, 0 replies; 13+ messages in thread From: Ivan T. Ivanov @ 2014-05-27 6:12 UTC (permalink / raw) To: linux-arm-kernel On Mon, 2014-05-26 at 16:47 +0300, Georgi Djakov wrote: > On 05/26/2014 03:49 PM, Paul Bolle wrote: > > On Fri, 2014-05-23 at 13:42 -0500, Kumar Gala wrote: > >> Ivan T. Ivanov (1): > >> ARM: debug: qcom: make UART address selection configuration option > > > > This one just landed in linux-next (next-20140526). > > > > It removed the Kconfig options DEBUG_MSM_UART1, DEBUG_MSM_UART3, and > > DEBUG_MSM_UART3. It did not touch these references to their macros: > > arch/arm/mach-msm/io.c:60:#if defined(CONFIG_DEBUG_MSM_UART1) || defined(CONFIG_DEBUG_MSM_UART2) || \ > > arch/arm/mach-msm/io.c:61: defined(CONFIG_DEBUG_MSM_UART3) > > arch/arm/mach-msm/io.c:79:#if defined(CONFIG_DEBUG_MSM_UART1) || defined(CONFIG_DEBUG_MSM_UART2) || \ > > arch/arm/mach-msm/io.c:80: defined(CONFIG_DEBUG_MSM_UART3) > > > > Is a patch to fix this queued somewhere? > > > > Not yet, but I'll send one in a minute. Right, I have missed this. Thanks. Ivan ^ permalink raw reply [flat|nested] 13+ messages in thread
* [GIT PULL] qcom SoC changes for v3.16 2014-05-26 12:49 ` Paul Bolle 2014-05-26 13:47 ` Georgi Djakov @ 2014-05-26 13:48 ` Ivan T. Ivanov 2014-05-26 13:58 ` [PATCH] ARM: msm: Update the references to DEBUG_MSM_UARTx Georgi Djakov 2 siblings, 0 replies; 13+ messages in thread From: Ivan T. Ivanov @ 2014-05-26 13:48 UTC (permalink / raw) To: linux-arm-kernel On Mon, 2014-05-26 at 14:49 +0200, Paul Bolle wrote: > On Fri, 2014-05-23 at 13:42 -0500, Kumar Gala wrote: > > Ivan T. Ivanov (1): > > ARM: debug: qcom: make UART address selection configuration option > > This one just landed in linux-next (next-20140526). > > It removed the Kconfig options DEBUG_MSM_UART1, DEBUG_MSM_UART3, and > DEBUG_MSM_UART3. It did not touch these references to their macros: > arch/arm/mach-msm/io.c:60:#if defined(CONFIG_DEBUG_MSM_UART1) || defined(CONFIG_DEBUG_MSM_UART2) || \ > arch/arm/mach-msm/io.c:61: defined(CONFIG_DEBUG_MSM_UART3) > arch/arm/mach-msm/io.c:79:#if defined(CONFIG_DEBUG_MSM_UART1) || defined(CONFIG_DEBUG_MSM_UART2) || \ > arch/arm/mach-msm/io.c:80: defined(CONFIG_DEBUG_MSM_UART3) > > Is a patch to fix this queued somewhere? > Hm, there was some conflicts, which where resolved by Stephen Rothwell [1], but maybe there is still something wrong. Have to check. Regards, Ivan [1] https://lkml.org/lkml/2014/5/25/147 > > Paul Bolle > ^ permalink raw reply [flat|nested] 13+ messages in thread
* [PATCH] ARM: msm: Update the references to DEBUG_MSM_UARTx 2014-05-26 12:49 ` Paul Bolle 2014-05-26 13:47 ` Georgi Djakov 2014-05-26 13:48 ` Ivan T. Ivanov @ 2014-05-26 13:58 ` Georgi Djakov 2014-05-26 14:58 ` Paul Bolle 2 siblings, 1 reply; 13+ messages in thread From: Georgi Djakov @ 2014-05-26 13:58 UTC (permalink / raw) To: linux-arm-kernel The Kconfig options DEBUG_MSM_UART1, DEBUG_MSM_UART2 and DEBUG_MSM_UART3 are removed, but they are still referenced in arch/arm/mach-msm/io.c Fix this by updating the reference to the new Kconfig option. Reported-by: Paul Bolle <pebolle@tiscali.nl> Signed-off-by: Georgi Djakov <gdjakov@mm-sol.com> --- arch/arm/mach-msm/io.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/arch/arm/mach-msm/io.c b/arch/arm/mach-msm/io.c index 34e0947..b042dca 100644 --- a/arch/arm/mach-msm/io.c +++ b/arch/arm/mach-msm/io.c @@ -57,8 +57,7 @@ static struct map_desc msm_io_desc[] __initdata = { .length = MSM_SHARED_RAM_SIZE, .type = MT_DEVICE, }, -#if defined(CONFIG_DEBUG_MSM_UART1) || defined(CONFIG_DEBUG_MSM_UART2) || \ - defined(CONFIG_DEBUG_MSM_UART3) +#if defined(CONFIG_DEBUG_MSM_UART) { /* Must be last: virtual and pfn filled in by debug_ll_addr() */ .length = SZ_4K, @@ -76,8 +75,7 @@ void __init msm_map_common_io(void) * pages are peripheral interface or not. */ asm("mcr p15, 0, %0, c15, c2, 4" : : "r" (0)); -#if defined(CONFIG_DEBUG_MSM_UART1) || defined(CONFIG_DEBUG_MSM_UART2) || \ - defined(CONFIG_DEBUG_MSM_UART3) +#if defined(CONFIG_DEBUG_MSM_UART) #ifdef CONFIG_MMU debug_ll_addr(&msm_io_desc[size - 1].pfn, &msm_io_desc[size - 1].virtual); -- 1.7.9.5 ^ permalink raw reply related [flat|nested] 13+ messages in thread
* [PATCH] ARM: msm: Update the references to DEBUG_MSM_UARTx 2014-05-26 13:58 ` [PATCH] ARM: msm: Update the references to DEBUG_MSM_UARTx Georgi Djakov @ 2014-05-26 14:58 ` Paul Bolle 2014-05-26 15:44 ` Georgi Djakov 2014-05-26 16:37 ` [PATCH v2] " Georgi Djakov 0 siblings, 2 replies; 13+ messages in thread From: Paul Bolle @ 2014-05-26 14:58 UTC (permalink / raw) To: linux-arm-kernel Hi Georgi, On Mon, 2014-05-26 at 16:58 +0300, Georgi Djakov wrote: > The Kconfig options DEBUG_MSM_UART1, DEBUG_MSM_UART2 and DEBUG_MSM_UART3 > are removed, but they are still referenced in arch/arm/mach-msm/io.c > Fix this by updating the reference to the new Kconfig option. > > Reported-by: Paul Bolle <pebolle@tiscali.nl> > Signed-off-by: Georgi Djakov <gdjakov@mm-sol.com> > --- > arch/arm/mach-msm/io.c | 6 ++---- > 1 file changed, 2 insertions(+), 4 deletions(-) > > diff --git a/arch/arm/mach-msm/io.c b/arch/arm/mach-msm/io.c > index 34e0947..b042dca 100644 > --- a/arch/arm/mach-msm/io.c > +++ b/arch/arm/mach-msm/io.c > @@ -57,8 +57,7 @@ static struct map_desc msm_io_desc[] __initdata = { > .length = MSM_SHARED_RAM_SIZE, > .type = MT_DEVICE, > }, > -#if defined(CONFIG_DEBUG_MSM_UART1) || defined(CONFIG_DEBUG_MSM_UART2) || \ > - defined(CONFIG_DEBUG_MSM_UART3) > +#if defined(CONFIG_DEBUG_MSM_UART) > { > /* Must be last: virtual and pfn filled in by debug_ll_addr() */ > .length = SZ_4K, > @@ -76,8 +75,7 @@ void __init msm_map_common_io(void) > * pages are peripheral interface or not. > */ > asm("mcr p15, 0, %0, c15, c2, 4" : : "r" (0)); > -#if defined(CONFIG_DEBUG_MSM_UART1) || defined(CONFIG_DEBUG_MSM_UART2) || \ > - defined(CONFIG_DEBUG_MSM_UART3) > +#if defined(CONFIG_DEBUG_MSM_UART) > #ifdef CONFIG_MMU > debug_ll_addr(&msm_io_desc[size - 1].pfn, > &msm_io_desc[size - 1].virtual); Should I now redo https://lkml.org/lkml/2014/5/15/249 by making the (outdated) checks in arch/arm/mach-msm/board-trout.c and arch/arm/mach-msm/board-trout-gpio.c also use CONFIG_DEBUG_MSM_UART? Paul Bolle ^ permalink raw reply [flat|nested] 13+ messages in thread
* [PATCH] ARM: msm: Update the references to DEBUG_MSM_UARTx 2014-05-26 14:58 ` Paul Bolle @ 2014-05-26 15:44 ` Georgi Djakov 2014-05-26 16:37 ` [PATCH v2] " Georgi Djakov 1 sibling, 0 replies; 13+ messages in thread From: Georgi Djakov @ 2014-05-26 15:44 UTC (permalink / raw) To: linux-arm-kernel On 05/26/2014 05:58 PM, Paul Bolle wrote: > Hi Georgi, Hi Paul, > > On Mon, 2014-05-26 at 16:58 +0300, Georgi Djakov wrote: >> The Kconfig options DEBUG_MSM_UART1, DEBUG_MSM_UART2 and DEBUG_MSM_UART3 >> are removed, but they are still referenced in arch/arm/mach-msm/io.c >> Fix this by updating the reference to the new Kconfig option. >> >> Reported-by: Paul Bolle <pebolle@tiscali.nl> >> Signed-off-by: Georgi Djakov <gdjakov@mm-sol.com> >> --- >> arch/arm/mach-msm/io.c | 6 ++---- >> 1 file changed, 2 insertions(+), 4 deletions(-) >> >> diff --git a/arch/arm/mach-msm/io.c b/arch/arm/mach-msm/io.c >> index 34e0947..b042dca 100644 >> --- a/arch/arm/mach-msm/io.c >> +++ b/arch/arm/mach-msm/io.c >> @@ -57,8 +57,7 @@ static struct map_desc msm_io_desc[] __initdata = { >> .length = MSM_SHARED_RAM_SIZE, >> .type = MT_DEVICE, >> }, >> -#if defined(CONFIG_DEBUG_MSM_UART1) || defined(CONFIG_DEBUG_MSM_UART2) || \ >> - defined(CONFIG_DEBUG_MSM_UART3) >> +#if defined(CONFIG_DEBUG_MSM_UART) >> { >> /* Must be last: virtual and pfn filled in by debug_ll_addr() */ >> .length = SZ_4K, >> @@ -76,8 +75,7 @@ void __init msm_map_common_io(void) >> * pages are peripheral interface or not. >> */ >> asm("mcr p15, 0, %0, c15, c2, 4" : : "r" (0)); >> -#if defined(CONFIG_DEBUG_MSM_UART1) || defined(CONFIG_DEBUG_MSM_UART2) || \ >> - defined(CONFIG_DEBUG_MSM_UART3) >> +#if defined(CONFIG_DEBUG_MSM_UART) >> #ifdef CONFIG_MMU >> debug_ll_addr(&msm_io_desc[size - 1].pfn, >> &msm_io_desc[size - 1].virtual); > > Should I now redo https://lkml.org/lkml/2014/5/15/249 by making the > (outdated) checks in arch/arm/mach-msm/board-trout.c and > arch/arm/mach-msm/board-trout-gpio.c also use CONFIG_DEBUG_MSM_UART? > Hm, not exactly. It seems that this board has multiple UARTs that can be used for debug, so we should also check for the UART PHYS address. I will prepare a patch to fix this too. Thanks, Georgi ^ permalink raw reply [flat|nested] 13+ messages in thread
* [PATCH v2] ARM: msm: Update the references to DEBUG_MSM_UARTx 2014-05-26 14:58 ` Paul Bolle 2014-05-26 15:44 ` Georgi Djakov @ 2014-05-26 16:37 ` Georgi Djakov 2014-06-19 8:16 ` Paul Bolle 1 sibling, 1 reply; 13+ messages in thread From: Georgi Djakov @ 2014-05-26 16:37 UTC (permalink / raw) To: linux-arm-kernel The Kconfig options DEBUG_MSM_UART1, DEBUG_MSM_UART2, DEBUG_MSM_UART3, MSM_DEBUG_UART1, MSM_DEBUG_UART2 and MSM_DEBUG_UART3 are removed, but they are still referenced in the following files: arch/arm/mach-msm/io.c, arch/arm/mach-msm/board-trout.c arch/arm/mach-msm/board-trout-gpio.c Fix this by updating the reference to the new Kconfig option. Reported-by: Paul Bolle <pebolle@tiscali.nl> Signed-off-by: Georgi Djakov <gdjakov@mm-sol.com> --- Changes since v1: - Fixed similar issue in board-trout.c and board-trout-gpio.c Applies to next-20140526. arch/arm/mach-msm/board-trout-gpio.c | 2 +- arch/arm/mach-msm/board-trout.c | 2 +- arch/arm/mach-msm/io.c | 6 ++---- 3 files changed, 4 insertions(+), 6 deletions(-) diff --git a/arch/arm/mach-msm/board-trout-gpio.c b/arch/arm/mach-msm/board-trout-gpio.c index 87e1d01..e24930e 100644 --- a/arch/arm/mach-msm/board-trout-gpio.c +++ b/arch/arm/mach-msm/board-trout-gpio.c @@ -94,7 +94,7 @@ static int trout_gpio_to_irq(struct gpio_chip *chip, unsigned offset) } static struct msm_gpio_chip msm_gpio_banks[] = { -#if defined(CONFIG_MSM_DEBUG_UART1) +#if defined(CONFIG_DEBUG_MSM_UART) && (CONFIG_DEBUG_UART_PHYS == 0xa9a00000) /* H2W pins <-> UART1 */ TROUT_GPIO_BANK("MISC2", 0x00, TROUT_GPIO_MISC2_BASE, 0x40), #else diff --git a/arch/arm/mach-msm/board-trout.c b/arch/arm/mach-msm/board-trout.c index 015d544..8369dfd 100644 --- a/arch/arm/mach-msm/board-trout.c +++ b/arch/arm/mach-msm/board-trout.c @@ -90,7 +90,7 @@ static void __init trout_map_io(void) msm_map_common_io(); iotable_init(trout_io_desc, ARRAY_SIZE(trout_io_desc)); -#ifdef CONFIG_MSM_DEBUG_UART3 +#if defined(CONFIG_DEBUG_MSM_UART) && (CONFIG_DEBUG_UART_PHYS == 0xa9c00000) /* route UART3 to the "H2W" extended usb connector */ writeb(0x80, TROUT_CPLD_BASE + 0x00); #endif diff --git a/arch/arm/mach-msm/io.c b/arch/arm/mach-msm/io.c index 34e0947..b042dca 100644 --- a/arch/arm/mach-msm/io.c +++ b/arch/arm/mach-msm/io.c @@ -57,8 +57,7 @@ static struct map_desc msm_io_desc[] __initdata = { .length = MSM_SHARED_RAM_SIZE, .type = MT_DEVICE, }, -#if defined(CONFIG_DEBUG_MSM_UART1) || defined(CONFIG_DEBUG_MSM_UART2) || \ - defined(CONFIG_DEBUG_MSM_UART3) +#if defined(CONFIG_DEBUG_MSM_UART) { /* Must be last: virtual and pfn filled in by debug_ll_addr() */ .length = SZ_4K, @@ -76,8 +75,7 @@ void __init msm_map_common_io(void) * pages are peripheral interface or not. */ asm("mcr p15, 0, %0, c15, c2, 4" : : "r" (0)); -#if defined(CONFIG_DEBUG_MSM_UART1) || defined(CONFIG_DEBUG_MSM_UART2) || \ - defined(CONFIG_DEBUG_MSM_UART3) +#if defined(CONFIG_DEBUG_MSM_UART) #ifdef CONFIG_MMU debug_ll_addr(&msm_io_desc[size - 1].pfn, &msm_io_desc[size - 1].virtual); -- 1.7.9.5 ^ permalink raw reply related [flat|nested] 13+ messages in thread
* [PATCH v2] ARM: msm: Update the references to DEBUG_MSM_UARTx 2014-05-26 16:37 ` [PATCH v2] " Georgi Djakov @ 2014-06-19 8:16 ` Paul Bolle 2014-09-04 22:38 ` Paul Bolle 0 siblings, 1 reply; 13+ messages in thread From: Paul Bolle @ 2014-06-19 8:16 UTC (permalink / raw) To: linux-arm-kernel On Mon, 2014-05-26 at 19:37 +0300, Georgi Djakov wrote: > The Kconfig options DEBUG_MSM_UART1, DEBUG_MSM_UART2, DEBUG_MSM_UART3, > MSM_DEBUG_UART1, MSM_DEBUG_UART2 and MSM_DEBUG_UART3 are removed, but > they are still referenced in the following files: > arch/arm/mach-msm/io.c, > arch/arm/mach-msm/board-trout.c > arch/arm/mach-msm/board-trout-gpio.c > > Fix this by updating the reference to the new Kconfig option. > > Reported-by: Paul Bolle <pebolle@tiscali.nl> > Signed-off-by: Georgi Djakov <gdjakov@mm-sol.com> > --- This patch is included neither in v3.16-rc1 nor in next-20140619. Did it run into problems or is it already queued somewhere? > Changes since v1: > - Fixed similar issue in board-trout.c and board-trout-gpio.c > > Applies to next-20140526. > > arch/arm/mach-msm/board-trout-gpio.c | 2 +- > arch/arm/mach-msm/board-trout.c | 2 +- > arch/arm/mach-msm/io.c | 6 ++---- > 3 files changed, 4 insertions(+), 6 deletions(-) > > diff --git a/arch/arm/mach-msm/board-trout-gpio.c b/arch/arm/mach-msm/board-trout-gpio.c > index 87e1d01..e24930e 100644 > --- a/arch/arm/mach-msm/board-trout-gpio.c > +++ b/arch/arm/mach-msm/board-trout-gpio.c > @@ -94,7 +94,7 @@ static int trout_gpio_to_irq(struct gpio_chip *chip, unsigned offset) > } > > static struct msm_gpio_chip msm_gpio_banks[] = { > -#if defined(CONFIG_MSM_DEBUG_UART1) > +#if defined(CONFIG_DEBUG_MSM_UART) && (CONFIG_DEBUG_UART_PHYS == 0xa9a00000) > /* H2W pins <-> UART1 */ > TROUT_GPIO_BANK("MISC2", 0x00, TROUT_GPIO_MISC2_BASE, 0x40), > #else > diff --git a/arch/arm/mach-msm/board-trout.c b/arch/arm/mach-msm/board-trout.c > index 015d544..8369dfd 100644 > --- a/arch/arm/mach-msm/board-trout.c > +++ b/arch/arm/mach-msm/board-trout.c > @@ -90,7 +90,7 @@ static void __init trout_map_io(void) > msm_map_common_io(); > iotable_init(trout_io_desc, ARRAY_SIZE(trout_io_desc)); > > -#ifdef CONFIG_MSM_DEBUG_UART3 > +#if defined(CONFIG_DEBUG_MSM_UART) && (CONFIG_DEBUG_UART_PHYS == 0xa9c00000) > /* route UART3 to the "H2W" extended usb connector */ > writeb(0x80, TROUT_CPLD_BASE + 0x00); > #endif > diff --git a/arch/arm/mach-msm/io.c b/arch/arm/mach-msm/io.c > index 34e0947..b042dca 100644 > --- a/arch/arm/mach-msm/io.c > +++ b/arch/arm/mach-msm/io.c > @@ -57,8 +57,7 @@ static struct map_desc msm_io_desc[] __initdata = { > .length = MSM_SHARED_RAM_SIZE, > .type = MT_DEVICE, > }, > -#if defined(CONFIG_DEBUG_MSM_UART1) || defined(CONFIG_DEBUG_MSM_UART2) || \ > - defined(CONFIG_DEBUG_MSM_UART3) > +#if defined(CONFIG_DEBUG_MSM_UART) > { > /* Must be last: virtual and pfn filled in by debug_ll_addr() */ > .length = SZ_4K, > @@ -76,8 +75,7 @@ void __init msm_map_common_io(void) > * pages are peripheral interface or not. > */ > asm("mcr p15, 0, %0, c15, c2, 4" : : "r" (0)); > -#if defined(CONFIG_DEBUG_MSM_UART1) || defined(CONFIG_DEBUG_MSM_UART2) || \ > - defined(CONFIG_DEBUG_MSM_UART3) > +#if defined(CONFIG_DEBUG_MSM_UART) > #ifdef CONFIG_MMU > debug_ll_addr(&msm_io_desc[size - 1].pfn, > &msm_io_desc[size - 1].virtual); Paul Bolle ^ permalink raw reply [flat|nested] 13+ messages in thread
* [PATCH v2] ARM: msm: Update the references to DEBUG_MSM_UARTx 2014-06-19 8:16 ` Paul Bolle @ 2014-09-04 22:38 ` Paul Bolle 2014-09-05 7:23 ` Georgi Djakov 0 siblings, 1 reply; 13+ messages in thread From: Paul Bolle @ 2014-09-04 22:38 UTC (permalink / raw) To: linux-arm-kernel On Thu, 2014-06-19 at 10:16 +0200, Paul Bolle wrote: > On Mon, 2014-05-26 at 19:37 +0300, Georgi Djakov wrote: > > The Kconfig options DEBUG_MSM_UART1, DEBUG_MSM_UART2, DEBUG_MSM_UART3, > > MSM_DEBUG_UART1, MSM_DEBUG_UART2 and MSM_DEBUG_UART3 are removed, but > > they are still referenced in the following files: > > arch/arm/mach-msm/io.c, > > arch/arm/mach-msm/board-trout.c > > arch/arm/mach-msm/board-trout-gpio.c > > > > Fix this by updating the reference to the new Kconfig option. > > > > Reported-by: Paul Bolle <pebolle@tiscali.nl> > > Signed-off-by: Georgi Djakov <gdjakov@mm-sol.com> > > --- > > This patch is included neither in v3.16-rc1 nor in next-20140619. Still true for v3.17-rc3 and next-20140904. > Did it run into problems or is it already queued somewhere? This issue was reported three months ago but is still present. Should I perhaps submit the trivial patch that removes the checks for these macros and the code they now hide? > > Changes since v1: > > - Fixed similar issue in board-trout.c and board-trout-gpio.c > > > > Applies to next-20140526. > > > > arch/arm/mach-msm/board-trout-gpio.c | 2 +- > > arch/arm/mach-msm/board-trout.c | 2 +- > > arch/arm/mach-msm/io.c | 6 ++---- > > 3 files changed, 4 insertions(+), 6 deletions(-) > > > > diff --git a/arch/arm/mach-msm/board-trout-gpio.c b/arch/arm/mach-msm/board-trout-gpio.c > > index 87e1d01..e24930e 100644 > > --- a/arch/arm/mach-msm/board-trout-gpio.c > > +++ b/arch/arm/mach-msm/board-trout-gpio.c > > @@ -94,7 +94,7 @@ static int trout_gpio_to_irq(struct gpio_chip *chip, unsigned offset) > > } > > > > static struct msm_gpio_chip msm_gpio_banks[] = { > > -#if defined(CONFIG_MSM_DEBUG_UART1) > > +#if defined(CONFIG_DEBUG_MSM_UART) && (CONFIG_DEBUG_UART_PHYS == 0xa9a00000) > > /* H2W pins <-> UART1 */ > > TROUT_GPIO_BANK("MISC2", 0x00, TROUT_GPIO_MISC2_BASE, 0x40), > > #else > > diff --git a/arch/arm/mach-msm/board-trout.c b/arch/arm/mach-msm/board-trout.c > > index 015d544..8369dfd 100644 > > --- a/arch/arm/mach-msm/board-trout.c > > +++ b/arch/arm/mach-msm/board-trout.c > > @@ -90,7 +90,7 @@ static void __init trout_map_io(void) > > msm_map_common_io(); > > iotable_init(trout_io_desc, ARRAY_SIZE(trout_io_desc)); > > > > -#ifdef CONFIG_MSM_DEBUG_UART3 > > +#if defined(CONFIG_DEBUG_MSM_UART) && (CONFIG_DEBUG_UART_PHYS == 0xa9c00000) > > /* route UART3 to the "H2W" extended usb connector */ > > writeb(0x80, TROUT_CPLD_BASE + 0x00); > > #endif > > diff --git a/arch/arm/mach-msm/io.c b/arch/arm/mach-msm/io.c > > index 34e0947..b042dca 100644 > > --- a/arch/arm/mach-msm/io.c > > +++ b/arch/arm/mach-msm/io.c > > @@ -57,8 +57,7 @@ static struct map_desc msm_io_desc[] __initdata = { > > .length = MSM_SHARED_RAM_SIZE, > > .type = MT_DEVICE, > > }, > > -#if defined(CONFIG_DEBUG_MSM_UART1) || defined(CONFIG_DEBUG_MSM_UART2) || \ > > - defined(CONFIG_DEBUG_MSM_UART3) > > +#if defined(CONFIG_DEBUG_MSM_UART) > > { > > /* Must be last: virtual and pfn filled in by debug_ll_addr() */ > > .length = SZ_4K, > > @@ -76,8 +75,7 @@ void __init msm_map_common_io(void) > > * pages are peripheral interface or not. > > */ > > asm("mcr p15, 0, %0, c15, c2, 4" : : "r" (0)); > > -#if defined(CONFIG_DEBUG_MSM_UART1) || defined(CONFIG_DEBUG_MSM_UART2) || \ > > - defined(CONFIG_DEBUG_MSM_UART3) > > +#if defined(CONFIG_DEBUG_MSM_UART) > > #ifdef CONFIG_MMU > > debug_ll_addr(&msm_io_desc[size - 1].pfn, > > &msm_io_desc[size - 1].virtual); Paul Bolle ^ permalink raw reply [flat|nested] 13+ messages in thread
* [PATCH v2] ARM: msm: Update the references to DEBUG_MSM_UARTx 2014-09-04 22:38 ` Paul Bolle @ 2014-09-05 7:23 ` Georgi Djakov 0 siblings, 0 replies; 13+ messages in thread From: Georgi Djakov @ 2014-09-05 7:23 UTC (permalink / raw) To: linux-arm-kernel On 09/05/2014 01:38 AM, Paul Bolle wrote: > On Thu, 2014-06-19 at 10:16 +0200, Paul Bolle wrote: >> On Mon, 2014-05-26 at 19:37 +0300, Georgi Djakov wrote: >>> The Kconfig options DEBUG_MSM_UART1, DEBUG_MSM_UART2, DEBUG_MSM_UART3, >>> MSM_DEBUG_UART1, MSM_DEBUG_UART2 and MSM_DEBUG_UART3 are removed, but >>> they are still referenced in the following files: >>> arch/arm/mach-msm/io.c, >>> arch/arm/mach-msm/board-trout.c >>> arch/arm/mach-msm/board-trout-gpio.c >>> >>> Fix this by updating the reference to the new Kconfig option. >>> >>> Reported-by: Paul Bolle <pebolle@tiscali.nl> >>> Signed-off-by: Georgi Djakov <gdjakov@mm-sol.com> >>> --- >> >> This patch is included neither in v3.16-rc1 nor in next-20140619. > > Still true for v3.17-rc3 and next-20140904. > Hi Paul, It was pulled yesterday into the msm/cleanup branch of the arm-soc tree. Thanks, Georgi ^ permalink raw reply [flat|nested] 13+ messages in thread
end of thread, other threads:[~2014-09-05 7:23 UTC | newest] Thread overview: 13+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2014-05-23 18:42 [GIT PULL] qcom SoC changes for v3.16 Kumar Gala 2014-05-23 20:10 ` Arnd Bergmann 2014-05-26 12:49 ` Paul Bolle 2014-05-26 13:47 ` Georgi Djakov 2014-05-27 6:12 ` Ivan T. Ivanov 2014-05-26 13:48 ` Ivan T. Ivanov 2014-05-26 13:58 ` [PATCH] ARM: msm: Update the references to DEBUG_MSM_UARTx Georgi Djakov 2014-05-26 14:58 ` Paul Bolle 2014-05-26 15:44 ` Georgi Djakov 2014-05-26 16:37 ` [PATCH v2] " Georgi Djakov 2014-06-19 8:16 ` Paul Bolle 2014-09-04 22:38 ` Paul Bolle 2014-09-05 7:23 ` Georgi Djakov
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).