On December 18, 2018 5:36:19 PM UTC, Dylan Baker <dy...@pnwbakers.com> wrote: > Quoting Ilia Mirkin (2018-12-06 09:39:25) > > On Thu, Dec 6, 2018 at 12:17 PM Eric Engestrom > <eric.engest...@intel.com> wrote: > > > > > > On Thursday, 2018-12-06 12:07:06 -0500, Ilia Mirkin wrote: > > > > Under what circumstances would tegra have a vdpau > implementation? > > > > > > I don't know about that, but this patch brings meson on par with > > > autotools. Are you saying autotools was wrong and it should be > removed > > > there instead? > > > > > > FtR, src/gallium/targets/vdpau/Makefile.am includes > > > src/gallium/drivers/tegra/Automake.inc which adds tegra to the > > > TARGET_DRIVERS, which in turn produces libvdpau_tegra.so > > > > > > If this was wrong, then that include on line 60 of > > > src/gallium/targets/vdpau/Makefile.am should be removed. > > > > > > This line was added by Thierry Reding (cc'ed) when he first > introduced > > > tegra support in 1755f608f5201e0a23f00 "tegra: Initial support". > > > > I'd like to hear tagr opine on this, but AFAIK tegra is a > > renderonly-style driver, which uses nouveau as its actual rendering > > backend. While nouveau does support VDPAU, the decoding engines are > > not available as part of the GPU device on Tegra platforms. They > still > > have decoders, they're just separate platform devices (which I > suspect > > operate very similarly to their desktop GPU counterparts, but that's > a > > separate matter). > > > > -ilia > > Ping?
I suggest we land this patch to bring meson on par with autotools, and if we decide later that autotools was wrong, we can just remove it from both places at once; ack? _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
