target/linux/mvebu/patches-4.14/511-clk-mvebu-armada-37xx-periph-Fix-wrong-return-value-.patch

   1 From 616bf80d381da13fbb392ebff06f46f946e3ee84 Mon Sep 17 00:00:00 2001
   2 From: Gregory CLEMENT <gregory.clement@bootlin.com>
   3 Date: Fri, 13 Jul 2018 12:27:26 +0200
   4 Subject: [PATCH] clk: mvebu: armada-37xx-periph: Fix wrong return value in
   5  get_parent
   6
   7 The return value of the get_parent operation is a u8, whereas a -EINVAL
   8 was returned. This wrong value was return if the value was bigger that
   9 the number of parent but this case was already handled by the core.
  10
  11 So we can just remove this chunk of code to fix the issue.
  12
  13 Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
  14 Fixes: 9818a7a4fd10 ("clk: mvebu: armada-37xx-periph: prepare cpu clk to
  15 be used with DVFS")
  16 Signed-off-by: Gregory CLEMENT <gregory.clement@bootlin.com>
  17 Signed-off-by: Stephen Boyd <sboyd@kernel.org>
  18 ---
  19  drivers/clk/mvebu/armada-37xx-periph.c | 3 ---
  20  1 file changed, 3 deletions(-)
  21
  22 --- a/drivers/clk/mvebu/armada-37xx-periph.c
  23 +++ b/drivers/clk/mvebu/armada-37xx-periph.c
  24 @@ -429,9 +429,6 @@ static u8 clk_pm_cpu_get_parent(struct c
  25                 val &= pm_cpu->mask_mux;
  26         }
  27
  28 -       if (val >= num_parents)
  29 -               return -EINVAL;
  30 -
  31         return val;
  32  }
  33