summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDave Jones <davej@redhat.com>2007-01-28 17:50:17 -0500
committerDave Jones <davej@redhat.com>2007-01-28 17:50:17 -0500
commit7707ea3b784195315366e6e4b5c73ca6933ff9b0 (patch)
treef8ad2fc44e1e851309bf2c22e6886ca6e34e18e4
parent87a17f31a3bc9bf0c7e7493add19ef200e741248 (diff)
downloadop-kernel-dev-7707ea3b784195315366e6e4b5c73ca6933ff9b0.zip
op-kernel-dev-7707ea3b784195315366e6e4b5c73ca6933ff9b0.tar.gz
[AGPGART] Remove pointless assignment.
No point in clearing local pointers then returning. Also fix up some CodingStyle nits. Signed-off-by: Dave Jones <davej@redhat.com>
-rw-r--r--drivers/char/agp/ati-agp.c10
1 files changed, 5 insertions, 5 deletions
diff --git a/drivers/char/agp/ati-agp.c b/drivers/char/agp/ati-agp.c
index 0994d35..9987dc2 100644
--- a/drivers/char/agp/ati-agp.c
+++ b/drivers/char/agp/ati-agp.c
@@ -124,18 +124,18 @@ static int ati_create_gatt_pages(int nr_tables)
for (i = 0; i < nr_tables; i++) {
entry = kzalloc(sizeof(struct ati_page_map), GFP_KERNEL);
if (entry == NULL) {
- while (i>0) {
- kfree (tables[i-1]);
+ while (i > 0) {
+ kfree(tables[i-1]);
i--;
}
- kfree (tables);
- tables = NULL;
+ kfree(tables);
retval = -ENOMEM;
break;
}
tables[i] = 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.gatt_pages = tables;
OpenPOWER on IntegriCloud