On Fri, Mar 03, 2023 at 09:32:50AM +0800, Yu Zhe wrote: > @@ -1157,7 +1155,7 @@ static long zcrypt_msgtype6_send_cprb(bool userspace, struct zcrypt_queue *zq, > struct ap_message *ap_msg) > { > int rc; > - struct response_type *rtype = (struct response_type *)(ap_msg->private); > + struct response_type *rtype = (ap_msg->private); If you're going to do this the get rid of the parentheses as well: struct response_type *rtype = ap_msg->private; Same in other places. regards, dan carpenter