These are chat archives for agherzan/meta-raspberrypi

1st
Mar 2016
Leon Anavi
@leon-anavi
Mar 01 2016 12:15
hi
Andrei Gherzan
@agherzan
Mar 01 2016 12:16
Hello @leon-anavi
Jonathan Liu
@net147
Mar 01 2016 12:26
hi
Andrei Gherzan
@agherzan
Mar 01 2016 12:27
@net147 Cheers mate. It seems that we still have a problem with the rpi-mkimage path.
But on dizzy.
Jonathan Liu
@net147
Mar 01 2016 12:29
what's the issue?
dabrain34
@dabrain34
Mar 01 2016 12:29
hello
i'm steven
Andrei Gherzan
@agherzan
Mar 01 2016 12:30
It fails for Stefan on dizzy. Even though I'm a little confused why.
export libexecdir = "${libdir}/${BPN}"
Jonathan Liu
@net147
Mar 01 2016 12:30
what's the error message?
dabrain34
@dabrain34
Mar 01 2016 12:30
the mkknlnimage is not copied in the right place
Andrei Gherzan
@agherzan
Mar 01 2016 12:31
What is the place where kernel is looking for the tool?
Checking the dizzy bitbake.conf it seems that it should work.
dabrain34
@dabrain34
Mar 01 2016 12:31
its in sysroot/usr/lib/rpi-mkimage/mkknlimg
Andrei Gherzan
@agherzan
Mar 01 2016 12:33
Correct = ${libdir}/${BPN}
If so, where is kernel looking for mkknlimg?
Jonathan Liu
@net147
Mar 01 2016 12:37
hmm....
Jonathan Liu
@net147
Mar 01 2016 12:46
I think it might have to do with expansion of BPN in dizzy branch
dabrain34
@dabrain34
Mar 01 2016 12:46
i'm building removing my local patch
Jonathan Liu
@net147
Mar 01 2016 12:49
can you do the following?
bitbake -e rpi-mkimage-native | grep '^STAGING_LIBEXECDIR_NATIVE'
bitbake -e linux-raspberrypi | grep '^STAGING_LIBEXECDIR_NATIVE'
dabrain34
@dabrain34
Mar 01 2016 12:51
ok
first is sysroots/x86_64-linux/usr/lib/rpi-mkimage
second is sysroots/x86_64-linux/usr/lib/linux-raspberrypi
Jonathan Liu
@net147
Mar 01 2016 12:54
so it is installing in right place but kernel is looking in wrong place
because BPN is different for kernel than rpi-mkimage
dabrain34
@dabrain34
Mar 01 2016 12:54
yes
because poky master branch (at least) changed the STAGING_LIBEXECDIR_NATIVE value
to libexec instead of BPN
Jonathan Liu
@net147
Mar 01 2016 13:07
we could perhaps have a variable defined in an include file like RPI_MKIMAGE_PATH
it could set it to the STAGING_LIBEXECDIR_NATIVE value for rpi-mkimage
dabrain34
@dabrain34
Mar 01 2016 13:10
yeah but it wont be poky wide
Andrei Gherzan
@agherzan
Mar 01 2016 13:12
What is the difference in between daisy and dizzy in regards to this issue. Both should fail the same. As they both use BPN which is recipe specific
dabrain34
@dabrain34
Mar 01 2016 13:13
i guess yes
Jonathan Liu
@net147
Mar 01 2016 13:13
I don't think there is any difference
dabrain34
@dabrain34
Mar 01 2016 13:14
why dont we use libdir/rpi-mkimage in kernel recipe ?
Andrei Gherzan
@agherzan
Mar 01 2016 13:14
We need to define our own path and use it as it is. Don't rely on anything
dabrain34
@dabrain34
Mar 01 2016 13:14
yes
we install the tools in libdir/rpi-mkimage
Jonathan Liu
@net147
Mar 01 2016 13:15
sounds like a plan
Andrei Gherzan
@agherzan
Mar 01 2016 13:15
It is not a library per se. So I would avoid having it in libdir.
Jonathan Liu
@net147
Mar 01 2016 13:15
can we make it use the new libexec path even for older branches?
Andrei Gherzan
@agherzan
Mar 01 2016 13:15
I would go for the classic STAGING_BINDIR_NATIVE
dabrain34
@dabrain34
Mar 01 2016 13:20
yes
i would prefer this solution less confusing
Jonathan Liu
@net147
Mar 01 2016 13:21
ok
dabrain34
@dabrain34
Mar 01 2016 13:22
can you do it liu ?
Jonathan Liu
@net147
Mar 01 2016 13:23
I am testing it
Andrei Gherzan
@agherzan
Mar 01 2016 13:23
Thanks guys for getting on this.
dabrain34
@dabrain34
Mar 01 2016 13:24
i'm trying too
i'm using bindir
and STAGING_BINDIR_NATIVE
dabrain34
@dabrain34
Mar 01 2016 13:28
yeah that's mine too
@agherzan do you think we can merge it quickly ?
Andrei Gherzan
@agherzan
Mar 01 2016 13:31
yup - send it ml
Jonathan Liu
@net147
Mar 01 2016 13:33
sending...
done
@dabrain34 do the posted patches look okay to you?
dabrain34
@dabrain34
Mar 01 2016 13:37
@net147 i'm sorry but you missed initramfs
Jonathan Liu
@net147
Mar 01 2016 13:37
oh that wasn't there last time I looked
dabrain34
@dabrain34
Mar 01 2016 13:38
yeah
i just happened yeasterday because of me :smile:
Jonathan Liu
@net147
Mar 01 2016 13:39
sent patch for that too
all good now?
dabrain34
@dabrain34
Mar 01 2016 13:42
I guess @agherzan what do you think ?
Andrei Gherzan
@agherzan
Mar 01 2016 13:42
There should be 2 changes in the kernel inc file.
dabrain34
@dabrain34
Mar 01 2016 13:42
yeah
Andrei Gherzan
@agherzan
Mar 01 2016 13:42
I don't see the second one in this file
@net147 please send a v2 with squashed uboot patches and change in both functions for kernel inc file.
Jonathan Liu
@net147
Mar 01 2016 13:45
which second change? I can't see it...
Andrei Gherzan
@agherzan
Mar 01 2016 13:46
line 77 and line 67
both need new paths
(git pull first / rebase)
Jonathan Liu
@net147
Mar 01 2016 13:48
should I separate the 2 u-boot changes
since one is for u-boot-rpi, other is for u-boot
Andrei Gherzan
@agherzan
Mar 01 2016 13:49
uh ok - didn't see that
But I don't have a u-boot recipe...
are you on current master?
Jonathan Liu
@net147
Mar 01 2016 13:50
yes I am on master
Jonathan Liu
@net147
Mar 01 2016 13:52
oh right... I still have non-upstreamed changes for u-boot, qt5, omx...
Andrei Gherzan
@agherzan
Mar 01 2016 13:54
That's why I was asking if you are on current master HEAD. It's fine. Just push a V2.
Jonathan Liu
@net147
Mar 01 2016 13:54
pushed
excluded my u-boot bbappend
would be great if there was a patchwork for meta-raspberrypi. then I could mark patches as superseded...
Andrei Gherzan
@agherzan
Mar 01 2016 13:57
1/2?
Shouldn't be 3 patches?
Jonathan Liu
@net147
Mar 01 2016 13:57
why 3?
Andrei Gherzan
@agherzan
Mar 01 2016 13:57
one on rpi-mkimage, one one kernel and one on uboot-rpi.
Jonathan Liu
@net147
Mar 01 2016 13:58
duh ok
V3 pushed
as clock approaches 1 am, mind dulls
Andrei Gherzan
@agherzan
Mar 01 2016 13:59
That's it. Go to sleep now :)
I'll hit a kernel build and merge.
Thanks @net147 and @dabrain34
Jonathan Liu
@net147
Mar 01 2016 14:00
thanks. sorry for all the noise.
Andrei Gherzan
@agherzan
Mar 01 2016 14:00
We used a filter. :)
Jonathan Liu
@net147
Mar 01 2016 14:01
later
dabrain34
@dabrain34
Mar 01 2016 14:10
you are welcome, thx a lot @net147 and @agherzan
Andrei Gherzan
@agherzan
Mar 01 2016 14:22
Merged to master. Enjoy.
dabrain34
@dabrain34
Mar 01 2016 14:23
gool !
:)
dabrain34
@dabrain34
Mar 01 2016 14:28
looks good on my side :)