Browse Source

staging: mt7621-mmc: Remove function uffs

The function uffs that is implemented by this driver returns the same
values as the kernel function ffs. So the uffs function is removed and
the calls to it are replaced with calls to ffs.

Signed-off-by: Christian Lütke-Stetzkamp <christian@lkamp.de>
Reviewed-by: NeilBrown <neil@brown.name>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Christian Lütke-Stetzkamp 7 years ago
parent
commit
35af3110d4
1 changed files with 2 additions and 31 deletions
  1. 2 31
      drivers/staging/mt7621-mmc/mt6575_sd.h

+ 2 - 31
drivers/staging/mt7621-mmc/mt6575_sd.h

@@ -950,35 +950,6 @@ struct msdc_host {
 	u32                         app_cmd_arg;
 };
 
-static inline unsigned int uffs(unsigned int x)
-{
-	unsigned int r = 1;
-
-	if (!x)
-		return 0;
-	if (!(x & 0xffff)) {
-		x >>= 16;
-		r += 16;
-	}
-	if (!(x & 0xff)) {
-		x >>= 8;
-		r += 8;
-	}
-	if (!(x & 0xf)) {
-		x >>= 4;
-		r += 4;
-	}
-	if (!(x & 3)) {
-		x >>= 2;
-		r += 2;
-	}
-	if (!(x & 1)) {
-		x >>= 1;
-		r += 1;
-	}
-	return r;
-}
-
 #define sdr_read8(reg)            readb(reg)
 #define sdr_read32(reg)           readl(reg)
 #define sdr_write8(reg, val)      writeb(val, reg)
@@ -1004,13 +975,13 @@ static inline void sdr_clr_bits(void __iomem *reg, u32 bs)
 do {								\
 	volatile unsigned int tv = sdr_read32(reg);			\
 	tv &= ~(field);							\
-	tv |= ((val) << (uffs((unsigned int)field) - 1));		\
+	tv |= ((val) << (ffs((unsigned int)field) - 1));		\
 	sdr_write32(reg, tv);						\
 } while (0)
 #define sdr_get_field(reg, field, val)					\
 do {								\
 	volatile unsigned int tv = sdr_read32(reg);			\
-	val = ((tv & (field)) >> (uffs((unsigned int)field) - 1));	\
+	val = ((tv & (field)) >> (ffs((unsigned int)field) - 1));	\
 } while (0)
 
 #endif