| From 5e2e2f9f76e157063a656351728703cb02b068f1 Mon Sep 17 00:00:00 2001 |
| From: Dan Carpenter <dan.carpenter@oracle.com> |
| Date: Thu, 23 Aug 2018 16:59:25 +0300 |
| Subject: PM / clk: signedness bug in of_pm_clk_add_clks() |
| |
| From: Dan Carpenter <dan.carpenter@oracle.com> |
| |
| commit 5e2e2f9f76e157063a656351728703cb02b068f1 upstream. |
| |
| "count" needs to be signed for the error handling to work. I made "i" |
| signed as well so they match. |
| |
| Fixes: 02113ba93ea4 (PM / clk: Add support for obtaining clocks from device-tree) |
| Cc: 4.6+ <stable@vger.kernel.org> # 4.6+ |
| Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> |
| Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com> |
| Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> |
| |
| --- |
| drivers/base/power/clock_ops.c | 2 +- |
| 1 file changed, 1 insertion(+), 1 deletion(-) |
| |
| --- a/drivers/base/power/clock_ops.c |
| +++ b/drivers/base/power/clock_ops.c |
| @@ -185,7 +185,7 @@ EXPORT_SYMBOL_GPL(of_pm_clk_add_clk); |
| int of_pm_clk_add_clks(struct device *dev) |
| { |
| struct clk **clks; |
| - unsigned int i, count; |
| + int i, count; |
| int ret; |
| |
| if (!dev || !dev->of_node) |