From: Stephen Warren <swarren@nvidia.com>
Date: Fri, 24 Jan 2014 17:16:21 +0000 (-0700)
Subject: ARM: tegra: MASK_BITS_ no longer needs specific values
X-Git-Tag: v2025.01-rc5-pxa1908~15470^2~160^2~32
X-Git-Url: http://git.dujemihanovic.xyz/repo?a=commitdiff_plain;h=04b8e8e7452a4cd00003e33a5a736bd077ff3471;p=u-boot.git

ARM: tegra: MASK_BITS_ no longer needs specific values

Since all code that sets or interprets MASK_BITS_* now uses the enums
to define/compare the values, there is no need for MASK_BITS_* to have
a specific integer value. In fact, having a specific integer value may
encourage people to hard-code those values, or interpret the values in
incorrect ways.

As such, remove the logic that assigns a specific value to the enum
values in order to make it completely clear that it's just an enum, not
something that directly represents some integer value.

Signed-off-by: Stephen Warren <swarren@nvidia.com>
Reviewed-by: Thierry Reding <treding@nvidia.com>
Tested-by: Thierry Reding <treding@nvidia.com>
Acked-by: Thierry Reding <treding@nvidia.com>
Signed-off-by: Tom Warren <twarren@nvidia.com>
---

diff --git a/arch/arm/include/asm/arch-tegra/clock.h b/arch/arm/include/asm/arch-tegra/clock.h
index 80825e30f8..2f85696a58 100644
--- a/arch/arm/include/asm/arch-tegra/clock.h
+++ b/arch/arm/include/asm/arch-tegra/clock.h
@@ -30,7 +30,7 @@ enum clock_osc_freq {
  * them all together and pretends they're all 31:28.
  */
 enum {
-	MASK_BITS_31_30	= 2,	/* num of bits used to specify clock source */
+	MASK_BITS_31_30,
 	MASK_BITS_31_29,
 	MASK_BITS_31_28,
 };