frankcrawford / it87

226 stars 46 forks source link

add: patch for linux 5.15.106 #7

Closed swordow closed 1 year ago

swordow commented 1 year ago

Add patch for linux 5.15.106 tested in Slackware15.0 with linux 5.15.106 in Gigabyte W480M motherboard

# sensors
hidpp_battery_0-hid-3-7
Adapter: HID adapter
in0:           3.81 V

acpitz-acpi-0
Adapter: ACPI interface
temp1:        +16.8°C  (crit = +20.8°C)
temp2:        +16.8°C  (crit = +20.8°C)
temp3:        +27.8°C  (crit = +119.0°C)

pch_cometlake-virtual-0
Adapter: Virtual device
temp1:        +28.0°C

nvme-pci-0400
Adapter: PCI adapter
Composite:    +25.9°C  (low  = -40.1°C, high = +83.8°C)
                       (crit = +87.8°C)
Sensor 1:     +33.9°C  (low  = -273.1°C, high = +65261.8°C)
Sensor 2:     +25.9°C  (low  = -273.1°C, high = +65261.8°C)

it8688-isa-0a40
Adapter: ISA adapter
in0:         708.00 mV (min =  +0.00 V, max =  +3.06 V)
in1:           2.04 V  (min =  +0.00 V, max =  +3.06 V)
in2:           2.03 V  (min =  +0.00 V, max =  +3.06 V)
in3:           2.00 V  (min =  +0.00 V, max =  +3.06 V)
in4:           0.00 V  (min =  +0.00 V, max =  +3.06 V)  ALARM
in5:           1.06 V  (min =  +0.00 V, max =  +3.06 V)
in6:           1.20 V  (min =  +0.00 V, max =  +3.06 V)
3VSB:          3.36 V  (min =  +0.00 V, max =  +6.12 V)
Vbat:          3.05 V
fan1:         969 RPM  (min =   10 RPM)
fan2:         847 RPM  (min =   10 RPM)
fan3:         775 RPM  (min =   10 RPM)
temp1:        +32.0°C  (low  =  +0.0°C, high = +60.0°C)  sensor = thermistor
temp2:        +30.0°C  (low  =  +0.0°C, high = +60.0°C)  sensor = thermistor
temp3:        +25.0°C  (low  =  +0.0°C, high = +80.0°C)  sensor = Intel PECI
temp4:        +28.0°C  (low  =  +0.0°C, high = +60.0°C)  sensor = thermistor
temp5:        +27.0°C  (low  =  +0.0°C, high = +60.0°C)  sensor = thermistor
temp6:        +29.0°C  (low  =  +0.0°C, high = +60.0°C)  sensor = thermistor
intrusion0:  ALARM

coretemp-isa-0000
Adapter: ISA adapter
Package id 0:  +26.0°C  (high = +82.0°C, crit = +100.0°C)
Core 0:        +25.0°C  (high = +82.0°C, crit = +100.0°C)
Core 1:        +24.0°C  (high = +82.0°C, crit = +100.0°C)
Core 2:        +26.0°C  (high = +82.0°C, crit = +100.0°C)
Core 3:        +25.0°C  (high = +82.0°C, crit = +100.0°C)
Core 4:        +25.0°C  (high = +82.0°C, crit = +100.0°C)
Core 5:        +24.0°C  (high = +82.0°C, crit = +100.0°C)
Core 6:        +27.0°C  (high = +82.0°C, crit = +100.0°C)
Core 7:        +24.0°C  (high = +82.0°C, crit = +100.0°C)

nvme-pci-0200
Adapter: PCI adapter
Composite:    +27.9°C

dmesg

[40757.688091] it87: it87 driver version it87
[40757.688374] it87: Found IT8688E chip at 0xa40, revision 1
[40757.688608] it87: Beeping is supported
[40757.688698] ACPI Warning: SystemIO range 0x0000000000000A45-0x0000000000000A46 conflicts with OpRegion 0x0000000000000A45-0x0000000000000A46 (\GSA1.SIO1) (20210730/utaddress-204)
[40757.688703] ACPI: OSL: Resource conflict; ACPI support missing from driver?
[40950.761856] i2c_dev: i2c /dev entries driver

/etc/modprobe.d/iit87.conf:

options it87 ignore_resource_conflict=1 force_id=0x8688
frankcrawford commented 1 year ago

@swordow thanks for the work here.

Unfortunately, you have based your changes on the Linux in-tree it87driver, which is well behind the one in this repo, and as such can't be merged into here.

However, what actual chip are you attempting to enable with your board, as it may already exist within the repo, and just not yet be available in the Linux mainline.

What I would suggest is you download a fresh copy of this repo, build it as an external module, and then unload the standard kernel it87 module and install the one built in here, without the force_id option. It may just work.

If the one build from this repo doesn't automatically work, we can run a few other checks to see what is happening and then update this repo to handle it.

Again, thanks for you work, and lets see if we can get it to work for you.

Regards Frank

frankcrawford commented 1 year ago

@swordow any update for this? If not, I'm going to reject this PR.

swordow commented 1 year ago

@swordow any update for this? If not, I'm going to reject this PR.

@frankcrawford
You are right. The actual chip is it8688 which is suppored in your repo and just not yet available in linux mainline. I will clone this repo and build as an external module later. I just think this patch may help some one that needs built in module in lts 5.15.x mainline, so i opend request. It does not matter if it is no necessary. Thanks~~

frankcrawford commented 1 year ago

@swordow FYI a lot of the simple changes are in this version will be in the kernel for 6.4, and 6.5. There is one item (multi-bank support) that the it8688 needs, that will be harder to get in, and so may even take until 6.6.

Getting it back ported will always be a much bigger issue. They normally will only take bug fixes, not enhancements.