linux-kernel - Re: [PATCH v2 0/9] thermal: exynos: various cleanups

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20140729125848.GA18746@developer>
Date:	Tue, 29 Jul 2014 08:58:48 -0400
From:	Eduardo Valentin <edubezval@...il.com>
To:	amit daniel kachhap <amit.daniel@...sung.com>
Cc:	Bartlomiej Zolnierkiewicz <b.zolnierkie@...sung.com>,
	Eduardo Valentin <eduardo.valentin@...com>,
	Zhang Rui <rui.zhang@...el.com>,
	Tomasz Figa <t.figa@...sung.com>,
	"Rafael J. Wysocki" <rjw@...ysocki.net>,
	Kyungmin Park <kyungmin.park@...sung.com>,
	"linux-samsung-soc@...r.kernel.org" 
	<linux-samsung-soc@...r.kernel.org>,
	"linux-pm@...r.kernel.org" <linux-pm@...r.kernel.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v2 0/9] thermal: exynos: various cleanups

On Mon, Jul 28, 2014 at 08:30:53PM +0530, amit daniel kachhap wrote:
> Hi Eduardo,

Hello Amit,

> 
> Please reject this entire series as this is not re-based recently.
> Actually two point trimming which this series seeks to remove is used
> by exynos3250 and is in Rui's tree
> (https://git.kernel.org/cgit/linux/kernel/git/rzhang/linux.git/commit/?h=next&id=1fe56dc16a3dab400206443f70ae158c8f595c42).
> 

Yeah, that was my original concern when I first looked this series.
However, as I saw a couple of your acks I thought you would be in sync
with Bartlomiej. 

I agree with Bartlomiej that we need to keep the kernel code in very
good shape, but if we are removing features for the sake of reducing
LOCs and we end up dropping support to known chips, I definitely won't
agree. 

Can you guys please reach a common agreement on these cleanups?

Bartlomiej, best thing to do is to sync your work on top of Rui's tree.

For now I am dropping this series from my -next branch. :-(

> Thanks,
> Amit D
> 
> 
> On Mon, Jul 28, 2014 at 8:07 PM, Eduardo Valentin <edubezval@...il.com> wrote:
> > On Tue, Jun 17, 2014 at 07:27:16PM +0200, Bartlomiej Zolnierkiewicz wrote:
> >> Hi,
> >>
> >> This patch series contains various cleanups for EXYNOS thermal
> >> driver.  Overall it decreases driver's LOC by 12%.  It is based
> >> on next-20140617 kernel.  It should not cause any functionality
> >> changes.
> >>
> >> Changes since v1:
> >> - 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
> >
> > Pulled the series.
> >
> > Thanks.
> >
> >>
> >> Best regards,
> >> --
> >> Bartlomiej Zolnierkiewicz
> >> Samsung R&D Institute Poland
> >> Samsung Electronics
> >>
> >>
> >> Bartlomiej Zolnierkiewicz (9):
> >>   thermal: exynos: remove unused struct exynos_tmu_registers entries
> >>   thermal: exynos: remove dead code for HW_MODE calibration
> >>   thermal: exynos: remove dead code for TYPE_TWO_POINT_TRIMMING
> >>     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            | 181 ++++--------------------
> >>  drivers/thermal/samsung/exynos_tmu.h            |  90 +-----------
> >>  drivers/thermal/samsung/exynos_tmu_data.c       |  64 +--------
> >>  drivers/thermal/samsung/exynos_tmu_data.h       |  33 +----
> >>  5 files changed, 41 insertions(+), 328 deletions(-)
> >>
> >> --
> >> 1.8.2.3
> >>
> >> --
> >> To unsubscribe from this list: send the line "unsubscribe linux-pm" in
> >> the body of a message to majordomo@...r.kernel.org
> >> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> > --
> > To unsubscribe from this list: send the line "unsubscribe linux-pm" in
> > the body of a message to majordomo@...r.kernel.org
> > More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/