<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugzilla.yoctoproject.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.6"
          urlbase="https://bugzilla.yoctoproject.org/"
          
          maintainer="it-coreprojects-helpdesk@linuxfoundation.org"
>

    <bug>
          <bug_id>7682</bug_id>
          
          <creation_ts>2015-04-30 15:16:39 +0000</creation_ts>
          <short_desc>acpitz-virtual-0 thermal device always returns 26.8 deg C</short_desc>
          <delta_ts>2017-06-13 18:08:08 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>12</classification_id>
          <classification>Hardware Platforms</classification>
          <product>MinnowBoard MAX Firmware</product>
          <component>minnowmax-edk2</component>
          <version>unspecified</version>
          <rep_platform>MinnowBoard Max</rep_platform>
          <op_sys>x86_64</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>MOVED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>Medium</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>Production Release</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Darren Hart">dvhart</reporter>
          <assigned_to name="Unassigned">unassigned</assigned_to>
          <cc>david.wei</cc>
    
    <cc>dvhart</cc>
    
    <cc>gerard.b</cc>
    
    <cc>shifeix.a.lu</cc>
    
    <cc>sjolley.yp.pm</cc>
          
          
          <cf_os>---</cf_os>
          <cf_regression_type>---</cf_regression_type>
          
          <cf_docchange>Don&apos;t know</cf_docchange>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>50664</commentid>
    <comment_count>0</comment_count>
    <who name="Darren Hart">dvhart</who>
    <bug_when>2015-04-30 15:16:39 +0000</bug_when>
    <thetext>On Linux 3.19.2 (yocto), a user on MinnowBoard is reporting the acpitz-virtual-0 device, accessed via /sys/class/thermal/thermal_zone0/temp, is always returning 26800 (26.8 deg C), regardless of core temp.

Another Debian user reported the following using the lm-sensors package:
sensors
acpitz-virtual-0
Adapter: Virtual device
temp1:        +26.8°C  (crit = +90.0°C)

coretemp-isa-0000
Adapter: ISA adapter
Core 0:       +50.0°C  (high = +110.0°C, crit = +110.0°C)
Core 2:       +51.0°C  (high = +110.0°C, crit = +110.0°C)

The core* temps using the coretemp driver are what I use, I&apos;m not sure what the acpitz-virtual-0 device is or what it reports. Since it&apos;s an ACPI device, I&apos;m filing under firmware for an initial review of this device and it&apos;s ACPI implementation.

Should this device be reported?

I have asked on list for the firmware version. The original reported is Cc&apos;d here. Gerard, if you can report your firmware version, that would be helpful.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>50668</commentid>
    <comment_count>1</comment_count>
    <who name="Darren Hart">dvhart</who>
    <bug_when>2015-04-30 15:38:13 +0000</bug_when>
    <thetext>Scanning through the edk2 sources, I see the following defaults for VLV2 devices, which explicitly returns this value (26.8 deg C) if &quot;both EC and DTS are unavailable&quot;.

If this always returns the same value, should we be removing this from the table? Or at least disabling it via _STA?

   // _TMP (Temperature)
    //
    // Return the highest of the CPU temperatures to the OS.
    //
    // Arguments: (0)
    //   None
    // Return Value:
    //   An Integer containing the current temperature of the thermal zone (in tenths of degrees Kelvin)
    //
    Method(_TMP,0,Serialized)
    {
      If(DTSE)
      {
        If(LGreater(DTS2, DTS1))
        {
          Store(DTS2,Local0)
        } else
        {
          Store(DTS1,Local0)
        }
        Return(Add(2732,Multiply(Local0,10)))
        //
        // Else return a static value if both EC and DTS are unavailable.
        //
      } Else
      {
        Return(3000) // (3000-2732)/10 = 26.8 degree C
      }
    }</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>50674</commentid>
    <comment_count>2</comment_count>
    <who name="Gerard Bucas">gerard.b</who>
    <bug_when>2015-04-30 16:58:24 +0000</bug_when>
    <thetext>Problem reported when using firmware: 
