Hi Andy, The purpose of the lpfc_cgn_update_tstamp routine is more than just printf. The *ts pointer argument is updated with cur_time, and is typically a pointer to a global statistics struct used by the device driver in various contexts. Sorry, but we can’t remove the lines suggested in this patch. Regards, Justin Tee