Repository navigation
85h50ai: build ultimate, for the CPU person detector Follow needs - #202
Merged
Merged
Conversation
The 85H50AI carries 16 MB of NOR (the lab unit's mtd table adds up to 0x1000000), not the 8 MB the profile and README said, so it can take the ultimate image -- and only ultimate ships the CPU person detector (cpu-models) that object detection and PTZ Follow run on, on a chip without an NPU. The defconfig is firmware's hi3516ev300_ultimate_defconfig verbatim, as the lite one was firmware's lite defconfig verbatim; it already keeps MAJESTIC_AF and MOTORS. The upgrade URL follows the new asset name. The lite asset stays on 'latest', frozen, for any camera still pointing at it; moving one across needs a repartition, which sysupgrade does not do.
openipc-ai
requested review from
ZigFisher,
cronyx,
flyrouter,
viktorxda and
ystinia
as code owners
October 14, 2026 13:06
Contributor
Author
|
/review |
Code Review by Qodo
1.
|
![qodo-free-for-open-source-projects[bot]](https://avatars.githubusercontent.com/in/1068000?s=60&v=4){kind=small}
The customizer runs once, on a camera with no settings, so one moved here from the lite profile with its settings kept keeps the lite address unless the move sets it. ipctool's layout bundle does, with -e upgrade=; on the lab unit fw_printenv upgrade showed the ultimate asset right after the move.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The XiongMai 85H50AI carries 16 MB of NOR, not the 8 MB the profile and README said. A lab unit's mtd table adds up to exactly 0x1000000, and its bootloader reports a W25Q128. So it can take the ultimate image, and only ultimate ships the CPU person detector (
cpu-models). On a chip with no NPU, object detection and PTZ Follow both run on that detector.Changes
devices/hi3516ev300_lite_xm-85h50aibecomesdevices/hi3516ev300_ultimate_xm-85h50ai. The defconfig is firmware'shi3516ev300_ultimate_defconfigverbatim, as the lite one was firmware's lite defconfig verbatim. It already keepsMAJESTIC_AFandMOTORS, and addsCPU_MODELS.<soc>_<flavour>.listnaming.customizer.shsets theupgradeURL to the new asset name.NOR_16M.Cameras already on lite
The old
hi3516ev300_lite_xm-85h50ai-nor.tgzstays onlatest, frozen. Moving a camera across needs a repartition (kernel 2 MB → 3 MB, rootfs 5 MB → 10 MB at a new offset), andsysupgradedoesn't do that. OpenIPC/ipctool'supgrade --layoutwork does it live and keeps the settings. A lab 85H50AI was moved that way to an image built from this branch.Tested
builder.sh hi3516ev300_ultimate_xm-85h50ai: uImage 2.0 MB, rootfs 7.6 MB (headroom: kernel 2090 KB, rootfs 8580 KB).person.pdnetis present, and people detection is offered.