Linux kernel stable tree https://git.rulkc.org/pub/scm/linux/kernel/git/stable/linux.git/atom?h=master 2020-10-22T16:38:21+00:00 2020-10-22T16:38:21+00:00 Ulf Hansson ulf.hansson@linaro.org 2020-10-06T16:05:13+00:00 urn:sha1:a7305e684fcfb33029fe3d0af6b7d8dc4c8ca7a1 The avs drivers are all SoC specific drivers that doesn't share any code. Instead they are located in a directory, mostly to keep similar functionality together. From a maintenance point of view, it makes better sense to collect SoC specific drivers like these, into the SoC specific directories. Therefore, let's move the qcom-cpr driver to the qcom directory. Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org> Acked-by: Bjorn Andersson <bjorn.andersson@linaro.org> Acked-by: Niklas Cassel <nks@flawful.org> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com> 2020-09-22T16:04:29+00:00 Liu Shixin liushixin2@huawei.com 2020-09-15T03:26:29+00:00 urn:sha1:3ffe2e7318ba705b8e0060d4696cafc75170e819 Simplify the return expression. Signed-off-by: Liu Shixin <liushixin2@huawei.com> Reviewed-by: Bjorn Andersson <bjorn.andersson@linaro.org> Acked-by: Kevin Hilman <khilman@baylibre.com> [ rjw: Minor subject edits ] Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com> 2020-01-31T10:07:43+00:00 Nathan Chancellor natechancellor@gmail.com 2020-01-30T01:41:30+00:00 urn:sha1:e093e53f4f70b126ff63bc422e5135e934e35a1b Clang warns (trimmed for brevity): ../drivers/power/avs/qcom-cpr.c:570:13: warning: variable 'reg_mask' is used uninitialized whenever 'if' condition is false [-Wsometimes-uninitialized] ../drivers/power/avs/qcom-cpr.c:520:13: warning: variable 'new_uV' is used uninitialized whenever 'if' condition is false [-Wsometimes-uninitialized] Due to the fact that Clang's static analysis happens before any optimization passes are taken into account, it cannot see that both branches in the if statement must be taken because dir cannot be something other than UP or DOWN due to the check at the top of this function. Change the else if condition to else to fix this false positive. Fixes: bf6910abf548 ("power: avs: Add support for CPR (Core Power Reduction)") Link: https://github.com/ClangBuiltLinux/linux/issues/840 Signed-off-by: Nathan Chancellor <natechancellor@gmail.com> Acked-by: Kevin Hilman <khilman@baylibre.com> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com> 2020-01-31T10:07:43+00:00 Niklas Cassel nks@flawful.org 2020-01-13T20:02:58+00:00 urn:sha1:03b10951e9f6e6297d967a903c8d743b245ae3b8 In order to easier inform the user that the driver has been initialized successfully, add a printout after the driver has been initialized. At the same time, remove a dev_dbg() that is now redundant. Signed-off-by: Niklas Cassel <nks@flawful.org> Reviewed-by: Bjorn Andersson <bjorn.andersson@linaro.org> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com> 2020-01-08T22:25:22+00:00 YueHaibing yuehaibing@huawei.com 2020-01-08T01:58:49+00:00 urn:sha1:f9900c178d61661c104c1600409ced81bf3a6037 Remove duplicated include. Signed-off-by: YueHaibing <yuehaibing@huawei.com> Acked-by: Niklas Cassel <nks@flawful.org> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com> 2020-01-07T10:57:45+00:00 Colin Ian King colin.king@canonical.com 2020-01-06T12:05:58+00:00 urn:sha1:5aa0c467519ccd862678a15885026660c271b5b0 Currently when the call cpr_read_fuse_uV returns an error the value in the uninitialized variable ret is returned. Fix this by instread returning the error value in the variable uV. Addresses-Coverity: ("Uninitialized scalar variable") Fixes: bf6910abf548 ("power: avs: Add support for CPR (Core Power Reduction)") Signed-off-by: Colin Ian King <colin.king@canonical.com> Reviewed-by: Bjorn Andersson <bjorn.andersson@linaro.org> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com> 2019-12-29T21:11:22+00:00 Niklas Cassel nks@flawful.org 2019-12-23T14:19:34+00:00 urn:sha1:207ca274cd4c46a418ecb250d555d712e387fce8 drivers/power/avs/qcom-cpr.c:1081:15: warning: no previous prototype for ‘cpr_get_opp_hz_for_req’ Detected when running make with W=1. Signed-off-by: Niklas Cassel <nks@flawful.org> Reviewed-by: Bjorn Andersson <bjorn.andersson@linaro.org> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com> 2019-12-29T21:11:22+00:00 Niklas Cassel nks@flawful.org 2019-12-23T14:19:33+00:00 urn:sha1:9b8c249ee9edd2666661c1c76bbebc3ec663e30c drivers/power/avs/qcom-cpr.c:896:35: warning: variable ‘prev’ set but not used Detected when running make with W=1. Signed-off-by: Niklas Cassel <nks@flawful.org> Reviewed-by: Bjorn Andersson <bjorn.andersson@linaro.org> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com> 2019-12-29T21:11:22+00:00 Niklas Cassel nks@flawful.org 2019-12-23T14:19:31+00:00 urn:sha1:cad12cdeea783e8b7fc4425848028d3606a4cba8 drivers/power/avs/qcom-cpr.c:1539:5-21: warning: unsigned expression compared with zero: drv->num_corners < 0 Reported-by: kbuild test robot <lkp@intel.com> Reported-by: Julia Lawall <julia.lawall@lip6.fr> Signed-off-by: Niklas Cassel <nks@flawful.org> Reviewed-by: Bjorn Andersson <bjorn.andersson@linaro.org> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com> 2019-12-29T21:11:22+00:00 Niklas Cassel nks@flawful.org 2019-12-23T14:19:30+00:00 urn:sha1:e808926b4dd1e1f1d3c842c1f6f2aae7c72543ae drivers/power/avs/qcom-cpr.c:838:15: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 6 has type ‘ssize_t’ Reported-by: kbuild test robot <lkp@intel.com> Signed-off-by: Niklas Cassel <nks@flawful.org> Reviewed-by: Bjorn Andersson <bjorn.andersson@linaro.org> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>