crypto: qat - remove unnecessary list iteration

bwallan · herbertx · commit caa8c508494c · 2015-07-28T15:03:35.000+08:00
There's no need to iterate through the list for instances in the
accel_table since the number of devices is already known in this file.

Signed-off-by: Bruce Allan &lt;bruce.w.allan@intel.com&gt;
Signed-off-by: Tadeusz Struk &lt;tadeusz.struk@intel.com&gt;
Signed-off-by: Herbert Xu &lt;herbert@gondor.apana.org.au&gt;
diff --git a/drivers/crypto/qat/qat_common/adf_dev_mgr.c b/drivers/crypto/qat/qat_common/adf_dev_mgr.c
@@ -182,12 +182,7 @@ int adf_devmgr_verify_id(uint32_t id)
 
 void adf_devmgr_get_num_dev(uint32_t *num)
 {
-	struct list_head *itr;
-
-	*num = 0;
-	list_for_each(itr, &accel_table) {
-		(*num)++;
-	}
+	*num = num_devices;
 }
 
 int adf_dev_in_use(struct adf_accel_dev *accel_dev)

Original file line number	Diff line number	Diff line change
`@@ -182,12 +182,7 @@ int adf_devmgr_verify_id(uint32_t id)`
`182`	`182`
`183`	`183`	`void adf_devmgr_get_num_dev(uint32_t *num)`
`184`	`184`	`{`
`185`		`- struct list_head *itr;`
`186`		`-`
`187`		`- *num = 0;`
`188`		`- list_for_each(itr, &accel_table) {`
`189`		`- (*num)++;`
`190`		`- }`
	`185`	`+ *num = num_devices;`
`191`	`186`	`}`
`192`	`187`
`193`	`188`	`int adf_dev_in_use(struct adf_accel_dev *accel_dev)`