Armbian 5.11 fails to build kernel

chradev · May 25, 2016

Hi to All,

I have just tried to compile Armbian after a few days pause and get error in:

â”‚ LD [M] drivers/usb/serial/usbserial.o â”‚
â”‚ LD drivers/usb/built-in.o â”‚
â”‚ LD drivers/built-in.o 

[ error ] ERROR in function compile_kernel [ common.sh:188 ]
[ error ] Kernel was not built [ @host ]

I test different ways including previous kernel branch (4.5.4).

Even starting from scratch was with no success.

Where could be a problem?

Best regards

Chris

Igor · May 25, 2016

You have some weird user settings?

- I made complete rebuild yesterday

- I just compile 4.5.5

Host: Xenial with 4.4.0-22-generic

Compiler: arm-linux-gnueabihf-gcc (Ubuntu/Linaro 5.3.1-14ubuntu2) 5.3.1 20160413

chradev · May 25, 2016

Hi Igor,

You have some weird user settings?

- I made complete rebuild yesterday

- I just compile 4.5.5

Host: Xenial with 4.4.0-22-generic

No, I have made a small change adding a patch to enable OneWire in Lime2 dts following http://linux-sunxi.org/1-Wire.

Try to compile 4.5.4 fails in the same way.

chradev · May 25, 2016

Hi Igor,

After a new try to build image a big amount of changes were added and new error is printed:

[ o.k. ] Preparing [ host ]
findmnt: unknown column: AVAIL
/data/armbian-cust/lib/general.sh: line 406: / 1024 / 1024 / 1024 : syntax error: operand expected (error token is "/ 1024 / 1024 / 1024 ")

The process is continued but stops again on the same place while compiling the kernel.

Could you point me a way to go back to the repository branch where Armbian build is possible?

Best regards

Chris

zador.blood.stained · May 25, 2016

[ o.k. ] Preparing [ host ]
findmnt: unknown column: AVAIL
/data/armbian-cust/lib/general.sh: line 406: / 1024 / 1024 / 1024 : syntax error: operand expected (error token is "/ 1024 / 1024 / 1024 ")

This shouldn't affect anything - this is free space checking that works on Xenial, I'll add error silencing in case this is running on Trusty later.

If you are experiencing errors with building kernel, then your patch is probably causing this. Wiki info may be outdated. You can add "PROGRESS_LOG_TO_FILE=yes" to compile.sh and then find exact error message in compilation log.

chradev · May 25, 2016

Thanks Zador,

You are right - disabling the patch makes kernel build possible.

It is strange because the patch is very simple and adds a few lines to arch/arm/boot/dts/sun7i-a20-olinuxino-lime2.dts.

Probably the problem comes from interference with other changes made by other patches to the same file.

I have added last changes into the file already patched by other patches and run 'git diff file > file.patch'.

After that I split the content to different files to be able to enable and disable changes separately.

Is there some rules or good practice how to split changes in one file into different patches?

Best regards

Chris

zador.blood.stained · May 25, 2016

Normally patch is smart enough to apply patches to changed source files.

If you want to split patch, you can duplicate file and remove unneeded hunks from different files while keeping common header.

Please post your 1-wire activation patch here (in code block), especially if it doesn't look like this.

Edit: changed link to diff format description

chradev · May 25, 2016

Thanks Zador,

I find an error in the first patch I have used so make a series of tests because it really interference with other changes.

The final patch I have used is:

diff --git a/arch/arm/boot/dts/sun7i-a20-olinuxino-lime2.dts b/arch/arm/boot/dts/sun7i-a20-olinuxino-lime2.dts
index d5c796c..93f3f62 100644
--- a/arch/arm/boot/dts/sun7i-a20-olinuxino-lime2.dts
+++ b/arch/arm/boot/dts/sun7i-a20-olinuxino-lime2.dts
@@ -235,6 +235,20 @@
                allwinner,drive = <SUN4I_PINCTRL_10_MA>;
                allwinner,pull = <SUN4I_PINCTRL_NO_PULL>;
        };
+
+       w1_pin: w1_pin@0 {
+               allwinner,pins = "PC21";
+               allwinner,function = "gpio_in";
+               allwinner,drive = <SUN4I_PINCTRL_10_MA>;
+               allwinner,pull = <SUN4I_PINCTRL_PULL_UP>;
+       };
+};
+
+&onewire_device {
+       compatible = "w1-gpio";
+       gpios = <&pio 2 21 GPIO_ACTIVE_HIGH>; /* PC21 */
+       pinctrl-names = "default";
+       pinctrl-0 = <&w1_pin>;
 };
 
 &reg_ahci_5v {

It is applied as a first patch to the fresh file from the repository but it is rejected.

Best regards

Chris