From: Martin Wilck [mailto:mwilck@xxxxxxxx] Subject: Re: [PATCH V3 22/25] smartpqi: update enclosure identifier in sysf > @@ -1841,7 +1841,6 @@ static void pqi_dev_info(struct pqi_ctrl_info > *ctrl_info, static void pqi_scsi_update_device(struct pqi_scsi_dev > *existing_device, > struct pqi_scsi_dev *new_device) { > - existing_device->devtype = new_device->devtype; > existing_device->device_type = new_device->device_type; > existing_device->bus = new_device->bus; > if (new_device->target_lun_valid) { > I don't get this. Why was it wrong to update the devtype field? Don: From patch Author... If we don't remove that statement, following issue will crop up. During initial device enumeration, the devtype attribute of the device(in current case enclosure device) is filled in slave_configure. But whenever a rescan occurs, the f/w would return zero for this field, and valid devtype is overwritten by zero, when device attributes are updated in pqi_scsi_update_device. Due to this lsscsi output shows wrong values.
