From f5698581ca1723bb59a2068a4c47eea741d7139e Mon Sep 17 00:00:00 2001 From: John Baldwin Date: Wed, 30 Jul 2008 17:21:32 +0000 Subject: [PATCH] SIS_SETBIT() already does a read/modify/write operation, so there isn't a reason to read the register twice. MFC after: 1 week --- sys/pci/if_sis.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/sys/pci/if_sis.c b/sys/pci/if_sis.c index 890224706c9..b22ff8d0232 100644 --- a/sys/pci/if_sis.c +++ b/sys/pci/if_sis.c @@ -2015,8 +2015,7 @@ sis_initl(struct sis_softc *sc) device_printf(sc->sis_dev, "Applying short cable fix (reg=%x)\n", reg); CSR_WRITE_4(sc, NS_PHY_TDATA, 0x00e8); - reg = CSR_READ_4(sc, NS_PHY_DSPCFG); - SIS_SETBIT(sc, NS_PHY_DSPCFG, reg | 0x20); + SIS_SETBIT(sc, NS_PHY_DSPCFG, 0x20); } CSR_WRITE_4(sc, NS_PHY_PAGE, 0); }