From bb4ab4336d4f6be66639058e70ac73de1f6d646e Mon Sep 17 00:00:00 2001 From: Philippe Saint-Pierre Date: Thu, 2 Jul 2015 13:20:44 -0400 Subject: [PATCH] Matrox accelerant: assignation to B_OK -> condition never true. Assignation to B_OK in if condition means it will never be true. Replace with ==. CID 991072, 991155 and 604410. --- src/add-ons/accelerants/matrox/engine/mga_acc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/add-ons/accelerants/matrox/engine/mga_acc.c b/src/add-ons/accelerants/matrox/engine/mga_acc.c index 74c1ecba04..eaf726d91f 100644 --- a/src/add-ons/accelerants/matrox/engine/mga_acc.c +++ b/src/add-ons/accelerants/matrox/engine/mga_acc.c @@ -44,7 +44,7 @@ status_t gx00_acc_init() /* used for convenience: MACCESS is a write only register! */ uint32 maccess = 0x00000000; /* if we were unable to read PINS, we have to assume something (keeping bit6 zero) */ - if ((si->ps.card_type >= G450) && (si->ps.pins_status = B_OK)) + if ((si->ps.card_type >= G450) && (si->ps.pins_status == B_OK)) { /* b7 v5_mem_type = done by Mark Watson. fixme: still confirm! (unknown bits) */ maccess |= ((((uint32)si->ps.v5_mem_type) & 0x80) >> 1);