Commit 1ee8da6d authored by Chris Wilson's avatar Chris Wilson

drm/i915: Convert intel_overlay.c to use native drm_i915_private pointers

Another day, another long overdue conversion. Not much to update inside
intel_overlay.c, but still

   text	   data	    bss	    dec	    hex	filename
6309547	3578778	 696320	10584645	 a18245	vmlinux
6309291	3578778	 696320	10584389	 a18145	vmlinux

a couple of hundred bytes of pointer misdirection.

Whilst here, rename the ioctl entry points to include the _ioctl suffix
so that the user entry points are clear (following the idiom).
Signed-off-by: default avatarChris Wilson <chris@chris-wilson.co.uk>
Link: http://patchwork.freedesktop.org/patch/msgid/1463053403-25086-1-git-send-email-chris@chris-wilson.co.ukReviewed-by: default avatarTvrtko Ursulin <tvrtko.ursulin@intel.com>
parent d5aab9d4
...@@ -1623,8 +1623,8 @@ const struct drm_ioctl_desc i915_ioctls[] = { ...@@ -1623,8 +1623,8 @@ const struct drm_ioctl_desc i915_ioctls[] = {
DRM_IOCTL_DEF_DRV(I915_GEM_GET_APERTURE, i915_gem_get_aperture_ioctl, DRM_RENDER_ALLOW), DRM_IOCTL_DEF_DRV(I915_GEM_GET_APERTURE, i915_gem_get_aperture_ioctl, DRM_RENDER_ALLOW),
DRM_IOCTL_DEF_DRV(I915_GET_PIPE_FROM_CRTC_ID, intel_get_pipe_from_crtc_id, 0), DRM_IOCTL_DEF_DRV(I915_GET_PIPE_FROM_CRTC_ID, intel_get_pipe_from_crtc_id, 0),
DRM_IOCTL_DEF_DRV(I915_GEM_MADVISE, i915_gem_madvise_ioctl, DRM_RENDER_ALLOW), DRM_IOCTL_DEF_DRV(I915_GEM_MADVISE, i915_gem_madvise_ioctl, DRM_RENDER_ALLOW),
DRM_IOCTL_DEF_DRV(I915_OVERLAY_PUT_IMAGE, intel_overlay_put_image, DRM_MASTER|DRM_CONTROL_ALLOW), DRM_IOCTL_DEF_DRV(I915_OVERLAY_PUT_IMAGE, intel_overlay_put_image_ioctl, DRM_MASTER|DRM_CONTROL_ALLOW),
DRM_IOCTL_DEF_DRV(I915_OVERLAY_ATTRS, intel_overlay_attrs, DRM_MASTER|DRM_CONTROL_ALLOW), DRM_IOCTL_DEF_DRV(I915_OVERLAY_ATTRS, intel_overlay_attrs_ioctl, DRM_MASTER|DRM_CONTROL_ALLOW),
DRM_IOCTL_DEF_DRV(I915_SET_SPRITE_COLORKEY, intel_sprite_set_colorkey, DRM_MASTER|DRM_CONTROL_ALLOW), DRM_IOCTL_DEF_DRV(I915_SET_SPRITE_COLORKEY, intel_sprite_set_colorkey, DRM_MASTER|DRM_CONTROL_ALLOW),
DRM_IOCTL_DEF_DRV(I915_GET_SPRITE_COLORKEY, drm_noop, DRM_MASTER|DRM_CONTROL_ALLOW), DRM_IOCTL_DEF_DRV(I915_GET_SPRITE_COLORKEY, drm_noop, DRM_MASTER|DRM_CONTROL_ALLOW),
DRM_IOCTL_DEF_DRV(I915_GEM_WAIT, i915_gem_wait_ioctl, DRM_AUTH|DRM_RENDER_ALLOW), DRM_IOCTL_DEF_DRV(I915_GEM_WAIT, i915_gem_wait_ioctl, DRM_AUTH|DRM_RENDER_ALLOW),
......
...@@ -16020,7 +16020,7 @@ void intel_modeset_gem_init(struct drm_device *dev) ...@@ -16020,7 +16020,7 @@ void intel_modeset_gem_init(struct drm_device *dev)
intel_modeset_init_hw(dev); intel_modeset_init_hw(dev);
intel_setup_overlay(dev); intel_setup_overlay(dev_priv);
/* /*
* Make sure any fbs we allocated at startup are properly * Make sure any fbs we allocated at startup are properly
...@@ -16093,7 +16093,7 @@ void intel_modeset_cleanup(struct drm_device *dev) ...@@ -16093,7 +16093,7 @@ void intel_modeset_cleanup(struct drm_device *dev)
drm_mode_config_cleanup(dev); drm_mode_config_cleanup(dev);
intel_cleanup_overlay(dev); intel_cleanup_overlay(dev_priv);
intel_cleanup_gt_powersave(dev_priv); intel_cleanup_gt_powersave(dev_priv);
......
...@@ -1435,13 +1435,13 @@ void intel_attach_aspect_ratio_property(struct drm_connector *connector); ...@@ -1435,13 +1435,13 @@ void intel_attach_aspect_ratio_property(struct drm_connector *connector);
/* intel_overlay.c */ /* intel_overlay.c */
void intel_setup_overlay(struct drm_device *dev); void intel_setup_overlay(struct drm_i915_private *dev_priv);
void intel_cleanup_overlay(struct drm_device *dev); void intel_cleanup_overlay(struct drm_i915_private *dev_priv);
int intel_overlay_switch_off(struct intel_overlay *overlay); int intel_overlay_switch_off(struct intel_overlay *overlay);
int intel_overlay_put_image(struct drm_device *dev, void *data, int intel_overlay_put_image_ioctl(struct drm_device *dev, void *data,
struct drm_file *file_priv); struct drm_file *file_priv);
int intel_overlay_attrs(struct drm_device *dev, void *data, int intel_overlay_attrs_ioctl(struct drm_device *dev, void *data,
struct drm_file *file_priv); struct drm_file *file_priv);
void intel_overlay_reset(struct drm_i915_private *dev_priv); void intel_overlay_reset(struct drm_i915_private *dev_priv);
......
...@@ -168,7 +168,7 @@ struct overlay_registers { ...@@ -168,7 +168,7 @@ struct overlay_registers {
}; };
struct intel_overlay { struct intel_overlay {
struct drm_device *dev; struct drm_i915_private *i915;
struct intel_crtc *crtc; struct intel_crtc *crtc;
struct drm_i915_gem_object *vid_bo; struct drm_i915_gem_object *vid_bo;
struct drm_i915_gem_object *old_vid_bo; struct drm_i915_gem_object *old_vid_bo;
...@@ -190,14 +190,13 @@ struct intel_overlay { ...@@ -190,14 +190,13 @@ struct intel_overlay {
static struct overlay_registers __iomem * static struct overlay_registers __iomem *
intel_overlay_map_regs(struct intel_overlay *overlay) intel_overlay_map_regs(struct intel_overlay *overlay)
{ {
struct drm_i915_private *dev_priv = to_i915(overlay->dev); struct drm_i915_private *dev_priv = overlay->i915;
struct i915_ggtt *ggtt = &dev_priv->ggtt;
struct overlay_registers __iomem *regs; struct overlay_registers __iomem *regs;
if (OVERLAY_NEEDS_PHYSICAL(overlay->dev)) if (OVERLAY_NEEDS_PHYSICAL(dev_priv))
regs = (struct overlay_registers __iomem *)overlay->reg_bo->phys_handle->vaddr; regs = (struct overlay_registers __iomem *)overlay->reg_bo->phys_handle->vaddr;
else else
regs = io_mapping_map_wc(ggtt->mappable, regs = io_mapping_map_wc(dev_priv->ggtt.mappable,
overlay->flip_addr, overlay->flip_addr,
PAGE_SIZE); PAGE_SIZE);
...@@ -207,7 +206,7 @@ intel_overlay_map_regs(struct intel_overlay *overlay) ...@@ -207,7 +206,7 @@ intel_overlay_map_regs(struct intel_overlay *overlay)
static void intel_overlay_unmap_regs(struct intel_overlay *overlay, static void intel_overlay_unmap_regs(struct intel_overlay *overlay,
struct overlay_registers __iomem *regs) struct overlay_registers __iomem *regs)
{ {
if (!OVERLAY_NEEDS_PHYSICAL(overlay->dev)) if (!OVERLAY_NEEDS_PHYSICAL(overlay->i915))
io_mapping_unmap(regs); io_mapping_unmap(regs);
} }
...@@ -233,14 +232,13 @@ static int intel_overlay_do_wait_request(struct intel_overlay *overlay, ...@@ -233,14 +232,13 @@ static int intel_overlay_do_wait_request(struct intel_overlay *overlay,
/* overlay needs to be disable in OCMD reg */ /* overlay needs to be disable in OCMD reg */
static int intel_overlay_on(struct intel_overlay *overlay) static int intel_overlay_on(struct intel_overlay *overlay)
{ {
struct drm_device *dev = overlay->dev; struct drm_i915_private *dev_priv = overlay->i915;
struct drm_i915_private *dev_priv = dev->dev_private;
struct intel_engine_cs *engine = &dev_priv->engine[RCS]; struct intel_engine_cs *engine = &dev_priv->engine[RCS];
struct drm_i915_gem_request *req; struct drm_i915_gem_request *req;
int ret; int ret;
WARN_ON(overlay->active); WARN_ON(overlay->active);
WARN_ON(IS_I830(dev) && !(dev_priv->quirks & QUIRK_PIPEA_FORCE)); WARN_ON(IS_I830(dev_priv) && !(dev_priv->quirks & QUIRK_PIPEA_FORCE));
req = i915_gem_request_alloc(engine, NULL); req = i915_gem_request_alloc(engine, NULL);
if (IS_ERR(req)) if (IS_ERR(req))
...@@ -267,8 +265,7 @@ static int intel_overlay_on(struct intel_overlay *overlay) ...@@ -267,8 +265,7 @@ static int intel_overlay_on(struct intel_overlay *overlay)
static int intel_overlay_continue(struct intel_overlay *overlay, static int intel_overlay_continue(struct intel_overlay *overlay,
bool load_polyphase_filter) bool load_polyphase_filter)
{ {
struct drm_device *dev = overlay->dev; struct drm_i915_private *dev_priv = overlay->i915;
struct drm_i915_private *dev_priv = dev->dev_private;
struct intel_engine_cs *engine = &dev_priv->engine[RCS]; struct intel_engine_cs *engine = &dev_priv->engine[RCS];
struct drm_i915_gem_request *req; struct drm_i915_gem_request *req;
u32 flip_addr = overlay->flip_addr; u32 flip_addr = overlay->flip_addr;
...@@ -336,8 +333,7 @@ static void intel_overlay_off_tail(struct intel_overlay *overlay) ...@@ -336,8 +333,7 @@ static void intel_overlay_off_tail(struct intel_overlay *overlay)
/* overlay needs to be disabled in OCMD reg */ /* overlay needs to be disabled in OCMD reg */
static int intel_overlay_off(struct intel_overlay *overlay) static int intel_overlay_off(struct intel_overlay *overlay)
{ {
struct drm_device *dev = overlay->dev; struct drm_i915_private *dev_priv = overlay->i915;
struct drm_i915_private *dev_priv = dev->dev_private;
struct intel_engine_cs *engine = &dev_priv->engine[RCS]; struct intel_engine_cs *engine = &dev_priv->engine[RCS];
struct drm_i915_gem_request *req; struct drm_i915_gem_request *req;
u32 flip_addr = overlay->flip_addr; u32 flip_addr = overlay->flip_addr;
...@@ -366,7 +362,7 @@ static int intel_overlay_off(struct intel_overlay *overlay) ...@@ -366,7 +362,7 @@ static int intel_overlay_off(struct intel_overlay *overlay)
intel_ring_emit(engine, flip_addr); intel_ring_emit(engine, flip_addr);
intel_ring_emit(engine, MI_WAIT_FOR_EVENT | MI_WAIT_FOR_OVERLAY_FLIP); intel_ring_emit(engine, MI_WAIT_FOR_EVENT | MI_WAIT_FOR_OVERLAY_FLIP);
/* turn overlay off */ /* turn overlay off */
if (IS_I830(dev)) { if (IS_I830(dev_priv)) {
/* Workaround: Don't disable the overlay fully, since otherwise /* Workaround: Don't disable the overlay fully, since otherwise
* it dies on the next OVERLAY_ON cmd. */ * it dies on the next OVERLAY_ON cmd. */
intel_ring_emit(engine, MI_NOOP); intel_ring_emit(engine, MI_NOOP);
...@@ -409,12 +405,11 @@ static int intel_overlay_recover_from_interrupt(struct intel_overlay *overlay) ...@@ -409,12 +405,11 @@ static int intel_overlay_recover_from_interrupt(struct intel_overlay *overlay)
*/ */
static int intel_overlay_release_old_vid(struct intel_overlay *overlay) static int intel_overlay_release_old_vid(struct intel_overlay *overlay)
{ {
struct drm_device *dev = overlay->dev; struct drm_i915_private *dev_priv = overlay->i915;
struct drm_i915_private *dev_priv = dev->dev_private;
struct intel_engine_cs *engine = &dev_priv->engine[RCS]; struct intel_engine_cs *engine = &dev_priv->engine[RCS];
int ret; int ret;
WARN_ON(!mutex_is_locked(&dev->struct_mutex)); lockdep_assert_held(&dev_priv->dev->struct_mutex);
/* Only wait if there is actually an old frame to release to /* Only wait if there is actually an old frame to release to
* guarantee forward progress. * guarantee forward progress.
...@@ -538,10 +533,10 @@ static int uv_vsubsampling(u32 format) ...@@ -538,10 +533,10 @@ static int uv_vsubsampling(u32 format)
} }
} }
static u32 calc_swidthsw(struct drm_device *dev, u32 offset, u32 width) static u32 calc_swidthsw(struct drm_i915_private *dev_priv, u32 offset, u32 width)
{ {
u32 mask, shift, ret; u32 mask, shift, ret;
if (IS_GEN2(dev)) { if (IS_GEN2(dev_priv)) {
mask = 0x1f; mask = 0x1f;
shift = 5; shift = 5;
} else { } else {
...@@ -549,7 +544,7 @@ static u32 calc_swidthsw(struct drm_device *dev, u32 offset, u32 width) ...@@ -549,7 +544,7 @@ static u32 calc_swidthsw(struct drm_device *dev, u32 offset, u32 width)
shift = 6; shift = 6;
} }
ret = ((offset + width + mask) >> shift) - (offset >> shift); ret = ((offset + width + mask) >> shift) - (offset >> shift);
if (!IS_GEN2(dev)) if (!IS_GEN2(dev_priv))
ret <<= 1; ret <<= 1;
ret -= 1; ret -= 1;
return ret << 2; return ret << 2;
...@@ -742,12 +737,12 @@ static int intel_overlay_do_put_image(struct intel_overlay *overlay, ...@@ -742,12 +737,12 @@ static int intel_overlay_do_put_image(struct intel_overlay *overlay,
int ret, tmp_width; int ret, tmp_width;
struct overlay_registers __iomem *regs; struct overlay_registers __iomem *regs;
bool scale_changed = false; bool scale_changed = false;
struct drm_device *dev = overlay->dev; struct drm_i915_private *dev_priv = overlay->i915;
u32 swidth, swidthsw, sheight, ostride; u32 swidth, swidthsw, sheight, ostride;
enum pipe pipe = overlay->crtc->pipe; enum pipe pipe = overlay->crtc->pipe;
WARN_ON(!mutex_is_locked(&dev->struct_mutex)); lockdep_assert_held(&dev_priv->dev->struct_mutex);
WARN_ON(!drm_modeset_is_locked(&dev->mode_config.connection_mutex)); WARN_ON(!drm_modeset_is_locked(&dev_priv->dev->mode_config.connection_mutex));
ret = intel_overlay_release_old_vid(overlay); ret = intel_overlay_release_old_vid(overlay);
if (ret != 0) if (ret != 0)
...@@ -770,7 +765,7 @@ static int intel_overlay_do_put_image(struct intel_overlay *overlay, ...@@ -770,7 +765,7 @@ static int intel_overlay_do_put_image(struct intel_overlay *overlay,
goto out_unpin; goto out_unpin;
} }
oconfig = OCONF_CC_OUT_8BIT; oconfig = OCONF_CC_OUT_8BIT;
if (IS_GEN4(overlay->dev)) if (IS_GEN4(dev_priv))
oconfig |= OCONF_CSC_MODE_BT709; oconfig |= OCONF_CSC_MODE_BT709;
oconfig |= pipe == 0 ? oconfig |= pipe == 0 ?
OCONF_PIPE_A : OCONF_PIPE_B; OCONF_PIPE_A : OCONF_PIPE_B;
...@@ -797,7 +792,7 @@ static int intel_overlay_do_put_image(struct intel_overlay *overlay, ...@@ -797,7 +792,7 @@ static int intel_overlay_do_put_image(struct intel_overlay *overlay,
tmp_width = params->src_w; tmp_width = params->src_w;
swidth = params->src_w; swidth = params->src_w;
swidthsw = calc_swidthsw(overlay->dev, params->offset_Y, tmp_width); swidthsw = calc_swidthsw(dev_priv, params->offset_Y, tmp_width);
sheight = params->src_h; sheight = params->src_h;
iowrite32(i915_gem_obj_ggtt_offset(new_bo) + params->offset_Y, &regs->OBUF_0Y); iowrite32(i915_gem_obj_ggtt_offset(new_bo) + params->offset_Y, &regs->OBUF_0Y);
ostride = params->stride_Y; ostride = params->stride_Y;
...@@ -807,9 +802,9 @@ static int intel_overlay_do_put_image(struct intel_overlay *overlay, ...@@ -807,9 +802,9 @@ static int intel_overlay_do_put_image(struct intel_overlay *overlay,
int uv_vscale = uv_vsubsampling(params->format); int uv_vscale = uv_vsubsampling(params->format);
u32 tmp_U, tmp_V; u32 tmp_U, tmp_V;
swidth |= (params->src_w/uv_hscale) << 16; swidth |= (params->src_w/uv_hscale) << 16;
tmp_U = calc_swidthsw(overlay->dev, params->offset_U, tmp_U = calc_swidthsw(dev_priv, params->offset_U,
params->src_w/uv_hscale); params->src_w/uv_hscale);
tmp_V = calc_swidthsw(overlay->dev, params->offset_V, tmp_V = calc_swidthsw(dev_priv, params->offset_V,
params->src_w/uv_hscale); params->src_w/uv_hscale);
swidthsw |= max_t(u32, tmp_U, tmp_V) << 16; swidthsw |= max_t(u32, tmp_U, tmp_V) << 16;
sheight |= (params->src_h/uv_vscale) << 16; sheight |= (params->src_h/uv_vscale) << 16;
...@@ -841,8 +836,7 @@ static int intel_overlay_do_put_image(struct intel_overlay *overlay, ...@@ -841,8 +836,7 @@ static int intel_overlay_do_put_image(struct intel_overlay *overlay,
overlay->old_vid_bo = overlay->vid_bo; overlay->old_vid_bo = overlay->vid_bo;
overlay->vid_bo = new_bo; overlay->vid_bo = new_bo;
intel_frontbuffer_flip(dev, intel_frontbuffer_flip(dev_priv->dev, INTEL_FRONTBUFFER_OVERLAY(pipe));
INTEL_FRONTBUFFER_OVERLAY(pipe));
return 0; return 0;
...@@ -853,12 +847,12 @@ static int intel_overlay_do_put_image(struct intel_overlay *overlay, ...@@ -853,12 +847,12 @@ static int intel_overlay_do_put_image(struct intel_overlay *overlay,
int intel_overlay_switch_off(struct intel_overlay *overlay) int intel_overlay_switch_off(struct intel_overlay *overlay)
{ {
struct drm_i915_private *dev_priv = overlay->i915;
struct overlay_registers __iomem *regs; struct overlay_registers __iomem *regs;
struct drm_device *dev = overlay->dev;
int ret; int ret;
WARN_ON(!mutex_is_locked(&dev->struct_mutex)); lockdep_assert_held(&dev_priv->dev->struct_mutex);
WARN_ON(!drm_modeset_is_locked(&dev->mode_config.connection_mutex)); WARN_ON(!drm_modeset_is_locked(&dev_priv->dev->mode_config.connection_mutex));
ret = intel_overlay_recover_from_interrupt(overlay); ret = intel_overlay_recover_from_interrupt(overlay);
if (ret != 0) if (ret != 0)
...@@ -898,15 +892,14 @@ static int check_overlay_possible_on_crtc(struct intel_overlay *overlay, ...@@ -898,15 +892,14 @@ static int check_overlay_possible_on_crtc(struct intel_overlay *overlay,
static void update_pfit_vscale_ratio(struct intel_overlay *overlay) static void update_pfit_vscale_ratio(struct intel_overlay *overlay)
{ {
struct drm_device *dev = overlay->dev; struct drm_i915_private *dev_priv = overlay->i915;
struct drm_i915_private *dev_priv = dev->dev_private;
u32 pfit_control = I915_READ(PFIT_CONTROL); u32 pfit_control = I915_READ(PFIT_CONTROL);
u32 ratio; u32 ratio;
/* XXX: This is not the same logic as in the xorg driver, but more in /* XXX: This is not the same logic as in the xorg driver, but more in
* line with the intel documentation for the i965 * line with the intel documentation for the i965
*/ */
if (INTEL_INFO(dev)->gen >= 4) { if (INTEL_GEN(dev_priv) >= 4) {
/* on i965 use the PGM reg to read out the autoscaler values */ /* on i965 use the PGM reg to read out the autoscaler values */
ratio = I915_READ(PFIT_PGM_RATIOS) >> PFIT_VERT_SCALE_SHIFT_965; ratio = I915_READ(PFIT_PGM_RATIOS) >> PFIT_VERT_SCALE_SHIFT_965;
} else { } else {
...@@ -949,7 +942,7 @@ static int check_overlay_scaling(struct put_image_params *rec) ...@@ -949,7 +942,7 @@ static int check_overlay_scaling(struct put_image_params *rec)
return 0; return 0;
} }
static int check_overlay_src(struct drm_device *dev, static int check_overlay_src(struct drm_i915_private *dev_priv,
struct drm_intel_overlay_put_image *rec, struct drm_intel_overlay_put_image *rec,
struct drm_i915_gem_object *new_bo) struct drm_i915_gem_object *new_bo)
{ {
...@@ -960,7 +953,7 @@ static int check_overlay_src(struct drm_device *dev, ...@@ -960,7 +953,7 @@ static int check_overlay_src(struct drm_device *dev,
u32 tmp; u32 tmp;
/* check src dimensions */ /* check src dimensions */
if (IS_845G(dev) || IS_I830(dev)) { if (IS_845G(dev_priv) || IS_I830(dev_priv)) {
if (rec->src_height > IMAGE_MAX_HEIGHT_LEGACY || if (rec->src_height > IMAGE_MAX_HEIGHT_LEGACY ||
rec->src_width > IMAGE_MAX_WIDTH_LEGACY) rec->src_width > IMAGE_MAX_WIDTH_LEGACY)
return -EINVAL; return -EINVAL;
...@@ -1012,14 +1005,14 @@ static int check_overlay_src(struct drm_device *dev, ...@@ -1012,14 +1005,14 @@ static int check_overlay_src(struct drm_device *dev,
return -EINVAL; return -EINVAL;
/* stride checking */ /* stride checking */
if (IS_I830(dev) || IS_845G(dev)) if (IS_I830(dev_priv) || IS_845G(dev_priv))
stride_mask = 255; stride_mask = 255;
else else
stride_mask = 63; stride_mask = 63;
if (rec->stride_Y & stride_mask || rec->stride_UV & stride_mask) if (rec->stride_Y & stride_mask || rec->stride_UV & stride_mask)
return -EINVAL; return -EINVAL;
if (IS_GEN4(dev) && rec->stride_Y < 512) if (IS_GEN4(dev_priv) && rec->stride_Y < 512)
return -EINVAL; return -EINVAL;
tmp = (rec->flags & I915_OVERLAY_TYPE_MASK) == I915_OVERLAY_YUV_PLANAR ? tmp = (rec->flags & I915_OVERLAY_TYPE_MASK) == I915_OVERLAY_YUV_PLANAR ?
...@@ -1064,13 +1057,13 @@ static int check_overlay_src(struct drm_device *dev, ...@@ -1064,13 +1057,13 @@ static int check_overlay_src(struct drm_device *dev,
* Return the pipe currently connected to the panel fitter, * Return the pipe currently connected to the panel fitter,
* or -1 if the panel fitter is not present or not in use * or -1 if the panel fitter is not present or not in use
*/ */
static int intel_panel_fitter_pipe(struct drm_device *dev) static int intel_panel_fitter_pipe(struct drm_i915_private *dev_priv)
{ {
struct drm_i915_private *dev_priv = dev->dev_private;
u32 pfit_control; u32 pfit_control;
/* i830 doesn't have a panel fitter */ /* i830 doesn't have a panel fitter */
if (INTEL_INFO(dev)->gen <= 3 && (IS_I830(dev) || !IS_MOBILE(dev))) if (INTEL_GEN(dev_priv) <= 3 &&
(IS_I830(dev_priv) || !IS_MOBILE(dev_priv)))
return -1; return -1;
pfit_control = I915_READ(PFIT_CONTROL); pfit_control = I915_READ(PFIT_CONTROL);
...@@ -1080,15 +1073,15 @@ static int intel_panel_fitter_pipe(struct drm_device *dev) ...@@ -1080,15 +1073,15 @@ static int intel_panel_fitter_pipe(struct drm_device *dev)
return -1; return -1;
/* 965 can place panel fitter on either pipe */ /* 965 can place panel fitter on either pipe */
if (IS_GEN4(dev)) if (IS_GEN4(dev_priv))
return (pfit_control >> 29) & 0x3; return (pfit_control >> 29) & 0x3;
/* older chips can only use pipe 1 */ /* older chips can only use pipe 1 */
return 1; return 1;
} }
int intel_overlay_put_image(struct drm_device *dev, void *data, int intel_overlay_put_image_ioctl(struct drm_device *dev, void *data,
struct drm_file *file_priv) struct drm_file *file_priv)
{ {
struct drm_intel_overlay_put_image *put_image_rec = data; struct drm_intel_overlay_put_image *put_image_rec = data;
struct drm_i915_private *dev_priv = dev->dev_private; struct drm_i915_private *dev_priv = dev->dev_private;
...@@ -1163,7 +1156,7 @@ int intel_overlay_put_image(struct drm_device *dev, void *data, ...@@ -1163,7 +1156,7 @@ int intel_overlay_put_image(struct drm_device *dev, void *data,
/* line too wide, i.e. one-line-mode */ /* line too wide, i.e. one-line-mode */
if (mode->hdisplay > 1024 && if (mode->hdisplay > 1024 &&
intel_panel_fitter_pipe(dev) == crtc->pipe) { intel_panel_fitter_pipe(dev_priv) == crtc->pipe) {
overlay->pfit_active = true; overlay->pfit_active = true;
update_pfit_vscale_ratio(overlay); update_pfit_vscale_ratio(overlay);
} else } else
...@@ -1197,7 +1190,7 @@ int intel_overlay_put_image(struct drm_device *dev, void *data, ...@@ -1197,7 +1190,7 @@ int intel_overlay_put_image(struct drm_device *dev, void *data,
goto out_unlock; goto out_unlock;
} }
ret = check_overlay_src(dev, put_image_rec, new_bo); ret = check_overlay_src(dev_priv, put_image_rec, new_bo);
if (ret != 0) if (ret != 0)
goto out_unlock; goto out_unlock;
params->format = put_image_rec->flags & ~I915_OVERLAY_FLAGS_MASK; params->format = put_image_rec->flags & ~I915_OVERLAY_FLAGS_MASK;
...@@ -1285,8 +1278,8 @@ static int check_gamma(struct drm_intel_overlay_attrs *attrs) ...@@ -1285,8 +1278,8 @@ static int check_gamma(struct drm_intel_overlay_attrs *attrs)
return 0; return 0;
} }
int intel_overlay_attrs(struct drm_device *dev, void *data, int intel_overlay_attrs_ioctl(struct drm_device *dev, void *data,
struct drm_file *file_priv) struct drm_file *file_priv)
{ {
struct drm_intel_overlay_attrs *attrs = data; struct drm_intel_overlay_attrs *attrs = data;
struct drm_i915_private *dev_priv = dev->dev_private; struct drm_i915_private *dev_priv = dev->dev_private;
...@@ -1310,7 +1303,7 @@ int intel_overlay_attrs(struct drm_device *dev, void *data, ...@@ -1310,7 +1303,7 @@ int intel_overlay_attrs(struct drm_device *dev, void *data,
attrs->contrast = overlay->contrast; attrs->contrast = overlay->contrast;
attrs->saturation = overlay->saturation; attrs->saturation = overlay->saturation;
if (!IS_GEN2(dev)) { if (!IS_GEN2(dev_priv)) {
attrs->gamma0 = I915_READ(OGAMC0); attrs->gamma0 = I915_READ(OGAMC0);
attrs->gamma1 = I915_READ(OGAMC1); attrs->gamma1 = I915_READ(OGAMC1);
attrs->gamma2 = I915_READ(OGAMC2); attrs->gamma2 = I915_READ(OGAMC2);
...@@ -1342,7 +1335,7 @@ int intel_overlay_attrs(struct drm_device *dev, void *data, ...@@ -1342,7 +1335,7 @@ int intel_overlay_attrs(struct drm_device *dev, void *data,
intel_overlay_unmap_regs(overlay, regs); intel_overlay_unmap_regs(overlay, regs);
if (attrs->flags & I915_OVERLAY_UPDATE_GAMMA) { if (attrs->flags & I915_OVERLAY_UPDATE_GAMMA) {
if (IS_GEN2(dev)) if (IS_GEN2(dev_priv))
goto out_unlock; goto out_unlock;
if (overlay->active) { if (overlay->active) {
...@@ -1372,37 +1365,36 @@ int intel_overlay_attrs(struct drm_device *dev, void *data, ...@@ -1372,37 +1365,36 @@ int intel_overlay_attrs(struct drm_device *dev, void *data,
return ret; return ret;
} }
void intel_setup_overlay(struct drm_device *dev) void intel_setup_overlay(struct drm_i915_private *dev_priv)
{ {
struct drm_i915_private *dev_priv = dev->dev_private;
struct intel_overlay *overlay; struct intel_overlay *overlay;
struct drm_i915_gem_object *reg_bo; struct drm_i915_gem_object *reg_bo;
struct overlay_registers __iomem *regs; struct overlay_registers __iomem *regs;
int ret; int ret;
if (!HAS_OVERLAY(dev)) if (!HAS_OVERLAY(dev_priv))
return; return;
overlay = kzalloc(sizeof(*overlay), GFP_KERNEL); overlay = kzalloc(sizeof(*overlay), GFP_KERNEL);
if (!overlay) if (!overlay)
return; return;
mutex_lock(&dev->struct_mutex); mutex_lock(&dev_priv->dev->struct_mutex);
if (WARN_ON(dev_priv->overlay)) if (WARN_ON(dev_priv->overlay))
goto out_free; goto out_free;
overlay->dev = dev; overlay->i915 = dev_priv;
reg_bo = NULL; reg_bo = NULL;
if (!OVERLAY_NEEDS_PHYSICAL(dev)) if (!OVERLAY_NEEDS_PHYSICAL(dev_priv))
reg_bo = i915_gem_object_create_stolen(dev, PAGE_SIZE); reg_bo = i915_gem_object_create_stolen(dev_priv->dev, PAGE_SIZE);
if (reg_bo == NULL) if (reg_bo == NULL)
reg_bo = i915_gem_object_create(dev, PAGE_SIZE); reg_bo = i915_gem_object_create(dev_priv->dev, PAGE_SIZE);
if (IS_ERR(reg_bo)) if (IS_ERR(reg_bo))
goto out_free; goto out_free;
overlay->reg_bo = reg_bo; overlay->reg_bo = reg_bo;
if (OVERLAY_NEEDS_PHYSICAL(dev)) { if (OVERLAY_NEEDS_PHYSICAL(dev_priv)) {
ret = i915_gem_object_attach_phys(reg_bo, PAGE_SIZE); ret = i915_gem_object_attach_phys(reg_bo, PAGE_SIZE);
if (ret) { if (ret) {
DRM_ERROR("failed to attach phys overlay regs\n"); DRM_ERROR("failed to attach phys overlay regs\n");
...@@ -1442,25 +1434,23 @@ void intel_setup_overlay(struct drm_device *dev) ...@@ -1442,25 +1434,23 @@ void intel_setup_overlay(struct drm_device *dev)
intel_overlay_unmap_regs(overlay, regs); intel_overlay_unmap_regs(overlay, regs);
dev_priv->overlay = overlay; dev_priv->overlay = overlay;
mutex_unlock(&dev->struct_mutex); mutex_unlock(&dev_priv->dev->struct_mutex);
DRM_INFO("initialized overlay support\n"); DRM_INFO("initialized overlay support\n");
return; return;
out_unpin_bo: out_unpin_bo:
if (!OVERLAY_NEEDS_PHYSICAL(dev)) if (!OVERLAY_NEEDS_PHYSICAL(dev_priv))
i915_gem_object_ggtt_unpin(reg_bo); i915_gem_object_ggtt_unpin(reg_bo);
out_free_bo: out_free_bo:
drm_gem_object_unreference(&reg_bo->base); drm_gem_object_unreference(&reg_bo->base);
out_free: out_free:
mutex_unlock(&dev->struct_mutex); mutex_unlock(&dev_priv->dev->struct_mutex);
kfree(overlay); kfree(overlay);
return; return;
} }
void intel_cleanup_overlay(struct drm_device *dev) void intel_cleanup_overlay(struct drm_i915_private *dev_priv)
{ {
struct drm_i915_private *dev_priv = dev->dev_private;
if (!dev_priv->overlay) if (!dev_priv->overlay)
return; return;
...@@ -1483,17 +1473,16 @@ struct intel_overlay_error_state { ...@@ -1483,17 +1473,16 @@ struct intel_overlay_error_state {
static struct overlay_registers __iomem * static struct overlay_registers __iomem *
intel_overlay_map_regs_atomic(struct intel_overlay *overlay) intel_overlay_map_regs_atomic(struct intel_overlay *overlay)
{ {
struct drm_i915_private *dev_priv = to_i915(overlay->dev); struct drm_i915_private *dev_priv = overlay->i915;
struct i915_ggtt *ggtt = &dev_priv->ggtt;
struct overlay_registers __iomem *regs; struct overlay_registers __iomem *regs;
if (OVERLAY_NEEDS_PHYSICAL(overlay->dev)) if (OVERLAY_NEEDS_PHYSICAL(dev_priv))
/* Cast to make sparse happy, but it's wc memory anyway, so /* Cast to make sparse happy, but it's wc memory anyway, so
* equivalent to the wc io mapping on X86. */ * equivalent to the wc io mapping on X86. */
regs = (struct overlay_registers __iomem *) regs = (struct overlay_registers __iomem *)
overlay->reg_bo->phys_handle->vaddr; overlay->reg_bo->phys_handle->vaddr;
else else
regs = io_mapping_map_atomic_wc(ggtt->mappable, regs = io_mapping_map_atomic_wc(dev_priv->ggtt.mappable,
overlay->flip_addr); overlay->flip_addr);
return regs; return regs;
...@@ -1502,11 +1491,10 @@ intel_overlay_map_regs_atomic(struct intel_overlay *overlay) ...@@ -1502,11 +1491,10 @@ intel_overlay_map_regs_atomic(struct intel_overlay *overlay)
static void intel_overlay_unmap_regs_atomic(struct intel_overlay *overlay, static void intel_overlay_unmap_regs_atomic(struct intel_overlay *overlay,
struct overlay_registers __iomem *regs) struct overlay_registers __iomem *regs)
{ {
if (!OVERLAY_NEEDS_PHYSICAL(overlay->dev)) if (!OVERLAY_NEEDS_PHYSICAL(overlay->i915))
io_mapping_unmap_atomic(regs); io_mapping_unmap_atomic(regs);
} }
struct intel_overlay_error_state * struct intel_overlay_error_state *
intel_overlay_capture_error_state(struct drm_i915_private *dev_priv) intel_overlay_capture_error_state(struct drm_i915_private *dev_priv)
{ {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment