Xiaomeng, > The list iterator 'fcport' is always non-NULL so it doesn't need to be > checked. Thus just remove the unnecessary NULL check. Also remove the > unnecessary initializer because the list iterator is always initialized. > And adjust the position of blank lines. Applied to 5.19/scsi-staging, thanks! -- Martin K. Petersen Oracle Linux Engineering