Hi Al, I’m good with the use of enum. For the if and else if blocks, would it be possible to help us out and convert those to switch case statements? For example, switch (kind) { case writeGuard : cnt = scnprintf(cbuf, 32, "%u\n", phba->lpfc_injerr_wgrd_cnt); break; case writeApp: cnt = scnprintf(cbuf, 32, "%u\n", phba->lpfc_injerr_wapp_cnt); break; … default: lpfc_printf_log(...); } Thanks, Justin