linux-kernel - Re: [PATCH v2 2/2] ARM: dts: sun8i-r40: Add thermal sensor and thermal zones

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: <20200109131343.mjyuj5ed2xwvmwd4@gilmour.lan>
Date:   Thu, 9 Jan 2020 14:13:43 +0100
From:   Maxime Ripard <mripard@...nel.org>
To:     Daniel Lezcano <daniel.lezcano@...aro.org>
Cc:     Yangtao Li <tiny.windzz@...il.com>, robh+dt@...nel.org,
        mark.rutland@....com, wens@...e.org, anarsoul@...il.com,
        rui.zhang@...el.com, amit.kucheria@...durent.com,
        megous@...ous.com, devicetree@...r.kernel.org,
        linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org,
        linux-pm@...r.kernel.org
Subject: Re: [PATCH v2 2/2] ARM: dts: sun8i-r40: Add thermal sensor and
 thermal zones

Hi Daniel,

On Thu, Jan 09, 2020 at 12:51:27PM +0100, Daniel Lezcano wrote:
> On 07/01/2020 08:58, Maxime Ripard wrote:
> > On Mon, Jan 06, 2020 at 05:46:39PM +0000, Yangtao Li wrote:
> >> There are two sensors, sensor0 for CPU, sensor1 for GPU.
> >>
> >> Signed-off-by: Yangtao Li <tiny.windzz@...il.com>
> >> Tested-by: Corentin Labbe <clabbe.montjoie@...il.com>
> >> Tested-on: sun8i-r40-bananapi-m2-ultra
> >
> > As far as I know, tested-on is not documented anywhere (and isn't
> > really used either). I've removed it and applied, thanks!
>
> I think this patch should go through my tree as it refers to a commit in
> my branch.

I'm not quite sure to get why. Even though that patch depends on the
one affecting the driver to be functional, it doesn't break anything
when merged through arm-soc, and similarly yours doesn't affect any
other tree if it's merged through your tree, so there's no dependency?

Maxime

Download attachment "signature.asc" of type "application/pgp-signature" (229 bytes)