diff mbox series

[07/10] ARM: dts: gemini: Remove "cooling-{min|max}-level" for gpio-fan node

Message ID 01ddf0ac8962feb63e8871a2c2c3498c763f649b.1518166039.git.viresh.kumar@linaro.org
State Accepted
Commit 953042d1536b0c089e00f096c69a75118b00b9f4
Headers show
Series None | expand

Commit Message

Viresh Kumar Feb. 9, 2018, 8:58 a.m. UTC
The "cooling-min-level" and "cooling-max-level" properties are not
parsed by any part of the kernel currently and the max cooling state of
gpio-fan cooling device is found by referring to the
"gpio-fan,speed-map" instead.

Remove the unused properties from the gpio-fan node.

Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>

---
 arch/arm/boot/dts/gemini-dlink-dns-313.dts | 2 --
 1 file changed, 2 deletions(-)

-- 
2.15.0.194.g9af6a3dea062

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

Comments

Arnd Bergmann March 27, 2018, 12:26 p.m. UTC | #1
On Tue, Mar 20, 2018 at 4:10 AM, Linus Walleij <linus.walleij@linaro.org> wrote:
> On Fri, Feb 9, 2018 at 9:58 AM, Viresh Kumar <viresh.kumar@linaro.org> wrote:

>

>> The "cooling-min-level" and "cooling-max-level" properties are not

>> parsed by any part of the kernel currently and the max cooling state of

>> gpio-fan cooling device is found by referring to the

>> "gpio-fan,speed-map" instead.

>>

>> Remove the unused properties from the gpio-fan node.

>>

>> Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>

>

> Acked-by: Linus Walleij <linus.walleij@linaro.org>

>

> I can queue it next week (different machine hosting the git tree)

> but it is also OK if the ARM SoC maintainers want to pick it

> directly for the ARM SoC DTS branch.


I've picked it up directly now, as we're about to run out of time
before the merge window.

       Arnd
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Linus Walleij March 27, 2018, 12:29 p.m. UTC | #2
On Tue, Mar 27, 2018 at 2:26 PM, Arnd Bergmann <arnd@arndb.de> wrote:
> On Tue, Mar 20, 2018 at 4:10 AM, Linus Walleij <linus.walleij@linaro.org> wrote:

>> On Fri, Feb 9, 2018 at 9:58 AM, Viresh Kumar <viresh.kumar@linaro.org> wrote:

>>

>>> The "cooling-min-level" and "cooling-max-level" properties are not

>>> parsed by any part of the kernel currently and the max cooling state of

>>> gpio-fan cooling device is found by referring to the

>>> "gpio-fan,speed-map" instead.

>>>

>>> Remove the unused properties from the gpio-fan node.

>>>

>>> Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>

>>

>> Acked-by: Linus Walleij <linus.walleij@linaro.org>

>>

>> I can queue it next week (different machine hosting the git tree)

>> but it is also OK if the ARM SoC maintainers want to pick it

>> directly for the ARM SoC DTS branch.

>

> I've picked it up directly now, as we're about to run out of time

> before the merge window.


Thanks, I was just about to queue it and send a pull request but now
I don't need to :)

Yours,
Linus Walleij
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox series

Patch

diff --git a/arch/arm/boot/dts/gemini-dlink-dns-313.dts b/arch/arm/boot/dts/gemini-dlink-dns-313.dts
index 08568ce24d06..727d16eb02d9 100644
--- a/arch/arm/boot/dts/gemini-dlink-dns-313.dts
+++ b/arch/arm/boot/dts/gemini-dlink-dns-313.dts
@@ -78,8 +78,6 @@ 
 		gpios = <&gpio0 11 GPIO_ACTIVE_HIGH>,
 			<&gpio0 12 GPIO_ACTIVE_HIGH>;
 		gpio-fan,speed-map = <0 0>, <3000 1>, <6000 2>;
-		cooling-min-level = <0>;
-		cooling-max-level = <2>;
 		#cooling-cells = <2>;
 	};