* [PATCH 1/2] ARM: davinci: fix for build breakage for tnetv107x platforms
2013-06-14 9:57 [PATCH 0/2] Fix build breakage for tnetv107x platforms Philip Avinash
@ 2013-06-14 9:57 ` Philip Avinash
2013-06-20 9:24 ` Sekhar Nori
2013-06-14 9:57 ` [PATCH 2/2] gpio: gpio-tnetv107x: Fix bulid breakge Philip Avinash
` (2 subsequent siblings)
3 siblings, 1 reply; 6+ messages in thread
From: Philip Avinash @ 2013-06-14 9:57 UTC (permalink / raw)
To: linux-arm-kernel
Fix the following build breakage and section mismatch error. This build
break will comes only on removal of NEED_MACH_GPIO_H select option for
ARCH-DAVINCI.
arch/arm/mach-davinci/board-tnetv107x-evm.c:283:15: error:
'davinci_timer_init' undeclared here (not in a function)
arch/arm/mach-davinci/board-tnetv107x-evm.c:285:15: error:
'davinci_init_late' undeclared here (not in a function)
make[1]: *** [arch/arm/mach-davinci/board-tnetv107x-evm.o] Error 1
Signed-off-by: Philip Avinash <avinashphilipk@gmail.com>
---
arch/arm/mach-davinci/board-tnetv107x-evm.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/arch/arm/mach-davinci/board-tnetv107x-evm.c b/arch/arm/mach-davinci/board-tnetv107x-evm.c
index ba79837..b5db980 100644
--- a/arch/arm/mach-davinci/board-tnetv107x-evm.c
+++ b/arch/arm/mach-davinci/board-tnetv107x-evm.c
@@ -30,6 +30,7 @@
#include <asm/mach/arch.h>
#include <asm/mach-types.h>
+#include <mach/common.h>
#include <mach/irqs.h>
#include <mach/edma.h>
#include <mach/mux.h>
@@ -147,7 +148,7 @@ static struct davinci_nand_pdata nand_config = {
.ecc_bits = 1,
};
-static struct davinci_uart_config serial_config __initconst = {
+static struct davinci_uart_config serial_config __initdata = {
.enabled_uarts = BIT(1),
};
@@ -245,7 +246,7 @@ static struct ti_ssp_data ssp_config = {
},
};
-static struct tnetv107x_device_info evm_device_info __initconst = {
+static struct tnetv107x_device_info evm_device_info __initdata = {
.serial_config = &serial_config,
.mmc_config[1] = &mmc_config, /* controller 1 */
.nand_config[0] = &nand_config, /* chip select 0 */
--
1.7.9.5
^ permalink raw reply related [flat|nested] 6+ messages in thread* [PATCH 1/2] ARM: davinci: fix for build breakage for tnetv107x platforms
2013-06-14 9:57 ` [PATCH 1/2] ARM: davinci: fix for " Philip Avinash
@ 2013-06-20 9:24 ` Sekhar Nori
0 siblings, 0 replies; 6+ messages in thread
From: Sekhar Nori @ 2013-06-20 9:24 UTC (permalink / raw)
To: linux-arm-kernel
On 6/14/2013 3:27 PM, Philip Avinash wrote:
> Fix the following build breakage and section mismatch error. This build
> break will comes only on removal of NEED_MACH_GPIO_H select option for
> ARCH-DAVINCI.
>
> arch/arm/mach-davinci/board-tnetv107x-evm.c:283:15: error:
> 'davinci_timer_init' undeclared here (not in a function)
> arch/arm/mach-davinci/board-tnetv107x-evm.c:285:15: error:
> 'davinci_init_late' undeclared here (not in a function)
> make[1]: *** [arch/arm/mach-davinci/board-tnetv107x-evm.o] Error 1
Can you note the section mismatch error you saw too?
Thanks,
Sekhar
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 2/2] gpio: gpio-tnetv107x: Fix bulid breakge
2013-06-14 9:57 [PATCH 0/2] Fix build breakage for tnetv107x platforms Philip Avinash
2013-06-14 9:57 ` [PATCH 1/2] ARM: davinci: fix for " Philip Avinash
@ 2013-06-14 9:57 ` Philip Avinash
2013-06-19 19:06 ` [PATCH 0/2] Fix build breakage for tnetv107x platforms Linus Walleij
2013-06-20 9:22 ` Sekhar Nori
3 siblings, 0 replies; 6+ messages in thread
From: Philip Avinash @ 2013-06-14 9:57 UTC (permalink / raw)
To: linux-arm-kernel
includes linux/io.h for fixing following build error. This build error
comes only after removing select option of NEED_MACH_GPIO_H for
ARCH_DAVINCI. linux/io.h is got included from mach/gpio-davinci.h on
selection of NEED_MACH_GPIO_H.
drivers/gpio/gpio-tnetv107x.c: In function 'tnetv107x_gpio_request':
drivers/gpio/gpio-tnetv107x.c:63:2: error: implicit declaration of
function '__raw_writel'
drivers/gpio/gpio-tnetv107x.c:63:2: error: implicit declaration of
function '__raw_readl'
drivers/gpio/gpio-tnetv107x.c: In function 'tnetv107x_gpio_setup':
drivers/gpio/gpio-tnetv107x.c:172:2: error: implicit declaration of
function 'ioremap'
drivers/gpio/gpio-tnetv107x.c:172:7: warning: assignment makes pointer
from integer without a cast
Signed-off-by: Philip Avinash <avinashphilipk@gmail.com>
---
drivers/gpio/gpio-tnetv107x.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/gpio/gpio-tnetv107x.c b/drivers/gpio/gpio-tnetv107x.c
index 3fa3e28..c7ed335 100644
--- a/drivers/gpio/gpio-tnetv107x.c
+++ b/drivers/gpio/gpio-tnetv107x.c
@@ -14,6 +14,7 @@
*/
#include <linux/kernel.h>
#include <linux/init.h>
+#include <linux/io.h>
#include <linux/gpio.h>
#include <mach/common.h>
--
1.7.9.5
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH 0/2] Fix build breakage for tnetv107x platforms
2013-06-14 9:57 [PATCH 0/2] Fix build breakage for tnetv107x platforms Philip Avinash
2013-06-14 9:57 ` [PATCH 1/2] ARM: davinci: fix for " Philip Avinash
2013-06-14 9:57 ` [PATCH 2/2] gpio: gpio-tnetv107x: Fix bulid breakge Philip Avinash
@ 2013-06-19 19:06 ` Linus Walleij
2013-06-20 9:22 ` Sekhar Nori
3 siblings, 0 replies; 6+ messages in thread
From: Linus Walleij @ 2013-06-19 19:06 UTC (permalink / raw)
To: linux-arm-kernel
On Fri, Jun 14, 2013 at 11:57 AM, Philip Avinash
<avinashphilipk@gmail.com> wrote:
> tnetv107x_defconfig build failing on removal selection of NEED_MACH_GPIO_H
> for ARCH_DAVINCI. This is due to header files inclusion from mach/gpio-davinci.h
> So this patch series fixes the build breakage on removal of NEED_MACH_GPIO_H.
>
> This patch series is based on Linux 3.10-rc4 and is availabel at [1] and dependency on 2.
>
> 1. https://github.com/avinashphilip/am335x_linux/tree/linux_davinci_v3.10_soc_gpio_v310-rc4
> 2. https://patchwork.kernel.org/patch/97853/
>
> Philip Avinash (2):
> ARM: davinci: fix for build breakage for tnetv107x platforms
> gpio: gpio-tnetv107x: Fix bulid breakge
The series:
Acked-by: Linus Walleij <linus.walleij@linaro.org>
Yours,
Linus Walleij
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 0/2] Fix build breakage for tnetv107x platforms
2013-06-14 9:57 [PATCH 0/2] Fix build breakage for tnetv107x platforms Philip Avinash
` (2 preceding siblings ...)
2013-06-19 19:06 ` [PATCH 0/2] Fix build breakage for tnetv107x platforms Linus Walleij
@ 2013-06-20 9:22 ` Sekhar Nori
3 siblings, 0 replies; 6+ messages in thread
From: Sekhar Nori @ 2013-06-20 9:22 UTC (permalink / raw)
To: linux-arm-kernel
On 6/14/2013 3:27 PM, Philip Avinash wrote:
> tnetv107x_defconfig build failing on removal selection of NEED_MACH_GPIO_H
> for ARCH_DAVINCI. This is due to header files inclusion from mach/gpio-davinci.h
> So this patch series fixes the build breakage on removal of NEED_MACH_GPIO_H.
So does this mean at the end of 7/7 of your "Convert GPIO Davinci to
platform driver" series, the build is broken?
If yes, then please include them when you repost that series before 7/7.
Having a patch break build is bad.
Thanks,
Sekhar
^ permalink raw reply [flat|nested] 6+ messages in thread