Re: [PATCH] 2.5.59 : drivers/net/fc/iph5526.c

Valdis.Kletnieks@vt.edu
Fri, 07 Feb 2003 12:23:00 -0500


--==_Exmh_-699087380P
Content-Type: text/plain; charset="us-ascii"
Content-Id: <6157.1044638580.1@turing-police.cc.vt.edu>

On Fri, 07 Feb 2003 12:23:22 EST, Frank Davis said:
>
> +++ linux/drivers/net/fc/iph5526.c 2003-02-07 02:13:43.000000000 -0500

> - for (i = 0; i < clone_list[i].vendor_id != 0; i++)
> + for (i = 0; ((i < clone_list[i].vendor_id) && (clone_list[i].vendor_id
!= 0)); i++)

'i < clone_list[i].vendor_id'?

Currently, clone_list only has 1 vendor listed:
#define PCI_VENDOR_ID_INTERPHASE 0x107e

I suspect this was intended:

for (i = 0; (clone_list[i].vendor_id != 0); i++)

-- 
				Valdis Kletnieks
				Computer Systems Senior Engineer
				Virginia Tech

--==_Exmh_-699087380P Content-Type: application/pgp-signature

-----BEGIN PGP SIGNATURE----- Version: GnuPG v1.2.1 (GNU/Linux) Comment: Exmh version 2.5 07/13/2001

iD8DBQE+Q+t0cC3lWbTT17ARAlJsAJ0eMdNPiNmhMSKTlfK6btm1tkrIZgCdHfzH 3lKFf/Sc1mggWyq9YLBIkt0= =JSqF -----END PGP SIGNATURE-----

--==_Exmh_-699087380P-- - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/