Re: [PATCH] ARM: dts: omap3: Include missing bandgap data for ti-soc-thermal driver
* Sebastian Reichel [160127 10:20]: > Hi, > > On Wed, Jan 27, 2016 at 08:09:46AM -0800, Tony Lindgren wrote: > > * Pali Rohár [160127 02:05]: > > > On Thursday 21 January 2016 09:15:01 Tony Lindgren wrote: > > > > * Pali Rohár [160121 01:48]: > > > > > On Saturday 26 December 2015 00:32:25 Pali Rohár wrote: > > > > > > Driver for omap3 with documentation is there since v4.4-rc1. > > > > ... > > > > > > > > > PING! Who can take this patch? > > > > > > > > Sounds like /me, but after -rc1 when any possible regressions > > > > are out of the way :) > > > > > > Great! Just to note that 4.5-rc1 is out. > > > > > > I just do not want to see this patch lost... > > > > So is this critical to have as a fix since it's thermal related? > > According to somebody from TI, who previously took part in the patch > discussion, the thermal sensors of omap3 are to imprecise to be used > for thermal decisions. The driver "just" informs userspace about the > temperatures. OK applying into omap-for-v4.6/dt then thanks. Tony
Re: [PATCH] ARM: dts: omap3: Include missing bandgap data for ti-soc-thermal driver
Hi, On Wed, Jan 27, 2016 at 08:09:46AM -0800, Tony Lindgren wrote: > * Pali Rohár [160127 02:05]: > > On Thursday 21 January 2016 09:15:01 Tony Lindgren wrote: > > > * Pali Rohár [160121 01:48]: > > > > On Saturday 26 December 2015 00:32:25 Pali Rohár wrote: > > > > > Driver for omap3 with documentation is there since v4.4-rc1. > > > ... > > > > > > > PING! Who can take this patch? > > > > > > Sounds like /me, but after -rc1 when any possible regressions > > > are out of the way :) > > > > Great! Just to note that 4.5-rc1 is out. > > > > I just do not want to see this patch lost... > > So is this critical to have as a fix since it's thermal related? According to somebody from TI, who previously took part in the patch discussion, the thermal sensors of omap3 are to imprecise to be used for thermal decisions. The driver "just" informs userspace about the temperatures. -- Sebastian signature.asc Description: PGP signature
Re: [PATCH] ARM: dts: omap3: Include missing bandgap data for ti-soc-thermal driver
* Pali Rohár [160127 02:05]: > On Thursday 21 January 2016 09:15:01 Tony Lindgren wrote: > > * Pali Rohár [160121 01:48]: > > > On Saturday 26 December 2015 00:32:25 Pali Rohár wrote: > > > > Driver for omap3 with documentation is there since v4.4-rc1. > > ... > > > > > PING! Who can take this patch? > > > > Sounds like /me, but after -rc1 when any possible regressions > > are out of the way :) > > Great! Just to note that 4.5-rc1 is out. > > I just do not want to see this patch lost... So is this critical to have as a fix since it's thermal related? Regards, Tony
Re: [PATCH] ARM: dts: omap3: Include missing bandgap data for ti-soc-thermal driver
On Thursday 21 January 2016 09:15:01 Tony Lindgren wrote: > * Pali Rohár [160121 01:48]: > > On Saturday 26 December 2015 00:32:25 Pali Rohár wrote: > > > Driver for omap3 with documentation is there since v4.4-rc1. > ... > > > PING! Who can take this patch? > > Sounds like /me, but after -rc1 when any possible regressions > are out of the way :) Great! Just to note that 4.5-rc1 is out. I just do not want to see this patch lost... -- Pali Rohár pali.ro...@gmail.com
Re: [PATCH] ARM: dts: omap3: Include missing bandgap data for ti-soc-thermal driver
* Pali Rohár [160121 01:48]: > On Saturday 26 December 2015 00:32:25 Pali Rohár wrote: > > Driver for omap3 with documentation is there since v4.4-rc1. ... > PING! Who can take this patch? Sounds like /me, but after -rc1 when any possible regressions are out of the way :) Regards, Tony
Re: [PATCH] ARM: dts: omap3: Include missing bandgap data for ti-soc-thermal driver
On Saturday 26 December 2015 00:32:25 Pali Rohár wrote:
> Driver for omap3 with documentation is there since v4.4-rc1.
>
> Signed-off-by: Pali Rohár
> ---
> arch/arm/boot/dts/omap34xx.dtsi |5 +
> arch/arm/boot/dts/omap36xx.dtsi |5 +
> 2 files changed, 10 insertions(+)
>
> diff --git a/arch/arm/boot/dts/omap34xx.dtsi b/arch/arm/boot/dts/omap34xx.dtsi
> index 4f6b2d5..c3f488f 100644
> --- a/arch/arm/boot/dts/omap34xx.dtsi
> +++ b/arch/arm/boot/dts/omap34xx.dtsi
> @@ -54,6 +54,11 @@
> #size-cells = <0>;
> };
> };
> +
> + bandgap {
> + reg = <0x48002524 0x4>;
> + compatible = "ti,omap34xx-bandgap";
> + };
> };
> };
>
> diff --git a/arch/arm/boot/dts/omap36xx.dtsi b/arch/arm/boot/dts/omap36xx.dtsi
> index 86253de..00f98c1 100644
> --- a/arch/arm/boot/dts/omap36xx.dtsi
> +++ b/arch/arm/boot/dts/omap36xx.dtsi
> @@ -86,6 +86,11 @@
> #size-cells = <0>;
> };
> };
> +
> + bandgap {
> + reg = <0x48002524 0x4>;
> + compatible = "ti,omap36xx-bandgap";
> + };
> };
> };
>
PING! Who can take this patch?
--
Pali Rohár
pali.ro...@gmail.com
Re: [PATCH] ARM: dts: omap3: Include missing bandgap data for ti-soc-thermal driver
On Thursday 31 December 2015 09:38:45 Eduardo Valentin wrote:
> > +
> > + bandgap {
> > + reg = <0x48002524 0x4>;
> > + compatible = "ti,omap36xx-bandgap";
>
> Can you please already add on both cases
>
> #thermal-sensor-cells = <0>;
> ?
>
> This way we can already use them to define thermal zones. Of course,
> that alone won't add the thermal zones. A separated patch would be
> needed to add the thermal zone for OMAP3.
Are you going to add thermal zone defines? If yes, then it would make
sense to add that #thermal line together with thermal zone defines...
--
Pali Rohár
pali.ro...@gmail.com
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] ARM: dts: omap3: Include missing bandgap data for ti-soc-thermal driver
Hello,
On Sat, Dec 26, 2015 at 12:32:25AM +0100, Pali Rohár wrote:
> Driver for omap3 with documentation is there since v4.4-rc1.
>
> Signed-off-by: Pali Rohár
> ---
> arch/arm/boot/dts/omap34xx.dtsi |5 +
> arch/arm/boot/dts/omap36xx.dtsi |5 +
> 2 files changed, 10 insertions(+)
>
> diff --git a/arch/arm/boot/dts/omap34xx.dtsi b/arch/arm/boot/dts/omap34xx.dtsi
> index 4f6b2d5..c3f488f 100644
> --- a/arch/arm/boot/dts/omap34xx.dtsi
> +++ b/arch/arm/boot/dts/omap34xx.dtsi
> @@ -54,6 +54,11 @@
> #size-cells = <0>;
> };
> };
> +
> + bandgap {
> + reg = <0x48002524 0x4>;
> + compatible = "ti,omap34xx-bandgap";
> + };
> };
> };
>
> diff --git a/arch/arm/boot/dts/omap36xx.dtsi b/arch/arm/boot/dts/omap36xx.dtsi
> index 86253de..00f98c1 100644
> --- a/arch/arm/boot/dts/omap36xx.dtsi
> +++ b/arch/arm/boot/dts/omap36xx.dtsi
> @@ -86,6 +86,11 @@
> #size-cells = <0>;
> };
> };
> +
> + bandgap {
> + reg = <0x48002524 0x4>;
> + compatible = "ti,omap36xx-bandgap";
Can you please already add on both cases
#thermal-sensor-cells = <0>;
?
This way we can already use them to define thermal zones. Of course,
that alone won't add the thermal zones. A separated patch would be
needed to add the thermal zone for OMAP3.
> + };
> };
> };
>
> --
> 1.7.9.5
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] ARM: dts: omap3: Include missing bandgap data for ti-soc-thermal driver
On Sat 2015-12-26 00:32:25, Pali Rohár wrote: > Driver for omap3 with documentation is there since v4.4-rc1. > > Signed-off-by: Pali Rohár Acked-by: Pavel Machek Tested-by: Pavel Machek Thanks! Pavel -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[PATCH] ARM: dts: omap3: Include missing bandgap data for ti-soc-thermal driver
Driver for omap3 with documentation is there since v4.4-rc1.
Signed-off-by: Pali Rohár
---
arch/arm/boot/dts/omap34xx.dtsi |5 +
arch/arm/boot/dts/omap36xx.dtsi |5 +
2 files changed, 10 insertions(+)
diff --git a/arch/arm/boot/dts/omap34xx.dtsi b/arch/arm/boot/dts/omap34xx.dtsi
index 4f6b2d5..c3f488f 100644
--- a/arch/arm/boot/dts/omap34xx.dtsi
+++ b/arch/arm/boot/dts/omap34xx.dtsi
@@ -54,6 +54,11 @@
#size-cells = <0>;
};
};
+
+ bandgap {
+ reg = <0x48002524 0x4>;
+ compatible = "ti,omap34xx-bandgap";
+ };
};
};
diff --git a/arch/arm/boot/dts/omap36xx.dtsi b/arch/arm/boot/dts/omap36xx.dtsi
index 86253de..00f98c1 100644
--- a/arch/arm/boot/dts/omap36xx.dtsi
+++ b/arch/arm/boot/dts/omap36xx.dtsi
@@ -86,6 +86,11 @@
#size-cells = <0>;
};
};
+
+ bandgap {
+ reg = <0x48002524 0x4>;
+ compatible = "ti,omap36xx-bandgap";
+ };
};
};
--
1.7.9.5
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
