On 11/21/22 21:17, John Garry wrote: > Instead of using hardcoded '0' as the do_scsi_scan_host() -> > scsi_scan_host_selected() rescan arg, use proper macro SCSI_SCAN_INITIAL. > > Signed-off-by: John Garry <john.g.garry@xxxxxxxxxx> Looks good. Reviewed-by: Damien Le Moal <damien.lemoal@xxxxxxxxxxxxxxxxxx> > --- > drivers/scsi/scsi_scan.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/scsi/scsi_scan.c b/drivers/scsi/scsi_scan.c > index 6cc974b382c1..920b145f80b7 100644 > --- a/drivers/scsi/scsi_scan.c > +++ b/drivers/scsi/scsi_scan.c > @@ -1920,7 +1920,7 @@ static void do_scsi_scan_host(struct Scsi_Host *shost) > msleep(10); > } else { > scsi_scan_host_selected(shost, SCAN_WILD_CARD, SCAN_WILD_CARD, > - SCAN_WILD_CARD, 0); > + SCAN_WILD_CARD, SCSI_SCAN_INITIAL); > } > } > -- Damien Le Moal Western Digital Research