ACK/Cmnt: [SRU][Resolute][Questing][Noble][PATCH 0/1] UBUNTU: SAUCE: ACPI: respect items already in honor_dep before skipping

[Yufeng Gao]

On 27/3/26 19:38, You-Sheng Yang wrote:
> BugLink: https://bugs.launchpad.net/bugs/2145171
>
> SRU Justification:
>
> [Impact]
>
> Upstream v6.18-rc1 commit 4405a214df146775338a1e6232701a29024b82e1
> "ACPI: scan: Add Intel CVS ACPI HIDs to acpi_ignore_dep_ids[]" (also
> backported to v6.17.8, v6.12.58), which in effect reverts a not yet
> upstreamed Intel MIPI camera patch titled "ACPI: scan: Update HID for
> new platform" meant for bug 2090932, and therefore they share the same
> symptom:
>
>    [ 4.484426] ov02c10 i2c-OVTI02C1:00: chip id mismatch: 560243!=0
>    [ 4.484432] ov02c10 i2c-OVTI02C1:00: failed to find sensor: -6
>
> [Fix]
>
> While this should really be fixed by upstream Intel's CVS driver and
> remove the two platforms from acpi_ignore_dep_ids, a sauced change is
> made to preserve the honor effect and yet don't introduce potential
> conflict with the upstream before any further progress is made.
>
> [Test Plan]
>
> 1. Install the kernel from -proposed pocket and reboot into it.
>
> 2. Check dmesg has no aforementioned error,
>
> 3. Make sure the device appears in the gstreamer device enumeration:
>
>     $ gst-inspect-1.0 icamerasrc
>     .....
>        device-name     : The input devices name queried from HAL
>                          flags: readable, writable
>                          Enum "GstCamerasrcDeviceName" Default: 0, "ov02c10-uf"
>                             (0): ov02c10-uf       - ov02c10 as sensor.
>
> [Where problems could occur]
>
> No. This restores the hardware enablement effort for the affected
> platforms only. For Ubuntu, CVS driver is always available along with
> the kernel, and therefore for a device declared _DEP to the two ACPI
> devices, the driver can be installed either through ubuntu-drivers or
> through oem meta packages.
>
> [Other Info]
>
> While this has been backported to v6.17.8, it affects R and Q. And while
> this has also been backported to v6.12.58, and linux-6.8/noble is
> tracking stable 6.8 fixes, Noble will be affected soon or later.
> Nominate for N, Q and R accordingly.
>
> You-Sheng Yang (1):
>    UBUNTU: SAUCE: ACPI: respect items already in honor_dep before
>      skipping
>
>   drivers/acpi/scan.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
Not sure if it is fine to have multiple series in the same patch set, 
but otherwise lgtm.

Acked-by: Yufeng Gao <yufeng.gao at canonical.com>