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
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 <[email protected]>
---
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);
Hi,
On Tue, Jun 18, 2013 at 9:19 AM, Naresh Kamboju
<[email protected]> 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
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 <[email protected]> wrote:
> Hi,
>
>
> On Tue, Jun 18, 2013 at 9:19 AM, Naresh Kamboju
> <[email protected]> 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