From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thierry Reding Subject: Re: [RFC PATCH 0/3] drm driver for baytrail's vxd392 Date: Wed, 15 Oct 2014 10:31:37 +0200 Message-ID: <20141015083136.GE12807@ulmo> References: <1413202502-19026-1-git-send-email-yao.cheng@intel.com> <20141014115259.GD5057@ulmo> <20141015081314.GC12807@ulmo> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============0209172766==" Return-path: In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: =?utf-8?B?U3TDqXBoYW5l?= Marchesin Cc: "intel-gfx@lists.freedesktop.org" , Fei Jiang , DRI mailing list , "Vetter, Daniel" List-Id: intel-gfx@lists.freedesktop.org --===============0209172766== Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="dFWYt1i2NyOo1oI9" Content-Disposition: inline --dFWYt1i2NyOo1oI9 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Wed, Oct 15, 2014 at 01:18:02AM -0700, St=C3=A9phane Marchesin wrote: > On Wed, Oct 15, 2014 at 1:13 AM, Thierry Reding > wrote: >=20 > > On Tue, Oct 14, 2014 at 08:50:35AM -0700, Sean V Kelley wrote: > > > On Tue, Oct 14, 2014 at 4:53 AM, Thierry Reding > > > wrote: > > > > On Mon, Oct 13, 2014 at 08:15:00PM +0800, Yao Cheng wrote: > > > >> drm/ipvr is a new GEM driver for baytrail's vxd392, which accelera= tes > > VP8 video decoding. > > > >> The driver name "ipvr" means the PowerVR's IP wrapped by Intel. In > > the future, ipvr may support other platforms such as Merrifield. > > > >> Code is placed at drivers/gpu/drm/ipvr and the following two new > > Kconfig are added: > > > >> CONFIG_DRM_IPVR: Build option for ipvr module > > > >> CONFIG_DRM_IPVR_EC: Experimental feature of error concealment > > > >> > > > >> User mode drm helper "libdrm_ipvr.so" and simple test are also > > included. > > > >> > > > >> Yao Cheng (3): > > > >> [1/3] drm/i915: add vxd392 bridge in i915 on baytrail > > > >> [2/3] drm/ipvr: ipvr drm driver for vxd392 > > > > > > > > If this is Intel-specific, why doesn't it live under the i915 drive= r? > > > > > > It is an entirely unrelated HW IP block, VXD392. Nothing to do with > > > GEN aside from DRM based. > > > > With GEN you're referring to the Intel integrated GPU? And VXD392 I take > > it is the IP block licensed by Imagination? Baytrail and others then > > wrap some additional logic around this as it is integrated into the SoC? > > > > How much wrapping actually happens here? I worry that this is going to > > lead to a lot of duplication if we ever want to support another SoC that > > uses the VXD392 IP. Could the code be split into a VXD392 "library" and > > some driver that implements the Intel-specific glue? > > > > Finally, if this IP block is a VP8 video decoding engine only, I'm not > > sure DRM is the best subsystem for it. Traditionally video decoding has > > been done primarily in V4L2. I'm not sure that's the best fit given that > > it was originally designed for video capturing, but they've evolved some > > infrastructure to deal with encoding/decoding, whereas we have nothing > > like that at all in DRM. > > >=20 > That isn't true. i915, nouveau and radeon drm drivers all support video > decoding user space in some form. Well yes, because they are GPUs that happen to have video decoding engines on the same board and use the same method of command-stream submission that they use for 2D or 3D work. For standalone video decoding hardware, the only drivers that I know of are in drivers/media. Thierry --dFWYt1i2NyOo1oI9 Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQIcBAEBAgAGBQJUPjDoAAoJEN0jrNd/PrOh7tUP/2aoOQEUE+HdHQqFAj93tQvx iSUeop8921F+2c5nRIABar6g4v4fNsj1iq9k+L6VyvoiP7E3t+aeg30dQg/P9Ut9 nTN4GzSb2Cah63weEhQBdCn7gn3GybpAwji/GvtjMGoKDQegADnjrVORuuwSsJj+ 2Kj2FSl3X0iGGCe8vFBkwv4yBaVCJ2Jq5an97RWHw3T6eiQWsFh+tnrCmI1tuL48 fkFbZQUquaFaAxNkc5HP5L9ZmNGLKFvKHTZcSh4b0oPeuu4G8evhYozNYky71/Eg Hezg3hUpKJxOD6Uyi394AMjVSfkqfzP6ZyaTBQBMDDFUH0sEoOHfRcIF+4F/v1RW RiJdJr8BijVsDTHvppv25WpQSp4sDwdMgsLXYntEaF35s5CCRSPbTwOxX6bCqkA2 4CE415rKA4IRmZVZHl5q8nVDDQ3K1cvYCC6a6gY8Trzx1sBteq0rqxHAzG4UJzM/ dsNKjeJdYgHXjFcMRg9pqEXmuVdLEVH9EcJ5p7xVYhB3Zz3Dx4rWFU5Oshjd0h8y p4uB1Ae2+1JxkqJ9g55byUhNo3Ub3NCBth4TpFw3y3VH/sa2o5iweGfICicFzK2i gkz9a5XOlO4GpYerVHPofhC1R6iaQXNAAkeM1Qg0zlF3wX4j+7NLj+gD+lnxNb7I bLqfLjwXuJD/inA+An6/ =Pewb -----END PGP SIGNATURE----- --dFWYt1i2NyOo1oI9-- --===============0209172766== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx --===============0209172766==--