From: sunil.kovvuri@xxxxxxxxx Date: Sun, 7 Oct 2018 20:29:14 +0530 > + req_hdr = (struct mbox_hdr *)(mdev->mbase + mbox->rx_start); I commented in the previous patch series version, for patch #4, that such casts are unnecessary and should be removed. You are in for a very long review process if you only consider feedback given to you for the specific patch for which it is given, rather than your entire series. Please put forth the effort to fix the problems pointed out to you in your entire series. Thank you.