Opened 3 years ago

Closed 3 years ago

#10954 closed task (fixed)

XO-1.75 runin CPU temperature support

Reported by: Quozl Owned by: buytenh
Priority: blocker Milestone: 1.75-software
Component: manufacturing process Version: not specified
Keywords: Cc: sascha_silbe
Blocked By: Blocking:
Deployments affected: Action Needed: no action
Verified: no

Description

runin-cpu_temp for XO-1.5 uses /sys/devices/platform/via_cputemp.0/temp1_input to determine CPU temperature.

Either we want something similar for XO-1.75, or confidence that we won't have to test CPU temperature.

Change History (18)

comment:1 Changed 3 years ago by Quozl

  • Blocking 10893 removed

(In #10893) runin-sus needs RTC support for rtcwake.

comment:2 Changed 3 years ago by Quozl

  • Blocking 10893 added

comment:3 Changed 3 years ago by Quozl

See #11000 for OpenFirmware implementation, in svn 2276. Comments there refer to a Linux driver, so I guess I just haven't found it yet.

comment:4 Changed 3 years ago by Quozl

Reproduced on os19.

comment:5 Changed 3 years ago by Quozl

Runin on XO-1.5 is designed to fail if the CPU temperature exceeds a characterisation point. Runin on XO-1.75 is missing kernel support for reading the CPU temperature. Runin inability to report this fail was added 15th August by Quanta as SW issue 10 to be completed before C-stage.

comment:6 Changed 3 years ago by pgf

driver for mmp2 is in drivers/hwmon/mmp2_thermal.c in any of the marvell kernel trees under /home/docs.

comment:7 Changed 3 years ago by sascha_silbe

  • Cc sascha_silbe added

comment:8 Changed 3 years ago by Quozl

comment:9 Changed 3 years ago by Quozl

there is no mmp2_thermal.c in olpc-kernel.

comment:10 Changed 3 years ago by martin.langhoff

I thought we had the sdkit implementation in git, perhaps even as an rpm.

Is there issue blocking the sdkit-based path?

comment:11 Changed 3 years ago by Quozl

Need a Fedora expert to figure out what is needed to package git://dev.laptop.org/users/quozl/runin branch master commit 4c9274f ... I know of no sdkit-arm implementation in git or rpm, and I don't yet know how sdkit-arm is built.

(for context, this is regarding the interim solution, the complete solution is to port the marvell mmp2_thermal.c to our kernel).

comment:12 Changed 3 years ago by Quozl

olpc-runin-tests 0.16.0 released with interim kernel CPU temperature support, tested on os10.

comment:13 Changed 3 years ago by Quozl

  • Action Needed changed from design to add to build

comment:14 Changed 3 years ago by martin.langhoff

  • Action Needed changed from add to build to test in build
  • Component changed from kernel to manufacturing process
  • Priority changed from normal to blocker
  • Summary changed from XO-1.75 kernel CPU temperature support to XO-1.75 runin CPU temperature support
  • Type changed from defect to task

Updated summary/component/pri/action to reflect that it's implemented in runin. Opened #11463 to track kernel support.

comment:15 Changed 3 years ago by Quozl

  • Blocking 10893 removed

comment:15 Changed 3 years ago by Quozl

no longer blocking

comment:16 Changed 3 years ago by Quozl

Still waiting for Quanta test, as at 23rd Nov, tracking list CL2 SW #10.

comment:17 Changed 3 years ago by Quozl

  • Action Needed changed from test in build to no action
  • Resolution set to fixed
  • Status changed from new to closed

Works.

Note: See TracTickets for help on using tickets.