thunderx2: i2c driver PEC and ACPI clock fixes

Bug #1738073 reported by dann frazier on 2017-12-13
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
linux (Ubuntu)
Undecided
dann frazier
Artful
Undecided
dann frazier

Bug Description

[Impact]
The i2c input frequency is currently hardcoded in the driver. This may not match the actual frequency used by the platform. Upstream has updated the driver to pull the frequency from firmware.

[Test Case]
The updated driver will detect the CAV9007 ACPI ID:

ubuntu@starbuck:~$ dmesg | grep xlp9xx-i2c
[ 47.723562] xlp9xx-i2c CAV9007:02: i2c transfer timed out!
                          ^^^^^^^
The old driver will not claim this device.

[Regression Risk]
The changes are restricted to a driver that appears to be only used on MIPS platforms (which we don't support) and Cavium ThunderX2 systems that we've specifically tested upon.

dann frazier (dannf) on 2017-12-13
Changed in linux (Ubuntu Artful):
status: New → In Progress
assignee: nobody → dann frazier (dannf)
Seth Forshee (sforshee) on 2017-12-14
Changed in linux (Ubuntu):
status: In Progress → Fix Committed
Changed in linux (Ubuntu Artful):
status: In Progress → Fix Committed

This bug is awaiting verification that the kernel in -proposed solves the problem. Please test the kernel and update this bug with the results. If the problem is solved, change the tag 'verification-needed-artful' to 'verification-done-artful'. If the problem still exists, change the tag 'verification-needed-artful' to 'verification-failed-artful'.

If verification is not done by 5 working days from today, this fix will be dropped from the source code, and this bug will be closed.

See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you!

tags: added: verification-needed-artful
dann frazier (dannf) wrote :

artful verification:
ubuntu@starbuck:~$ dmesg | grep xlp9xx
[ 46.890471] ipmi_ssif: Trying SPMI-specified SSIF interface at i2c address 0x10, adapter xlp9xx-i2c, slave address 0x0

tags: added: verification-done-artful
removed: verification-needed-artful
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers