Skip to content

sof-kernel-log-check: ignore ACPI _ADR workaround - #590

Closed
plbossart wants to merge 1 commit into
thesofproject:masterfrom
plbossart:fix/rt701-workaround
Closed

plbossart wants to merge 1 commit into
thesofproject:masterfrom
plbossart:fix/rt701-workaround

Conversation

@plbossart

Copy link
Copy Markdown
Member

PR thesofproject/linux#2701 introduces a
workaround which leads to errors being thrown when the DSDT contains a
device with the 0x701 part_id.

Ignore the error since it's a normal error flow.

Signed-off-by: Pierre-Louis Bossart pierre-louis.bossart@linux.intel.com

PR thesofproject/linux#2701 introduces a
workaround which leads to errors being thrown when the DSDT contains a
device with the 0x701 part_id.

Ignore the error since it's a normal error flow.

Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
@plbossart
plbossart requested a review from a team as a code owner January 22, 2021 20:27
@plbossart

Copy link
Copy Markdown
Member Author

@marc-hb GitActions issue here unrelated to this PR?

@marc-hb marc-hb left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@marc-hb GitActions issue here unrelated to this PR?

Pylint failure is indeed unrelated; you didn't submit any Python. The Github Ubuntu VM is out of date, see #588 hack if you're interested (you're not)

# https://github.com/thesofproject/linux/pull/2701
ignore_str="$ignore_str"'|rt711 sdw:[0-3]:25d:701:0: Probe of rt711 failed: -19'
ignore_str="$ignore_str"'|rt1308 sdw:[0-3]:25d:701:0: Probe of rt1308 failed: -19'

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you guard this with:

if [ "$platform" = xxx ] ; then
  ignore_str="$ignore_str"'|rt711 ...'
  ignore_str="$ignore_str"'|rt1308 ...'
fi 

The PR URL is super useful but... very long! Can you also add just a couple words like:

# some BIOSes are  buggy so we need to probe like mad.

This helps a lot with future maintenance of this file.

@plbossart

Copy link
Copy Markdown
Member Author

no longer needed, we chose a different implementation in the Linux kernel.

@plbossart plbossart closed this Feb 3, 2021
@marc-hb marc-hb added area:non-audio Failure False positives: failing when we don't want to area:dmesg filter labels Oct 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:non-audio Failure False positives: failing when we don't want to

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants