Re: [PATCH] Driver for omap34xx temperature sensor. second version.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1


Op 29 mei 2008, om 01:12 heeft Tony Lindgren het volgende geschreven:

* Koen Kooi <k.kooi@xxxxxxxxxxxxxxxxxx> [080528 16:08]:

Op 23 mei 2008, om 12:49 heeft Peter 'p2' De Schrijver het volgende
geschreven:

Hi,

This patch adds a hwmon driver for the OMAP34xx internal temperature
sensor. I get unusually high values (80 degrees and more) from this
sensor though.

Same high values here as well:

root@beagleboard:/sys/class/hwmon/hwmon0/device# cat temp1_input
96

does anyone from TI have a hint if the chip is really that hot, or
that we need to scale the value?

Maybe it's in Fahrenheit?

That was my first thought, but looking that the omap3530 TRM (spruf98.pdf) on page 836:

ADC code  Temperature °C
                    From    To
0                 -40         -40
32               -0.7        0.8
64                44.9      46.3
96                89.3      90.6

And that maps to the adc_to_temp var at line 69 from http://source.mvista.com/git/gitweb.cgi?p=linux-omap-2.6.git;a=blob;f=drivers/hwmon/omap34xx_temp.c
Even if it didn't map, the raw ADC value is only ±6 °C off from the real temperature around that point. So the adc value the kernel reads gets translated correctly to °C, so if we think it's incorrect, the kernel might be reading the ADC wrong.

regards,

Koen




Tony


-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.5 (Darwin)

iD8DBQFIPlmiMkyGM64RGpERAgfsAJ9H6kOLLP3T4Nlt03UwmwXMsl4lBQCfcJVz
j8wEOY13ggcCsXY4rU7FOuA=
=//co
-----END PGP SIGNATURE-----
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux