Repository navigation
Fix OV08X40 media links for IPU6 CSI2 - #184
Conversation
|
Jimmy verified this changes on CVS board and Non CVS board, camera is working fine |
There was a problem hiding this comment.
I checked CVS & non-CVS devices.
Intel CVS configurations are not necessary.
Could you help to check line 18, 19, 33 & 34 still needed for currently media control?
There was a problem hiding this comment.
ok, updated patch set accordingly
|
@saragsapre I had verified this PR on MTL & ARL devices. it works on CVS & non-CVS devices. |
sujimmy
left a comment
sujimmy
left a comment
There was a problem hiding this comment.
this PR is working on MTL & ARL devices with different sensors, like ov08x & ov05c etc.
There was a problem hiding this comment.
The patch doesn't remove CVS/Sabre support, it removes the hardcoded "Intel CVS" links from the sensor's static media config XML, and instead relies on MediaControl.cpp to dynamically re-target links to the ISVC/Sabre entity at runtime, only if it's actually present.
If no CVS/Sabre device is present (bypass rework, per your diagram) → the fmt entries referencing it are skipped (continue), and the driver just uses the direct sensor → IPU6 CSI-2 link that's now in the XML.
If CVS/Sabre is present, ivsc is found, and it patches the format's entity/entityName fields to point at the real ISVC device, effectively re-inserting it into the pipeline at runtime.
Since this patch has been already tested with CVS and without CVS devices. hence approving.
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
4 participants
No description provided.