The check for zero sized resources is already done earlier. So don't redo it
authorStefan Reinauer <stepan@coresystems.de>
Wed, 28 Oct 2009 14:57:14 +0000 (14:57 +0000)
committerStefan Reinauer <stepan@openbios.org>
Wed, 28 Oct 2009 14:57:14 +0000 (14:57 +0000)
here. I think we don't ever want to drop the extra check, since it indicates
that the components involved need fixing.

Signed-off-by: Stefan Reinauer <stepan@coresystems.de>
Acked-by: Stefan Reinauer <stepan@coresystems.de>
git-svn-id: svn://svn.coreboot.org/coreboot/trunk@4876 2b7e53f0-3cfb-0310-b3e9-8179ed1497e1

src/devices/device.c

index 2323850ae6c35fb198110506fcafa9a16f31a106..9ce58c52c9420765004b1c4ea683b9340fbb50a5 100644 (file)
@@ -576,8 +576,7 @@ static void constrain_resources(struct device *dev, struct constraints* limits)
                        continue;
 
                /* Is it already outside the limits? */
-               if (res->size && (((res->base + res->size -1) < lim->base) ||
-                                 (res->base > lim->limit)))
+               if (((res->base + res->size -1) < lim->base) || (res->base > lim->limit))
                        continue;
 
                /* Choose to be above or below fixed resources.  This