Re: [PATCH v4 24/41] drm/i915: Implement HDCP2.2 repeater authentication

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

 



Hi Ramalingam,

Thank you for the patch! Perhaps something to improve:

[auto build test WARNING on drm-intel/for-linux-next]
[also build test WARNING on v4.17-rc6 next-20180517]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url:    https://github.com/0day-ci/linux/commits/Ramalingam-C/drm-i915-Implement-HDCP2-2/20180523-031938
base:   git://anongit.freedesktop.org/drm-intel for-linux-next
reproduce:
        # apt-get install sparse
        make ARCH=x86_64 allmodconfig
        make C=1 CF=-D__CHECK_ENDIAN__


sparse warnings: (new ones prefixed by >>)

   drivers/gpu/drm/i915/i915_drv.h:3663:16: sparse: expression using sizeof(void)
>> drivers/gpu/drm/i915/intel_hdcp.c:1247:30: sparse: incorrect type in assignment (different base types) @@    expected restricted __be16 [assigned] [usertype] k @@    got e] k @@
   drivers/gpu/drm/i915/intel_hdcp.c:1247:30:    expected restricted __be16 [assigned] [usertype] k
   drivers/gpu/drm/i915/intel_hdcp.c:1247:30:    got int
   drivers/gpu/drm/i915/intel_hdcp.c:1343:12: warning: 'hdcp2_authenticate_sink' defined but not used [-Wunused-function]
    static int hdcp2_authenticate_sink(struct intel_connector *connector)
               ^~~~~~~~~~~~~~~~~~~~~~~

vim +1247 drivers/gpu/drm/i915/intel_hdcp.c

  1228	
  1229	static
  1230	int hdcp2_propagate_stream_management_info(struct intel_connector *connector)
  1231	{
  1232		struct intel_digital_port *intel_dig_port = conn_to_dig_port(connector);
  1233		struct intel_hdcp *hdcp = &connector->hdcp;
  1234		union {
  1235			struct hdcp2_rep_stream_manage stream_manage;
  1236			struct hdcp2_rep_stream_ready stream_ready;
  1237		} msgs;
  1238		const struct intel_hdcp_shim *shim = hdcp->hdcp_shim;
  1239		int ret;
  1240	
  1241		/* Prepare RepeaterAuth_Stream_Manage msg */
  1242		msgs.stream_manage.msg_id = HDCP_2_2_REP_STREAM_MANAGE;
  1243		reverse_endianness(msgs.stream_manage.seq_num_m, HDCP_2_2_SEQ_NUM_LEN,
  1244				   (u8 *)&hdcp->seq_num_m);
  1245	
  1246		/* K no of streams is fixed as 1. Stored as big-endian. */
> 1247		msgs.stream_manage.k = __swab16(1);
  1248	
  1249		/* For HDMI this is forced to be 0x0. For DP SST also this is 0x0. */
  1250		msgs.stream_manage.streams[0].stream_id = 0;
  1251		msgs.stream_manage.streams[0].stream_type = hdcp->content_type;
  1252	
  1253		/* Send it to Repeater */
  1254		ret = shim->write_2_2_msg(intel_dig_port, &msgs.stream_manage,
  1255					  sizeof(msgs.stream_manage));
  1256		if (ret < 0)
  1257			return ret;
  1258	
  1259		ret = shim->read_2_2_msg(intel_dig_port, HDCP_2_2_REP_STREAM_READY,
  1260					 &msgs.stream_ready, sizeof(msgs.stream_ready));
  1261		if (ret < 0)
  1262			return ret;
  1263	
  1264		hdcp->mei_data.seq_num_m = hdcp->seq_num_m;
  1265		hdcp->mei_data.streams[0].stream_type = hdcp->content_type;
  1266	
  1267		ret = hdcp2_verify_mprime(hdcp,	&msgs.stream_ready);
  1268		if (ret < 0)
  1269			return ret;
  1270	
  1271		hdcp->seq_num_m++;
  1272	
  1273		if (hdcp->seq_num_m > HDCP_2_2_SEQ_NUM_MAX) {
  1274			DRM_DEBUG_KMS("seq_num_m roll over.\n");
  1275			return -1;
  1276		}
  1277	
  1278		return 0;
  1279	}
  1280	

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation
_______________________________________________
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