diff options
| author | Theo de Raadt <deraadt@cvs.openbsd.org> | 2013-11-26 20:33:21 +0000 |
|---|---|---|
| committer | Theo de Raadt <deraadt@cvs.openbsd.org> | 2013-11-26 20:33:21 +0000 |
| commit | adaa84b5b57cf328ccf6e345b38ec6706c692895 (patch) | |
| tree | 7981ac434a72299baff1a6d37d0dc095dac8ffec /sys/dev/pci/tcpcib.c | |
| parent | a259a6aeceab099f25670e0fbdd7d90fa1ec9bfa (diff) | |
1 << 31 cleanup. Eitan Adler pointed out that there has been a
resurrection of the bad idiom in the tree.
sufficient review by miod, kettenis, tedu
Diffstat (limited to 'sys/dev/pci/tcpcib.c')
| -rw-r--r-- | sys/dev/pci/tcpcib.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sys/dev/pci/tcpcib.c b/sys/dev/pci/tcpcib.c index 0b6f93d56a3..5f1be2ec10c 100644 --- a/sys/dev/pci/tcpcib.c +++ b/sys/dev/pci/tcpcib.c @@ -1,4 +1,4 @@ -/* $OpenBSD: tcpcib.c,v 1.5 2013/05/30 16:15:02 deraadt Exp $ */ +/* $OpenBSD: tcpcib.c,v 1.6 2013/11/26 20:33:17 deraadt Exp $ */ /* * Copyright (c) 2012 Matt Dainty <matt@bodgit-n-scarper.com> @@ -220,7 +220,7 @@ tcpcib_attach(struct device *parent, struct device *self, void *aux) reg = pci_conf_read(pa->pa_pc, pa->pa_tag, E600_LPC_WDTBA); wdtbase = reg & 0xffff; sc->sc_wdt_iot = pa->pa_iot; - if (reg & (1 << 31) && wdtbase) { + if (reg & (1U << 31) && wdtbase) { if (PCI_MAPREG_IO_ADDR(wdtbase) == 0 || bus_space_map(sc->sc_wdt_iot, PCI_MAPREG_IO_ADDR(wdtbase), E600_WDT_SIZE, 0, &sc->sc_wdt_ioh)) { |