On Jun 30, 2022, at 3:51 PM, Ville Syrjala <ville.syrj...@linux.intel.com<mailto:ville.syrj...@linux.intel.com>> wrote:
From: Ville Syrjälä <ville.syrj...@linux.intel.com<mailto:ville.syrj...@linux.intel.com>> Stop using the 'TRUE' define. This ultimately gets defined by acpi/actypes.h that gets included here via a convoluted chain of other headers. drm_crtc.h is part of that chain, and I'm trying to eliminate all unnecessary includes from it to avoid pointless rebuilds. v2: Split out from the bigger patch Cc: Zack Rusin <za...@vmware.com<mailto:za...@vmware.com>> Cc: VMware Graphics Reviewers <linux-graphics-maintai...@vmware.com<mailto:linux-graphics-maintai...@vmware.com>> Acked-by: Sam Ravnborg <s...@ravnborg.org<mailto:s...@ravnborg.org>> Acked-by: Jani Nikula <jani.nik...@intel.com<mailto:jani.nik...@intel.com>> Signed-off-by: Ville Syrjälä <ville.syrj...@linux.intel.com<mailto:ville.syrj...@linux.intel.com>> --- drivers/gpu/drm/vmwgfx/vmwgfx_kms.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_kms.c b/drivers/gpu/drm/vmwgfx/vmwgfx_kms.c index 693028c31b6b..ff2f735bbe7a 100644 --- a/drivers/gpu/drm/vmwgfx/vmwgfx_kms.c +++ b/drivers/gpu/drm/vmwgfx/vmwgfx_kms.c @@ -252,7 +252,7 @@ static void vmw_cursor_update_position(struct vmw_private *dev_priv, vmw_write(dev_priv, SVGA_REG_CURSOR4_Y, y); vmw_write(dev_priv, SVGA_REG_CURSOR4_SCREEN_ID, SVGA3D_INVALID_ID); vmw_write(dev_priv, SVGA_REG_CURSOR4_ON, svga_cursor_on); - vmw_write(dev_priv, SVGA_REG_CURSOR4_SUBMIT, TRUE); + vmw_write(dev_priv, SVGA_REG_CURSOR4_SUBMIT, 1); } else if (vmw_is_cursor_bypass3_enabled(dev_priv)) { vmw_fifo_mem_write(dev_priv, SVGA_FIFO_CURSOR_ON, svga_cursor_on); vmw_fifo_mem_write(dev_priv, SVGA_FIFO_CURSOR_X, x); Hi, Ville. Sorry for the delay. Looks great. In case you haven’t pushed it to drm-misc-next yet: Reviewed-by: Zack Rusin <za...@vmware.com<mailto:za...@vmware.com>> z
