Skip to content

PTL MR2 fix - #77

Open
syafiqnizam25 wants to merge 1 commit into
intel:mainfrom
syafiqnizam25:PTL_MR5_OV13B
Open

syafiqnizam25 wants to merge 1 commit into
intel:mainfrom
syafiqnizam25:PTL_MR5_OV13B

Conversation

@syafiqnizam25

Copy link
Copy Markdown
Contributor
  • ov13b10-wf cannot stream.

It loops over candidate sensor entities (ov13b10 1-0010, ov13b10 2-0010), but validates reachability using only the generic name prefix "ov13b10 " instead of the specific candidate's exact name.

root@localhost:/home/user/drivers.camera.scaling.sensor# gst-launch-1.0 icamerasrc num-buffers=-1 scene-mode=normal device-name=ov13b10-wf printfps=true io-mode=dma_mode ! 'video/x-raw(memory:DMABuf),drm-format=NV12,width=1280,height=960' ! glimagesink sync=false
[10-06 17:45:01.279] CamHAL[INF] CameraParserInvoker: getAvailableSensors, Found IPU: IPU7
[10-06 17:45:01.279] CamHAL[INF] CameraParserInvoker: parseSensors: I will Load config file: sensors/ov13b10-wf.json
[10-06 17:45:01.280] CamHAL[INF] CameraSensorsParser: resolveCsiPortAndI2CBus: I2CBus:2-0010 <=> CSI Port:0
[10-06 17:45:01.281] CamHAL[ERR] CameraSensorsParser: Failed to find NVM directory
[10-06 17:45:01.281] CamHAL[INF] CameraParserInvoker: parseSensors, sensors/ov13b10-wf.json loaded!
[10-06 17:45:01.281] CamHAL[INF] CameraParserInvoker: parseSensors: I will Load config file: sensors/ov13b10-uf.json
[10-06 17:45:01.282] CamHAL[INF] CameraSensorsParser: resolveCsiPortAndI2CBus: I2CBus:2-0010 <=> CSI Port:2
[10-06 17:45:01.282] CamHAL[ERR] CameraSensorsParser: Failed to find NVM directory
[10-06 17:45:01.282] CamHAL[INF] CameraParserInvoker: parseSensors, sensors/ov13b10-uf.json loaded!
Setting pipeline to PAUSED ...
error: XDG_RUNTIME_DIR is invalid or not set in the environment.
ATTENTION: default value of option vblank_mode overridden by environment.
[10-06 17:45:01.417] CamHAL[INF] AiqInitData: aiqb file name OV13B10_09B13_PTL.aiqb
[10-06 17:45:01.418] CamHAL[INF] AiqInitData: aiqb file name OV13B10_09B13_PTL.aiqb
[10-06 17:45:01.418] CamHAL[INF] AiqInitData: aiqb file name OV13B10_09B13_PTL.aiqb
[10-06 17:45:01.418] CamHAL[INF] AiqInitData: aiqb file name OV13B10_09B13_PTL.aiqb
[10-06 17:45:01.430] CamHAL[ERR] V4l2_device_cc: GetControl: Device node /dev/v4l-subdev4 IOCTL VIDIOC_G_EXT_CTRLS error: Invalid argument
Pipeline is live and does not need PREROLL ...
Got context from element 'sink': gst.gl.GLDisplay=context, gst.gl.GLDisplay=(GstGLDisplay)"(GstGLDisplayX11)\ gldisplayx11-0";
Pipeline is PREROLLED ...
Setting pipeline to PLAYING ...
New clock: GstSystemClock
[10-06 17:45:01.464] CamHAL[WAR] MediaControl: failed to setup Link ov13b10 2-0010 [233:0] ==> Intel IPU7 CSI2 0 [129:0] enable.
[10-06 17:45:01.570] CamHAL[ERR] SensorHwCtrl: sensor output sub device is not set
[10-06 17:45:01.579] CamHAL[ERR] V4l2_video_node_cc: Start: Device node /dev/video0 IOCTL VIDIOC_STREAMON error: Broken pipe
[10-06 17:45:01.579] CamHAL[ERR] CaptureUnit: Device:Generic stream on failed.
[10-06 17:45:01.579] CamHAL[ERR] CaptureUnit: Devices stream on failed:-38
[10-06 17:45:01.579] CamHAL[ERR] CameraDevice: Start capture unit failed with ret:-38.
[10-06 17:45:01.579] CamHAL[ERR] CameraDevice: Camera device starts failed.
[10-06 17:45:01.579] CamHAL[WAR] CaptureUnit: @stop: device not started
[10-06 17:45:03.572] CamHAL[WAR] RequestThread: wait event time out, 1 requests processing, 5 requests in HAL
[10-06 17:45:03.573] CamHAL[WAR] Scheduler: video_bb: wait trigger time out
[10-06 17:45:03.573] CamHAL[WAR] Scheduler: still_full: wait trigger time out
[10-06 17:45:03.573] CamHAL[WAR] Scheduler: video_lb: wait trigger time out

@hao-yao

hao-yao commented Oct 8, 2026

Copy link
Copy Markdown
Contributor

Hi @syafiqnizam25 , what's the behavior without the change "getI2CBussAddress miss ov13b10-wf"?

@syafiqnizam25

syafiqnizam25 commented Oct 8, 2026 •

Copy link
Copy Markdown
Contributor Author

@hao-yao , cannot stream using ov13b10-wf device name, can refer log attached in pr description.

@hao-yao

hao-yao commented Oct 8, 2026

Copy link
Copy Markdown
Contributor

Hi @syafiqnizam25 , on which version/tag/commit did you find this issue?

@hao-yao

hao-yao commented Oct 9, 2026

Copy link
Copy Markdown
Contributor

Hi @syafiqnizam25 , how about rebasing your commit 4ac5050 onto 11d8aff ? Will merge your commit first.

- Fix getI2CBusAddress matching wrong sensor instance when multiple entities share a name prefix (e.g. ov13b10-wf/uf)

Signed-off-by: syafiqnizam25 <muhammadsyafiqnizamuddinx.binshafie@intel.com>
@syafiqnizam25

Copy link
Copy Markdown
Contributor Author

Hye @hao-yao , already rebase

@hao-yao

hao-yao commented Oct 9, 2026 •

Copy link
Copy Markdown
Contributor

Hye @hao-yao , already rebase

@syafiqnizam25 My local rebasing conflicted, could you help to check if this is good for you?
https://github.com/hao-yao/ipu7-camera-hal/commits/main/
directly commited your change to the release PR to avoid conflict.

@syafiqnizam25

Copy link
Copy Markdown
Contributor Author

@hao-yao, my side is fine, since this PR has been rebase onto 11d8aff from https://github.com/intel/ipu7-camera-hal/commits/main/ as requested. Could u try apply my commit first before db6769f in https://github.com/hao-yao/ipu7-camera-hal/commits/main/ ?

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.

3 participants