* linux-next: origen build failed at drivers/pinctrl/pinctrl-exynos.c
@ 2013-06-18 16:19 Naresh Kamboju
2013-06-18 16:58 ` Arnd Bergmann
2013-06-18 17:03 ` Olof Johansson
0 siblings, 2 replies; 4+ messages in thread
From: Naresh Kamboju @ 2013-06-18 16:19 UTC (permalink / raw)
To: linux-arm-kernel
Hi,
linux-next branch failed to build origen build
defconfig: exynos_defconfig
full build log:
----------------
https://ci.linaro.org/jenkins/job/linux-next/hwpack=origen,label=kernel_cloud/72/console
build error log:
--------------------
05:02:12 /mnt/ci_build/workspace/linux-next/hwpack/origen/label/kernel_cloud/drivers/pinctrl/pinctrl-exynos.c:
In function ?exynos_wkup_irq_mask?:
05:02:12 /mnt/ci_build/workspace/linux-next/hwpack/origen/label/kernel_cloud/drivers/pinctrl/pinctrl-exynos.c:299:3:
error: implicit declaration of function ?exynos_wkup_irq_ack?
[-Werror=implicit-function-declaration]
05:02:12 exynos_wkup_irq_ack(irqd);
05:02:12 ^
05:02:12 /mnt/ci_build/workspace/linux-next/hwpack/origen/label/kernel_cloud/drivers/pinctrl/pinctrl-exynos.c:
At top level:
05:02:12 /mnt/ci_build/workspace/linux-next/hwpack/origen/label/kernel_cloud/drivers/pinctrl/pinctrl-exynos.c:310:13:
warning: conflicting types for ?exynos_wkup_irq_ack? [enabled by
default]
05:02:12 static void exynos_wkup_irq_ack(struct irq_data *irqd)
05:02:12 ^
05:02:12 /mnt/ci_build/workspace/linux-next/hwpack/origen/label/kernel_cloud/drivers/pinctrl/pinctrl-exynos.c:310:13:
error: static declaration of ?exynos_wkup_irq_ack? follows non-static
declaration
05:02:12 /mnt/ci_build/workspace/linux-next/hwpack/origen/label/kernel_cloud/drivers/pinctrl/pinctrl-exynos.c:299:3:
note: previous implicit declaration of ?exynos_wkup_irq_ack? was here
05:02:12 exynos_wkup_irq_ack(irqd);
05:02:12 ^
05:02:12 cc1: some warnings being treated as errors
05:02:12 make[3]: *** [drivers/pinctrl/pinctrl-exynos.o] Error 1
05:02:12 make[2]: *** [drivers/pinctrl] Error 2
Best regards
Naresh kamboju
^ permalink raw reply [flat|nested] 4+ messages in thread
* linux-next: origen build failed at drivers/pinctrl/pinctrl-exynos.c
2013-06-18 16:19 linux-next: origen build failed at drivers/pinctrl/pinctrl-exynos.c Naresh Kamboju
@ 2013-06-18 16:58 ` Arnd Bergmann
2013-06-18 17:03 ` Olof Johansson
1 sibling, 0 replies; 4+ messages in thread
From: Arnd Bergmann @ 2013-06-18 16:58 UTC (permalink / raw)
To: linux-arm-kernel
On Tuesday 18 June 2013, Naresh Kamboju wrote:
> Hi,
>
> linux-next branch failed to build origen build
>
> defconfig: exynos_defconfig
>
> full build log:
> ----------------
> https://ci.linaro.org/jenkins/job/linux-next/hwpack=origen,label=kernel_cloud/72/console
>
> build error log:
> --------------------
> 05:02:12 /mnt/ci_build/workspace/linux-next/hwpack/origen/label/kernel_cloud/drivers/pinctrl/pinctrl-exynos.c:
> In function ?exynos_wkup_irq_mask?:
> 05:02:12 /mnt/ci_build/workspace/linux-next/hwpack/origen/label/kernel_cloud/drivers/pinctrl/pinctrl-exynos.c:299:3:
> error: implicit declaration of function ?exynos_wkup_irq_ack?
> [-Werror=implicit-function-declaration]
> 05:02:12 exynos_wkup_irq_ack(irqd);
> 05:02:12 ^
> 05:02:12 /mnt/ci_build/workspace/linux-next/hwpack/origen/label/kernel_cloud/drivers/pinctrl/pinctrl-exynos.c:
> At top level:
> 05:02:12 /mnt/ci_build/workspace/linux-next/hwpack/origen/label/kernel_cloud/drivers/pinctrl/pinctrl-exynos.c:310:13:
> warning: conflicting types for ?exynos_wkup_irq_ack? [enabled by
> default]
> 05:02:12 static void exynos_wkup_irq_ack(struct irq_data *irqd)
> 05:02:12 ^
> 05:02:12 /mnt/ci_build/workspace/linux-next/hwpack/origen/label/kernel_cloud/drivers/pinctrl/pinctrl-exynos.c:310:13:
> error: static declaration of ?exynos_wkup_irq_ack? follows non-static
> declaration
> 05:02:12 /mnt/ci_build/workspace/linux-next/hwpack/origen/label/kernel_cloud/drivers/pinctrl/pinctrl-exynos.c:299:3:
> note: previous implicit declaration of ?exynos_wkup_irq_ack? was here
> 05:02:12 exynos_wkup_irq_ack(irqd);
> 05:02:12 ^
> 05:02:12 cc1: some warnings being treated as errors
> 05:02:12 make[3]: *** [drivers/pinctrl/pinctrl-exynos.o] Error 1
> 05:02:12 make[2]: *** [drivers/pinctrl] Error 2
>
> Best regards
> Naresh kamboju
>
I got the same thing and used this patch to fix it up.
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
---
Linus, could you fold that into Doug's patch?
diff --git a/drivers/pinctrl/pinctrl-exynos.c b/drivers/pinctrl/pinctrl-exynos.c
index ef75321..00cd89a 100644
--- a/drivers/pinctrl/pinctrl-exynos.c
+++ b/drivers/pinctrl/pinctrl-exynos.c
@@ -279,6 +279,15 @@ err_domains:
return ret;
}
+static void exynos_wkup_irq_ack(struct irq_data *irqd)
+{
+ struct samsung_pin_bank *b = irq_data_get_irq_chip_data(irqd);
+ struct samsung_pinctrl_drv_data *d = b->drvdata;
+ unsigned long pend = d->ctrl->weint_pend + b->eint_offset;
+
+ writel(1 << irqd->hwirq, d->virt_base + pend);
+}
+
static void exynos_wkup_irq_mask(struct irq_data *irqd)
{
struct samsung_pin_bank *b = irq_data_get_irq_chip_data(irqd);
@@ -296,15 +305,6 @@ static void exynos_wkup_irq_mask(struct irq_data *irqd)
spin_unlock_irqrestore(&b->slock, flags);
}
-static void exynos_wkup_irq_ack(struct irq_data *irqd)
-{
- struct samsung_pin_bank *b = irq_data_get_irq_chip_data(irqd);
- struct samsung_pinctrl_drv_data *d = b->drvdata;
- unsigned long pend = d->ctrl->weint_pend + b->eint_offset;
-
- writel(1 << irqd->hwirq, d->virt_base + pend);
-}
-
static void exynos_wkup_irq_unmask(struct irq_data *irqd)
{
struct samsung_pin_bank *b = irq_data_get_irq_chip_data(irqd);
^ permalink raw reply related [flat|nested] 4+ messages in thread* linux-next: origen build failed at drivers/pinctrl/pinctrl-exynos.c
2013-06-18 16:19 linux-next: origen build failed at drivers/pinctrl/pinctrl-exynos.c Naresh Kamboju
2013-06-18 16:58 ` Arnd Bergmann
@ 2013-06-18 17:03 ` Olof Johansson
2013-06-19 6:27 ` Naresh Kamboju
1 sibling, 1 reply; 4+ messages in thread
From: Olof Johansson @ 2013-06-18 17:03 UTC (permalink / raw)
To: linux-arm-kernel
Hi,
On Tue, Jun 18, 2013 at 9:19 AM, Naresh Kamboju
<naresh.kamboju@linaro.org> wrote:
> Hi,
>
> linux-next branch failed to build origen build
>
> defconfig: exynos_defconfig
This was broken yesterday and fixed today. Getting linux-next build
error reports that are 24+ hours old isn't very useful, unfortunately.
It'd also be convenient if you supplied the "git describe" version
number of the source tree in these reports, so it'd be trivial to find
out by glancing at the report, instead of going to the build log.
(Besides that, thanks for the bug report!)
-Olof
^ permalink raw reply [flat|nested] 4+ messages in thread
* linux-next: origen build failed at drivers/pinctrl/pinctrl-exynos.c
2013-06-18 17:03 ` Olof Johansson
@ 2013-06-19 6:27 ` Naresh Kamboju
0 siblings, 0 replies; 4+ messages in thread
From: Naresh Kamboju @ 2013-06-19 6:27 UTC (permalink / raw)
To: linux-arm-kernel
Hi Olof,
I will give you Kernel head commit ID of linux-next.
Thanks for the feedback.
Best regards
Naresh Kamboju
On 18 June 2013 22:33, Olof Johansson <olof@lixom.net> wrote:
> Hi,
>
>
> On Tue, Jun 18, 2013 at 9:19 AM, Naresh Kamboju
> <naresh.kamboju@linaro.org> wrote:
>> Hi,
>>
>> linux-next branch failed to build origen build
>>
>> defconfig: exynos_defconfig
>
> This was broken yesterday and fixed today. Getting linux-next build
> error reports that are 24+ hours old isn't very useful, unfortunately.
>
> It'd also be convenient if you supplied the "git describe" version
> number of the source tree in these reports, so it'd be trivial to find
> out by glancing at the report, instead of going to the build log.
>
> (Besides that, thanks for the bug report!)
>
>
> -Olof
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2013-06-19 6:27 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-06-18 16:19 linux-next: origen build failed at drivers/pinctrl/pinctrl-exynos.c Naresh Kamboju
2013-06-18 16:58 ` Arnd Bergmann
2013-06-18 17:03 ` Olof Johansson
2013-06-19 6:27 ` Naresh Kamboju
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox