On 12/01/2021 18:38, Joe Perches wrote: > static void > _base_display_OEMs_branding(struct MPT3SAS_ADAPTER *ioc) > { > + const char *b = NULL; /* brand */ > + const char *v = NULL; /* vendor */ Any reason you didn't spell out brand and vendor as the variable names?