Re: [PATCH] ARM: dts: veyron: enable the tsadc on pinky board

Hi Caesar,

Am Freitag, 25. März 2016, 08:58:01 schrieb Caesar Wang:

在 2016年03月25日 05:12, Heiko Stübner 写道:

quoted

Am Sonntag, 20. Dezember 2015, 18:30:34 schrieb Caesar Wang:

quoted

I think the tsadc is fakly hand on pinky board,
maybe that's fixed in newest kernel.
As the following patch is a example:
https://patchwork.kernel.org/patch/7472051/

I don't have meet this issue on pinky board.
Let me know if that's still hanging on pinky board.

Signed-off-by: Caesar Wang <redacted>

I just gave that a try again on my pinky-rev2. And while the tsadc did
produce some results and didn't hang the system in my tries, the
temperature values seemed to be frozen (only changing once on every
reboot).

Running the same kernel on a Jerry produced the expected results of the
temperature changing over time and differing cpu frequencies.

Pinky also is a sort of dinosaur and only you and me seem to still like
them> 
:-) . And something is definitly strange here, and I'd tend to keep the
:tsadc> 
disabled for the time being.

I remember the same issue on
https://code.google.com/p/chrome-os-partner/issues/detail?id=35060 in
the past

"thermal can 't work if  the hardware C1106 isn't removed in the
attachment."

So I think you can change this early board.

As I'm a total noob when it comes to soldering (especially on such a tiny 
scale), I don't think that is an option ;-) .

Reading that bug report again, brought back memories, that exactly this was 
the issue why suspend didn't work correctly for me then.

So I still think I'll keep the tsadc on pinky as it is now - as I said before, 
it looks like you and me are the only people still caring for pinky and 
everybody else most likely put theirs into the trash already, so nothing 
terrible important is affected by this :-)


Heiko
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html