diff options
| author |   daniel <daniel@openbsd.org> | 2014-11-05 01:02:10 +0000 |
|---|---|---|
| committer | daniel <daniel@openbsd.org> | 2014-11-05 01:02:10 +0000 |
| commit | 9e38834c0301c06d97e8d83a66ea0d1a2d909e7f (patch) | |
| tree | d81e3ea59a359abdbebaa7003ea9e418005b1941 /sys | |
| parent | i forgot the :wq bit after applying jmcs tweaks. (diff) | |
| download | wireguard-openbsd-9e38834c0301c06d97e8d83a66ea0d1a2d909e7f.tar.xz wireguard-openbsd-9e38834c0301c06d97e8d83a66ea0d1a2d909e7f.zip | |
Functions that say they return void should not try to return something.
ok deraadt@
"Even though I admire the chutzpah of return(void), I agree". ok krw@
Diffstat (limited to 'sys')
| -rw-r--r-- | sys/dev/ic/ami.c | 5 | ||||
| -rw-r--r-- | sys/dev/ic/bwivar.h | 4 |
2 files changed, 5 insertions, 4 deletions
diff --git a/sys/dev/ic/ami.c b/sys/dev/ic/ami.c index 98afef3411f..1b654df2cce 100644 --- a/sys/dev/ic/ami.c +++ b/sys/dev/ic/ami.c @@ -1,4 +1,4 @@ -/* $OpenBSD: ami.c,v 1.229 2014/09/14 14:17:24 jsg Exp $ */ +/* $OpenBSD: ami.c,v 1.230 2014/11/05 01:02:10 daniel Exp $ */ /* * Copyright (c) 2001 Michael Shalayeff @@ -1364,7 +1364,8 @@ ami_scsi_cmd(struct scsi_xfer *xs) cmd = &ccb->ccb_cmd; cmd->acc_cmd = AMI_FLUSH; - return (ami_start_xs(sc, ccb, xs)); + ami_start_xs(sc, ccb, xs); + return; case TEST_UNIT_READY: /* save off sd? after autoconf */ diff --git a/sys/dev/ic/bwivar.h b/sys/dev/ic/bwivar.h index d55a72f9140..fd6e28ef26e 100644 --- a/sys/dev/ic/bwivar.h +++ b/sys/dev/ic/bwivar.h @@ -1,4 +1,4 @@ -/* $OpenBSD: bwivar.h,v 1.30 2014/07/20 11:59:12 stsp Exp $ */ +/* $OpenBSD: bwivar.h,v 1.31 2014/11/05 01:02:10 daniel Exp $ */ /* * Copyright (c) 2007 The DragonFly Project. All rights reserved. @@ -725,7 +725,7 @@ bwi_rf_calc_rssi(struct bwi_mac *_mac, const struct bwi_rxbuf_hdr *_hdr) static __inline void bwi_rf_lo_update(struct bwi_mac *_mac) { - return (_mac->mac_rf.rf_lo_update(_mac)); + _mac->mac_rf.rf_lo_update(_mac); } #define RF_WRITE(mac, ofs, val) bwi_rf_write((mac), (ofs), (val)) |