search

edison-fw / meta-intel-edison

Here is the meta-intel-edison that builds, tries to stay up to date. Master is based on Yocto Poky Gatesgarth LTS 5.10.yy vanilla kernels. It builds a 32bit kernel (Gatesgarth branch 64bit) with ACPI enabled and corresponding rootfs. Telegram group: https://t.me/IntelEdison Web-site:
https://edison-fw.github.io/meta-intel-edison/
MIT License
60 stars 38 forks source link

Add proper initramfs image generation #9

Closed alext-mkrs closed 6 years ago

alext-mkrs commented 6 years ago

As we've discussed in the other thread, I have to test this first, however as this would take me a while (need to do the u-boot build and update on my Edison first) and it actually looks ok, by inspecting the produced files - I figured I'll send the PR in parallel, in case you'd be willing/able to test this sooner. If not, no worries, I'll post a comment when I confirm this works.

The separate config is no longer needed and I've moved the relevant pieces of it to the machine config, as per Yocto manual recommendations. It actually says to put one piece into local.conf and another - into machine conf, but IMHO they could be naturally stored as a single logical piece in the machine one.

I'll also take care of updating wiki build instructions when/if this is merged.

htot commented 6 years ago

Today I did a clean build and this builds nicely. Will install tomorrow and test.

htot commented 6 years ago

When I build the sdk I get hit by this bug: https://bugzilla.yoctoproject.org/show_bug.cgi?id=12335. Not sure if this is caused by this patch, or by the clean build.

htot commented 6 years ago

When I overlay the recipe with the newer one from rocko the error goes away. What is your opinion, copy new recipe into meta-intel-edison-distro/recipes-core/glibc, let setup apply a patch, or upgrade to rocko?

alext-mkrs commented 6 years ago

I'd go with pulling this to meta-intel-edison:pyro for now - it's IMHO better to do all the tweaks and cleanups like this one with initramfs generation on pyro, which is stable, than to alter one more variable by moving straight to rocko. This is unlikely to be caused by this PR - as it doesn't really introduce anything new, just moves that standalone conf file into proper place (so you'd have had it previously as well). Besides, that -native package is really outside of the build path affected here. On my Ubuntu 16.04 it worked fine, by the way - so those bitbake errors about untested distro are not entirely meaningless :)

alext-mkrs commented 6 years ago

And BTW, I'm going to test this binary today - the week turned out busier than I anticipated, so I've got time to do this only over the weekend. I have u-boot built and I've got through update instructions, so I should report back shortly.

htot commented 6 years ago

The image 4.14 kernel built using this patch (with ftrace enabled etc) seems to be working very well for me.

alext-mkrs commented 6 years ago

Oh, that's wonderful, thanks for confirming. Not that I expected anything else than that, but nothing substitutes a real-life test :)

alext-mkrs commented 6 years ago

Thanks for merging. Just to round this up - on pyro and kernel 4.13 it works fine as well, just tested.