> -----Original Message-----
> From: Daniel Stone <dan...@fooishbar.org>
> Sent: 25 February 2020 13:00
> To: Laxminarayan Bharadiya, Pankaj
> <pankaj.laxminarayan.bharad...@intel.com>
> Cc: Jani Nikula <jani.nik...@linux.intel.com>; Daniel Vetter
> <dan...@ffwll.ch>; intel-gfx <intel-gfx@lists.freedesktop.org>; dri-devel
> <dri-de...@lists.freedesktop.org>; Ville Syrjälä
> <ville.syrj...@linux.intel.com>; David Airlie <airl...@linux.ie>; Maarten
> Lankhorst <maarten.lankho...@linux.intel.com>; tzimmerm...@suse.de;
> Maxime Ripard <mrip...@kernel.org>; mihail.atanas...@arm.com; Joonas
> Lahtinen <joonas.lahti...@linux.intel.com>; Vivi, Rodrigo
> <rodrigo.v...@intel.com>; Chris Wilson <ch...@chris-wilson.co.uk>; Souza,
> Jose <jose.so...@intel.com>; De Marchi, Lucas
> <lucas.demar...@intel.com>; Roper, Matthew D
> <matthew.d.ro...@intel.com>; Deak, Imre <imre.d...@intel.com>;
> Shankar, Uma <uma.shan...@intel.com>; Nautiyal, Ankit K
> <ankit.k.nauti...@intel.com>; Linux Kernel Mailing List <linux-
> ker...@vger.kernel.org>
> Subject: Re: [Intel-gfx] [RFC][PATCH 5/5] drm/i915/display: Add Nearest-
> neighbor based integer scaling support
>
> Hi,
>
> On Tue, 25 Feb 2020 at 07:17, Pankaj Bharadiya
> <pankaj.laxminarayan.bharad...@intel.com> wrote:
> > @@ -415,18 +415,26 @@ skl_program_scaler(struct intel_plane *plane,
> > u16 y_vphase, uv_rgb_vphase;
> > int hscale, vscale;
> > const struct drm_plane_state *state = &plane_state->uapi;
> > + u32 src_w = drm_rect_width(&plane_state->uapi.src) >> 16;
> > + u32 src_h = drm_rect_height(&plane_state->uapi.src) >> 16;
> > u32 scaling_filter = PS_FILTER_MEDIUM;
> > + struct drm_rect dst;
> >
> > if (state->scaling_filter ==
> DRM_SCALING_FILTER_NEAREST_NEIGHBOR) {
> > scaling_filter = PS_FILTER_PROGRAMMED;
> > + skl_setup_nearest_neighbor_filter(dev_priv, pipe,
> > + scaler_id);
> > +
> > + /* Make the scaling window size to integer multiple of
> > source
> > + * TODO: Should userspace take desision to round scaling
> > window
> > + * to integer multiple?
> > + */
> > + crtc_w = rounddown(crtc_w, src_w);
> > + crtc_h = rounddown(crtc_h, src_h);
>
> The kernel should absolutely not be changing the co-ordinates that
> userspace requested.
Thanks, Will get rid of this in V2.
Thanks,
Pankaj
>
> Cheers,
> Daniel
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
