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

mediatek-filogic: add support for OpenWRT One #3446

Merged
merged 1 commit into from
Mar 9, 2025

Conversation

ambassador86
Copy link
Contributor

Specification:

  • MT7981 CPU using 2.4GHz and 5GHz WiFi (both AX)
  • 1GB RAM
  • 16MB NOR
  • 128MB NAND
  • 3 LEDs (red, green, blue, white)
  • 2 buttons (reset, user defined)
  • 1 2.5Gbit WAN port (Airoha EN8811h)
  • 1 1Gbit LAN ports
  • 1 single lane M.2 SSD slot
  • 1 mikroBus socket
  • externel HW WDT (25s refresh time)
  • i2c RTC (with battery backup)

Serial Interface

  • UBS-C CDC-ACM
  • 3 Pins GND, RX, TX
  • Settings: 115200, 8N1

Test Device: https://map.chemnitz.freifunk.net/#!v:m;n:2005b6ff13f0

  • Must be flashable from vendor firmware
  • Must support upgrade mechanism
    • Must have working sysupgrade
      • Must keep/forget configuration (sysupgrade [-n], firstboot)
    • Gluon profile name matches autoupdater image name
      (lua -e 'print(require("platform_info").get_image_name())') = openwrt-one
  • Reset button must return device into config mode
  • Primary MAC address should match address on device label (or packaging)
  • Wired network
    • should support all network ports on the device
    • must have correct port assignment (WAN/LAN)
      • 2.5G WAN / 1G LAN
  • Wireless network (if applicable)
    • Association with AP must be possible on all radios
    • Association with 802.11s mesh must work on all radios
    • AP+mesh mode must work in parallel on all radios
  • LED mapping
  • Docs:
    • Added Device to docs/user/supported_devices.rst

@github-actions github-actions bot added 3. topic: docs Topic: Documentation 3. topic: hardware Topic: Hardware Support labels Mar 5, 2025
Copy link
Member

@AiyionPrime AiyionPrime left a comment

Choose a reason for hiding this comment

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

What an annoyance :D
Just borrowed an OpenWrt this week to get this in, only to be beaten by @ambassador86 :D

Anyway, the logs looked good on my end.
The performance lies around 333Mbit/s with WireGuard and the MAC address matches.
https://hannover.freifunk.net/karte/#!/en/map/2005b6ff00bc

Thanks for your work :)

@AiyionPrime
Copy link
Member

Just gotta say, the USB C console port is one smooth experience I don't wanna miss on other devices anyore already.

Specification:
 - MT7981 CPU using 2.4GHz and 5GHz WiFi (both AX)
 - 1GB RAM
 - 16MB NOR
 - 128MB NAND
 - 3 LEDs (red, green, blue, white)
 - 2 buttons (reset, user defined)
 - 1 2.5Gbit WAN port (Airoha EN8811h)
 - 1 1Gbit LAN ports
 - 1 single lane M.2 SSD slot
 - 1 mikroBus socket
 - externel HW WDT (25s refresh time)
 - i2c RTC (with battery backup)

Serial Interface
 - UBS-C CDC-ACM
 - 3 Pins GND, RX, TX
 - Settings: 115200, 8N1
@AiyionPrime
Copy link
Member

I can rebuild this tonight, but'd be fine with the results and looking forward for some more experiences from deployments.
@blocktrron any further suggestions?
Or is this good to go in your opinion as well?

@rotanid
Copy link
Member

rotanid commented Mar 7, 2025

@ambassador86 thanks for the changes, your build seems to be failing though, please investigate

@ambassador86
Copy link
Contributor Author

Hello @rotanid
I already looked through the logs. The only thing i found so far: 'No space space left on device'
This sounds like a runner problem to me and is not related to the PR

@AiyionPrime
Copy link
Member

I can confirm, the build ran without issues for us yesterday on our machines.

@rotanid
Copy link
Member

rotanid commented Mar 8, 2025

i rebuilt again, anyone has an idea on how to deal/fix the problem in the CI ?

@blocktrron
Copy link
Member

GitHub provides runners with quite an opinionated package-set (See https://github.com/actions/runner-images/blob/main/images/ubuntu/Ubuntu2404-Readme.md )

Let me just push something so the CI removes these default packages and see if this fixes the build issue.

@herbetom
Copy link
Contributor

herbetom commented Mar 9, 2025

Is it possible that we just reached the point where the 21G of usable space on / of the runners isn't enough anymore?

@herbetom
Copy link
Contributor

herbetom commented Mar 9, 2025

What i find sometimes quite helpful for debugging is adding something like https://github.com/marketplace/actions/debugging-with-tmate to the workflow so that one can take a look at the state of the runner interactively. And also watch for example for disk space usage. But probably better to do that in the action run of someones local fork of this branch.

@blocktrron
Copy link
Member

@rotanid Test-build with this PR on top is here, as "allow changes by maintainers" was not checked for this PR: https://github.com/blocktrron/gluon/actions/runs/13743697613

Once this completed, feel free to merge #3448 before this and we should be set.

@blocktrron blocktrron merged commit 55d654f into freifunk-gluon:main Mar 9, 2025
12 of 16 checks passed
@rotanid
Copy link
Member

rotanid commented Mar 9, 2025

@blocktrron thanks

@ambassador86 ambassador86 deleted the openwrtone branch March 16, 2025 16:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. topic: docs Topic: Documentation 3. topic: hardware Topic: Hardware Support
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants