From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753559AbaIALFe (ORCPT ); Mon, 1 Sep 2014 07:05:34 -0400 Received: from mail-lb0-f174.google.com ([209.85.217.174]:62767 "EHLO mail-lb0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752722AbaIALF3 (ORCPT ); Mon, 1 Sep 2014 07:05:29 -0400 MIME-Version: 1.0 In-Reply-To: References: <1406826667-2289-1-git-send-email-b.zolnierkie@samsung.com> <20140828144921.GB18194@developer> Date: Mon, 1 Sep 2014 07:05:27 -0400 Message-ID: Subject: Re: [PATCH v3 0/8] thermal: exynos: various cleanups From: "edubezval@gmail.com" To: amit daniel kachhap Cc: Bartlomiej Zolnierkiewicz , Eduardo Valentin , Zhang Rui , Tomasz Figa , "Rafael J. Wysocki" , Kyungmin Park , "linux-samsung-soc@vger.kernel.org" , "linux-pm@vger.kernel.org" , "linux-kernel@vger.kernel.org" Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Amit On Mon, Sep 1, 2014 at 6:53 AM, amit daniel kachhap wrote: > On Thu, Aug 28, 2014 at 8:19 PM, Eduardo Valentin wrote: >> Amit, >> >> On Thu, Jul 31, 2014 at 07:10:58PM +0200, Bartlomiej Zolnierkiewicz wrote: >>> Hi, >>> >>> This patch series contains various cleanups for EXYNOS thermal >>> driver. Overall it decreases driver's LOC by 9%. It is based >>> on next-20140731 kernel. It should not cause any functionality >>> changes. >>> >> >> Did you have the time to test this series? > > Hi Eduardo/Bartlomiej, > > For the whole series tested on exynos5250 based arndale board. > Tested-by: Amit Daniel Kachhap > Thanks! I will pull the series on my tree. > Sorry for the delay as I was busy in some internal work. > > Thanks, > Amit D >> >>> Changes since v2 (https://lkml.org/lkml/2014/6/17/436): >>> - synced patches against next-20140731 >>> - dropped patch "thermal: exynos: remove dead code for >>> TYPE_TWO_POINT_TRIMMING calibration" (newly added Exynos3250 >>> SoC support uses TYPE_TWO_POINT_TRIMMING calibration) >>> - updated patch description for patch #2 >>> - dropped Reviewed-by from Amit from patch #8 (due to changed >>> scope of the patch) >>> >>> Changes since v1 (https://lkml.org/lkml/2014/5/5/194): >>> - synced patches against next-20140617 >>> - merged patch "thermal: exynos: remove unused defines" into >>> "thermal: exynos: remove unused struct exynos_tmu_registers >>> entries" one (per request from Eduardo) >>> - improved patch descriptions for patches #1-5 >>> - fixed documentation for pdata->gain and pdata->reference_voltage >>> - added Reviewed-by from Amit to patches #6, #7 and #10 >>> - added missing Acked-by from Kyungmin Park >>> >>> Best regards, >>> -- >>> Bartlomiej Zolnierkiewicz >>> Samsung R&D Institute Poland >>> Samsung Electronics >>> >>> >>> Bartlomiej Zolnierkiewicz (8): >>> thermal: exynos: remove unused struct exynos_tmu_registers entries >>> thermal: exynos: remove dead code for HW_MODE calibration >>> thermal: exynos: remove redundant pdata checks from >>> exynos_tmu_initialize() >>> thermal: exynos: remove redundant threshold_code checks from >>> exynos_tmu_initialize() >>> thermal: exynos: simplify temp_to_code() and code_to_temp() >>> thermal: exynos: cache non_hw_trigger_levels in pdata >>> thermal: exynos: remove redundant pdata checks from >>> exynos_tmu_control() >>> thermal: exynos: remove identical values from exynos*_tmu_registers >>> structures >>> >>> drivers/thermal/samsung/exynos_thermal_common.h | 1 - >>> drivers/thermal/samsung/exynos_tmu.c | 126 +++++------------------- >>> drivers/thermal/samsung/exynos_tmu.h | 74 +------------- >>> drivers/thermal/samsung/exynos_tmu_data.c | 56 ++--------- >>> drivers/thermal/samsung/exynos_tmu_data.h | 31 +----- >>> 5 files changed, 35 insertions(+), 253 deletions(-) >>> >>> -- >>> 1.8.2.3 >>> >>> -- >>> To unsubscribe from this list: send the line "unsubscribe linux-pm" in >>> the body of a message to majordomo@vger.kernel.org >>> More majordomo info at http://vger.kernel.org/majordomo-info.html >> -- >> To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in >> the body of a message to majordomo@vger.kernel.org >> More majordomo info at http://vger.kernel.org/majordomo-info.html -- Eduardo Bezerra Valentin From mboxrd@z Thu Jan 1 00:00:00 1970 From: "edubezval@gmail.com" Subject: Re: [PATCH v3 0/8] thermal: exynos: various cleanups Date: Mon, 1 Sep 2014 07:05:27 -0400 Message-ID: References: <1406826667-2289-1-git-send-email-b.zolnierkie@samsung.com> <20140828144921.GB18194@developer> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Return-path: In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: amit daniel kachhap Cc: Bartlomiej Zolnierkiewicz , Eduardo Valentin , Zhang Rui , Tomasz Figa , "Rafael J. Wysocki" , Kyungmin Park , "linux-samsung-soc@vger.kernel.org" , "linux-pm@vger.kernel.org" , "linux-kernel@vger.kernel.org" List-Id: linux-pm@vger.kernel.org Amit On Mon, Sep 1, 2014 at 6:53 AM, amit daniel kachhap wrote: > On Thu, Aug 28, 2014 at 8:19 PM, Eduardo Valentin wrote: >> Amit, >> >> On Thu, Jul 31, 2014 at 07:10:58PM +0200, Bartlomiej Zolnierkiewicz wrote: >>> Hi, >>> >>> This patch series contains various cleanups for EXYNOS thermal >>> driver. Overall it decreases driver's LOC by 9%. It is based >>> on next-20140731 kernel. It should not cause any functionality >>> changes. >>> >> >> Did you have the time to test this series? > > Hi Eduardo/Bartlomiej, > > For the whole series tested on exynos5250 based arndale board. > Tested-by: Amit Daniel Kachhap > Thanks! I will pull the series on my tree. > Sorry for the delay as I was busy in some internal work. > > Thanks, > Amit D >> >>> Changes since v2 (https://lkml.org/lkml/2014/6/17/436): >>> - synced patches against next-20140731 >>> - dropped patch "thermal: exynos: remove dead code for >>> TYPE_TWO_POINT_TRIMMING calibration" (newly added Exynos3250 >>> SoC support uses TYPE_TWO_POINT_TRIMMING calibration) >>> - updated patch description for patch #2 >>> - dropped Reviewed-by from Amit from patch #8 (due to changed >>> scope of the patch) >>> >>> Changes since v1 (https://lkml.org/lkml/2014/5/5/194): >>> - synced patches against next-20140617 >>> - merged patch "thermal: exynos: remove unused defines" into >>> "thermal: exynos: remove unused struct exynos_tmu_registers >>> entries" one (per request from Eduardo) >>> - improved patch descriptions for patches #1-5 >>> - fixed documentation for pdata->gain and pdata->reference_voltage >>> - added Reviewed-by from Amit to patches #6, #7 and #10 >>> - added missing Acked-by from Kyungmin Park >>> >>> Best regards, >>> -- >>> Bartlomiej Zolnierkiewicz >>> Samsung R&D Institute Poland >>> Samsung Electronics >>> >>> >>> Bartlomiej Zolnierkiewicz (8): >>> thermal: exynos: remove unused struct exynos_tmu_registers entries >>> thermal: exynos: remove dead code for HW_MODE calibration >>> thermal: exynos: remove redundant pdata checks from >>> exynos_tmu_initialize() >>> thermal: exynos: remove redundant threshold_code checks from >>> exynos_tmu_initialize() >>> thermal: exynos: simplify temp_to_code() and code_to_temp() >>> thermal: exynos: cache non_hw_trigger_levels in pdata >>> thermal: exynos: remove redundant pdata checks from >>> exynos_tmu_control() >>> thermal: exynos: remove identical values from exynos*_tmu_registers >>> structures >>> >>> drivers/thermal/samsung/exynos_thermal_common.h | 1 - >>> drivers/thermal/samsung/exynos_tmu.c | 126 +++++------------------- >>> drivers/thermal/samsung/exynos_tmu.h | 74 +------------- >>> drivers/thermal/samsung/exynos_tmu_data.c | 56 ++--------- >>> drivers/thermal/samsung/exynos_tmu_data.h | 31 +----- >>> 5 files changed, 35 insertions(+), 253 deletions(-) >>> >>> -- >>> 1.8.2.3 >>> >>> -- >>> To unsubscribe from this list: send the line "unsubscribe linux-pm" in >>> the body of a message to majordomo@vger.kernel.org >>> More majordomo info at http://vger.kernel.org/majordomo-info.html >> -- >> To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in >> the body of a message to majordomo@vger.kernel.org >> More majordomo info at http://vger.kernel.org/majordomo-info.html -- Eduardo Bezerra Valentin