From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751068AbcB2Fyd (ORCPT ); Mon, 29 Feb 2016 00:54:33 -0500 Received: from hqemgate16.nvidia.com ([216.228.121.65]:5745 "EHLO hqemgate16.nvidia.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750778AbcB2Fya convert rfc822-to-8bit (ORCPT ); Mon, 29 Feb 2016 00:54:30 -0500 X-PGP-Universal: processed; by hqnvupgp07.nvidia.com on Sun, 28 Feb 2016 21:53:19 -0800 Message-ID: <56D3DD1A.40208@nvidia.com> Date: Mon, 29 Feb 2016 13:54:34 +0800 From: Wei Ni User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.7.0 MIME-Version: 1.0 To: , , CC: , , , , Subject: Re: [PATCH v6 00/12] Add T210 support in Tegra soctherm References: <1456128340-28800-1-git-send-email-wni@nvidia.com> In-Reply-To: <1456128340-28800-1-git-send-email-wni@nvidia.com> X-Originating-IP: [10.19.224.146] X-ClientProxiedBy: DRBGMAIL104.nvidia.com (10.18.16.23) To HKMAIL101.nvidia.com (10.18.16.10) Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi, Does anyone have comments on this series? Thanks. Wei. On 2016年02月22日 16:05, Wei Ni wrote: > This patchset adds following functions for tegra_soctherm driver: > 1. add T210 support. > 2. export debugfs to show some registers. > 3. add thermtrip funciton. > 4. add suspend/resume function. > > The v5 serial is in: > http://www.spinics.net/lists/linux-tegra/msg25079.html > The v4 serial is in: > http://www.spinics.net/lists/linux-tegra/msg24972.html > The V3 serial is in: > http://www.spinics.net/lists/linux-tegra/msg24911.html > The V2 serial is in: > http://www.spinics.net/lists/linux-tegra/msg24901.html > The V1 serial is in: > http://www.spinics.net/lists/linux-tegra/msg24808.html > > Main changes from V5: > 1. Change to use linux thermal framework to implement > thermtrip funciton, per Rob's comment. > 2. Add .set_trip_temp() in of-thermal driver, so that > we can set trips on hardware. > > Main changes from V4: > 1. Change description of devicetree binding per Rob's comment. > 2. Call of_node_put to decrement refcount of the node. > > Main changes from V3: > 1. Change structures to "const" in chip specific files. > 2. Minor changes per Thieery's comments. > > Main changes from V2: > 1. Fix build error in patch [1/11]. > 2. Use of_get_child_by_name instead of of_find_node_by_name in patch [8/11]. > 3. Use debugfs_remove_recursive to remove debugfs in patch [6/11]. > > Main changes from V1: > 1. Use the new type to handl different Tegra chips in one driver, which suggested by Thierry. > 2. Changes per Thieery's other comments. > > Wei Ni (12): > thermal: tegra: move tegra thermal files into tegra directory > thermal: tegra: combine sensor group-related data > thermal: tegra: get rid of PDIV/HOTSPOT hack > thermal: tegra: split tegra_soctherm driver > thermal: tegra: add Tegra210 specific SOC_THERM driver > thermal: tegra: add a debugfs to show registers > thermal: of-thermal: allow setting trip_temp on hardware > of: add notes of critical trips for soctherm > thermal: tegra: add thermtrip function > thermal: tegra: add PM support > arm64: tegra: add soctherm node for Tegra210 > arm: tegra: set critical trips for Tegra124 > > .../devicetree/bindings/thermal/tegra-soctherm.txt | 12 + > arch/arm/boot/dts/tegra124.dtsi | 16 + > arch/arm64/boot/dts/nvidia/tegra210.dtsi | 60 ++ > drivers/thermal/Kconfig | 12 +- > drivers/thermal/Makefile | 2 +- > drivers/thermal/of-thermal.c | 8 + > drivers/thermal/tegra/Kconfig | 13 + > drivers/thermal/tegra/Makefile | 5 + > drivers/thermal/tegra/soctherm-fuse.c | 169 +++++ > drivers/thermal/tegra/soctherm.c | 685 +++++++++++++++++++++ > drivers/thermal/tegra/soctherm.h | 123 ++++ > drivers/thermal/tegra/tegra124-soctherm.c | 196 ++++++ > drivers/thermal/tegra/tegra210-soctherm.c | 197 ++++++ > drivers/thermal/tegra_soctherm.c | 476 -------------- > include/dt-bindings/thermal/tegra124-soctherm.h | 1 + > include/linux/thermal.h | 1 + > 16 files changed, 1489 insertions(+), 487 deletions(-) > create mode 100644 drivers/thermal/tegra/Kconfig > create mode 100644 drivers/thermal/tegra/Makefile > create mode 100644 drivers/thermal/tegra/soctherm-fuse.c > create mode 100644 drivers/thermal/tegra/soctherm.c > create mode 100644 drivers/thermal/tegra/soctherm.h > create mode 100644 drivers/thermal/tegra/tegra124-soctherm.c > create mode 100644 drivers/thermal/tegra/tegra210-soctherm.c > delete mode 100644 drivers/thermal/tegra_soctherm.c >