Assigning a static to zero will cause the checkpatch tool to complain that you are unnecessarily providing a value that is the default.@@ -214,6 +214,24 @@ void xe_guc_log_snapshot_print(struct xe_guc_log_snapshot *snapshot, struct drm_ } } +/** + * xe_guc_log_print_dmesg - dump a copy of the GuC log to dmesg + * @log: GuC log structure + */ +void xe_guc_log_print_dmesg(struct xe_guc_log *log) +{ + struct xe_gt *gt = log_to_gt(log); + static int g_count; + struct drm_printer ip = xe_gt_info_printer(gt); + struct drm_printer lp = drm_line_printer(&ip, "Capture", ++g_count);Increment of non-assigned value "g_count". It is static so zero allocated but suggest to assign to 0 for clarity.
Reviewed-by: Julia Filipchuk <julia.filipchuk@intel.com>