Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

projects/Amlogic: update kernel version and kernel config #2404

Merged
merged 2 commits into from
Jan 10, 2018

Conversation

kszaq
Copy link
Contributor

@kszaq kszaq commented Jan 9, 2018

This PR:

  • bumps amlogic-3.14 kernel version to latest linux-amlogic revision
  • updates kernel config for projects/Amlogic and removes an already applied patch

Reasons for kernel options are listed in commit message.

As a bonus, projects/Amlogic now compiles. Runtime tested on a S905X device.

@kszaq
Copy link
Contributor Author

kszaq commented Jan 9, 2018

Added WIP label to discuss kernel options for media drivers.

My suggestion is to add IR drivers (especially ir-meson) and not include other modules, e.g. DVB as these are outdated in 3.14 and users should download one of media addons instead.

@Ray-future
Copy link
Contributor

Currently we have no dvb addons upstreamed for AML. Even though they are outdated they might be more stable than those experimental media build ones. I'd like to keep the config similar to C2, WH and WP2 so those can be moved under the Amlogic project someday.

@spycat88
Copy link
Contributor

spycat88 commented Jan 10, 2018

I agree with you @kszaq media drivers should not be included, they should be handled by crazycat, I also agree with @Raybuntu the config should be similar to C2/Wetek to support potentially merging them in the future.

On a side note, none of the dvb supporting code has been commited/PR'd yet

edit 1: I just noticed you disabled CONFIG_VM_EVENT_COUNTERS I think @MilhouseVH enabled this previously so he may want that retained

edit 2: can also confirm config now gives a booting kernel, tested on LePotato

edit 3: I couldn't get a U9-H (S912) to boot with this with #2400

@kszaq
Copy link
Contributor Author

kszaq commented Jan 10, 2018

Thank you @adamg88 for your review.

I will re-enable VM_EVENT_COUNTERS, you are correct that they were enabled in all projects.

I will check what is missing to support S912.

Re media drivers, I think it will be a good idea to include RC drivers, e.g. meson-ir to let users use their remotes without having to install addon drivers. Everything else, i.e. DVB should be installed from media_build(s) as 3.14 drivers are outdated and many of them unusable.

@kszaq
Copy link
Contributor Author

kszaq commented Jan 10, 2018

@adamg88 Have you compiled a device tree for U9 specifically for this kernel? Unfortunately device trees changed again and you can't use these: https://kszaq.libreelec.tv/s912/8.2/device_trees/

@spycat88
Copy link
Contributor

@kszaq no I used the one from the link above, I will test again later and let you know thanks

Copy link
Contributor

@spycat88 spycat88 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@kszaq I just tried the updated DTB and the S912 box booted with #2400

@kszaq
Copy link
Contributor Author

kszaq commented Jan 10, 2018

Thank you @adamg88 for confirming that it works. I have updated kernel config re-enabling VM_EVENT_COUNTERS and rc drivers. Will test is and should be good to go.

@kszaq
Copy link
Contributor Author

kszaq commented Jan 10, 2018

This PR is good to go if there are no objections.

@adamg88 I uploaded all device trees for the updated kernel: https://kszaq.libreelec.tv/s905/9.0/device_trees/

@Ray-future
Copy link
Contributor

Imho it would be good to have some basic media/dvb support like we have on C2, WH and WP2. Mediabuild will always be experimental. For instance I'm using Sundtek DVB devices and therefore don't need any mediabuild.
So either add those now or in a separate PR before we enable crazycat for S905 later. I can also do this in my crazycat PR if that's better?

@spycat88
Copy link
Contributor

Ray I think it would be better to leave it till it's enabled in crazycat, community builds can fill the void until then seeing as it will be some time before there is an Official LE release for Aml.

@kszaq
Copy link
Contributor Author

kszaq commented Jan 10, 2018

Sure, I can align media config to WeTek/C2.

Most of changes are related to Nougat kernel update and aligning config to other projects.

Some of the options are required to add S912-on-libhybris:
* CONFIG_HMP_VARIABLE_SCALE
* CONFIG_NR_CPUS
* CONFIG_ARM_BIG_LITTLE_CPUFREQ
* CONFIG_FB_OSD_SUPPORT_SYNC_FENCE
* CONFIG_FB_OSD2_CURSOR
* CONFIG_ASHMEM
* CONFIG_SW_SYNC
* CONFIG_PSTORE
@kszaq
Copy link
Contributor Author

kszaq commented Jan 10, 2018

Media config copied from Odroid_C2 and redacted by running make menuconfig.

Builds and runs.

Copy link
Contributor

@Ray-future Ray-future left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks.

@Ray-future Ray-future merged commit 883f1ba into LibreELEC:master Jan 10, 2018
@kszaq kszaq deleted the amlogic_kernel_update branch January 10, 2018 20:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants