On Fri, Jun 15, 2018 at 08:44:04PM +0300, Ville Syrjala wrote: > From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > > No point in having this extra indireciton for the cursor max size. > So drop the defines and just write out the raw numbers. Makes it > easier to see what's going on. > > Signed-off-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> Reviewed-by: Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx> > --- > drivers/gpu/drm/i915/intel_display.c | 8 ++++---- > drivers/gpu/drm/i915/intel_drv.h | 6 ------ > 2 files changed, 4 insertions(+), 10 deletions(-) > > diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c > index 8251e189a8bb..f6655f482b67 100644 > --- a/drivers/gpu/drm/i915/intel_display.c > +++ b/drivers/gpu/drm/i915/intel_display.c > @@ -15054,11 +15054,11 @@ int intel_modeset_init(struct drm_device *dev) > dev->mode_config.cursor_width = IS_I845G(dev_priv) ? 64 : 512; > dev->mode_config.cursor_height = 1023; > } else if (IS_GEN2(dev_priv)) { > - dev->mode_config.cursor_width = GEN2_CURSOR_WIDTH; > - dev->mode_config.cursor_height = GEN2_CURSOR_HEIGHT; > + dev->mode_config.cursor_width = 64; > + dev->mode_config.cursor_height = 64; > } else { > - dev->mode_config.cursor_width = MAX_CURSOR_WIDTH; > - dev->mode_config.cursor_height = MAX_CURSOR_HEIGHT; > + dev->mode_config.cursor_width = 256; > + dev->mode_config.cursor_height = 256; > } > > dev->mode_config.fb_base = ggtt->gmadr.start; > diff --git a/drivers/gpu/drm/i915/intel_drv.h b/drivers/gpu/drm/i915/intel_drv.h > index 8840108749a5..2d09f08e5e0c 100644 > --- a/drivers/gpu/drm/i915/intel_drv.h > +++ b/drivers/gpu/drm/i915/intel_drv.h > @@ -158,12 +158,6 @@ > #define MAX_OUTPUTS 6 > /* maximum connectors per crtcs in the mode set */ > > -/* Maximum cursor sizes */ > -#define GEN2_CURSOR_WIDTH 64 > -#define GEN2_CURSOR_HEIGHT 64 > -#define MAX_CURSOR_WIDTH 256 > -#define MAX_CURSOR_HEIGHT 256 > - > #define INTEL_I2C_BUS_DVO 1 > #define INTEL_I2C_BUS_SDVO 2 > > -- > 2.16.4 > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@xxxxxxxxxxxxxxxxxxxxx > https://lists.freedesktop.org/mailman/listinfo/intel-gfx _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx