| Summary: | IRQ request for Intel SoC DTS Thermal fails | ||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | [Hardware Platforms] MinnowBoard MAX Firmware | Reporter: | Marcel Holtmann <marcel> | ||||||||
| Component: | minnowmax-edk2 | Assignee: | Marcel Holtmann <marcel> | ||||||||
| Status: | RESOLVED FIXED | QA Contact: | |||||||||
| Severity: | normal | ||||||||||
| Priority: | Medium | CC: | danders, david.wei, marcel, mike.wu, shifeix.a.lu, sjolley.yp.pm, tim.he, warthog9, yoctoproject | ||||||||
| Version: | unspecified | ||||||||||
| Target Milestone: | Q4 2014 | ||||||||||
| Hardware: | MinnowBoard Max | ||||||||||
| OS: | Multiple | ||||||||||
| Whiteboard: | |||||||||||
| OS type for building Yocto: | --- | Type of Regression: | --- | ||||||||
| Verified: | Documentation change: | No (bug/feature does not impact docs) | |||||||||
| Attachments: |
|
||||||||||
Maybe this is related to the thermal zone information: [ 1.889377] ACPI: Invalid active0 threshold [ 1.891743] thermal LNXTHERM:00: registered as thermal_zone0 [ 1.891758] ACPI: Thermal Zone [TZ01] (27 C) Hi Marcel, 1. I can not duplicate information"intel_soc_dts_thermal: request_threaded_irq ret -22 " with Yocto 3.14.4 version OS. [ 14.519602] intel_soc_dts_thermal: request_threaded_irq ret -22 [ 14.564178] intel_soc_dts_thermal: request_threaded_irq ret -22 Can you tell me which version Yocto's OS and BIOS can duplicate this issue? 2.In the other words,we can duplicate information" [ 1.889377] ACPI: Invalid active0 threshold [ 1.891743] thermal LNXTHERM:00: registered as thermal_zone0 [ 1.891758] ACPI: Thermal Zone [TZ01] (27 C)" Can we removed the method "_AC0" in BIOS ASL code? Here is more detail: Since there is no active cooling device on Minnow Max, so Shifei plans to remove _AC0 so that following warning would disappear. [ 1.889377] ACPI: Invalid active0 threshold David, thanks. I'm assigning this to you as a firmware issue. Please proxy for who is working on it, or have them get a BZ account and assign to them. Thanks. Created attachment 2239 [details]
Modified firmware's log with _AC0 Removed
Hi Darren,
Warning message “ACPI: Invalid active0 threshold” would disappear, see log changes between “6714before.log” and “6714after.log”.
Can you test attachment’s firmware image” MNW2MAX1.X64.0074.R03.1411071600.zip” on your side? Thanks a lot!
Created attachment 2240 [details]
Original Firmware's log
Created attachment 2241 [details]
test Firmware Image
Design and implemeantation are completed and code is checked in. Will be available by 0.75 release. Assign back to Marcel. Please help test the attached test image from OTC side and give feedback. Please respond I'm still seeing the original reported error on 3.19 with 0.77 [ 17.645160] intel_soc_dts_thermal: request_threaded_irq ret -22 (mind you I'm only seeing one now) I am not seeing: [ 1.889377] ACPI: Invalid active0 threshold so I'm going to close this. If I'm incorrect Marcel please re-open. |
Within the init procedure of drivers/thermal/intel_soc_dts_thermal.c we get failures of EINVAL when requesting the IRQ: [ 14.519602] intel_soc_dts_thermal: request_threaded_irq ret -22 [ 14.564178] intel_soc_dts_thermal: request_threaded_irq ret -22 The board is matched by the following: { X86_VENDOR_INTEL, X86_FAMILY_ANY, 0x37, 0, BYT_SOC_DTS_APIC_IRQ}, Is this suppose to work or are we exposing functionality that is not present on this board? What I can tell is that the thermal zone for each CPU core is found and registered, but since the IRQ request fails, it gets later on unregistered.