From 6db5dbffc5aab25edf7b880a2ff6a00e34709f27 Mon Sep 17 00:00:00 2001 From: krw Date: Tue, 22 Mar 2022 16:29:58 +0000 Subject: [PATCH] Nuke increasingly pointless comment. Shorten a line and compare pointer to NULL instead of 0. No functional change. --- sys/scsi/scsiconf.c | 12 +++--------- 1 file changed, 3 insertions(+), 9 deletions(-) diff --git a/sys/scsi/scsiconf.c b/sys/scsi/scsiconf.c index b085af4f53a..941fdd6661f 100644 --- a/sys/scsi/scsiconf.c +++ b/sys/scsi/scsiconf.c @@ -1,4 +1,4 @@ -/* $OpenBSD: scsiconf.c,v 1.245 2022/03/21 12:57:46 krw Exp $ */ +/* $OpenBSD: scsiconf.c,v 1.246 2022/03/22 16:29:58 krw Exp $ */ /* $NetBSD: scsiconf.c,v 1.57 1996/05/02 01:09:01 neil Exp $ */ /* @@ -472,12 +472,6 @@ scsi_probe_lun(struct scsibus_softc *sb, int target, int lun) return scsi_probe_link(sb, target, lun, 0); } -/* - * Given a target and lun, ask the device what it is, and find the correct - * driver table entry. - * - * Return 0 if further LUNs are possible, EINVAL if not. - */ int scsi_probe_link(struct scsibus_softc *sb, int target, int lun, int dumbscan) { @@ -681,8 +675,8 @@ scsi_probe_link(struct scsibus_softc *sb, int target, int lun, int dumbscan) sa.sa_sc_link = link; - if ((cf = config_search(scsibussubmatch, (struct device *)sb, - &sa)) == 0) { + cf = config_search(scsibussubmatch, (struct device *)sb, &sa); + if (cf == NULL) { scsibussubprint(&sa, sb->sc_dev.dv_xname); printf(" not configured\n"); goto free_devid; -- 2.20.1