svn commit: r349461 - head/sys/sys
Andriy Gapon
avg at FreeBSD.org
Thu Jun 27 15:51:51 UTC 2019
Author: avg
Date: Thu Jun 27 15:51:50 2019
New Revision: 349461
URL: https://svnweb.freebsd.org/changeset/base/349461
Log:
revert r349460, printf -> KASSERT in bus.h, until I can fix it
I tested only kernel builds naively assuming that sys/bus.h cannot
affect userland builds.
Pointyhat to: me
Modified:
head/sys/sys/bus.h
Modified: head/sys/sys/bus.h
==============================================================================
--- head/sys/sys/bus.h Thu Jun 27 15:46:06 2019 (r349460)
+++ head/sys/sys/bus.h Thu Jun 27 15:51:50 2019 (r349461)
@@ -35,7 +35,6 @@
#include <machine/_bus.h>
#include <sys/_bus_dma.h>
#include <sys/ioccom.h>
-#include <sys/systm.h>
/**
* @defgroup NEWBUS newbus - a generic framework for managing devices
@@ -814,9 +813,12 @@ static __inline type varp ## _get_ ## var(device_t dev
int e; \
e = BUS_READ_IVAR(device_get_parent(dev), dev, \
ivarp ## _IVAR_ ## ivar, &v); \
- KASSERT(e == 0, ("%s failed for %s on bus %s, error = %d", \
- __func__, device_get_nameunit(dev), \
- device_get_nameunit(device_get_parent(dev)), e)); \
+ if (e != 0) { \
+ device_printf(dev, "failed to read ivar " \
+ __XSTRING(ivarp ## _IVAR_ ## ivar) " on bus %s, " \
+ "error = %d\n", \
+ device_get_nameunit(device_get_parent(dev)), e); \
+ } \
return ((type) v); \
} \
\
@@ -826,9 +828,12 @@ static __inline void varp ## _set_ ## var(device_t dev
int e; \
e = BUS_WRITE_IVAR(device_get_parent(dev), dev, \
ivarp ## _IVAR_ ## ivar, v); \
- KASSERT(e == 0, ("%s failed for %s on bus %s, error = %d", \
- __func__, device_get_nameunit(dev), \
- device_get_nameunit(device_get_parent(dev)), e)); \
+ if (e != 0) { \
+ device_printf(dev, "failed to write ivar " \
+ __XSTRING(ivarp ## _IVAR_ ## ivar) " on bus %s, " \
+ "error = %d\n", \
+ device_get_nameunit(device_get_parent(dev)), e); \
+ } \
}
/**
More information about the svn-src-all
mailing list