ll-20140607.0 - ftbfs on beaglebone

Bug #1328776 reported by Fathi Boudra
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Linaro Linux Baseline
Fix Committed
Undecided
Andrey Konovalov

Bug Description

https://ci.linaro.org/view/engineering-builds/job/package-and-publish-linux-linaro/hwpack=beaglebone,label=build/470/consoleText

  CC arch/arm/mm/proc-syms.o
/home/buildslave/workspace/package-and-publish-linux-linaro/kernel_build/test_build/linux/arch/arm/mach-highbank/highbank.c: In function ‘highbank_init’:
/home/buildslave/workspace/package-and-publish-linux-linaro/kernel_build/test_build/linux/arch/arm/mach-highbank/highbank.c:170:2: error: implicit declaration of function ‘pl320_ipc_register_notifier’ [-Werror=implicit-function-declaration]
  pl320_ipc_register_notifier(&hb_keys_nb);
  ^
cc1: some warnings being treated as errors
make[3]: *** [arch/arm/mach-highbank/highbank.o] Error 1
make[2]: *** [arch/arm/mach-highbank] Error 2

Fathi Boudra (fboudra)
Changed in linaro-linux-baseline:
status: New → Confirmed
assignee: nobody → Andrey Konovalov (andrey-konovalov)
Revision history for this message
Fathi Boudra (fboudra) wrote :
Revision history for this message
Andrey Konovalov (andrey-konovalov) wrote :

The compilation error was due to mailbox framework patches in the integration-linaro-vexpress topic - they broke multi_v7_defconfig build. beaglebone kernel configuration is based on multi_v7_defconfig, so the beaglebone build was broken.
In particular:
1) "mailbox: rename pl320-ipc specific mailbox.h" patch is the older version (which doesn't update arch/arm/mach-highbank/highbank.c). The newer, corrected version of this patch exists. Anyway, this is a single line change renaming the header file.
2) "mailbox: pl320: Introduce common API driver" patch removes pl320_ipc_register_notifier() function which is called from arch/arm/mach-highbank/highbank.c without changing the highbank.c.

The current solution by ARM LT is to drop the mailbox framework patches from their integration-linaro-vexpress topic branch (thus dropping cpu_freq support for Juno). This has been done, and the updated integration-linaro-vexpress topic has been merged into linux-linaro, the ll-20140612.0 tag.

Revision history for this message
Andrey Konovalov (andrey-konovalov) wrote :
Changed in linaro-linux-baseline:
status: Confirmed → Fix Committed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.