On Fri, Feb 23, 2018 at 11:43:29AM -0500, Sean Paul wrote: > On Fri, Feb 23, 2018 at 11:25:11AM -0500, Rob Clark wrote: > > On Fri, Feb 23, 2018 at 10:59 AM, Sean Paul <seanpaul@xxxxxxxxxxxx> wrote: > > > > > > Have we considered hiding writeback behind a client cap instead? > > > > It is kinda *almost* unneeded, since the connector reports itself as > > disconnected. > > > > I'm not sure what the reason was to drop the cap, but I think it would > > be better to have a cap so WB connectors don't show up in, for ex, > > xrandr > > Yeah, the disconnected hack is kind of gross, IMO. I hate to introduce churn in > the patch series given that it was initially introduced with the client cap. Haha, that's the reverse of Daniel's position: https://lists.freedesktop.org/archives/dri-devel/2016-October/120519.html > > There are also cases where we might want to make writeback unavailable, such as > when content protection is enabled. In those cases, it's conceivable that we > might want to use disconnected as a signal to u/s. I suppose we could also just > fail the check, so most of this is just academic. Not sure what other hardware out there does, but on Mali DP's case you would be outputing the protected content by putting the display processor in secure mode, which automatically disables writeback for us. Or to put in another way, you don't need a writeback framebuffer if you are in non-secure mode as you can get access to the framebuffer used for the plane anyway. Best regards, Liviu > > Sean > > > > > > BR, > > -R > > -- > Sean Paul, Software Engineer, Google / Chromium OS -- ==================== | I would like to | | fix the world, | | but they're not | | giving me the | \ source code! / --------------- ¯\_(ツ)_/¯ -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
