Skip to content

Enable ov08x40 CVS media path for IPU8#59

Open
arunt1 wants to merge 1 commit into
intel:mainfrom
arunt1:cvs
Open

Enable ov08x40 CVS media path for IPU8#59
arunt1 wants to merge 1 commit into
intel:mainfrom
arunt1:cvs

Conversation

@arunt1

@arunt1 arunt1 commented Jul 1, 2026

Copy link
Copy Markdown

Add ov08x40-uf-0 to the available sensor list and update the ov08x40-uf IPU8 sensor topology to include the Intel CVS entity between the ov08x40 sensor and the Intel IPU7 CSI2 receiver.

The media format remains SGRBG10 3856x2176, but the link graph now matches the actual hardware path:

ov08x40 -> Intel CVS -> Intel IPU7 CSI2 -> Intel IPU7 ISYS Capture

This allows Camera HAL/GStreamer to configure the correct media pipeline for ov08x40-uf on the CVS board.

Comment thread src/v4l2/MediaControl.cpp Outdated
entityName = entity.info.name;
break;
}
if (strcmp(sinkEntityName.c_str(), entity.info.name) == 0) {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I remember that similar feature was implemented on IPU6 HAL, could you help to check if that works for this case?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

@hao-yao , as you suggested, updated similar changes from ipu6-camera-hal to ipu7-camera-hal, Please review the PR and merge the change

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@arunt1 do you have verified full function, include suspend/resume ?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

@sujimmy , Yes verified , basic functionalities check like image capture, video recording, suspend and resume, it got passed.
verified stability test also (cold boot, warmboot, S0ix and S4 test), it got passed.

@arunt1 arunt1 force-pushed the cvs branch 2 times, most recently from c24753a to 4109fa9 Compare July 8, 2026 07:12
Comment thread src/v4l2/MediaControl.cpp Outdated
char devname[32];
};

static const string ivscName = "Intel IVSC CSI";

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Better to change the name to something like CVS

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

@hao-yao updated the comments as you suggested.

@arunt1 arunt1 left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

@hao-yao , updated changes as you commented

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