linux-kernel - Re: [PATCH V2] Thermal: imx: add i.mx6sx thermal support

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-next>] [day] [month] [year] [list]

Date:	Wed, 6 Aug 2014 08:56:33 -0400
From:	"edubezval@...il.com" <edubezval@...il.com>
To:	"Anson.Huang@...escale.com" <Anson.Huang@...escale.com>
Cc:	Varka Bhadram <varkabhadram@...il.com>,
	"shawn.guo@...aro.org" <shawn.guo@...aro.org>,
	"rui.zhang@...el.com" <rui.zhang@...el.com>,
	"linux-pm@...r.kernel.org" <linux-pm@...r.kernel.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>
Subject: Re: [PATCH V2] Thermal: imx: add i.mx6sx thermal support

Hello Anson,

On Wed, Aug 6, 2014 at 3:05 AM, Anson.Huang@...escale.com
<Anson.Huang@...escale.com> wrote:
> Sorry for leaving such warnings, I thought we did NOT care about the long
> line issue now, will fix them in V3 patch.
>

Lines over 80 columns are not an error of style. We simply suggest to
keep this. By now, I think it really depends on which subsystem you
are working with. I would prefer you keep this rule, as most of the
code inside the thermal framework follows it.

>
>
> Thanks Varka.
>
>
>
> Best regards!
>
> Anson Huang
>
>
>
> From: Varka Bhadram [mailto:varkabhadram@...il.com]
> Sent: 2014-08-06 3:01 PM
> To: Huang Yongcai-B20788; edubezval@...il.com; shawn.guo@...aro.org;
> rui.zhang@...el.com
> Cc: linux-pm@...r.kernel.org; linux-kernel@...r.kernel.org;
> devicetree@...r.kernel.org
> Subject: Re: [PATCH V2] Thermal: imx: add i.mx6sx thermal support
>
>
>
> On 08/06/2014 10:47 AM, Anson Huang wrote:
>
> i.MX6SX has some new features of thermal interrupt function,
>
> there are LOW, HIGH and PANIC irq for thermal sensor, so add
>
> platform data to separate different thermal version;
>
>
>
> The reset value of LOW ALARM is 0 which means the highest
>
> temp, so the LOW ALARM will be triggered once irq is enabled,
>
> so we need to correct it before enabling thermal irq;
>
>
>
> Enable PANIC ALARM as critical trip point, it will trigger
>
> system reset via SRC module once PANIC IRQ is triggered, it
>
> is pure hardware function, so use it instead of software
>
> reset by cooling device.
>
>
>
> Signed-off-by: Anson Huang <b20788@...escale.com>
>
> ---
>
>  .../devicetree/bindings/thermal/imx-thermal.txt    |    5 +-
>
>  drivers/thermal/imx_thermal.c                      |   89
> +++++++++++++++++---
>
>  2 files changed, 80 insertions(+), 14 deletions(-)
>
>
>
> ./scripts/checkpatch.pl --show-types
> V2-Thermal-imx-add-i.mx6sx-thermal-support.patch
>
> WARNING:LONG_LINE: line over 80 characters
>
> #109: FILE: drivers/thermal/imx_thermal.c:185:
>
> +    /* Update alarm value to next higher trip point, only TEMPMON_IMX6Q
> needs it */
>
>
>
> WARNING:LONG_LINE: line over 80 characters
>
> #111: FILE: drivers/thermal/imx_thermal.c:187:
>
> +        if (data->alarm_temp == data->temp_passive && *temp >=
> data->temp_passive)
>
>
>
> WARNING:LONG_LINE: line over 80 characters
>
> #113: FILE: drivers/thermal/imx_thermal.c:189:
>
> +        if (data->alarm_temp == data->temp_critical && *temp <
> data->temp_passive) {
>
>
>
> --
>
> Regards,
>
> Varka Bhadram.



-- 
Eduardo Bezerra Valentin
--
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/