Enable OV08X40 camera sensor for Intel MTL IPU6 MIPI Camera

Bug #2060101 reported by You-Sheng Yang
16
This bug affects 2 people
Affects Status Importance Assigned to Milestone
HWE Next
New
Undecided
Unassigned
ipu6-drivers (Ubuntu)
Status tracked in Oracular
Jammy
Won't Fix
Undecided
Unassigned
Noble
Fix Committed
High
You-Sheng Yang
Oracular
Fix Released
High
You-Sheng Yang
linux (Ubuntu)
Status tracked in Oracular
Jammy
Invalid
Undecided
Unassigned
Noble
In Progress
High
You-Sheng Yang
Oracular
Triaged
High
You-Sheng Yang
linux-oem-6.10 (Ubuntu)
Status tracked in Oracular
Jammy
Invalid
Undecided
Unassigned
Noble
In Progress
High
You-Sheng Yang
Oracular
Invalid
Undecided
Unassigned
linux-oem-6.5 (Ubuntu)
Status tracked in Oracular
Jammy
In Progress
High
You-Sheng Yang
Noble
Invalid
Undecided
You-Sheng Yang
Oracular
Invalid
Undecided
You-Sheng Yang
linux-oem-6.8 (Ubuntu)
Status tracked in Oracular
Jammy
Invalid
Undecided
Unassigned
Noble
In Progress
High
You-Sheng Yang
Oracular
Invalid
Undecided
Unassigned

Bug Description

[SRU Justification]

BugLink: https://bugs.launchpad.net/bugs/2060101

[Impact]

ov08x40 camera sensor is not working on IPU6 platforms.

[Fix]

A few changes from upstream kernel as well as a not yet upstreamed patch in the Intel ipu6-drivers GitHub repo.
* commit b1a42fde ("media: ov08x40: Avoid sensor probing in D0 state")
* commit eea8f730 ("media: ov08x40: Modify the tline calculation in different modes")
* https://github.com/intel/ipu6-drivers/blob/Release_20240624/patch/v6.8/0001-media-ov08x40-Add-Tline-calculation-and-handshake-pi.patch

[Test Case]

The proposed kernel should have been built along with the updated ipu6-drivers dkms. To verify ipu6-drivers alone, one will need experimental kernels from https://launchpad.net/~canonical-hwe-team/+archive/ubuntu/intermediate-kernel.

Here are steps for verifying kernels from -proposed pocket:

1. test camera after rebooting into a proposed kernel,
2. run suspend/resume stress test plan.
3. verify camera still working

[Where problems could occur]

These fixes are quite limited to ipu6/ivsc platforms, and affected no power-consumption or other noticible problem.

[Other Info]

The fixing commits/patches were meant for in-tree driver, but yet it changes device power up sequence and may affect platforms not interested in IPU6 enablement. It is then duplicated into ipu6-drivers dkms to avoid any unexpected side effect.

While some of the fixes are still under upstream review, meaning they're not yet available in any kernel release, linux-unstable/oracular/noble and oem-6.8/6.1x will be nominated for a rebuild against new dkms.

