lk/linux-kernel-test
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

[AGPGART] Remove pointless assignment.

Browse Source 
No point in clearing local pointers then returning.
Also fix up some CodingStyle nits.

Signed-off-by: Dave Jones <davej@redhat.com>
This commit is contained in:
Dave Jones
2007-01-28 17:50:17 -05:00
parent 87a17f31a3
commit 7707ea3b78
1 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
drivers/char/agp/ati-agp.c
View File

@@ -124,18 +124,18 @@ static int ati_create_gatt_pages(int nr_tables)
for (i = 0; i < nr_tables; i++) { for (i = 0; i < nr_tables; i++) {
entry = kzalloc(sizeof(struct ati_page_map), GFP_KERNEL); entry = kzalloc(sizeof(struct ati_page_map), GFP_KERNEL);
if (entry == NULL) { if (entry == NULL) {
while (i>0) { while (i > 0) {
kfree (tables[i-1]); kfree(tables[i-1]);
i--; i--;
} }
kfree (tables); kfree(tables);
tables = NULL;
retval = -ENOMEM; retval = -ENOMEM;
break; break;
} }
tables[i] = entry; tables[i] = entry;
retval = ati_create_page_map(entry); retval = ati_create_page_map(entry);
if (retval != 0) break; if (retval != 0)
break;
} }
ati_generic_private.num_tables = nr_tables; ati_generic_private.num_tables = nr_tables;
ati_generic_private.gatt_pages = tables; ati_generic_private.gatt_pages = tables;