* [PATCH] ARM: dts: veyron: enable the tsadc on pinky board
@ 2015-12-20 10:30 ` Caesar Wang
0 siblings, 0 replies; 9+ messages in thread
From: Caesar Wang @ 2015-12-20 10:30 UTC (permalink / raw)
To: heiko
Cc: linux-kernel, linux-arm-kernel, linux-rockchip, devicetree, Caesar Wang
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 <wxt@rock-chips.com>
---
arch/arm/boot/dts/rk3288-veyron-pinky.dts | 5 -----
1 file changed, 5 deletions(-)
diff --git a/arch/arm/boot/dts/rk3288-veyron-pinky.dts b/arch/arm/boot/dts/rk3288-veyron-pinky.dts
index 94b56e3..b587ebf 100644
--- a/arch/arm/boot/dts/rk3288-veyron-pinky.dts
+++ b/arch/arm/boot/dts/rk3288-veyron-pinky.dts
@@ -121,8 +121,3 @@
&sdmmc_wp_gpio &sdmmc_bus4>;
wp-gpios = <&gpio7 10 GPIO_ACTIVE_HIGH>;
};
-
-&tsadc {
- /* Some connection is flaky making the tsadc hang the system */
- status = "disabled";
-};
--
1.9.1
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH] ARM: dts: veyron: enable the tsadc on pinky board
@ 2015-12-20 10:30 ` Caesar Wang
0 siblings, 0 replies; 9+ messages in thread
From: Caesar Wang @ 2015-12-20 10:30 UTC (permalink / raw)
To: heiko-4mtYJXux2i+zQB+pC5nmwQ
Cc: linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Caesar Wang,
linux-kernel-u79uwXL29TY76Z2rM5mHXA,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
devicetree-u79uwXL29TY76Z2rM5mHXA
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 <wxt-TNX95d0MmH7DzftRWevZcw@public.gmane.org>
---
arch/arm/boot/dts/rk3288-veyron-pinky.dts | 5 -----
1 file changed, 5 deletions(-)
diff --git a/arch/arm/boot/dts/rk3288-veyron-pinky.dts b/arch/arm/boot/dts/rk3288-veyron-pinky.dts
index 94b56e3..b587ebf 100644
--- a/arch/arm/boot/dts/rk3288-veyron-pinky.dts
+++ b/arch/arm/boot/dts/rk3288-veyron-pinky.dts
@@ -121,8 +121,3 @@
&sdmmc_wp_gpio &sdmmc_bus4>;
wp-gpios = <&gpio7 10 GPIO_ACTIVE_HIGH>;
};
-
-&tsadc {
- /* Some connection is flaky making the tsadc hang the system */
- status = "disabled";
-};
--
1.9.1
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH] ARM: dts: veyron: enable the tsadc on pinky board
@ 2015-12-20 10:30 ` Caesar Wang
0 siblings, 0 replies; 9+ messages in thread
From: Caesar Wang @ 2015-12-20 10:30 UTC (permalink / raw)
To: linux-arm-kernel
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 <wxt@rock-chips.com>
---
arch/arm/boot/dts/rk3288-veyron-pinky.dts | 5 -----
1 file changed, 5 deletions(-)
diff --git a/arch/arm/boot/dts/rk3288-veyron-pinky.dts b/arch/arm/boot/dts/rk3288-veyron-pinky.dts
index 94b56e3..b587ebf 100644
--- a/arch/arm/boot/dts/rk3288-veyron-pinky.dts
+++ b/arch/arm/boot/dts/rk3288-veyron-pinky.dts
@@ -121,8 +121,3 @@
&sdmmc_wp_gpio &sdmmc_bus4>;
wp-gpios = <&gpio7 10 GPIO_ACTIVE_HIGH>;
};
-
-&tsadc {
- /* Some connection is flaky making the tsadc hang the system */
- status = "disabled";
-};
--
1.9.1
^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH] ARM: dts: veyron: enable the tsadc on pinky board
@ 2016-03-24 21:12 ` Heiko Stübner
0 siblings, 0 replies; 9+ messages in thread
From: Heiko Stübner @ 2016-03-24 21:12 UTC (permalink / raw)
To: Caesar Wang; +Cc: linux-kernel, linux-arm-kernel, linux-rockchip, devicetree
Hi Caesar,
Am Sonntag, 20. Dezember 2015, 18:30:34 schrieb Caesar Wang:
> 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 <wxt@rock-chips.com>
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.
Heiko
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] ARM: dts: veyron: enable the tsadc on pinky board
@ 2016-03-24 21:12 ` Heiko Stübner
0 siblings, 0 replies; 9+ messages in thread
From: Heiko Stübner @ 2016-03-24 21:12 UTC (permalink / raw)
To: Caesar Wang
Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
devicetree-u79uwXL29TY76Z2rM5mHXA
Hi Caesar,
Am Sonntag, 20. Dezember 2015, 18:30:34 schrieb Caesar Wang:
> 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 <wxt-TNX95d0MmH7DzftRWevZcw@public.gmane.org>
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.
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
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH] ARM: dts: veyron: enable the tsadc on pinky board
@ 2016-03-24 21:12 ` Heiko Stübner
0 siblings, 0 replies; 9+ messages in thread
From: Heiko Stübner @ 2016-03-24 21:12 UTC (permalink / raw)
To: linux-arm-kernel
Hi Caesar,
Am Sonntag, 20. Dezember 2015, 18:30:34 schrieb Caesar Wang:
> 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 <wxt@rock-chips.com>
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.
Heiko
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] ARM: dts: veyron: enable the tsadc on pinky board
@ 2016-03-27 21:30 ` Heiko Stübner
0 siblings, 0 replies; 9+ messages in thread
From: Heiko Stübner @ 2016-03-27 21:30 UTC (permalink / raw)
To: Caesar Wang; +Cc: linux-kernel, linux-arm-kernel, linux-rockchip, devicetree
Hi Caesar,
Am Freitag, 25. März 2016, 08:58:01 schrieb Caesar Wang:
> 在 2016年03月25日 05:12, Heiko Stübner 写道:
> > Am Sonntag, 20. Dezember 2015, 18:30:34 schrieb Caesar Wang:
> >> 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 <wxt@rock-chips.com>
> >
> > 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
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] ARM: dts: veyron: enable the tsadc on pinky board
@ 2016-03-27 21:30 ` Heiko Stübner
0 siblings, 0 replies; 9+ messages in thread
From: Heiko Stübner @ 2016-03-27 21:30 UTC (permalink / raw)
To: Caesar Wang
Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
devicetree-u79uwXL29TY76Z2rM5mHXA
Hi Caesar,
Am Freitag, 25. März 2016, 08:58:01 schrieb Caesar Wang:
> 在 2016年03月25日 05:12, Heiko Stübner 写道:
> > Am Sonntag, 20. Dezember 2015, 18:30:34 schrieb Caesar Wang:
> >> 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 <wxt-TNX95d0MmH7DzftRWevZcw@public.gmane.org>
> >
> > 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
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH] ARM: dts: veyron: enable the tsadc on pinky board
@ 2016-03-27 21:30 ` Heiko Stübner
0 siblings, 0 replies; 9+ messages in thread
From: Heiko Stübner @ 2016-03-27 21:30 UTC (permalink / raw)
To: linux-arm-kernel
Hi Caesar,
Am Freitag, 25. M?rz 2016, 08:58:01 schrieb Caesar Wang:
> ? 2016?03?25? 05:12, Heiko St?bner ??:
> > Am Sonntag, 20. Dezember 2015, 18:30:34 schrieb Caesar Wang:
> >> 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 <wxt@rock-chips.com>
> >
> > 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
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2016-03-27 21:30 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-12-20 10:30 [PATCH] ARM: dts: veyron: enable the tsadc on pinky board Caesar Wang
2015-12-20 10:30 ` Caesar Wang
2015-12-20 10:30 ` Caesar Wang
2016-03-24 21:12 ` Heiko Stübner
2016-03-24 21:12 ` Heiko Stübner
2016-03-24 21:12 ` Heiko Stübner
[not found] ` <56F48D19.1050702@gmail.com>
2016-03-27 21:30 ` Heiko Stübner
2016-03-27 21:30 ` Heiko Stübner
2016-03-27 21:30 ` Heiko Stübner
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.