GitHub PushEvent at 2014-10-03 14:37:32
Repository: lks
Actor: khiemnguyen
Payload: {"shas":[["3585453daba3b8be18deb1e472e779961be17f77","912cf7eb7d8018e2943586ae6657d21fd4e38239@linaro.org","thermal: cpu_cooling: fix 'descend' check in get_property()\n\nThe variable 'descend' is initialized as -1 in function get_property(),\nand will never get any chance to be updated by the following code.\n\n\tif (freq != CPUFREQ_ENTRY_INVALID && descend != -1)\n\t\tdescend = !!(freq > table[i].frequency);\n\nThis makes function get_property() return the wrong frequency for given\ncooling level if the frequency table is sorted in ascending.  Fix it\nby correcting the 'descend' check in if-condition to 'descend == -1'.\n\nSigned-off-by: Shawn Guo <shawn.guo@linaro.org>\nSigned-off-by: Zhang Rui <rui.zhang@intel.com>\n(cherry picked from commit 24c7a381720843f17efb42de81f7e85aefd6f616)\nSigned-off-by: Alex Shi <alex.shi@linaro.org>","Shawn Guo",false],["d05bc7105a1b8f6df6569b7c1a89a2fe466f8688","e00c330fbe547eef303c59736cf80909d6b41757@nvidia.com","thermal: Fix binding problem when there is thermal zone params\n\nThe thermal zone params can be used to set governor\nto specific thermal governor for thermal zone device.\nBut if the thermal zone params has only governor name\nwithout thermal bind params, then the thermal zone device\nwill not be binding to cooling device. Because tz->ops->bind\noperator is not invoked in bind_tz() and bind_cdev() when\nthere is thermal zone params.\n\nSigned-off-by: Wei Ni <wni@nvidia.com>\nSigned-off-by: Jinyoung Park <jinyoungp@nvidia.com>\nSigned-off-by: Zhang Rui <rui.zhang@intel.com>\n(cherry picked from commit a9f2d19ba7be38590c84487359891d45a66b62f4)\nSigned-off-by: Alex Shi <alex.shi@linaro.org>","Ni Wade",false],["e86c742e299cbe321870aa00bcbd4985d1f628fe","977df7d8e3020f3c4acf757f6ad9c6cf2bc8f2d7@ti.com","thermal: fix cpu_cooling max_level behavior\n\nAs per Documentation/thermal/sysfs-api.txt, max_level\nis an index, not a counter. Thus, in case a CPU has\n3 valid frequencies, max_level is expected to be 2, for instance.\n\nThe current code makes max_level == number of valid frequencies,\nwhich is bogus. This patch fix the cpu_cooling device by\nranging max_level properly.\n\nReported-by: Carlos Hernandez <ceh@ti.com>\nSigned-off-by: Eduardo Valentin <eduardo.valentin@ti.com>\nSigned-off-by: Zhang Rui <rui.zhang@intel.com>\n(cherry picked from commit 1c9573a40c1d34494419f32560f28c763c504d79)\nSigned-off-by: Alex Shi <alex.shi@linaro.org>","Eduardo Valentin",false],["25a434675eaa351f90b51866a982accac5ee6c61","f2a4cf793007be8f81a774dc7806c1ffea7b4a2c@intel.com","Thermal cpu cooling: return error if no valid cpu frequency entry\n\nSigned-off-by: Zhang Rui <rui.zhang@intel.com>\n(cherry picked from commit a116776f7b6052599df0c67db29c30ea9d69d7ee)\nSigned-off-by: Alex Shi <alex.shi@linaro.org>","Zhang Rui",false],["fe174a059c03451611cbefa828f6efc76fc5ac78","f2a4cf793007be8f81a774dc7806c1ffea7b4a2c@intel.com","Thermal: thermal zone governor fix\n\nThis patch does a cleanup about the thermal zone govenor,\nsetting and make the following rule.\n1. For thermal zone devices that are registered w/o tz->tzp,\n   they can use the default thermal governor only.\n2. For thermal zone devices w/ governor name specified in\n   tz->tzp->governor_name, we will use the default govenor\n   if the governor specified is not available at the moment,\n   and update tz->governor when the matched governor is registered.\n\nThis also fixes a problem that OF registered thermal zones\nare running with no governor.\n\nSigned-off-by: Zhang Rui <rui.zhang@intel.com>\nAcked-by: Javi Merino <javi.merino@arm.com>\n(cherry picked from commit f2234bcd03ad031225d7dc37dd18852a2f2ff2bf)\nSigned-off-by: Alex Shi <alex.shi@linaro.org>","Zhang Rui",false],["ed835be313220380ccacadc959f7d4d64324d9ee","e00c330fbe547eef303c59736cf80909d6b41757@nvidia.com","Thermal: Allow first update of cooling device state\n\nIn initialization, if the cooling device is initialized at\nmax cooling state, and the thermal zone temperature is below\nthe first trip point, then the cooling state can't be updated\nto the right state, untill the first trip point be triggered.\n\nTo fix this issue, allow first update of cooling device state\nduring registration, initialized \"updated\" device field as\n\"false\" (instead of \"true\").\n\nSigned-off-by: Wei Ni <wni@nvidia.com>\nSigned-off-by: Zhang Rui <rui.zhang@intel.com>\n(cherry picked from commit 5ca0cce5622bf476e3e6bf627fe8e9381d6ae174)\nSigned-off-by: Alex Shi <alex.shi@linaro.org>","Ni Wade",false],["f8b3519d0a85467970e662f75d1c12d39563c2de","93fe5f620cf5f235dea09f543baa20a9b3210b07@arm.com","thermal: Bind cooling devices with the correct arguments\n\nWhen binding cooling devices to thermal zones created from the device\ntree the minimum and maximum cooling states are in the wrong order\nleading to failure to bind.\n\nFix the order of cooling states in the call to\nthermal_zone_bind_cooling_device to fix this.\n\nCc:Zhang Rui <rui.zhang@intel.com>\nSigned-off-by: Punit Agrawal <punit.agrawal@arm.com>\nReviewed-by: Stephen Boyd <sboyd@codeaurora.org>\nSigned-off-by: Zhang Rui <rui.zhang@intel.com>\n(cherry picked from commit dd354b84d47ec8ca53686bdb3cc1aecdeb75bef5)\nSigned-off-by: Alex Shi <alex.shi@linaro.org>","Punit Agrawal",false],["fca2d1a60af983180816385fc8238ce818cb72c3","4221d0916eb5c472a0ab46f0107445bbb3e8a589@intel.com","Thermal/cpu_cooling: Return directly for the cpu out of allowed_cpus in the cpufreq_thermal_notifier()\n\ncpufreq_thermal_notifier() is to change the cpu's cpufreq in the allowed_cpus mask\nwhen associated thermal-cpufreq cdev's cooling state is changed. It's a cpufreq policy\nnotifier handler and it will be triggered even if those cpus out of allowed_cpus has\nchanged freq policy.\n\ncpufreq_thermal_notifier() checks the policy->cpu. If it belongs to allowed_cpus,\nchange max_freq(default to 0) to the desire cpufreq value and pass 0 and max_freq\nto cpufreq_verify_within_limits() as cpufreq scope. But if not, do nothing and\nmax_freq will remain 0. This will cause the cpufreq scope to become 0~0. This\nis not right. This patch is to return directly after finding cpu not belonging\nto allowed_cpus.\n\nSigned-off-by: Lan Tianyu <tianyu.lan@intel.com>\nSigned-off-by: Zhang Rui <rui.zhang@intel.com>\n(cherry picked from commit 044d5c26da262fa433dacbe1c6962459050d6b06)\nSigned-off-by: Alex Shi <alex.shi@linaro.org>","Lan Tianyu",false],["3a68305f1f92040bc57d6ea94bedde1b694d0356","4221d0916eb5c472a0ab46f0107445bbb3e8a589@intel.com","Thermal: update thermal zone device after setting emul_temp\n\nThis patch is to update thermal zone device after setting emul_temp\nin order to make governor work according to input temperature immediately.\n\nSigned-off-by: Lan Tianyu <tianyu.lan@intel.com>\nSigned-off-by: Zhang Rui <rui.zhang@intel.com>\n(cherry picked from commit 800744bf31df54b0cd4d1104ccfa426d3f578f0e)\nSigned-off-by: Alex Shi <alex.shi@linaro.org>","lan,Tianyu",false],["c60fd301f395adc8f788be8b192a58f0246ddbae","010521127f513270fe503d86ab8316ac5147f4b7@codeaurora.org","thermal: Add braces around suspect code\n\nIt looks like this code is missing braces, otherwise the if\nstatement shouldn't have been indented. Fix it.\n\nSigned-off-by: Stephen Boyd <sboyd@codeaurora.org>\nSigned-off-by: Zhang Rui <rui.zhang@intel.com>\n(cherry picked from commit ca9521b770c988bb6bb8eea1241f7a487dab6ff1)\nSigned-off-by: Alex Shi <alex.shi@linaro.org>","Stephen Boyd",false],["e6ac9c53208c2871e3704e5b911971522cda8a56","b8f9cab8be13de37b9588aedad10a20fc3a68783@trendmicro.com.cn","thermal: cpu_cooling: fix return value check in cpufreq_cooling_register()\n\nIn case of error, the function thermal_cooling_device_register() returns\nERR_PTR() and never returns NULL. The NULL test in the return value check\nshould be replaced with IS_ERR().\n\nSigned-off-by: Wei Yongjun <yongjun_wei@trendmicro.com.cn>\nSigned-off-by: Zhang Rui <rui.zhang@intel.com>\n(cherry picked from commit 73b9bcd76d13716cc0e0ab053f8c1ae41f47636e)\nSigned-off-by: Alex Shi <alex.shi@linaro.org>\n\nConflicts:\n\tdrivers/thermal/cpu_cooling.c","Wei Yongjun",false],["d49a5d426a7906927351ca58dd14c6870975b166","977df7d8e3020f3c4acf757f6ad9c6cf2bc8f2d7@ti.com","drivers: thermal: add check when unregistering cpu cooling\n\nThis patch avoids NULL pointer accesses while unregistering\ncpu cooling devices, in case a NULL pointer is received.\n\nCc: Zhang Rui <rui.zhang@intel.com>\nCc: linux-pm@vger.kernel.org\nCc: linux-kernel@vger.kernel.org\nSigned-off-by: Eduardo Valentin <eduardo.valentin@ti.com>\n(cherry picked from commit 50e66c7ed8a1cd7e933628f9f5cf2617394adf5a)\nSigned-off-by: Alex Shi <alex.shi@linaro.org>","Eduardo Valentin",false],["0a32b0ba838f343d06db33b96ccc12859d0180e5","1abc5746028550adcf7742cb5d47df86766285d8@linaro.org","Merge branch 'thermal-framework' into linux-linaro-lsk","Alex Shi",false],["5c4e2425d94aa1545ce9d5feb9324ca85f62ef90","5ff32272b3d9f86512eddc8e0af523fc6f7924e5@linaro.org","cpufreq: arm_big_little: set 'physical_cluster' for each CPU\n\nWe have a per-CPU variable for managing which cluster a CPU belongs to.\nCurrently, physical_cluster is set for policy->cpu only which leads to\nthe following on some SoC's:\n\n - There are two clusters:\n   - Cluster 0 has four ARM Cortex A7 CPUs (slower ones): 0,1,2,3\n   - Cluster 1 has four ARM Cortex A15 CPUs (faster ones): 4,5,6,7\n - CPUs are booted in order 0,1..7 and so initially policy->cpu for A7 cluster\n   would be 0 and for A15 cluster would be 4.\n - Now CPU4 (i.e. A15_0) is hotplugged out and so policy->cpu for A15 cluster\n   becomes 5 (i.e. A15_1).\n - But physical cluster is only set for CPU0 and CPU4 in ARM big LITTLE driver\n   and isn't updated.\n - Now freq change request comes for A15 cluster and we would try to update freq\n   of physical_cluster of CPU5, i.e. A15_1. And it is currently set to zero\n   (default value of uninitialized global variables).\n - And so we actually try to change freq of A7 cluster instead of A15.\n - This also results in kernel crash as sometimes we might request freq above\n   A7's limit and CPU may behave badly..\n\nFix this by initializing physical_cluster for all CPUs of a policy.\n\nBased on previous work by Xin Wang.\n\nSigned-off-by: Viresh Kumar <viresh.kumar@linaro.org>\nSigned-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>\n(cherry picked from commit 8f3ba3d3257be80636ed15cc221d6a2efb6a6e82)\nSigned-off-by: Mark Brown <broonie@kernel.org>","viresh kumar",false],["54cd4bea5a18f7bd5921a76d31eef4c61a68d8e7","b51b9a92386687a9ac927cebfa0f978adeb8cea5@kernel.org","Merge remote-tracking branch 'lsk/v3.10/topic/tc2' into linux-linaro-lsk","Mark Brown",false]],"size":15,"ref":"refs/heads/linux-linaro-lsk","head":"54cd4bea5a18f7bd5921a76d31eef4c61a68d8e7"}