On Wed, 14 Dec 2022, Suraj Kandpal <suraj.kandpal@xxxxxxxxx> wrote: > From: Anshuman Gupta <anshuman.gupta@xxxxxxxxx> > > Change the include/drm/i915_mei_hdcp_interface.h to > include/drm/i915_cp_fw_hdcp_interface.h This is now stale. > > Cc: Tomas Winkler <tomas.winkler@xxxxxxxxx> > Cc: Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx> > Cc: Uma Shankar <uma.shankar@xxxxxxxxx> > Cc: Ankit Nautiyal <ankit.k.nautiyal@xxxxxxxxx> > Signed-off-by: Anshuman Gupta <anshuman.gupta@xxxxxxxxx> > Signed-off-by: Suraj Kandpal <suraj.kandpal@xxxxxxxxx> > Acked-by: Tomas Winkler <tomas.winkler@xxxxxxxxx> > --- > .../drm/i915/display/intel_display_types.h | 2 +- > drivers/misc/mei/hdcp/mei_hdcp.c | 2 +- > include/drm/i915_mei_hdcp_interface.h | 184 ------------------ > 3 files changed, 2 insertions(+), 186 deletions(-) > delete mode 100644 include/drm/i915_mei_hdcp_interface.h > > diff --git a/drivers/gpu/drm/i915/display/intel_display_types.h b/drivers/gpu/drm/i915/display/intel_display_types.h > index 32e8b2fc3cc6..81d195ef5e57 100644 > --- a/drivers/gpu/drm/i915/display/intel_display_types.h > +++ b/drivers/gpu/drm/i915/display/intel_display_types.h > @@ -43,7 +43,7 @@ > #include <drm/drm_rect.h> > #include <drm/drm_vblank.h> > #include <drm/drm_vblank_work.h> > -#include <drm/i915_mei_hdcp_interface.h> > +#include <drm/i915_hdcp_interface.h> > #include <media/cec-notifier.h> > > #include "i915_vma.h" > diff --git a/drivers/misc/mei/hdcp/mei_hdcp.c b/drivers/misc/mei/hdcp/mei_hdcp.c > index e889a8bd7ac8..cbad27511899 100644 > --- a/drivers/misc/mei/hdcp/mei_hdcp.c > +++ b/drivers/misc/mei/hdcp/mei_hdcp.c > @@ -23,7 +23,7 @@ > #include <linux/component.h> > #include <drm/drm_connector.h> > #include <drm/i915_component.h> > -#include <drm/i915_mei_hdcp_interface.h> > +#include <drm/i915_hdcp_interface.h> > > #include "mei_hdcp.h" > > diff --git a/include/drm/i915_mei_hdcp_interface.h b/include/drm/i915_mei_hdcp_interface.h > deleted file mode 100644 This patch now *deletes* the file. The next patch adds it back. Please be more careful before sending. Please check that all commits build before sending, run something like: $ git rebase -i $tip --exec="make -j8" This should be standard practice for all contributions. BR, Jani. > index f441cbcd95a4..000000000000 > --- a/include/drm/i915_mei_hdcp_interface.h > +++ /dev/null > @@ -1,184 +0,0 @@ > -/* SPDX-License-Identifier: (GPL-2.0+) */ > -/* > - * Copyright © 2017-2019 Intel Corporation > - * > - * Authors: > - * Ramalingam C <ramalingam.c@xxxxxxxxx> > - */ > - > -#ifndef _I915_MEI_HDCP_INTERFACE_H_ > -#define _I915_MEI_HDCP_INTERFACE_H_ > - > -#include <linux/mutex.h> > -#include <linux/device.h> > -#include <drm/display/drm_hdcp.h> > - > -/** > - * enum hdcp_port_type - HDCP port implementation type defined by ME FW > - * @HDCP_PORT_TYPE_INVALID: Invalid hdcp port type > - * @HDCP_PORT_TYPE_INTEGRATED: In-Host HDCP2.x port > - * @HDCP_PORT_TYPE_LSPCON: HDCP2.2 discrete wired Tx port with LSPCON > - * (HDMI 2.0) solution > - * @HDCP_PORT_TYPE_CPDP: HDCP2.2 discrete wired Tx port using the CPDP (DP 1.3) > - * solution > - */ > -enum hdcp_port_type { > - HDCP_PORT_TYPE_INVALID, > - HDCP_PORT_TYPE_INTEGRATED, > - HDCP_PORT_TYPE_LSPCON, > - HDCP_PORT_TYPE_CPDP > -}; > - > -/** > - * enum hdcp_wired_protocol - HDCP adaptation used on the port > - * @HDCP_PROTOCOL_INVALID: Invalid HDCP adaptation protocol > - * @HDCP_PROTOCOL_HDMI: HDMI adaptation of HDCP used on the port > - * @HDCP_PROTOCOL_DP: DP adaptation of HDCP used on the port > - */ > -enum hdcp_wired_protocol { > - HDCP_PROTOCOL_INVALID, > - HDCP_PROTOCOL_HDMI, > - HDCP_PROTOCOL_DP > -}; > - > -enum mei_fw_ddi { > - MEI_DDI_INVALID_PORT = 0x0, > - > - MEI_DDI_B = 1, > - MEI_DDI_C, > - MEI_DDI_D, > - MEI_DDI_E, > - MEI_DDI_F, > - MEI_DDI_A = 7, > - MEI_DDI_RANGE_END = MEI_DDI_A, > -}; > - > -/** > - * enum mei_fw_tc - ME Firmware defined index for transcoders > - * @MEI_INVALID_TRANSCODER: Index for Invalid transcoder > - * @MEI_TRANSCODER_EDP: Index for EDP Transcoder > - * @MEI_TRANSCODER_DSI0: Index for DSI0 Transcoder > - * @MEI_TRANSCODER_DSI1: Index for DSI1 Transcoder > - * @MEI_TRANSCODER_A: Index for Transcoder A > - * @MEI_TRANSCODER_B: Index for Transcoder B > - * @MEI_TRANSCODER_C: Index for Transcoder C > - * @MEI_TRANSCODER_D: Index for Transcoder D > - */ > -enum mei_fw_tc { > - MEI_INVALID_TRANSCODER = 0x00, > - MEI_TRANSCODER_EDP, > - MEI_TRANSCODER_DSI0, > - MEI_TRANSCODER_DSI1, > - MEI_TRANSCODER_A = 0x10, > - MEI_TRANSCODER_B, > - MEI_TRANSCODER_C, > - MEI_TRANSCODER_D > -}; > - > -/** > - * struct hdcp_port_data - intel specific HDCP port data > - * @fw_ddi: ddi index as per ME FW > - * @fw_tc: transcoder index as per ME FW > - * @port_type: HDCP port type as per ME FW classification > - * @protocol: HDCP adaptation as per ME FW > - * @k: No of streams transmitted on a port. Only on DP MST this is != 1 > - * @seq_num_m: Count of RepeaterAuth_Stream_Manage msg propagated. > - * Initialized to 0 on AKE_INIT. Incremented after every successful > - * transmission of RepeaterAuth_Stream_Manage message. When it rolls > - * over re-Auth has to be triggered. > - * @streams: struct hdcp2_streamid_type[k]. Defines the type and id for the > - * streams > - */ > -struct hdcp_port_data { > - enum mei_fw_ddi fw_ddi; > - enum mei_fw_tc fw_tc; > - u8 port_type; > - u8 protocol; > - u16 k; > - u32 seq_num_m; > - struct hdcp2_streamid_type *streams; > -}; > - > -/** > - * struct i915_hdcp_component_ops- ops for HDCP2.2 services. > - * @owner: Module providing the ops > - * @initiate_hdcp2_session: Initiate a Wired HDCP2.2 Tx Session. > - * And Prepare AKE_Init. > - * @verify_receiver_cert_prepare_km: Verify the Receiver Certificate > - * AKE_Send_Cert and prepare > - AKE_Stored_Km/AKE_No_Stored_Km > - * @verify_hprime: Verify AKE_Send_H_prime > - * @store_pairing_info: Store pairing info received > - * @initiate_locality_check: Prepare LC_Init > - * @verify_lprime: Verify lprime > - * @get_session_key: Prepare SKE_Send_Eks > - * @repeater_check_flow_prepare_ack: Validate the Downstream topology > - * and prepare rep_ack > - * @verify_mprime: Verify mprime > - * @enable_hdcp_authentication: Mark a port as authenticated. > - * @close_hdcp_session: Close the Wired HDCP Tx session per port. > - * This also disables the authenticated state of the port. > - */ > -struct i915_hdcp_component_ops { > - /** > - * @owner: mei_hdcp module > - */ > - struct module *owner; > - > - int (*initiate_hdcp2_session)(struct device *dev, > - struct hdcp_port_data *data, > - struct hdcp2_ake_init *ake_data); > - int (*verify_receiver_cert_prepare_km)(struct device *dev, > - struct hdcp_port_data *data, > - struct hdcp2_ake_send_cert > - *rx_cert, > - bool *km_stored, > - struct hdcp2_ake_no_stored_km > - *ek_pub_km, > - size_t *msg_sz); > - int (*verify_hprime)(struct device *dev, > - struct hdcp_port_data *data, > - struct hdcp2_ake_send_hprime *rx_hprime); > - int (*store_pairing_info)(struct device *dev, > - struct hdcp_port_data *data, > - struct hdcp2_ake_send_pairing_info > - *pairing_info); > - int (*initiate_locality_check)(struct device *dev, > - struct hdcp_port_data *data, > - struct hdcp2_lc_init *lc_init_data); > - int (*verify_lprime)(struct device *dev, > - struct hdcp_port_data *data, > - struct hdcp2_lc_send_lprime *rx_lprime); > - int (*get_session_key)(struct device *dev, > - struct hdcp_port_data *data, > - struct hdcp2_ske_send_eks *ske_data); > - int (*repeater_check_flow_prepare_ack)(struct device *dev, > - struct hdcp_port_data *data, > - struct hdcp2_rep_send_receiverid_list > - *rep_topology, > - struct hdcp2_rep_send_ack > - *rep_send_ack); > - int (*verify_mprime)(struct device *dev, > - struct hdcp_port_data *data, > - struct hdcp2_rep_stream_ready *stream_ready); > - int (*enable_hdcp_authentication)(struct device *dev, > - struct hdcp_port_data *data); > - int (*close_hdcp_session)(struct device *dev, > - struct hdcp_port_data *data); > -}; > - > -/** > - * struct i915_hdcp_component_master - Used for communication between i915 > - * and mei_hdcp drivers for the HDCP2.2 services > - * @mei_dev: device that provide the HDCP2.2 service from MEI Bus. > - * @hdcp_ops: Ops implemented by mei_hdcp driver, used by i915 driver. > - */ > -struct i915_hdcp_comp_master { > - struct device *mei_dev; > - const struct i915_hdcp_component_ops *ops; > - > - /* To protect the above members. */ > - struct mutex mutex; > -}; > - > -#endif /* _I915_MEI_HDCP_INTERFACE_H_ */ -- Jani Nikula, Intel Open Source Graphics Center