Re: [PATCH v3 3/9] drm: Add Content Protection property

2017-12-06 Thread Ramalingam C
On Wednesday 06 December 2017 09:56 PM, Sean Paul wrote: I'd rather keep the property as-is and expose an HDCP version property alongside it (or perhaps something more elaborate that includes bksv and the downstream bksvs). The reason I prefer that is it will also cover the 1.2 vs 1.4 differenc

Re: [PATCH v3 3/9] drm: Add Content Protection property

2017-12-06 Thread Sean Paul
-devel ; Hans Verkuil >> >> Subject: Re: [PATCH v3 3/9] drm: Add Content Protection property >> >> On Tue, Dec 5, 2017 at 9:04 AM, Ramalingam C >> wrote: >> > >> > >> > On Tuesday 05 December 2017 01:37 PM, Hans Verkuil wrote: >> >

RE: [PATCH v3 3/9] drm: Add Content Protection property

2017-12-05 Thread C, Ramalingam
Best Regards, Ramalingam C > -Original Message- > From: Sean Paul [mailto:seanp...@chromium.org] > Sent: Tuesday, December 5, 2017 8:07 PM > To: C, Ramalingam > Cc: dri-devel ; Hans Verkuil > > Subject: Re: [PATCH v3 3/9] drm: Add Content Protection property >

Re: [PATCH v3 3/9] drm: Add Content Protection property

2017-12-05 Thread Hans Verkuil
On 12/05/17 15:36, Sean Paul wrote: > On Tue, Dec 5, 2017 at 9:04 AM, Ramalingam C wrote: >> >> >> On Tuesday 05 December 2017 01:37 PM, Hans Verkuil wrote: >> >> On 12/05/2017 06:15 AM, Sean Paul wrote: >> >> This patch adds a new optional connector property to allow userspace to >> enable >> pro

Re: [Intel-gfx] [PATCH v3 3/9] drm: Add Content Protection property

2017-12-05 Thread Chris Wilson
Quoting Daniel Vetter (2017-12-05 15:34:36) > Two bits missing imo: > - Should explain that userspace should poll this property to detect a > change from ENABLED to DESIRED (and take adequate actions and e.g. stop > the stream). No uevent will be sent out because the HDCP specs require > poll

Re: [PATCH v3 3/9] drm: Add Content Protection property

2017-12-05 Thread Daniel Vetter
On Tue, Dec 05, 2017 at 12:15:02AM -0500, Sean Paul wrote: > This patch adds a new optional connector property to allow userspace to enable > protection over the content it is displaying. This will typically be > implemented > by the driver using HDCP. > > The property is a tri-state with the fol

Re: [Intel-gfx] [PATCH v3 3/9] drm: Add Content Protection property

2017-12-05 Thread Daniel Vetter
On Tue, Dec 05, 2017 at 09:07:58AM +0100, Hans Verkuil wrote: > On 12/05/2017 06:15 AM, Sean Paul wrote: > > This patch adds a new optional connector property to allow userspace to > > enable > > protection over the content it is displaying. This will typically be > > implemented > > by the drive

Re: [PATCH v3 3/9] drm: Add Content Protection property

2017-12-05 Thread Sean Paul
On Tue, Dec 5, 2017 at 9:04 AM, Ramalingam C wrote: > > > On Tuesday 05 December 2017 01:37 PM, Hans Verkuil wrote: > > On 12/05/2017 06:15 AM, Sean Paul wrote: > > This patch adds a new optional connector property to allow userspace to > enable > protection over the content it is displaying. This

Re: [PATCH v3 3/9] drm: Add Content Protection property

2017-12-05 Thread Ramalingam C
On Tuesday 05 December 2017 01:37 PM, Hans Verkuil wrote: On 12/05/2017 06:15 AM, Sean Paul wrote: This patch adds a new optional connector property to allow userspace to enable protection over the content it is displaying. This will typically be implemented by the driver using HDCP. The prop

Re: [PATCH v3 3/9] drm: Add Content Protection property

2017-12-05 Thread Hans Verkuil
On 12/05/2017 06:15 AM, Sean Paul wrote: > This patch adds a new optional connector property to allow userspace to enable > protection over the content it is displaying. This will typically be > implemented > by the driver using HDCP. > > The property is a tri-state with the following values: > -

[PATCH v3 3/9] drm: Add Content Protection property

2017-12-04 Thread Sean Paul
This patch adds a new optional connector property to allow userspace to enable protection over the content it is displaying. This will typically be implemented by the driver using HDCP. The property is a tri-state with the following values: - OFF: Self explanatory, no content protection - DESIRED: