From: "Ville Syrjälä" <ville.syrjala@linux.intel.com>
To: Jim Zhang <Zhang_Jim@cat.com>
Cc: "intel-gfx@lists.freedesktop.org"
<intel-gfx@lists.freedesktop.org>,
"dri-devel@lists.freedesktop.org"
<dri-devel@lists.freedesktop.org>
Subject: Re: colorkey support for intel i915 gpu driver
Date: Tue, 9 Apr 2019 17:17:46 +0300 [thread overview]
Message-ID: <20190409141746.GP3888@intel.com> (raw)
In-Reply-To: <SN6PR08MB45764E2AF36C02ACCCB175BA882D0@SN6PR08MB4576.namprd08.prod.outlook.com>
On Tue, Apr 09, 2019 at 01:59:21PM +0000, Jim Zhang wrote:
> Nice, do you have any sample code for it?
https://cgit.freedesktop.org/xorg/driver/xf86-video-intel/tree/src/sna/sna_video_sprite.c
is the only userspace code we have that uses the colorkey.
>
> Thanks,
>
> Jim
>
>
> Caterpillar: Confidential Green
>
> -----Original Message-----
> From: Ville Syrjälä <ville.syrjala@linux.intel.com>
> Sent: Tuesday, April 9, 2019 8:57 AM
> To: Jim Zhang <Zhang_Jim@cat.com>
> Cc: intel-gfx@lists.freedesktop.org; dri-devel@lists.freedesktop.org
> Subject: Re: [Intel-gfx] colorkey support for intel i915 gpu driver
>
> On Tue, Apr 09, 2019 at 04:46:24PM +0300, Ville Syrjälä wrote:
> > On Tue, Apr 09, 2019 at 01:29:41PM +0000, Jim Zhang wrote:
> > > Villie:
> > >
> > > What is Intel's plan for the colorkey patch? Does Intel have any plan to review and release?
> >
> > There is no real plan at this time. But if you have a use case for it
> > I can try to harass people until someone reviews it :)
> >
> > > If I go with custom ioctl, and my custom ioctl will only used in Baytrail product, could it be atomic for Baytrail only?
> >
> > No. We would need to define a new api for it.
>
> I should clarify that you can pre-configure the colorkey before turning on the sprite plane(s). So unless you really need to change the colorkey while the sprite plane(s) are already enabled you may not even need an atomic api for this.
>
> >
> > >
> > > Thanks,
> > >
> > > Jim
> > >
> > >
> > > Caterpillar: Confidential Green
> > >
> > > -----Original Message-----
> > > From: Ville Syrjälä <ville.syrjala@linux.intel.com>
> > > Sent: Tuesday, April 2, 2019 7:32 AM
> > > To: Jim Zhang <Zhang_Jim@cat.com>
> > > Cc: dri-devel@lists.freedesktop.org; intel-gfx@lists.freedesktop.org
> > > Subject: Re: colorkey support for intel i915 gpu driver
> > >
> > > On Mon, Apr 01, 2019 at 08:18:13PM +0000, Jim Zhang wrote:
> > > > Hi Sir/Madam:
> > > >
> > > > I am using the open source Baytrail gpu drm driver.
> > > >
> > > > Linux kernel version 3.10.61:
> > > > Libdrm package: 2.4.97
> > > >
> > > > When calling function
> > > > properties = drmModeObjectGetProperties(drmfd, plane_id,
> > > > DRM_MODE_OBJECT_PLANE); it only returns only one property:
> > > >
> > > > This property is:
> > > > property->name = "rotation", property->prop_id =4 It looks like
> > > > that the Baytrail gpu drm driver does not support colorkey.
> > >
> > > There are two problems currently:
> > > - Destination colorkey is not implemented on BYT/CHV. I have
> > > patches for it but they have not been reviewed by anyone:
> > >
> > > https://urldefense.proofpoint.com/v2/url?u=https-3A__patchwork.freed
> > > esktop.org_series_43902_&d=DwIDAw&c=p0oa49nxxGtbbM2qgM-GB4r4m9OlGg-s
> > > Ep8sXylY2aQ&r=kszNssoSc2KF2GeTYwo7za6kdvLoemctuEIYtXbA4PI&m=ztfgUCy9
> > > ePzHa0zagoDF75AfJJVbElfjXUWmbpBRM58&s=aXK10RLMgC4i_nBRGS6Jzzbv3pXo50
> > > PX79myDO5gEYA&e=
> > > - Colorkey can only be set via a custom i915 specific ioctl
> > > (DRM_I915_SET_SPRITE_COLORKEY). There have been a few attempts at
> > > a generic property based API that never really went anywhere. It's
> > > a rather difficult problem making this generic as each hardware has
> > > its own peculiar way of specifying colorkeying. The main problem
> > > with the custom ioctl is that it's not atomic with other screen
> > > updates.
> > >
> > > So, what kind of use case do you have in mind?
> > >
> > > --
> > > Ville Syrjälä
> > > Intel
> >
> > --
> > Ville Syrjälä
> > Intel
> > _______________________________________________
> > Intel-gfx mailing list
> > Intel-gfx@lists.freedesktop.org
> > https://urldefense.proofpoint.com/v2/url?u=https-3A__lists.freedesktop
> > .org_mailman_listinfo_intel-2Dgfx&d=DwIDAw&c=p0oa49nxxGtbbM2qgM-GB4r4m
> > 9OlGg-sEp8sXylY2aQ&r=kszNssoSc2KF2GeTYwo7za6kdvLoemctuEIYtXbA4PI&m=Oyn
> > sqGRzmnTMX2pxog1nBd_nnCa413-CM6loSIjLXXo&s=fjXVFmFDB_LEKe_W5PHzmzkzNPQ
> > cu3xTPSCqb5Dnpwk&e=
>
> --
> Ville Syrjälä
> Intel
--
Ville Syrjälä
Intel
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
next prev parent reply other threads:[~2019-04-09 14:17 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-04-01 20:18 colorkey support for intel i915 gpu driver Jim Zhang
2019-04-02 12:31 ` Ville Syrjälä
2019-04-09 13:29 ` Jim Zhang
2019-04-09 13:46 ` Ville Syrjälä
2019-04-09 13:56 ` Ville Syrjälä
2019-04-09 13:59 ` [Intel-gfx] " Jim Zhang
2019-04-09 14:17 ` Ville Syrjälä [this message]
2019-04-09 14:14 ` Jim Zhang
2019-04-09 14:19 ` Ville Syrjälä
2019-04-09 14:24 ` [Intel-gfx] " Jim Zhang
2019-04-09 15:01 ` Ville Syrjälä
2019-04-09 15:17 ` [Intel-gfx] " Jim Zhang
2019-04-09 19:15 ` Jim Zhang
2019-04-09 19:44 ` Ville Syrjälä
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=20190409141746.GP3888@intel.com \
--to=ville.syrjala@linux.intel.com \
--cc=Zhang_Jim@cat.com \
--cc=dri-devel@lists.freedesktop.org \
--cc=intel-gfx@lists.freedesktop.org \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).