You-Sheng Yang (vicamo)
tags: added: oem-priority originate-from-2044254 sutton
Changed in linux-oem-6.5 (Ubuntu):
assignee: nobody → You-Sheng Yang (vicamo)
Changed in linux-oem-6.5 (Ubuntu Jammy):
assignee: nobody → You-Sheng Yang (vicamo)
Changed in ipu6-drivers (Ubuntu Noble):
assignee: nobody → You-Sheng Yang (vicamo)
You-Sheng Yang (vicamo)
Changed in ipu6-drivers (Ubuntu Jammy):
status: New → Won't Fix
Changed in ipu6-drivers (Ubuntu Mantic):
assignee: nobody → You-Sheng Yang (vicamo)
importance: Undecided → High
status: New → In Progress
Changed in ipu6-drivers (Ubuntu Noble):
importance: Undecided → High
status: New → In Progress
Changed in linux (Ubuntu Jammy):
status: New → Invalid
Changed in linux (Ubuntu Mantic):
status: New → Won't Fix
Changed in linux (Ubuntu Noble):
assignee: nobody → You-Sheng Yang (vicamo)
importance: Undecided → High
status: New → In Progress
Changed in linux-oem-6.5 (Ubuntu Jammy):
importance: Undecided → High
status: New → In Progress
Changed in linux-oem-6.5 (Ubuntu Mantic):
status: New → Invalid
Changed in linux-oem-6.5 (Ubuntu Noble):
status: New → Invalid
You-Sheng Yang (vicamo)
description: updated
no longer affects: ipu6-drivers (Ubuntu Mantic)
no longer affects: linux (Ubuntu Mantic)
no longer affects: linux-oem-6.10 (Ubuntu Mantic)
no longer affects: linux-oem-6.5 (Ubuntu Mantic)
no longer affects: linux-oem-6.8 (Ubuntu Mantic)
Changed in linux-oem-6.8 (Ubuntu Jammy):
status: New → Invalid
Changed in linux-oem-6.8 (Ubuntu Noble):
assignee: nobody → You-Sheng Yang (vicamo)
importance: Undecided → High
status: New → In Progress
Changed in linux-oem-6.8 (Ubuntu Oracular):
status: New → Invalid
Changed in linux-oem-6.10 (Ubuntu Jammy):
status: New → Invalid
Changed in linux-oem-6.10 (Ubuntu Noble):
assignee: nobody → You-Sheng Yang (vicamo)
importance: Undecided → High
status: New → In Progress
Changed in linux-oem-6.10 (Ubuntu Oracular):
status: New → Invalid
Revision history for this message
You-Sheng Yang (vicamo) wrote :

linux/oracular is still at 6.8.

Changed in linux (Ubuntu Oracular):
status: In Progress → Triaged
Revision history for this message
You-Sheng Yang (vicamo) wrote :

ipu6-drivers SRU is in progress in https://bugs.launchpad.net/bugs/2067364

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ipu6-drivers - 0~git202406240945.aecec2aa-0ubuntu1

---------------
ipu6-drivers (0~git202406240945.aecec2aa-0ubuntu1) oracular; urgency=medium

  [ You-Sheng Yang ]
  * upstream Release_20240624 release (LP: #2067364)
  * UBUNTU: SAUCE: [IPU6] skip hwcfg checking for platforms with none/broken
    graph port nodes (LP: #2070251)
  * UBUNTU: SAUCE: media: ipu6: Fix compilation with kernels >= 6.10
  * ov08x40: add back driver source (LP: #2060101)

 -- You-Sheng Yang <email address hidden> Mon, 24 Jun 2024 22:01:05 +0800

Changed in ipu6-drivers (Ubuntu Oracular):
status: In Progress → Fix Released
Revision history for this message
Timo Aaltonen (tjaalton) wrote :

too late for oem-6.5, let's focus on noble: linux/hwe-6.8

Changed in linux-oem-6.5 (Ubuntu Jammy):
status: In Progress → Won't Fix
Revision history for this message
Timo Aaltonen (tjaalton) wrote : Please test proposed package

Hello You-Sheng, or anyone else affected,

Accepted ipu6-drivers into noble-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/ipu6-drivers/0~git202406240945.aecec2aa-0ubuntu1~22.04.1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, what testing has been performed on the package and change the tag from verification-needed-noble to verification-done-noble. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-noble. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

Changed in ipu6-drivers (Ubuntu Noble):
status: In Progress → Fix Committed
tags: added: verification-needed verification-needed-noble
Revision history for this message
Anthony Wong (anthonywong) wrote :

It's late but let's try to have this in oem-6.5 first because it'll take longer for the fix to land in hwe-6.8

Changed in linux-oem-6.5 (Ubuntu Jammy):
status: Won't Fix → In Progress
Revision history for this message
Timo Aaltonen (tjaalton) wrote :

Hello You-Sheng, or anyone else affected,

Accepted ipu6-drivers into noble-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/ipu6-drivers/0~git202406240945.aecec2aa-0ubuntu1~24.04.1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, what testing has been performed on the package and change the tag from verification-needed-noble to verification-done-noble. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-noble. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.