<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Oct 15, 2014 at 1:13 AM, Thierry Reding <span dir="ltr"><<a href="mailto:thierry.reding@gmail.com" target="_blank">thierry.reding@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Tue, Oct 14, 2014 at 08:50:35AM -0700, Sean V Kelley wrote:<br>
> On Tue, Oct 14, 2014 at 4:53 AM, Thierry Reding<br>
> <<a href="mailto:thierry.reding@gmail.com">thierry.reding@gmail.com</a>> wrote:<br>
> > On Mon, Oct 13, 2014 at 08:15:00PM +0800, Yao Cheng wrote:<br>
> >> drm/ipvr is a new GEM driver for baytrail's vxd392, which accelerates VP8 video decoding.<br>
> >> The driver name "ipvr" means the PowerVR's IP wrapped by Intel. In the future, ipvr may support other platforms such as Merrifield.<br>
> >> Code is placed at drivers/gpu/drm/ipvr and the following two new Kconfig are added:<br>
> >>   CONFIG_DRM_IPVR: Build option for ipvr module<br>
> >>   CONFIG_DRM_IPVR_EC: Experimental feature of error concealment<br>
> >><br>
> >> User mode drm helper "libdrm_ipvr.so" and simple test are also included.<br>
> >><br>
> >> Yao Cheng (3):<br>
> >>  [1/3] drm/i915: add vxd392 bridge in i915 on baytrail<br>
> >>  [2/3] drm/ipvr: ipvr drm driver for vxd392<br>
> ><br>
> > If this is Intel-specific, why doesn't it live under the i915 driver?<br>
><br>
> It is an entirely unrelated HW IP block, VXD392.  Nothing to do with<br>
> GEN aside from DRM based.<br>
<br>
</span>With GEN you're referring to the Intel integrated GPU? And VXD392 I take<br>
it is the IP block licensed by Imagination? Baytrail and others then<br>
wrap some additional logic around this as it is integrated into the SoC?<br>
<br>
How much wrapping actually happens here? I worry that this is going to<br>
lead to a lot of duplication if we ever want to support another SoC that<br>
uses the VXD392 IP. Could the code be split into a VXD392 "library" and<br>
some driver that implements the Intel-specific glue?<br>
<br>
Finally, if this IP block is a VP8 video decoding engine only, I'm not<br>
sure DRM is the best subsystem for it. Traditionally video decoding has<br>
been done primarily in V4L2. I'm not sure that's the best fit given that<br>
it was originally designed for video capturing, but they've evolved some<br>
infrastructure to deal with encoding/decoding, whereas we have nothing<br>
like that at all in DRM.<br></blockquote><div><br></div><div>That isn't true. i915, nouveau and radeon drm drivers all support video decoding user space in some form.</div><div><br></div><div>Stéphane</div><div><br></div></div></div></div>