Paths

Table of Contentst

ig4(4): Use calculated value when sda_hold is zero in ACPI
Needs ReviewPublic
Actions

Authored by allanjude on Feb 17 2021, 9:35 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Thu, May 21, 9:42 AM

	Unknown Object (File)
	Wed, May 20, 7:01 AM

	Unknown Object (File)
	Tue, May 19, 5:18 AM

	Unknown Object (File)
	Sun, May 17, 11:32 AM

	Unknown Object (File)
	Sat, May 16, 7:30 AM

	Unknown Object (File)
	Sat, May 2, 3:41 AM

	Unknown Object (File)
	Sat, May 2, 2:52 AM

	Unknown Object (File)
	Mon, Apr 27, 10:45 AM

Subscribers

Details

Reviewers

scottph

Summary

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 37123
Build 34012: arc lint + arc unit

Event Timeline

allanjude created this revision.Feb 17 2021, 9:35 PM

Herald added a subscriber: imp. · View Herald TranscriptFeb 17 2021, 9:35 PM

allanjude requested review of this revision.Feb 17 2021, 9:35 PM

Harbormaster completed remote builds in B37123: Diff 84143.Feb 17 2021, 9:35 PM

allanjude added a parent revision: D28747: ig4(4): Increase timeout to about 1 second.Feb 17 2021, 9:35 PM

allanjude added a child revision: D28749: ipmi_ssif: Fix inverted for the end of multi-part reads.

allanjude mentioned this in D28707: Fix attachment of IPMI driver on ARM64.Feb 17 2021, 9:38 PM

Is there any docs on what the acpi objects should contain ?

allanjude added a reviewer: scottph.Feb 18 2021, 3:53 PM

So this isn't needed for what I'm doing... it looks interesting. What's the status?

Is this a workaround for a broken ACPI implementation, possibly in early firmware? I notice Linux's equivalent i2c_dw_acpi_params does not do this. I do see Linux explicitly skips ACPI here for Dell Inspiron 7348.

Revision Contents
Changeset List

Path

Size

sys/

dev/

ichiic/

ig4_iic.c

8 lines

Diff 84143

View Options

ig4(4): Use calculated value when sda_hold is zero in ACPINeeds ReviewPublicActions