video: tegra: dc: collapse CURSOR_POSITION() and CURSOR_POSITION_124()
Andy Ritger [Fri, 19 Jul 2013 06:47:07 +0000 (23:47 -0700)]
The CURSOR_POSITION() macro was defined to limit its x and y arguments
to 16-bits each.

The CURSOR_POSITION_124() macro was later defined to limit its x and y
arguments to 14-bits each.

DC_DISP_CURSOR_POSITION only has 14-bits per component on all Tegra chips,
so there is no need for a separate T124 macro.

Change-Id: Icd74f7b8c47523089d34f153d1c6c0e3fe59ae65
Signed-off-by: Andy Ritger <aritger@nvidia.com>
Reviewed-on: http://git-master/r/251262
Reviewed-by: Automatic_Commit_Validation_User
Reviewed-by: Jon Mayo <jmayo@nvidia.com>

drivers/video/tegra/dc/dc_reg.h
drivers/video/tegra/dc/ext/cursor.c

index 0e2c571..05af5df 100644 (file)
 
 #define DC_DISP_CURSOR_POSITION                        0x440
 #define   CURSOR_POSITION(_x, _y)              \
-       (((_x) & ((1 << 16) - 1)) |             \
-       (((_y) & ((1 << 16) - 1)) << 16))
-
-#define   CURSOR_POSITION_124(_x, _y)          \
        (((_x) & ((1 << 14) - 1)) |             \
        (((_y) & ((1 << 14) - 1)) << 16))
 
index 6da58e4..d497a36 100644 (file)
@@ -433,7 +433,7 @@ int tegra_dc_ext_set_cursor_low_latency(struct tegra_dc_ext_user *user,
 
        cursor_start = tegra_dc_readl(dc, DC_DISP_CURSOR_START_ADDR);
 
-       tegra_dc_writel(dc, CURSOR_POSITION_124(args->x, args->y),
+       tegra_dc_writel(dc, CURSOR_POSITION(args->x, args->y),
                DC_DISP_CURSOR_POSITION);
 
        reg = tegra_dc_readl(dc, DC_CMD_REG_ACT_CONTROL);