Skip to content

ambarella: s3l: ship majestic - #2785

Merged
widgetii merged 2 commits into
masterfrom
s3l-majestic
Nov 7, 2026
Merged

widgetii merged 2 commits into
masterfrom
s3l-majestic

Conversation

@widgetii

@widgetii widgetii commented Nov 7, 2026

Copy link
Copy Markdown
Member

Enables BR2_PACKAGE_MAJESTIC and BR2_PACKAGE_MAJESTIC_FONTS in s3l_lite_defconfig.

The s3l lite majestic build is now published to S3 as majestic.s3l.lite.master.tar.bz2, the name majestic.mk asks for with OPENIPC_SOC_FAMILY=s3l and OPENIPC_MAJESTIC=lite.

Verified:

  • make BOARD=s3l_lite defconfig resolves BR2_PACKAGE_MAJESTIC=y with JSON_C, LIBYAML and ZLIB selected.
  • The published binary is hard-float musl (/lib/ld-musl-armhf.so.1). Every library it links (json-c, libyaml, zlib, opus, ogg, mbedtls, libevent 2.2) is in the image.
  • That same binary runs on an S3Lm + AR0237 camera:
    • The chip and sensor are detected and 3A starts.
    • RTSP gives H.265 1920x1080 at 25 fps, and /image.jpg returns a snapshot.

The s3l lite build of majestic is now published
(majestic.s3l.lite.master.tar.bz2), so the image can carry it like
every other lite board, with its fonts. BR2_PACKAGE_MAJESTIC selects
json-c and libyaml; the rest of what the binary links (libevent,
mbedtls, opus, ogg, zlib) was already in the defconfig.
@qodo-free-for-open-source-projects

Copy link
Copy Markdown

PR Summary by Qodo

Enable Majestic and fonts in the Ambarella S3L lite image

⚙️ Configuration changes ✨ Enhancement 🕐 Less than 10 minutes

Grey Divider

AI Description

• Enable Majestic and its fonts in the S3L lite firmware image.
• Use the published S3L lite binary through the existing Majestic package.
Diagram

graph TD
  Board["S3L lite config"] --> Options["Package options"] --> Majestic["Majestic package"] --> Image["Firmware image"]
  Artifact["Published binary"] --> Majestic
  Options --> Fonts["Fonts package"] --> Image
Loading
High-Level Assessment

Board-level opt-in is appropriate: the existing Majestic package already resolves the S3L lite archive, while a global default would enable it on unrelated boards.

Files changed (1) +2 / -0

Other (1) +2 / -0
s3l_lite_defconfigShip Majestic and fonts on S3L lite +2/-0

Ship Majestic and fonts on S3L lite

• Enables the Majestic streamer and its fonts alongside the already-enabled web UI. The existing package configuration selects Majestic's json-c, libyaml, and zlib requirements.

br-ext-chip-ambarella/configs/s3l_lite_defconfig

@qodo-free-for-open-source-projects

Copy link
Copy Markdown

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

Tip of the day
💡 Did you know, you can hide the parts of a finding you never read, like the evidence or the agent prompt

More tips ↗ | Customize Qodo ↗ | Qodo docs ↗

Grey Divider

Qodo Logo

@widgetii
widgetii merged commit b08259f into master Nov 7, 2026
43 checks passed
@widgetii
widgetii deleted the s3l-majestic branch November 7, 2026 07:53
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.

1 participant