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

Pr neutis n5 support v2.19.0+rev1 #60

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

AD-Aleksandrov
Copy link

Hello!

I would like to add resin-os support for Emlid Neutis N5.
This is a SoM based on Allwinner H5, it has 8GB eMMC and 512MB DDR3, WiFi+BT (Ampak AP6212A).
Please visit neutis.io for the detailed information.

I met one issue - usb-modeswitch build error, attach a log below.
log-usb-modeswitch.zip

This neutis n5 build works okay, if I use the recipe usb-modeswitch from meta-openembedded.

I would be happy to hear your thoughts about these commits.

Thank you for your time!
Aleksandr

Aleksandr Aleksandrov added 4 commits October 11, 2018 12:40
- add neutis BSP layer as a git submodule
- add neutis BSP layers to bblayers sample
- resin-image.inc: appends for neutis n5
- add linux-sunxi bbappend to make neutis linux be
  compatible with resin kernel recipe
- add neutis-n5.coffee, based on nanopi-neo-air file

Signed-off-by: Aleksandr Aleksandrov <[email protected]>
Resin image can't be built with usb-modeswitch from resin layer.
gcc native should be lower than version 8, rust-llvm has an error
if you use gcc-8.

Signed-off-by: Aleksandr Aleksandrov <[email protected]>
- add brcm firmware for Ampak AP6212A
- copy all neutis supported device tree overlays to image

Signed-off-by: Aleksandr Aleksandrov <[email protected]>
- share resin u-boot patches with u-boot_2018.07
- don't apply 0001-Add-Resin-specific-boot-command.patch,
  boot commands are not compatible with arm64 u-boot
- add boot.cmd for neutis u-boot
- update dtb location in boot partition for neutis
- add resin-specific-env-integration-kconfig patch for v2018.07

Signed-off-by: Aleksandr Aleksandrov <[email protected]>
@resin-jenkins
Copy link

Can one of the admins verify this patch?

@floion
Copy link
Contributor

floion commented Oct 16, 2018

Hi @AD-Aleksandrov , thanks for your contribution.
@shaunmulligan do you think we should add this here as this PR does it, or think about a new emlid repository specifically for this board?
Adding this here would introduce a new BSP layer alongside meta-sunxi which might make the resin-allwinner repository feel a bit bloated.

@shaunmulligan
Copy link
Contributor

@floion its up to you, I don't think it will cause too much bloat and its nice to keep things centralised, but ultimately its up to you to do what you think is most maintainable. I am very excited to get the Neutis board supported, thanks @AD-Aleksandrov it looks like a great piece of hardware


# FILESEXTRAPATHS_prepend := "${THISDIR}/files:"

# SRC_URI += "file://0001-Add-Resin-specific-boot-command.patch"
Copy link
Contributor

Choose a reason for hiding this comment

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

Hi. What is the usb modeswitch error you get?

Copy link
Contributor

Choose a reason for hiding this comment

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

I see. what is the gcc version you have on your development machine?

Image-sun50i-h5-usbhost3.dtbo:/allwinner/overlay/sun50i-h5-usbhost3.dtbo \
Image-sun50i-h5-hdmi.dtbo:/allwinner/overlay/sun50i-h5-hdmi.dtbo \
Image-sun50i-h5-camera-status-okay.dtbo:/allwinner/overlay/sun50i-h5-camera-status-okay.dtbo \
Image-sun50i-h5-emlid-neutis-n5-devboard.dtb:/dtb/allwinner/sun50i-h5-emlid-neutis-n5-devboard.dtb \
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you amend the previous commit in which you introduced the copying of the overlays so that you do this change directly in that commit?

setenv bootargs ${resin_kernel_root} rootfstype=ext4 rootwait console=$console;
fatload ${resin_dev_type} ${resin_dev_index}:${resin_boot_part} ${resin_kernel_load_addr} Image;
fatload ${resin_dev_type} ${resin_dev_index}:${resin_boot_part} ${fdt_addr_r} dtb/${fdtfile};
booti ${resin_kernel_load_addr} - ${fdt_addr_r}
Copy link
Contributor

Choose a reason for hiding this comment

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

Why can't this be patched like for other boards? What is the problem with 64 bits u-boot?
Just as a side note, here's how we do it for other boards. https://github.com/balena-os/resin-artik710/blob/master/layers/meta-resin-artik710/recipes-bsp/u-boot/patches/0007-artik710_raptor.h-Board-specific-integration-with-re.patch (this also happens to be 64 bits)

@floion
Copy link
Contributor

floion commented Nov 11, 2018

Can you use these contributing guidelines in your PR please?
https://github.com/balena-os/resin-jetson-tx2#contributing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants