10 10
zador.blood.stained

testers wanted Testers wanted: sunxi Device Tree overlays

Recommended Posts

1 hour ago, technik007_cz said:

Kernel version 4.19.13-sunxi does not accept param_w1_pin=

As I said in the other thread, it is not the kernel that manage those DT parameters, it is u-boot responsible to load them, so, we need to see in early u-boot log (before kernel started) if the fixup.scr apply succeeded or not .

You need USB-TTL Serial dongle to watch those early logs ...

Share this post


Link to post
Share on other sites

I had submitted a PR last year that was committed to the tree at https://github.com/armbian/sunxi-DT-overlays but this doesn't seem to have made it to the mainline nor is it included in any pine64 pre-built kernels.  It is a very simple one-line correction from 

 

 

For now I've been using a local user patch when rebuilding kernels.  Would I need to submit a separate PR for the linux mainline as well?

 

Or would a PR to add a bundled patch in the armbian build tree be better/preferred?

 

Share this post


Link to post
Share on other sites
28 minutes ago, windysea said:

but this doesn't seem to have made it to the mainline

The sunxi-DT-overlays is some kind of backup repository, but the real place that you fix should be in the patches, in the present case here :

https://github.com/armbian/build/blob/master/patch/kernel/sunxi-dev/general-sunxi-overlays.patch#L5071

Feel free to submit another PR for that file, we will merge it quickly ...

Share this post


Link to post
Share on other sites

Thanks!

 

I've submitted the a PR with two separate commits, one each for -dev and -next, with the exact same change.  I didn't know if you would accept PRs on -next and meant to create two PRs but the second commit was merged with the first PR that had already been submitted. . .

Share this post


Link to post
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now
10 10