Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Intel-SIG] MTL ACPI driver updates sync with upstream (6.9) #246

Merged

Conversation

quanxianwang
Copy link
Contributor

1e518e8,ACPI: scan: Defer enumeration of devices with a _DEP pointing to IVSC device,2024-02-12 18:55:40 +0100,Wentong Wu [email protected],v6.9-rc1

… device

mainline:1e518e8333ca2134a53b96bc835432083f274350
severity: Moderate

upstream commit 1e518e8 ("ACPI: scan: Defer enumeration of devices with a _DEP pointing to IVSC device")

Inside IVSC, switching ownership requires an interface with two
different hardware modules, ACE and CSI. The software interface
to these modules is based on Intel MEI framework. Usually mei
client devices are dynamically created, so the info of consumers
depending on mei client devices is not present in the firmware
tables.

This causes problems with the probe ordering with respect to
drivers for consumers of these MEI client devices. But on these
camera sensor devices, the ACPI nodes describing the sensors all
have a _DEP dependency on the matching MEI bus ACPI device, so
adding IVSC MEI bus ACPI device to acpi_honor_dep_ids allows
solving the probe-ordering problem by deferring the enumeration of
ACPI-devices which have a _DEP dependency on an IVSC mei bus ACPI
device.

Add INTC10CF, the HID of IVSC MEI bus ACPI device on MTL platform,
to acpi_honor_dep_ids.

Intel-SIG: commit 1e518e8 ("ACPI: scan: Defer enumeration of devices with a _DEP pointing to IVSC device")

Signed-off-by: Wentong Wu <[email protected]>
Signed-off-by: Rafael J. Wysocki <[email protected]>
[ Quanxian Wang: amend commit log ]
Signed-off-by: Quanxian Wang <[email protected]>
@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign liujianqiang-niu for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@deepin-ci-robot
Copy link

deepin pr auto review

关键摘要:

  • 检查是否有对新增的ACPI设备ID "INTC10CF" 的兼容性测试和文档更新
  • 确认新增的设备ID "INTC10CF" 是否已经在其他代码部分得到了充分的测试

是否建议立即修改:

  • 是,确保代码的更改不会引入新的兼容性问题,并且与整个项目的测试和文档保持一致。

@matrix-wsk matrix-wsk merged commit 5aa746f into deepin-community:linux-6.6.y Jun 14, 2024
2 of 3 checks passed
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.

4 participants