| 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
| ||
|
Date: Tue, 31 Mar 2015 02:12:12 +0000 From: "Kweh, Hock Leong" <hock.leong.kweh@...el.com> To: "Ong, Boon Leong" <boon.leong.ong@...el.com>, "Zhang, Rui" <rui.zhang@...el.com>, "edubezval@...il.com" <edubezval@...il.com> CC: "linux-pm@...r.kernel.org" <linux-pm@...r.kernel.org>, "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>, "pure.logic@...us-software.ie" <pure.logic@...us-software.ie>, "andy.shevchenko@...il.com" <andy.shevchenko@...il.com> Subject: RE: [PATCHv3 0/1] Intel Quark X1000 DTS thermal driver > -----Original Message----- > From: Ong, Boon Leong > Sent: Tuesday, March 31, 2015 7:58 AM > To: Zhang, Rui; edubezval@...il.com > Cc: linux-pm@...r.kernel.org; linux-kernel@...r.kernel.org; Ong, Boon > Leong; pure.logic@...us-software.ie; Kweh, Hock Leong; > andy.shevchenko@...il.com > Subject: RE: [PATCHv3 0/1] Intel Quark X1000 DTS thermal driver > > Dear maintainer, gentle ping if this driver is ready for more review/inclusion > into Linux v4.1 ? > Thank you very much. > > >-----Original Message----- > >From: linux-kernel-owner@...r.kernel.org [mailto:linux-kernel- > >owner@...r.kernel.org] On Behalf Of Ong Boon Leong > >Sent: Monday, March 9, 2015 3:43 PM > >To: Zhang, Rui; edubezval@...il.com; pure.logic@...us-software.ie; > >Kweh, Hock Leong; andy.shevchenko@...il.com > >Cc: linux-pm@...r.kernel.org; linux-kernel@...r.kernel.org > >Subject: [PATCHv3 0/1] Intel Quark X1000 DTS thermal driver > > > >Dear maintainers & communities, > > > >This patch introduces DTS thermal driver for Intel Quark X1000. > >The code implementation is based on intel_soc_dts_thermal.c. > > > >Intel Quark X1000 has one on-die DTS with two configurable trip points: > >critical and hot trip points. However, todate, UEFI BIOS for Quark > >X1000 uses only critical trip point. UEFI BIOS always lock DTS register > >before hand-over to Linux kernel. > > > >The minimalist thermal design is meant to trigger Linux distro to > >gracefully power-down the system when its DTS temperature exceeds the > >configured critical trip point. > > > >In anticipation that other variant of Quark platform may come with UEFI > >BIOS that does not lock DTS register during hand-over, this DTS driver > >is built with logics to handle such case too. > > > >I have tested v1 of the patch on Intel Galileo Gen v2 board and found > >it satisfactory with logs below: > > > > root@...rk:/sys/class/thermal/thermal_zone0# echo disabled > mode > > [ 46.276881] intel_quark_dts_thermal: DTS is locked. Cannot disable DTS > > -sh: echo: write error: Operation not permitted > > root@...rk:/sys/class/thermal/thermal_zone0# > > root@...rk:/sys/class/thermal/thermal_zone0# cat temp > > 53 > > root@...rk:/sys/class/thermal/thermal_zone0# cat trip_point_0_temp > > 105 > > root@...rk:/sys/class/thermal/thermal_zone0# cat trip_point_0_type > > critical root@...rk:/sys/class/thermal/thermal_zone0# cat > > trip_point_1_temp > > 20 > > root@...rk:/sys/class/thermal/thermal_zone0# cat trip_point_1_type > > hot root@...rk:/sys/class/thermal/thermal_zone0# cat type quark_dts > > > > root@...rk:/sys/class/thermal/thermal_zone0# echo 105 > emul_temp > > [ 179.372981] thermal thermal_zone0: critical temperature reached(0 > >C),shutting down > > root@...rk:/sys/class/thermal/thermal_zone0# > > [ OK ] Stopped target Multi-User System. > > Stopping Telephony service... > > Stopping Lightning Fast Webserver With Light System Requirements... > > Stopping Target Communication Framework agent... > > Stopping Galileo Arduino Layer... > > [ OK ] Stopped target Login Prompts. > > Stopping Getty on tty1... > > Stopping Serial Getty on ttyS1... > > Stopping Login Service... > > Stopping D-Bus System Message Bus... > > Starting Store Sound Card State... > > [ OK ] Stopped Telephony service. > > [ OK ] Stopped Galileo Arduino Layer. > > [ OK ] Stopped Login Service. > > [ OK ] Stopped D-Bus System Message Bus. > > [ OK ] Stopped Target Communication Framework agent. > > [ OK ] Stopped Lightning Fast Webserver With Light System Requirements. > > [ OK ] Stopped WPA supplicant. > > [ OK ] Stopped Getty on tty1. > > [ OK ] Stopped Serial Getty on ttyS1. > > > >Please kindly review the patch at your convenient time and provide me > >feedback for improvement. Appreciate your time and effort. > > > >Thank You > >Ong Boon Leong > >Intel Corp. > > > >--- > >Changes in v3: > >* Kconfig dependency changed to X86_INTEL_QUARK > > > >Changes in v2: > >* Fix several commit write-up grammar, choice of words. > >* Ensure "int ret" in correct order > >* Add comment to explain DTS register field read/write bit operation > >* Change to Dual BSD/GPL license > >* Add logic to ensure safe trip point threshold value being set > > > >Ong Boon Leong (1): > > thermal: intel Quark SoC X1000 DTS thermal driver > > > > drivers/thermal/Kconfig | 10 > > drivers/thermal/Makefile | 1 > > drivers/thermal/intel_quark_dts_thermal.c | 434 > >+++++++++++++++++++++++++++++ > > 3 files changed, 445 insertions(+) > > create mode 100644 drivers/thermal/intel_quark_dts_thermal.c > > > >-- > >1.7.9.5 > > > >-- > >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/ I am fine with it. Thanks. Reviewed-by: Kweh, Hock Leong <hock.leong.kweh@...el.com> -- 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/
Powered by blists - more mailing lists