Re: [PATCH v11 39/42] misc/mei/hdcp: Component framework for I915 Interface

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

 



> -----Original Message-----
> From: Winkler, Tomas
> Sent: Friday, February 8, 2019 3:42 AM
> To: C, Ramalingam <ramalingam.c@xxxxxxxxx>; intel-gfx@xxxxxxxxxxxxxxxxxxxxx;
> dri-devel@xxxxxxxxxxxxxxxxxxxxx; daniel.vetter@xxxxxxxx; Shankar, Uma
> <uma.shankar@xxxxxxxxx>
> Subject: RE: [PATCH v11 39/42] misc/mei/hdcp: Component framework for I915
> Interface
> 
> > Mei hdcp driver is designed as component slave for the I915 component
> > master.
> >
> > v2: Rebased.
> > v3:
> >   Notifier chain is adopted for cldev state update [Tomas]
> > v4:
> >   Made static dummy functions as inline in mei_hdcp.h
> >   API for polling client device status
> >   IS_ENABLED used in header, for config status for mei_hdcp.
> > v5:
> >   Replacing the notifier with component framework. [Daniel]
> > v6:
> >   Rebased on the I915 comp master redesign.
> > v7:
> >   mei_hdcp_component_registered is made static [Uma]
> >   Need for global static variable mei_cldev is removed.
> > v8:
> >   master comp is added to be matched with i915 subcomponent [daniel]
> > v9:
> >   only comp_master is set and retrieved as driver_data [Daniel]
> >   Reviewed-by Daniel.
> >
> > Signed-off-by: Ramalingam C <ramalingam.c@xxxxxxxxx>
> > Reviewed-by: Daniel Vetter <daniel.vetter@xxxxxxxx>
> > ---
> >  drivers/misc/mei/hdcp/mei_hdcp.c | 83
> > +++++++++++++++++++++++++++++++++++++++-
> >  1 file changed, 81 insertions(+), 2 deletions(-)
> >
> > diff --git a/drivers/misc/mei/hdcp/mei_hdcp.c
> > b/drivers/misc/mei/hdcp/mei_hdcp.c
> > index 615eed149837..beba6fb4d2e2 100644
> > --- a/drivers/misc/mei/hdcp/mei_hdcp.c
> > +++ b/drivers/misc/mei/hdcp/mei_hdcp.c
> > @@ -23,6 +23,7 @@
> >  #include <linux/slab.h>
> >  #include <linux/uuid.h>
> >  #include <linux/mei_cl_bus.h>
> > +#include <linux/component.h>
> >  #include <drm/drm_connector.h>
> >  #include <drm/i915_component.h>
> >  #include <drm/i915_mei_hdcp_interface.h> @@ -706,8 +707,7 @@
> > mei_hdcp_close_session(struct device *dev, struct hdcp_port_data *data)
> >  	return 0;
> >  }
> >
> > -static __attribute__((unused))
> > -struct i915_hdcp_component_ops mei_hdcp_ops = {
> > +static struct i915_hdcp_component_ops mei_hdcp_ops = {
> >  	.owner = THIS_MODULE,
> >  	.initiate_hdcp2_session = mei_hdcp_initiate_session,
> >  	.verify_receiver_cert_prepare_km =
> > @@ -724,20 +724,99 @@ struct i915_hdcp_component_ops mei_hdcp_ops =
> {
> >  	.close_hdcp_session = mei_hdcp_close_session,  };
> >
> > +static int mei_component_master_bind(struct device *dev) {
> > +	struct mei_cl_device *cldev = to_mei_cl_device(dev);
> > +	struct i915_hdcp_comp_master *comp_master =
> > +
> > 	mei_cldev_get_drvdata(cldev);
> > +	int ret;
> > +
> > +	dev_info(dev, "%s\n", __func__);
> > +	comp_master->ops = &mei_hdcp_ops;
> > +	comp_master->mei_dev = dev;
> > +	ret = component_bind_all(dev, comp_master);
> > +	if (ret < 0)
> > +		return ret;
> > +
> > +	return 0;
> > +}
> > +
> > +static void mei_component_master_unbind(struct device *dev) {
> > +	struct mei_cl_device *cldev = to_mei_cl_device(dev);
> > +	struct i915_hdcp_comp_master *comp_master =
> > +
> > 	mei_cldev_get_drvdata(cldev);
> > +
> > +	dev_info(dev, "%s\n", __func__);
> > +	component_unbind_all(dev, comp_master); }
> > +
> > +static const struct component_master_ops mei_component_master_ops = {
> > +	.bind = mei_component_master_bind,
> > +	.unbind = mei_component_master_unbind, };
> > +
> > +static int mei_hdcp_component_match(struct device *dev, int
> subcomponent,
> > +				    void *data)
> > +{
> > +	return !strcmp(dev->driver->name, "i915") &&
> > +	       subcomponent == I915_COMPONENT_HDCP; }
> > +
> >  static int mei_hdcp_probe(struct mei_cl_device *cldev,
> >  			  const struct mei_cl_device_id *id)  {
> > +	struct i915_hdcp_comp_master *comp_master;
> > +	struct component_match *master_match;
> >  	int ret;
> >
> >  	ret = mei_cldev_enable(cldev);
> >  	if (ret < 0)
> >  		dev_err(&cldev->dev, "mei_cldev_enable Failed. %d\n", ret);
> >
> > +	comp_master = kzalloc(sizeof(*comp_master), GFP_KERNEL);
> > +	if (!comp_master) {
> > +		ret = -ENOMEM;
> > +		goto comp_master_exit;
> > +	}
> > +
> > +	master_match = NULL;
> > +	component_match_add_typed(&cldev->dev, &master_match,
> > +				  mei_hdcp_component_match,
> > comp_master);
> > +	if (IS_ERR_OR_NULL(master_match)) {
> > +		ret = -ENOMEM;
> > +		goto match_add_exit;
> > +	}
> > +
> > +	mei_cldev_set_drvdata(cldev, comp_master);
> > +	ret = component_master_add_with_match(&cldev->dev,
> > +					      &mei_component_master_ops,
> > +					      master_match);
> > +	if (ret < 0) {
> > +		dev_err(&cldev->dev, "Master comp add failed %d\n", ret);
> > +		mei_cldev_set_drvdata(cldev, NULL);
> Can be done in exit label.


I will move it.
> > +		goto match_add_exit;
> > +	}
> > +
> > +	return 0;
> > +
> > +match_add_exit:
> > +	kfree(comp_master);
> > +comp_master_exit:
> 
> You can use a single label as  kfree is NULL safe.
Sure.

-Ram
> > +	mei_cldev_disable(cldev);
> > +
> >  	return ret;
> >  }
> >
> >  static int mei_hdcp_remove(struct mei_cl_device *cldev)  {
> > +	struct i915_hdcp_comp_master *comp_master =
> > +
> > 	mei_cldev_get_drvdata(cldev);
> > +
> > +	component_master_del(&cldev->dev, &mei_component_master_ops);
> > +	kfree(comp_master);
> > +	mei_cldev_set_drvdata(cldev, NULL);
> > +
> >  	return mei_cldev_disable(cldev);
> >  }
> >
> > --
> > 2.7.4

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




[Index of Archives]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux