mips: mt76x8: ddr_cal: Change types from u32 to int in dqs_find_min/max
authorStefan Roese <sr@denx.de>
Fri, 6 Mar 2020 14:14:04 +0000 (15:14 +0100)
committerDaniel Schwierzeck <daniel.schwierzeck@gmail.com>
Mon, 27 Apr 2020 18:30:13 +0000 (20:30 +0200)
This change is made to enable comparison of integer variables, which
might be negative in the next patch. No functional change is intended
in this patch.

Signed-off-by: Stefan Roese <sr@denx.de>
Cc: Weijie Gao <weijie.gao@mediatek.com>
Cc: Daniel Schwierzeck <daniel.schwierzeck@gmail.com>
Reviewed-by: Weijie Gao <weijie.gao@mediatek.com>
arch/mips/mach-mtmips/ddr_cal.c

index b934c4a854a92ca7e314e7f522a5769a9b1571c9..80a058d693d0db6b10712b53bb696260a839f4dc 100644 (file)
@@ -71,10 +71,11 @@ static inline bool dqs_test_error(void __iomem *memc, u32 memsize, u32 dqsval,
        return false;
 }
 
-static inline u32 dqs_find_max(void __iomem *memc, u32 memsize, u32 initval,
-                              u32 maxval, u32 shift, u32 regval)
+static inline int dqs_find_max(void __iomem *memc, u32 memsize, int initval,
+                              int maxval, int shift, u32 regval)
 {
-       u32 fieldval = initval, dqsval;
+       int fieldval = initval;
+       u32 dqsval;
 
        do {
                dqsval = regval | (fieldval << shift);
@@ -88,10 +89,11 @@ static inline u32 dqs_find_max(void __iomem *memc, u32 memsize, u32 initval,
        return fieldval;
 }
 
-static inline u32 dqs_find_min(void __iomem *memc, u32 memsize, u32 initval,
-                              u32 minval, u32 shift, u32 regval)
+static inline int dqs_find_min(void __iomem *memc, u32 memsize, int initval,
+                              int minval, int shift, u32 regval)
 {
-       u32 fieldval = initval, dqsval;
+       int fieldval = initval;
+       u32 dqsval;
 
        while (fieldval > minval) {
                dqsval = regval | (fieldval << shift);