Re: [PATCH v2 07/42] misc/mei/hdcp: Get & Put for mei cl_device

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

 



> Interfaces to obtain and release the cl_device reference is developed.
> Using these interfaces intel hdcp driver will get the reference to the mei
> client devices, so that hdcp2.2 service calls can be routed to that client
> device.
> 
> During registration, call back function will be registered with mei_hdcp driver
> so that when the client device is removed intel hdcp driver can be informed.
> 
> At a time only one reference is allowed in this interfaces.
> 
> v2:
>   Rebased.
Linux kernel already provide notification chain,  please use that. 
This is not needed and I'm not sure it will ever work.

Tomas

 
> Signed-off-by: Ramalingam C <ramalingam.c@xxxxxxxxx>
> ---
>  drivers/misc/mei/hdcp/mei_hdcp.c | 55
> +++++++++++++++++++++++++++++++++++++++-
>  drivers/misc/mei/hdcp/mei_hdcp.h |  9 +++++++
>  include/linux/mei_hdcp.h         | 47 ++++++++++++++++++++++++++++++++++
>  3 files changed, 110 insertions(+), 1 deletion(-)  create mode 100644
> include/linux/mei_hdcp.h
> 
> diff --git a/drivers/misc/mei/hdcp/mei_hdcp.c
> b/drivers/misc/mei/hdcp/mei_hdcp.c
> index 25df7034cfb4..63f77800a6f7 100644
> --- a/drivers/misc/mei/hdcp/mei_hdcp.c
> +++ b/drivers/misc/mei/hdcp/mei_hdcp.c
> @@ -55,18 +55,71 @@ static int mei_hdcp_probe(struct mei_cl_device
> *cldev,
>  	mei_cldev_set_drvdata(cldev, &mei_hdcp);
> 
>  	ret = mei_cldev_enable(cldev);
> -	if (ret < 0)
> +	if (ret < 0) {
>  		dev_err(&cldev->dev, "mei_cldev_enable Failed. %d\n", ret);
> +		goto err;
> +	}
> +
> +	if (mei_hdcp.notify_on_cldev_change)
> +		mei_hdcp.notify_on_cldev_change(mei_hdcp.client, cldev);
> +
> +	return 0;
> +err:
> +	if (mei_hdcp.notify_on_cldev_change)
> +		mei_hdcp.notify_on_cldev_change(mei_hdcp.client, NULL);
> 
>  	return ret;
>  }
> 
>  static int mei_hdcp_remove(struct mei_cl_device *cldev)  {
> +	struct mei_hdcp *mei_hdcp = mei_cldev_get_drvdata(cldev);
> +
> +	if (mei_hdcp->notify_on_cldev_change)
> +		mei_hdcp->notify_on_cldev_change(mei_hdcp->client, NULL);
> +
>  	mei_cldev_disable(cldev);
> +
>  	return 0;
>  }
> 
> +int mei_hdcp_cldev_get_reference(void *client_data,
> +				 struct mei_cl_device **cldev,
> +				 void (*notify_change)(void *client,
> +						       struct mei_cl_device
> +						       *cldev))
> +{
> +	if (!notify_change || !client_data)
> +		return -EINVAL;
> +
> +	if (mei_hdcp.ref_cnt)
> +		return -EBUSY;
> +
> +	if (!mei_cldev_active_and_enabled(mei_hdcp.cldev)) {
> +		if (!notify_change)
> +			return -EAGAIN;
> +	} else {
> +		*cldev = mei_hdcp.cldev;
> +	}
> +
> +	mei_hdcp.ref_cnt++;
> +	mei_hdcp.client = client_data;
> +	mei_hdcp.notify_on_cldev_change = notify_change;
> +
> +	return 0;
> +}
> +EXPORT_SYMBOL(mei_hdcp_cldev_get_reference);
> +
> +void mei_hdcp_cldev_put_reference(struct mei_cl_device *cldev) {
> +	if (cldev == mei_hdcp.cldev) {
> +		mei_hdcp.ref_cnt--;
> +		mei_hdcp.client = NULL;
> +		mei_hdcp.notify_on_cldev_change = NULL;
> +	}
> +}
> +EXPORT_SYMBOL(mei_hdcp_cldev_put_reference);
> +
>  #define WIDI_HECI_CLIENT_GUID	UUID_LE(0xB638AB7E, 0x94E2,
> 0x4EA2, 0xA5, \
>  					0x52, 0xD1, 0xC5, 0x4B, \
>  					0x62, 0x7F, 0x04)
> diff --git a/drivers/misc/mei/hdcp/mei_hdcp.h
> b/drivers/misc/mei/hdcp/mei_hdcp.h
> index c06c0d767c4f..7d792b5ad703 100644
> --- a/drivers/misc/mei/hdcp/mei_hdcp.h
> +++ b/drivers/misc/mei/hdcp/mei_hdcp.h
> @@ -27,6 +27,15 @@
> 
>  struct mei_hdcp {
>  	struct mei_cl_device *cldev;
> +
> +	/* Reference to the HDCP2.2 service consumer */
> +	void *client;
> +
> +	/* Callback function for the consumer on cl_device state change */
> +	void (*notify_on_cldev_change)(void *client,
> +				      struct mei_cl_device *cldev);
> +
> +	int ref_cnt;
>  };
> 
>  #endif /* __MEI_HDCP_H__ */
> diff --git a/include/linux/mei_hdcp.h b/include/linux/mei_hdcp.h new file
> mode 100644 index 000000000000..774b26da0c26
> --- /dev/null
> +++ b/include/linux/mei_hdcp.h
> @@ -0,0 +1,47 @@
> +/*
> + * 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.
> + */
> +
> +#ifndef _LINUX_MEI_HDCP_H
> +#define _LINUX_MEI_HDCP_H
> +
> +#ifdef CONFIG_INTEL_MEI_HDCP
> +int mei_hdcp_cldev_get_reference(void *client_data,
> +				 struct mei_cl_device **cldev,
> +				 void (*notify_change)(void *client,
> +						       struct mei_cl_device
> +						       *cldev));
> +void mei_hdcp_cldev_put_reference(struct mei_cl_device *cldev); #else
> +static inline int mei_hdcp_cldev_get_reference(void *client_data,
> +				 struct mei_cl_device **cldev,
> +				 void (*notify_change)(void *client,
> +						       struct mei_cl_device
> +						       *cldev))
> +{
> +	return -ENODEV;
> +}
> +static inline
> +void mei_hdcp_cldev_put_reference(struct mei_cl_device *cldev) {}
> +#endif /* defined (CONFIG_INTEL_MEI_HDCP) */ #endif /* defined
> +(_LINUX_MEI_HDCP_H) */
> --
> 2.7.4

_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux