Re: [Intel-gfx] [RFC v1 01/20] drm/hdcp: HDCP bitmask property for connectors

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Jul 12, 2017 at 01:58:45PM +0530, Ramalingam C wrote:
> DRM connector property is created as bitmask to receive
> HDCP enable/disable request along with content type.
> 
> And also there are Read only status bits for
> 	1. HDCP spec capability of the connector + panel
> 	2. HDCP encryption status of the connector
> 
> Signed-off-by: Ramalingam C <ramalingam.c@xxxxxxxxx>
> ---
>  drivers/gpu/drm/drm_connector.c | 30 +++++++++++++++++++++
>  include/drm/drm_hdcp.h          | 58 +++++++++++++++++++++++++++++++++++++++++
>  include/drm/drm_mode_config.h   |  5 ++++
>  3 files changed, 93 insertions(+)
>  create mode 100644 include/drm/drm_hdcp.h
> 
> diff --git a/drivers/gpu/drm/drm_connector.c b/drivers/gpu/drm/drm_connector.c
> index 8072e6e..04f8cf8 100644
> --- a/drivers/gpu/drm/drm_connector.c
> +++ b/drivers/gpu/drm/drm_connector.c
> @@ -24,6 +24,7 @@
>  #include <drm/drm_connector.h>
>  #include <drm/drm_edid.h>
>  #include <drm/drm_encoder.h>
> +#include <drm/drm_hdcp.h>
>  
>  #include "drm_crtc_internal.h"
>  #include "drm_internal.h"
> @@ -617,6 +618,28 @@ static const struct drm_prop_enum_list drm_link_status_enum_list[] = {
>  };
>  DRM_ENUM_NAME_FN(drm_get_link_status_name, drm_link_status_enum_list)
>  
> +static const struct drm_prop_enum_list drm_hdcp_enum_list[] = {
> +	{ __builtin_ffs(DRM_HDCP_ENABLE) - 1,
> +					"Enable HDCP Encryption" },
> +	{ __builtin_ffs(DRM_HDCP_TYPE_BIT0) - 1,
> +					"HDCP Content type bit0" },
> +	{ __builtin_ffs(DRM_HDCP_TYPE_BIT1) - 1,
> +					"HDCP Content type bit1" },
> +	{ __builtin_ffs(DRM_HDCP1_SUPPORTED) - 1,
> +					"HDCP1.x Supported" },
> +	{ __builtin_ffs(DRM_HDCP2_SUPPORTED) - 1,
> +					"HDCP2.x Supported" },
> +	{ __builtin_ffs(DRM_HDCP_WIP) - 1,
> +					"HDCP work in progress" },
> +	{ __builtin_ffs(DRM_HDCP_AUTH_FAILED) - 1,
> +					"HDCP Authentication Failed" },
> +	{ __builtin_ffs(DRM_HDCP_LINK_INTEGRITY_FAILED) - 1,
> +					"HDCP Link Integrity Failed" },
> +	{ __builtin_ffs(DRM_HDCP_REAUTH_REQUESTED) - 1,
> +					"HDCP Reauthentication Requested" },

Why all these intermediate steps and different failure modes? Either hdcp
works, or it doesnt (and we can split up with the type 0 or type 1 if
needed), but I don't know what userspace would do with all the other
stuff?

This also doesn't seem to do the lockdown mode to force hdcp mode. And
afair it also doesn't match what CrOS currently uses, which means we don't
have userspace for this.

I think the best approach would be if we simply try to upstream _exactly_
the property CrOS currently uses (not even bothering with type 0 vs type 1
if that's not required), since that is the currently only open source
userspace that asks for this. Going with a much more complicated or
different interface just because only risks a massive discussion and
subsequent long delays.
-Daniel

> +};
> +DRM_ENUM_NAME_FN(drm_get_hdcp_name, drm_hdcp_enum_list)
> +
>  /**
>   * drm_display_info_set_bus_formats - set the supported bus formats
>   * @info: display info to store bus formats in
> @@ -789,6 +812,13 @@ int drm_connector_create_standard_properties(struct drm_device *dev)
>  		return -ENOMEM;
>  	dev->mode_config.link_status_property = prop;
>  
> +	prop = drm_property_create_bitmask(dev, 0, "hdcp", drm_hdcp_enum_list,
> +					   ARRAY_SIZE(drm_hdcp_enum_list),
> +					   DRM_HDCP_PROP_SUPPORTED_BITS);
> +	if (!prop)
> +		return -ENOMEM;
> +	dev->mode_config.hdcp_property = prop;
> +
>  	return 0;
>  }
>  
> diff --git a/include/drm/drm_hdcp.h b/include/drm/drm_hdcp.h
> new file mode 100644
> index 0000000..7cebf0f
> --- /dev/null
> +++ b/include/drm/drm_hdcp.h
> @@ -0,0 +1,58 @@
> +/*
> + * Copyright (c) 2017 Intel Corporation
> + *
> + * Permission to use, copy, modify, distribute, and sell this software and its
> + * documentation for any purpose is hereby granted without fee, provided that
> + * the above copyright notice appear in all copies and that both that copyright
> + * notice and this permission notice appear in supporting documentation, and
> + * that the name of the copyright holders not be used in advertising or
> + * publicity pertaining to distribution of the software without specific,
> + * written prior permission.  The copyright holders make no representations
> + * about the suitability of this software for any purpose.  It is provided "as
> + * is" without express or implied warranty.
> + *
> + * THE COPYRIGHT HOLDERS DISCLAIM ALL WARRANTIES WITH REGARD TO THIS SOFTWARE,
> + * INCLUDING ALL IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS, IN NO
> + * EVENT SHALL THE COPYRIGHT HOLDERS BE LIABLE FOR ANY SPECIAL, INDIRECT OR
> + * CONSEQUENTIAL DAMAGES OR ANY DAMAGES WHATSOEVER RESULTING FROM LOSS OF USE,
> + * DATA OR PROFITS, WHETHER IN AN ACTION OF CONTRACT, NEGLIGENCE OR OTHER
> + * TORTIOUS ACTION, ARISING OUT OF OR IN CONNECTION WITH THE USE OR PERFORMANCE
> + * OF THIS SOFTWARE.
> + */
> +
> +/*
> + * Header for HDCP specific data
> + */
> +
> +#ifndef __DRM_HDCP_H__
> +#define __DRM_HDCP_H__
> +
> +/**
> + * HDCP property related information
> + */
> +/* RW: HDCP Encryption Requests and Status bits */
> +#define DRM_HDCP_ENABLE				BIT(0)
> +#define DRM_HDCP_TYPE_BIT0			BIT(4)
> +#define DRM_HDCP_TYPE_BIT1			BIT(5)
> +
> +/* RO: HDCP Version supported on Platform + panel */
> +#define DRM_HDCP1_SUPPORTED			BIT(12)
> +#define DRM_HDCP2_SUPPORTED			BIT(13)
> +
> +/* RO: Status of the requested operations */
> +#define DRM_HDCP_WIP				BIT(20)
> +#define DRM_HDCP_AUTH_FAILED			BIT(21)
> +
> +/* RO: Error Status From HDCP sink */
> +#define DRM_HDCP_LINK_INTEGRITY_FAILED		BIT(22)
> +#define DRM_HDCP_REAUTH_REQUESTED		BIT(23)
> +
> +#define DRM_HDCP_PROP_SUPPORTED_BITS	(DRM_HDCP_ENABLE | DRM_HDCP_TYPE_BIT0 \
> +					| DRM_HDCP_TYPE_BIT1 | \
> +					DRM_HDCP1_SUPPORTED | \
> +					DRM_HDCP2_SUPPORTED | DRM_HDCP_WIP | \
> +					DRM_HDCP_AUTH_FAILED | \
> +					DRM_HDCP_LINK_INTEGRITY_FAILED | \
> +					DRM_HDCP_REAUTH_REQUESTED)
> +
> +#endif	/* __DRM_HDCP_H__ */
> diff --git a/include/drm/drm_mode_config.h b/include/drm/drm_mode_config.h
> index 4298171..0c5bb90 100644
> --- a/include/drm/drm_mode_config.h
> +++ b/include/drm/drm_mode_config.h
> @@ -538,6 +538,11 @@ struct drm_mode_config {
>  	 */
>  	struct drm_property *link_status_property;
>  	/**
> +	 * @hdcp_property: Default connector property for HDCP
> +	 * of a connector
> +	 */
> +	struct drm_property *hdcp_property;
> +	/**
>  	 * @plane_type_property: Default plane property to differentiate
>  	 * CURSOR, PRIMARY and OVERLAY legacy uses of planes.
>  	 */
> -- 
> 2.7.4
> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx

-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux