<div dir="ltr"><div>Hi Hans,</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, May 11, 2020 at 10:47 AM Hans de Goede <<a href="mailto:hdegoede@redhat.com">hdegoede@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi All,<br>
<br>
This RFC takes Rajat's earlier patch for adding privacy-screen properties<br>
infra to drm_connector.c and then adds the results of the discussion from<br>
the "RFC: Drm-connector properties managed by another driver / privacy<br>
screen support" mail thread on top, hence the v2.<br></blockquote><div><br></div><div>Thank you so much for doing this. I was following the said discussion and eventually it became quite complex for me to understand and follow :-)</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
The most important thing here is big kernel-doc comment which gets added in<br>
the first patch-chunk modifying drm_connector.c, this summarizes, or at<br>
least tries to summarize, the conclusions of our previous discussion on<br>
the userspace API and lays down the ground rules for how the 2 new<br>
"privacy-screen sw-state" and  "privacy-screen hw-state" properties are<br>
to be used both from the driver side as well as from the userspace side.<br>
<br>
Other then that this modifies Rajat's patch to add 2 properties instead<br>
of one, without much other changes.<br>
<br>
Rajat, perhaps you can do a new version of your patch-set integration /<br>
using this version of the properties and then if everyone is ok with<br>
the proposed userspace API Jani can hopefully merge the whole set<br>
through the i915 tree sometime during the 5.9 cycle.<br></blockquote><div><br></div><div>SGTM. I have actually moved to working on something else now, so I will most likely wait for this patch to get merged, before rebasing my other / remaining patches on top of that.</div><div><br></div><div>Thanks & Best Regards,</div><div><br></div><div>Rajat</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">This RFC takes Rajat's earlier patch for adding privacy-screen properties<br>infra to drm_connector.c and then adds the results of the discussion from<br>the "RFC: Drm-connector properties managed by another driver / privacy<br>screen support" mail thread on top, hence the v2.<br><br>The most important thing here is big kernel-doc comment which gets added in<br>the first patch-chunk modifying drm_connector.c, this summarizes, or at<br>least tries to summarize, the conclusions of our previous discussion on<br>the userspace API and lays down the ground rules for how the 2 new<br>"privacy-screen sw-state" and  "privacy-screen hw-state" properties are<br>to be used both from the driver side as well as from the userspace side.<br><br>Other then that this modifies Rajat's patch to add 2 properties instead<br>of one, without much other changes.<br><br>Rajat, perhaps you can do a new version of your patch-set integration /<br>using this version of the properties and then if everyone is ok with<br>the proposed userspace API Jani can hopefully merge the whole set<br>through the i915 tree sometime during the 5.9 cycle.<br>
<br>
Regards,<br>
<br>
Hans<br>
<br>
p.s.<br>
<br>
I plan to start working on the lcdshadow subsystem next. As discussed the<br>
plan for this subsystem is to allow drivers outside of the DRM subsys, such<br>
as for example the thinkpad_acpi driver, to register a lcdshadow device,<br>
which DRM drivers can then get a reference to and use to implement these<br>
properties.<br>
<br>
</blockquote></div></div>