From: "Ville Syrjälä" <ville.syrjala@linux.intel.com>
To: "Sharma, Shashank" <shashank.sharma@intel.com>
Cc: "intel-gfx@lists.freedesktop.org"
<intel-gfx@lists.freedesktop.org>,
"dri-devel@lists.freedesktop.org"
<dri-devel@lists.freedesktop.org>
Subject: Re: [Intel-gfx] [PATCH 1/3] drm: Introduce scaling filter mode property
Date: Tue, 22 Oct 2019 15:25:11 +0300 [thread overview]
Message-ID: <20191022122511.GK1208@intel.com> (raw)
In-Reply-To: <FF3DDC77922A8A4BB08A3BC48A1EA8CB8E03C761@BGSMSX101.gar.corp.intel.com>
On Tue, Oct 22, 2019 at 10:12:29AM +0000, Sharma, Shashank wrote:
> Hey Daniel,
>
> > -----Original Message-----
> > From: Daniel Vetter <daniel.vetter@ffwll.ch> On Behalf Of Daniel Vetter
> > Sent: Tuesday, October 22, 2019 3:39 PM
> > To: Sharma, Shashank <shashank.sharma@intel.com>
> > Cc: dri-devel@lists.freedesktop.org; intel-gfx@lists.freedesktop.org
> > Subject: Re: [Intel-gfx] [PATCH 1/3] drm: Introduce scaling filter mode property
> >
> > On Tue, Oct 22, 2019 at 03:29:44PM +0530, Shashank Sharma wrote:
> > > This patch adds a scaling filter mode porperty to allow:
> > > - A driver/HW to showcase it's scaling filter capabilities.
> > > - A userspace to pick a desired effect while scaling.
> > >
> > > This option will be particularly useful in the scenarios where Integer
> > > mode scaling is possible, and a UI client wants to pick filters like
> > > Nearest-neighbor applied for non-blurry outputs.
> > >
> > > There was a RFC patch series published, to discus the request to
> > > enable Integer mode scaling by some of the gaming communities, which
> > > can be found here:
> > > https://patchwork.freedesktop.org/series/66175/
> > >
> > > Signed-off-by: Shashank Sharma <shashank.sharma@intel.com>
> >
> > Two things:
> >
> > - needs real property docs for this in the kms property chapter
> Got it,
> > - would be good if we could somehow subsume the panel fitter prop into
> > this? Or at least document possible interactions with that.
> >
> Sure, let me see what can I do here.
The scaling mode prop has nothing really to do with the filter being
used. The scaling mode prop just provides a bad mechanism for
configuring the destination coordinates of the scaler.
Trying to shoehorn these things into one prop would be a mistake IMO.
--
Ville Syrjälä
Intel
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
next prev parent reply other threads:[~2019-10-22 12:25 UTC|newest]
Thread overview: 38+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-10-22 9:59 [PATCH 0/3] Add scaling filters in DRM layer Shashank Sharma
2019-10-22 9:59 ` [PATCH 1/3] drm: Introduce scaling filter mode property Shashank Sharma
2019-10-22 10:08 ` [Intel-gfx] " Daniel Vetter
2019-10-22 10:12 ` Sharma, Shashank
2019-10-22 12:25 ` Ville Syrjälä [this message]
2019-10-23 12:44 ` Jani Nikula
2019-10-23 12:44 ` [Intel-gfx] " Jani Nikula
2019-10-23 12:44 ` Jani Nikula
2019-10-24 12:06 ` Daniel Vetter
2019-10-24 12:06 ` Daniel Vetter
2019-10-24 12:12 ` Jani Nikula
2019-10-24 12:12 ` Jani Nikula
2019-10-24 12:14 ` Daniel Vetter
2019-10-24 12:14 ` Daniel Vetter
2019-10-24 12:23 ` Jani Nikula
2019-10-24 12:23 ` [Intel-gfx] " Jani Nikula
2019-10-24 12:23 ` Jani Nikula
2019-10-22 12:20 ` Ville Syrjälä
2019-10-22 14:06 ` Harry Wentland
2019-10-22 15:28 ` Sharma, Shashank
2019-10-22 15:42 ` Ville Syrjälä
2019-10-22 15:18 ` Sharma, Shashank
2019-10-23 7:34 ` Pekka Paalanen
2019-10-23 12:41 ` Ville Syrjälä
2019-10-23 12:41 ` [Intel-gfx] " Ville Syrjälä
2019-10-23 12:41 ` Ville Syrjälä
2019-10-22 12:26 ` Ville Syrjälä
2019-10-22 15:21 ` Sharma, Shashank
2019-10-22 15:38 ` Ville Syrjälä
2019-10-22 13:26 ` Mihail Atanassov
2019-10-22 13:32 ` Mihail Atanassov
2019-10-22 15:25 ` Sharma, Shashank
2019-10-22 9:59 ` [PATCH 2/3] drm/i915: Add support for scaling filters Shashank Sharma
2019-10-22 9:59 ` [PATCH 3/3] drm/i915: Handle nearest-neighbor scaling filter Shashank Sharma
2019-10-22 13:18 ` ✗ Fi.CI.CHECKPATCH: warning for Add scaling filters in DRM layer Patchwork
2019-10-22 13:21 ` ✗ Fi.CI.SPARSE: " Patchwork
2019-10-22 13:50 ` ✓ Fi.CI.BAT: success " Patchwork
2019-10-22 23:06 ` ✗ Fi.CI.IGT: failure " Patchwork
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20191022122511.GK1208@intel.com \
--to=ville.syrjala@linux.intel.com \
--cc=dri-devel@lists.freedesktop.org \
--cc=intel-gfx@lists.freedesktop.org \
--cc=shashank.sharma@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.