Ticket #12243 (closed task: fixed)

Opened 23 months ago

Last modified 18 months ago

XO-4 outdoor light sensor support

Reported by: dsd Owned by: pgf
Priority: normal Milestone: 13.1.0
Component: power manager (powerd) Version: not specified
Keywords: Cc:
Action Needed: test in build Verified: no
Deployments affected: Blocked By:
Blocking:

Description

Just so that we have this in track: outdoor light sensor support needs implementing on XO-4.

Change History

Changed 23 months ago by pgf

it is implemented.

is it broken?

Changed 23 months ago by dsd

It didn't work with a very quick test at the weekend and someone else added it as a known issue to the 13.1.0 page. So I think it might be.

I'll do better testing/diagnosis this afternoon.

Changed 23 months ago by pgf

i've tested, you're right. the name of the device node in /sys has changed from /sys/devices/ols.3 (when i added support) to /sys/devices/ols.4, so powerd doesn't find it.

Changed 23 months ago by dsd

  • component changed from kernel to power manager (powerd)

Just ran exactly the same experiment at exactly the same time. Changing the path solves the issue.

Changed 22 months ago by pgf

there are two parts to this fix. one is establishing uniform device naming from OFW. that's in q7b05 (might have been in q7b04). the other part is changing powerd to use the new, stable, name to reference the OLS (/sys/devices/0.ols). i'm waiting on semi-functional suspend/resume before doing a new powerd release.

Changed 21 months ago by pgf

  • next_action changed from code to add to build

fixed in powerd-106, will probably appear in os20.

Changed 21 months ago by dsd

Thanks. It missed build 20 but will be in the following one.

Changed 20 months ago by dsd

  • next_action changed from add to build to test in build

Test in 13.1.0 build 21.

Changed 18 months ago by pgf

  • status changed from new to closed
  • resolution set to fixed

this has been fixed for some time. just verified in os30-something.

Note: See TracTickets for help on using tickets.