MNW2MAX1.X64.0079.R01.1504081555

- Gerard Bucas (gerard.b@tekmagic.net)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>50950</commentid>
    <comment_count>3</comment_count>
    <who name="David_Wei">david.wei</who>
    <bug_when>2015-05-13 06:18:05 +0000</bug_when>
    <thetext>We are trying to enable Digital Thermal Sensor to detect the temperature.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>51217</commentid>
    <comment_count>4</comment_count>
    <who name="David_Wei">david.wei</who>
    <bug_when>2015-05-22 06:08:44 +0000</bug_when>
    <thetext>Firware team is optimizing Digital Thermal Sensor driver to make it able to detect smaller changes of processor temperature.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>51277</commentid>
    <comment_count>5</comment_count>
    <who name="David_Wei">david.wei</who>
    <bug_when>2015-05-26 23:23:38 +0000</bug_when>
    <thetext>Shifei has finished adding back Digital Thermal Sensor and tested its function. V0.81 release will include this feature.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>52062</commentid>
    <comment_count>6</comment_count>
    <who name="David_Wei">david.wei</who>
    <bug_when>2015-06-30 23:34:50 +0000</bug_when>
    <thetext>This issue has been fixed by V0.81 release. Please review it.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>52088</commentid>
    <comment_count>7</comment_count>
    <who name="Gerard Bucas">gerard.b</who>
    <bug_when>2015-07-02 05:47:09 +0000</bug_when>
    <thetext>Ok - tested it on a minnowboard MAX with latest 0.81 firmware. 

It now behaves differently and after Linux starts in root shell it returns 60000 (= 60.0C). At that point the sensors command returns around 65C

However, then I run some heavy CPU/GPU content and the value returned by temp only rises to 70000 (70.0C) and then stats there, even though &apos;sensors&apos; now reports 92C.

So the &apos;virtual device&apos; seems to get stuck at a max of 70000, even though CPU (die) temp has gone up to 92C (stabilises around 92C with the content I am running). 

So my conclusion is that the behavior has changed (no longer returns a fixed/constant value like before) but it still does not report correct temp and always returns a value between 60000 and 70000.

A good &apos;check&apos; is to compare the results of the Linux &apos;sensors&apos; command - which I did.

Hope this helps. 

Regards
Gerard Bucas
Gerard.b@tekmagic.net</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>52328</commentid>
    <comment_count>8</comment_count>
    <who name="David_Wei">david.wei</who>
    <bug_when>2015-07-14 06:14:29 +0000</bug_when>
    <thetext>Assign to Shifei to check the ACPI thermal zone.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>52822</commentid>
    <comment_count>9</comment_count>
    <who name="Lu_Shifei">shifeix.a.lu</who>
    <bug_when>2015-08-05 07:13:10 +0000</bug_when>
    <thetext>Please double check with V0.82 Firmware Binary.Thanks a lot!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>52957</commentid>
    <comment_count>10</comment_count>
    <who name="Gerard Bucas">gerard.b</who>
    <bug_when>2015-08-10 16:50:29 +0000</bug_when>
    <thetext>OK - I checked it out with 0.82 firmware and linux 3.19 kernel (ubuntu &amp; yocto) running on a minnowboard MAX.

Seems to work and reports similar results now to &quot;sensors&quot; command.

Thanks for fixing this - much appreciated.

- Gerard</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>53006</commentid>
    <comment_count>11</comment_count>
    <who name="David_Wei">david.wei</who>
    <bug_when>2015-08-11 23:25:48 +0000</bug_when>
    <thetext>Hi Dareen,
Gerard has confirmed this fix. Could you help change this bug status to &quot;resolved&quot;?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>74107</commentid>
    <comment_count>12</comment_count>
    <who name="Stephen K Jolley">sjolley.yp.pm</who>
    <bug_when>2017-06-13 18:08:08 +0000</bug_when>
    <thetext>We are no longer tracking in Yocto Project Bugzilla MinnowBoard and MinnowBoard-MAX HW and firmware bugs.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>