RE: [PATCH v11 3/6] drm: Extend I915 mei interface for transcoder info
Looks good to me, From display side, Please feel free to use Reviewed-by: Shashank Sharma Regards Shashank > -Original Message- > From: Winkler, Tomas > Sent: Wednesday, August 28, 2019 1:49 PM > To: C, Ramalingam > Cc: intel-gfx ; dri-devel de...@lists.freedesktop.org>; Sharma, Shashank ; > Daniel Vetter ; Nikula, Jani > Subject: RE: [PATCH v11 3/6] drm: Extend I915 mei interface for transcoder > info > > > > > > > On 2019-08-28 at 13:28:06 +0530, Winkler, Tomas wrote: > > > > I915 needs to send the index of the transcoder as per ME FW. > > > > > > > > To support this, define enum mei_fw_tc and add as a member into > > > > the struct hdcp_port_data. > > > > > > > > v2: > > > > Typo in commit msg is fixed [Shashank] > > > > v3: > > > > kdoc is added for mei_fw_tc [Tomas] > > > > s/MEI_TC_x/MEI_TRANSCODER_x > > > > > > > > Signed-off-by: Ramalingam C > > > > Acked-by: Jani Nikula > > > > > > LGTM, just need to updated the copyright year. > > Tomas, > > > > Copyright year for this i915_mei_hdcp_interface.h is updated in the > > 1st patch itself. > > Okay. > > > > -Ram > > > > > > > --- > > > > include/drm/i915_mei_hdcp_interface.h | 24 > > > > > > 1 file changed, 24 insertions(+) > > > > > > > > diff --git a/include/drm/i915_mei_hdcp_interface.h > > > > b/include/drm/i915_mei_hdcp_interface.h > > > > index 08670aa650d4..4d48de8890ca 100644 > > > > --- a/include/drm/i915_mei_hdcp_interface.h > > > > +++ b/include/drm/i915_mei_hdcp_interface.h > > > > @@ -54,9 +54,32 @@ enum mei_fw_ddi { > > > > 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 @@ - > > > > 69,6 +92,7 @@ enum mei_fw_ddi { > > > > */ > > > > struct hdcp_port_data { > > > > enum mei_fw_ddi fw_ddi; > > > > + enum mei_fw_tc fw_tc; > > > > u8 port_type; > > > > u8 protocol; > > > > u16 k; > > > > -- > > > > 2.20.1 > > > ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
RE: [PATCH v11 3/6] drm: Extend I915 mei interface for transcoder info
> > On 2019-08-28 at 13:28:06 +0530, Winkler, Tomas wrote: > > > I915 needs to send the index of the transcoder as per ME FW. > > > > > > To support this, define enum mei_fw_tc and add as a member into the > > > struct hdcp_port_data. > > > > > > v2: > > > Typo in commit msg is fixed [Shashank] > > > v3: > > > kdoc is added for mei_fw_tc [Tomas] > > > s/MEI_TC_x/MEI_TRANSCODER_x > > > > > > Signed-off-by: Ramalingam C > > > Acked-by: Jani Nikula > > > > LGTM, just need to updated the copyright year. > Tomas, > > Copyright year for this i915_mei_hdcp_interface.h is updated in the 1st patch > itself. Okay. > > -Ram > > > > > --- > > > include/drm/i915_mei_hdcp_interface.h | 24 > > > > 1 file changed, 24 insertions(+) > > > > > > diff --git a/include/drm/i915_mei_hdcp_interface.h > > > b/include/drm/i915_mei_hdcp_interface.h > > > index 08670aa650d4..4d48de8890ca 100644 > > > --- a/include/drm/i915_mei_hdcp_interface.h > > > +++ b/include/drm/i915_mei_hdcp_interface.h > > > @@ -54,9 +54,32 @@ enum mei_fw_ddi { > > > 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 @@ - > > > 69,6 +92,7 @@ enum mei_fw_ddi { > > > */ > > > struct hdcp_port_data { > > > enum mei_fw_ddi fw_ddi; > > > + enum mei_fw_tc fw_tc; > > > u8 port_type; > > > u8 protocol; > > > u16 k; > > > -- > > > 2.20.1 > > ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
Re: [PATCH v11 3/6] drm: Extend I915 mei interface for transcoder info
On 2019-08-28 at 13:28:06 +0530, Winkler, Tomas wrote: > > I915 needs to send the index of the transcoder as per ME FW. > > > > To support this, define enum mei_fw_tc and add as a member into the struct > > hdcp_port_data. > > > > v2: > > Typo in commit msg is fixed [Shashank] > > v3: > > kdoc is added for mei_fw_tc [Tomas] > > s/MEI_TC_x/MEI_TRANSCODER_x > > > > Signed-off-by: Ramalingam C > > Acked-by: Jani Nikula > > LGTM, just need to updated the copyright year. Tomas, Copyright year for this i915_mei_hdcp_interface.h is updated in the 1st patch itself. -Ram > > > --- > > include/drm/i915_mei_hdcp_interface.h | 24 > > 1 file changed, 24 insertions(+) > > > > diff --git a/include/drm/i915_mei_hdcp_interface.h > > b/include/drm/i915_mei_hdcp_interface.h > > index 08670aa650d4..4d48de8890ca 100644 > > --- a/include/drm/i915_mei_hdcp_interface.h > > +++ b/include/drm/i915_mei_hdcp_interface.h > > @@ -54,9 +54,32 @@ enum mei_fw_ddi { > > 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 @@ > > - > > 69,6 +92,7 @@ enum mei_fw_ddi { > > */ > > struct hdcp_port_data { > > enum mei_fw_ddi fw_ddi; > > + enum mei_fw_tc fw_tc; > > u8 port_type; > > u8 protocol; > > u16 k; > > -- > > 2.20.1 > ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
RE: [PATCH v11 3/6] drm: Extend I915 mei interface for transcoder info
> I915 needs to send the index of the transcoder as per ME FW. > > To support this, define enum mei_fw_tc and add as a member into the struct > hdcp_port_data. > > v2: > Typo in commit msg is fixed [Shashank] > v3: > kdoc is added for mei_fw_tc [Tomas] > s/MEI_TC_x/MEI_TRANSCODER_x > > Signed-off-by: Ramalingam C > Acked-by: Jani Nikula LGTM, just need to updated the copyright year. > --- > include/drm/i915_mei_hdcp_interface.h | 24 > 1 file changed, 24 insertions(+) > > diff --git a/include/drm/i915_mei_hdcp_interface.h > b/include/drm/i915_mei_hdcp_interface.h > index 08670aa650d4..4d48de8890ca 100644 > --- a/include/drm/i915_mei_hdcp_interface.h > +++ b/include/drm/i915_mei_hdcp_interface.h > @@ -54,9 +54,32 @@ enum mei_fw_ddi { > 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 @@ - > 69,6 +92,7 @@ enum mei_fw_ddi { > */ > struct hdcp_port_data { > enum mei_fw_ddi fw_ddi; > + enum mei_fw_tc fw_tc; > u8 port_type; > u8 protocol; > u16 k; > -- > 2.20.1 ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel