Dear maintainer, gentle ping if this driver is ready for more review/inclusion into Linux v4.1 ? Thank you very much.
>-----Original Message----- >From: [email protected] [mailto:linux-kernel- >[email protected]] On Behalf Of Ong Boon Leong >Sent: Monday, March 9, 2015 3:43 PM >To: Zhang, Rui; [email protected]; [email protected]; Kweh, >Hock Leong; [email protected] >Cc: [email protected]; [email protected] >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@quark:/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@quark:/sys/class/thermal/thermal_zone0# > root@quark:/sys/class/thermal/thermal_zone0# cat temp > 53 > root@quark:/sys/class/thermal/thermal_zone0# cat trip_point_0_temp > 105 > root@quark:/sys/class/thermal/thermal_zone0# cat trip_point_0_type > critical > root@quark:/sys/class/thermal/thermal_zone0# cat trip_point_1_temp > 20 > root@quark:/sys/class/thermal/thermal_zone0# cat trip_point_1_type > hot > root@quark:/sys/class/thermal/thermal_zone0# cat type > quark_dts > > root@quark:/sys/class/thermal/thermal_zone0# echo 105 > emul_temp > [ 179.372981] thermal thermal_zone0: critical temperature reached(0 >C),shutting down > root@quark:/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 [email protected] More majordomo info at >http://vger.kernel.org/majordomo-info.html >Please read the FAQ at http://www.tux.org/lkml/ -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
