Re: port-prep/55003

[T <bobs%thelibertytree.org@localhost>]

The following reply was made to PR port-prep/55003; it has been noted by GNATS.

From: T <bobs%thelibertytree.org@localhost>
To: gnats-bugs%netbsd.org@localhost
Cc: 
Subject: Re: port-prep/55003
Date: Mon, 24 Feb 2020 02:54:17 -0700

 Hello, I have a modification for this ticket. There are a couple more 
 devices that should be blacklisted and I figured out a related problem 
 in the same area that overlaps with this function. The problem is 
 described in this mailing list post: 
 http://mail-index.netbsd.org/port-prep/2020/02/24/msg000180.html 
 Attached is a new patch so the prior one can be ignored.
 
 
 --- /usr/src/sys/arch/prep/prep/autoconf.c     2020-02-24 
 02:34:49.760979074 -0700
 +++ /usr/src/sys/arch/prep/prep/autoconf.c  2020-02-24 
 02:30:18.387739603 -0700
 @@ -117,7 +117,9 @@
  Â Â Â Â Â Â Â Â Â Â Â  device_is_a(dev, "pckbc") || device_is_a(dev, "pckbd") ||
  Â Â Â Â Â Â Â Â Â Â Â  device_is_a(dev, "vga") || device_is_a(dev, "wsdisplay") ||
  Â Â Â Â Â Â Â Â Â Â Â  device_is_a(dev, "wskbd") || device_is_a(dev, "wsmouse") ||
 -Â Â Â Â Â Â Â Â Â Â  device_is_a(dev, "pms") || device_is_a(dev, "cpu"))
 +Â Â Â Â Â Â Â Â Â Â  device_is_a(dev, "pms") || device_is_a(dev, "cpu") ||
 +Â Â Â Â Â Â Â Â Â Â  device_is_a(dev, "nvram") || device_is_a(dev, "isabeep") ||
 +Â Â Â Â Â Â Â Â Â Â  device_is_a(dev, "audio"))
  Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â  return;
 
  Â Â Â Â Â Â Â  if (device_is_a(dev, "pchb")) {
 @@ -218,6 +220,11 @@
  Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â  n += snprintf(devpath + n, sizeof(devpath) - n, "%d",
  Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â  device_unit(dev));
  Â Â Â Â Â Â Â  }
 +Â Â Â Â Â Â  /* Catch buses so they do not show up in the fw-path. DV_BUS? */
 +Â Â Â Â Â Â  if (device_is_a(dev, "isa") || device_is_a(dev, "isapnp") ||
 +Â Â Â Â Â Â Â Â Â Â  device_is_a(dev, "scsibus") || device_is_a(dev, "umass") ||
 +Â Â Â Â Â Â Â Â Â Â  device_is_a(dev, "usb") || device_is_a(dev, "uhub"))
 +Â Â Â Â Â Â Â Â Â Â Â Â Â Â  n = snprintf(devpath, sizeof(devpath), "");
 
  Â Â Â Â Â Â Â  str1 = prop_string_create_cstring(devpath);
  Â Â Â Â Â Â Â  KASSERT(str1 != NULL);