src/sys/dev/ichsmb, branch releng/14.2

Add smbus and i2c device IDs for Meteor Lake

2024-10-17T16:32:38+00:00

Reviewed by:	emaste, Daniel Schaefer 
MFC after:	3 days
Sponsored by:	Framework Computer Inc
Sponsored by:	The FreeBSD Foundation
Differential Revision:	https://reviews.freebsd.org/D46249

(cherry picked from commit d3b05d0ea10abe059dc63c6fb6ef3f061b758af2)

ichsmb: fix block read operation

2024-02-17T14:15:11+00:00

First of all and unlike I2C, it's not the master that dictates how many
bytes to read in block read operation.  It's the device that informs the
master how many bytes it's sending back.

Thus, for ichsmb_bread() the count parameter is purely an output
parameter.  The code has been changed to reflect that.
The sanity checking of the response length is now done once it (the
first byte of the response) is received.

While here, handling of ICH_HST_STA_FAILED status bit has been added.
Plus some code style improvements and some new code comments in the
vicinity of the changed code.

(cherry picked from commit cbf7c81b608bb9311e50df9481447dc843083a0e)

sys: Remove $FreeBSD$: one-line .c pattern

2023-08-16T17:54:36+00:00

Remove /^[\s*]*__FBSDID\("\$FreeBSD\$"\);?\s*\n/

sys: Remove $FreeBSD$: two-line .h pattern

2023-08-16T17:54:11+00:00

Remove /^\s*\*\n \*\s+\$FreeBSD\$$\n/

sys/dev/ichsmb: Silence unhandled SMBALERT device_printfs

2023-06-26T08:29:34+00:00

The ichsmb driver does not actually handle SMBALERT, other than by logging the
first 16 occurences of the ICH_HST_STA_SMBALERT_STS_SMBALERT status
flag. Because the SMBALERT is not acknowledged by the host, clearing it in the
host status register does not appear to work as long as some slave device is
pulling the SMBALERT line low, at least for C2000 chips. As a result, if a slave
device does pull SMBALERT low the interrupt handler will always loop its maximum
of 16 times attempting to clear all status register flags and device_printf the
status register. The result is the kernel message buffer is littered with these
device_printfs at every interrupt.

To remedy the problem, the ICH_HST_STA_SMBALERT_STS flag is zeroed in the read
host status register value, just as with ICH_HST_STA_INUSE_STS and
ICH_HST_STA_HOST_BUSY. This allows the loop to break when no other flags that
must be handled are set in the host status register. Additionally, because the
SMBALERT is not actually handled the SMBALERT logging is omitted as it has no
actual function at this time.

Reviewed by:	imp
Sponsored by:	Rubicon Communications, LLC ("Netgate")
Differential Revision:	https://reviews.freebsd.org/D39966

ichsmb/imcsmb/intpm/ismt: Remove unused devclass arguments to DRIVER_MODULE.

2022-05-09T19:22:02+00:00

Remove unused smbus_devclass.

2022-05-06T22:39:30+00:00

Remove check for __CC_SUPPORTS_FUNC.

2022-04-12T17:06:04+00:00

__func__ is part of C99.

PR:		263102 (exp-run)
Reviewed by:	brooks, imp, emaste
Differential Revision:	https://reviews.freebsd.org/D34801

ichsmb: Add PCI IDs for Intel Alder Lake SMBus controller

2021-11-24T18:07:58+00:00

Submitted by:	Dmitry Luhtionov 
MFC after:	2 weeks

ichsmb: Add PCI ID for Intel Gemini Lake SMBus controller

2021-04-14T00:58:07+00:00

Submitted by:	Dmitry Luhtionov 
MFC after:	2 weeks