hisilicon-opensdk: bump 073e6e8 → 31a5e9f0 (cv500 magenta-cast + ring_depth knob)#2156
Merged
Conversation
Pulls in two upstream openhisilicon changes: - 67d793c2 (#199): cv500/V4/V4A — move MIPI_FS push from MIPI vsync IRQ to ISP_INT_FE_FSTART edge-detect. Fixes hi3516av300_lite magenta-cast + intermittent `Timeout from venc channel 0` (V4A regression report on openipc-hi3516av300.dlab.torturelabs.com) by avoiding W1C of the MIPI controller's vsync bit in atomic context. Non-disruptive replacement for the cv500-specific source from #155; preserves the openipc_frame_ts feature on all three platforms (cv500, V4, V4A). - 31a5e9f0 (#201): openipc_frame_ts — tunable ring_depth module_param (default 256, range 16..65536, rounded up to power-of-2). Per-channel rings now kmalloc'd in __init. Lets per-frame consumers raise the buffer past the 256 default that the SR-anchor consumer in #155 was sized for. Verified on cv500: depth=4096 absorbs 15 s reader pauses (was 23 drops/s at depth=256). Default keeps the binary behaviour identical for existing builds.
3 tasks
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.
Summary
Bumps
HISILICON_OPENSDK_VERSIONto pick up two openhisilicon changes:67d793c2): cv500/V4/V4A — moveOPENIPC_FT_EVT_MIPI_FSpush from the MIPI vsync IRQ (which W1C-clearedMIPI_CTRL_INT_RAWin atomic context and desynced the controller under 4K30 load) to anISP_INT_FE_FSTARTedge-detect alongside the existing FEND push. Fixes the cv500 / hi3516av300_lite magenta-cast + intermittentTimeout from venc channel 0regression and preserves theopenipc_frame_tsfeature on all three platforms (cv500, V4 hi3516ev200/gk7205v200, V4A hi3516av300).31a5e9f0):openipc_frame_ts— tunablering_depthmodule_param (default 256, range 16..65536, power-of-2). Per-channel rings now allocated in__init. Lets per-frame consumers raise the buffer past the 256-default sized for the SR-anchor consumer in работа с OSD #155. Default keeps binary behaviour identical for existing builds; operators wanting per-frame(pts_us, wall_ns)under bursty reader load caninsmod open_openipc_frame_ts.ko ring_depth=4096or persist via/etc/modprobe.d/.Test plan
openipc-hi3516av300.dlab.torturelabs.com, imx415 20 fps): add missing .mk files #199 verified — no magenta cast, FSTART/FEND alternating at ~40 Hz (20 of each),droppedaccumulates only at expected rates.cat /sys/module/open_openipc_frame_ts/parameters/ring_depthreflects insmod param; atring_depth=4096, 8 s / 15 s / 60 s reader pauses produce 0 drops/s wherering_depth=256produced 8.1 / 23.0 drops/s respectively.HISILICON_OPENSDK_SKIP_UNUSED_EV200gate was removed in hisilicon-opensdk: bump e22d315 → b958db4 (V4 mipi_rx vsync IRQ fix) #2140); add missing .mk files #199's FSTART edge-detect is in the V4 generickernel/isp/mkp/src/isp.cand was verified earlier in this thread.Related
🤖 Generated with Claude Code