atm: forever loop loading ambassador firmware
authorDan Carpenter <dan.carpenter@oracle.com>
Tue, 27 Nov 2012 07:29:35 +0000 (07:29 +0000)
committerDavid S. Miller <davem@davemloft.net>
Wed, 28 Nov 2012 16:38:11 +0000 (11:38 -0500)
There was a forever loop introduced here when we converted this to
request_firmware() back in 2008.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Acked-by: Chas Williams <chas@cmf.nrl.navy.mil>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/atm/ambassador.c

index 89b30f32ba68b4f1e5cd58fc5911a95213abf163..ff7bb8a42ed62b042bff5f786b94775f8bfb87bc 100644 (file)
@@ -1961,6 +1961,7 @@ static int __devinit ucode_init (loader_block * lb, amb_dev * dev) {
     res = loader_verify(lb, dev, rec);
     if (res)
       break;
+    rec = ihex_next_binrec(rec);
   }
   release_firmware(fw);
   if (!res)