On Tue, 2021-01-12 at 17:40 +0000, Johannes Thumshirn wrote: > 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? 80 column lines.