video: tegra: dc: fix DC_DISP_DISP_WIN_OPTIONS programming
authorAndy Ritger <aritger@nvidia.com>
Fri, 19 Jul 2013 06:59:54 +0000 (23:59 -0700)
committerDan Willemsen <dwillemsen@nvidia.com>
Sat, 14 Sep 2013 20:33:04 +0000 (13:33 -0700)
commitbfbb526e5ab31b9d6d681b8d69fb4d7af0c2fb23
treeed8ac91869ce7abcf1b5eed9c54a2b32d752f2e3
parentb8fe3a7db16ee4499bcc8275f2fc578b6f8c3a52
video: tegra: dc: fix DC_DISP_DISP_WIN_OPTIONS programming

tegra_dc_ext_set_cursor_low_latency() made several mistakes in programming
DC_DISP_DISP_WIN_OPTIONS:

* The logic to decide whether to program DC_DISP_DISP_WIN_OPTIONS was
  duplicated: once near the top of the function and once near the bottom
  of the function.

* The first instance of the logic to decide whether to program
  DC_DISP_DISP_WIN_OPTIONS used the 'enable' variable without initializing
  it.

* The second instance of the logic to decide whether to program
  DC_DISP_DISP_WIN_OPTIONS initialized the 'enable' variable
  using a bitwise AND with TEGRA_DC_EXT_CURSOR_FLAGS_VISIBLE, and
  then compared 'enable' with a boolean expression.  Fortunately,
  TEGRA_DC_EXT_CURSOR_FLAGS_VISIBLE is defined as (1 << 0), so the
  values worked out.  However, it is still clearer to convert the
  result of the bitwise operation to a boolean with '!!', as is done
  in tegra_dc_ext_set_cursor().

Change-Id: Ie84572bc68f31ae94baaaddc6147c496636b2769
Signed-off-by: Andy Ritger <aritger@nvidia.com>
Reviewed-on: http://git-master/r/251263
Reviewed-by: Automatic_Commit_Validation_User
Reviewed-by: Jon Mayo <jmayo@nvidia.com>
drivers/video/tegra/dc/ext/cursor.c