On 3/29/21 11:40 PM, Viswas G wrote: > +static ssize_t ctl_raae_count_show(struct device *cdev, > + struct device_attribute *attr, char *buf) > +{ > + struct Scsi_Host *shost = class_to_shost(cdev); > + struct sas_ha_struct *sha = SHOST_TO_SAS_HA(shost); > + struct pm8001_hba_info *pm8001_ha = sha->lldd_ha; > + unsigned int raaecnt; > + int c; > + > + raaecnt = pm8001_mr32(pm8001_ha->general_stat_tbl_addr, 12); > + c = sysfs_emit(buf, "0x%08x\n", raaecnt); > + return c; > +} Please remove the variable 'c' from this patch and also from subsequent patches. Thanks, Bart.