Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
	id S1754716AbcCOFqz (ORCPT <rfc822;w@1wt.eu>);
	Tue, 15 Mar 2016 01:46:55 -0400
Received: from hqemgate15.nvidia.com ([216.228.121.64]:4613 "EHLO
	hqemgate15.nvidia.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
	with ESMTP id S1752838AbcCOFqs convert rfc822-to-8bit (ORCPT
	<rfc822;linux-kernel@vger.kernel.org>);
	Tue, 15 Mar 2016 01:46:48 -0400
X-PGP-Universal: processed;
	by hqnvupgp08.nvidia.com on Mon, 14 Mar 2016 22:46:02 -0700
Message-ID: <56E7A1CF.4060302@nvidia.com>
Date: Tue, 15 Mar 2016 13:46:55 +0800
From: Wei Ni <wni@nvidia.com>
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.7.0
MIME-Version: 1.0
To: Eduardo Valentin <edubezval@gmail.com>
CC: <rui.zhang@intel.com>, <thierry.reding@gmail.com>,
        <MLongnecker@nvidia.com>, <swarren@wwwdotorg.org>,
        <mikko.perttunen@kapsi.fi>, <linux-tegra@vger.kernel.org>,
        <linux-pm@vger.kernel.org>, <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH V7 00/12] Add T210 support in Tegra soctherm
References: <1457665763-29781-1-git-send-email-wni@nvidia.com> <20160314210152.GA4717@localhost.localdomain>
In-Reply-To: <20160314210152.GA4717@localhost.localdomain>
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: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
Content-Length: 934
Lines: 44



On 2016年03月15日 05:01, Eduardo Valentin wrote:
> * PGP Signed by an unknown key
> 
> On Fri, Mar 11, 2016 at 11:09:11AM +0800, Wei Ni wrote:
>> This patchset adds following functions for tegra_soctherm driver:
>> 1. add T210 support.
> 
> It would be good to update the compatible string in the binding
> documentation.

The commit "193c9d23a0f0 Documentation: DT bindings: add more Tegra chip
compatible strings" already add compatible string for Tegra210.

> 
>> 2. export debugfs to show some registers.
>> 3. add thermtrip funciton.
>> 4. add suspend/resume function.
> 
> Thanks for your effort in keeping up with upstream. I think overall the
> series is good, I had only very few comments (in response to each
> patch).

Thanks for your review. I will check these comments, and update my patches in
next few days.

Wei.

> 
> BR,
> 
> Eduardo Valentin
> 
> 
>>
>>
>> -- 
>> 1.9.1
>>
> 
> * Unknown Key
> * 0x7DA4E256
>