clock: tegra: fix typecast
Bo Yan [Sat, 8 Nov 2014 01:49:57 +0000 (17:49 -0800)]
Remove unnecessary typecast which actually causes Sparse warning.

The "uintptr_t" typecast is only needed when there is a need to
temporarily save pointer to an integer which is later converted
to pointer again. Here the pointer is directly used for MMIO access,
therefore there is no need to do extra typecasting.

bug 200032218

Change-Id: I4476aa8cabfba05315d7b027e385bdd4a8c547de
Signed-off-by: Bo Yan <byan@nvidia.com>
Reviewed-on: http://git-master/r/598937
Reviewed-by: Sachin Nikam <snikam@nvidia.com>
Tested-by: Sachin Nikam <snikam@nvidia.com>

drivers/platform/tegra/clock.c

index 62aa16e..d8781a7 100644 (file)
@@ -1257,14 +1257,11 @@ unsigned long tegra_clk_measure_input_freq(void)
        if (osc_freq)
                return osc_freq;
 
-       writel(OSC_FREQ_DET_TRIG | 1,
-               (void *)((uintptr_t)clk_base + OSC_FREQ_DET));
-       do {} while (readl(
-               (void *)((uintptr_t)clk_base + OSC_FREQ_DET_STATUS))
+       writel(OSC_FREQ_DET_TRIG | 1, clk_base + OSC_FREQ_DET);
+       do {} while (readl(clk_base + OSC_FREQ_DET_STATUS)
                & OSC_FREQ_DET_BUSY);
 
-       clock_autodetect = readl(
-               (void *)((uintptr_t)clk_base + OSC_FREQ_DET_STATUS));
+       clock_autodetect = readl(clk_base + OSC_FREQ_DET_STATUS);
        if (clock_autodetect >= 732 - 3 && clock_autodetect <= 732 + 3) {
                osc_freq = 12000000;
        } else if (clock_autodetect >= 794 - 3 && clock_autodetect <= 794 + 3) {