diff options
| author | Dave Jones <davej@redhat.com> | 2007-01-28 17:50:17 -0500 |
|---|---|---|
| committer | Dave Jones <davej@redhat.com> | 2007-01-28 17:50:17 -0500 |
| commit | 7707ea3b784195315366e6e4b5c73ca6933ff9b0 (patch) | |
| tree | f8ad2fc44e1e851309bf2c22e6886ca6e34e18e4 | |
| parent | 87a17f31a3bc9bf0c7e7493add19ef200e741248 (diff) | |
| download | op-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.c | 10 |
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; |
