On Thu, Feb 06, 2025 at 10:16:09AM -0800, Justin Tee wrote: > 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. Ah, you are right, the ts is used somewhere else. -- With Best Regards, Andy Shevchenko