Bug 16584 - ambient light sensor readout broken
: ambient light sensor readout broken
Status: VERIFIED FIXED
Product: SB Radio
Classification: Unclassified
Component: OS/Bootloader
: Include FW version in comment
: PC Other
: P1 critical with 2 votes (vote)
: 7.5.1
Assigned To: Ryan
http://forums.slimdevices.com/showthr...
: factory_mp
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2010-09-30 06:43 UTC by Felix Mueller
Modified: 2011-08-18 00:00 UTC (History)
7 users (show)

See Also:
Category: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Felix Mueller 2010-09-30 06:43:04 UTC
After a factory reset (and re-programming msp430) I get ok readouts from the ambient light sensor. Values of a thousand the darker it is in the room.

cat /sys/bus/i2c/devices/1-0010/ambient

However after shutdown and rebooting (w/o factory reset / re-porgramming) I only get values 0 or 1.

7.6 r9136
Comment 1 Felix Mueller 2010-09-30 06:55:47 UTC
You can also see the ambient light sensor readout in Settings / Advanced / Factory Test / Ambient Light Sensor

BTW: Even in the 'good' case the value isn't going high enough in dark room situations, it should go up to 100000, but I only see something like 34464

Goodish readout: bright room: 0 dark room: 34464 (should be 100000)

Bad readout: bright room: 0 dark room: 1
Comment 2 Felix Mueller 2010-09-30 07:13:13 UTC
After factory reset:

7.5.1 r9009 (msp430 1316) - 0 to 100000

7.6.0 r9136 (msp430 1336) - 0 or 34464


After reboot:

7.5.1 r9009 (msp430 1316) - 0 to 100000

7.6.0 r9136 (msp430 1336) - 0 or 1
Comment 3 Felix Mueller 2010-09-30 07:20:43 UTC
The problem manifests itself even without starting SP and only reading from /sys/bus/i2c/devices/1-0010/ambient I already get the wrong values.
Comment 4 Felix Mueller 2010-09-30 07:40:54 UTC
I've just tested this with 7.5.1 r9136 (msp430 1403) and it's broken too.

After factory reset: 0 to 34464

After reboot : 0 or 1
Comment 5 Vahid Fereydouny 2010-09-30 11:06:32 UTC
Working on this bug. Planning to fix it before end of the day
Comment 6 Vahid Fereydouny 2010-09-30 11:59:02 UTC
I fixed the bug and tested the Ambient values by covering the Baby with a black cover. The Ambient goes all the way up to 100,000 which is the expected value. The changes are checked into 7.5.1 and will be integrated into 7.6
Comment 7 Felix Mueller 2010-09-30 12:59:53 UTC
7.5.1 r9142 (msp430 1406) still shows the issue after a reboot.

After a factory reset (reprogramming msp430) I get 100000 as expected.

But after a simple power off / power on I still only get the 0 or 1.
Comment 8 Felix Mueller 2010-09-30 13:09:01 UTC
*** Bug 16585 has been marked as a duplicate of this bug. ***
Comment 9 Vahid Fereydouny 2010-10-01 00:15:19 UTC
This is an intermittent problem that is caused by the changes that has been made in the initialization sequence. In progress.
Comment 10 Michael 2010-10-08 10:59:33 UTC
With the latest firmware 7.60 r9155: Still not working.

It dims a little, but nowhere near as dark as before. in a completely dark room, brightness is still about 30% or so ...
Comment 11 Vahid Fereydouny 2010-10-13 13:17:45 UTC
Fixed in both 7.5 and 7.6. Felix: Could you please let me know the result.
Comment 12 Felix Mueller 2010-10-13 23:17:04 UTC
These are my results for 7.5.1 r9172:

I had to force an msp430 reprogramming via factory reset. Have you maybe forgotten to increase the msp430 version?

1. Downloaded 7.5.1 r9172 from parabuild and installed it on SB Radio.
2. According to the log msp430 1406 was found (no reprogramming)
3. Ambient light sensor did _not_ work and I only got 0 or 1
4. Factory reset SB Radio
5. According to the log msp430 1406 got programmed
6. Ambient light sensor ok
7. Hard power down
8. Power up
9. According to the log msp430 1406 is found (no reprogramming)
10. Ambient light sensor ok

I repeated steps 7 to 10 about five times and the ambient light sensor was ok every time.
Comment 13 Felix Mueller 2010-10-13 23:28:27 UTC
These are my results for 7.6.0 r9174:

This time on factory reset was necessary as I assume msp430 was already running the correct firmware.

1. Downloaded 7.6.0 r9174 from parabuild and installed it on SB Radio.
2. According to the log msp430 1406 was found (no reprogramming)
3. Ambient light sensor ok
4. Hard power down
5. Power up
6. According to the log msp430 1406 is found (no reprogramming)
7. Ambient light sensor ok

I repeated steps 4 to 7 about five times and the ambient light sensor was ok
every time.
Comment 14 Felix Mueller 2010-10-13 23:29:25 UTC
Ryan - may I ask you to double-check this fix?

Thanks
Felix
Comment 15 Michael 2010-10-13 23:53:54 UTC
Please excuse a maybe dumb question, but what is Parabuild? I would like to check this firmware myself.
Comment 16 Felix Mueller 2010-10-14 00:02:41 UTC
Hello Michael

Sorry for the confusion. Parabuild is our internal build server system.

I am afraid you need to wait until our QA team has reviewed this firmware and then you'll be able to get it via nightly build.

Felix
Comment 17 Michael 2010-10-14 00:05:16 UTC
Alright, thanks für the quick response Felix.

Michael
Comment 18 Bill Burns 2010-10-21 18:00:47 UTC
When will 7.6.0 r9174 be released to field testers?  I'm running 7.6.0 - r31450 but have not been offered Radio firmware beyond 7.6.0-r9155.

My Touch updates its firmware every day or so, and is at 7.6.0-r9190

Bill
Comment 19 Bill Burns 2010-10-23 07:12:46 UTC
Firmware 7.6.0-r9197 for the Radio, pushed yesterday, has restored the original auto dimming capability.

Bill
Comment 20 Kris Murphy 2010-11-17 15:28:49 UTC
Verified fixed in Baby fw 9219
Comment 21 Dennis Mutsaers 2011-08-17 22:34:05 UTC
Seems to be broken again in latest 7.6.1 releases...
Comment 22 Felix Mueller 2011-08-18 00:00:27 UTC
Hello Dennis

Could you please be more specific about the release and also give me the build revision, i.e. r9xxx?

Please also go into Settings - Advanced - Diagnostics - Power and report back the MSP version.

I've checked the automatic brightness on my SB Radio running 7.6.1 r9468 (MSP 1463) and it works as it should.

Thanks
Felix