diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2011-10-18 02:07:51 -0400 |
|---|---|---|
| committer | Ben Skeggs <bskeggs@redhat.com> | 2011-11-09 17:59:43 -0500 |
| commit | ef5ced4bfe48ba2acb16e867ceb9c473dd0ef192 (patch) | |
| tree | 5e3323ee6add369c79d464f34fc65c5761c86dff /drivers/gpu | |
| parent | 12b6d9d881ccb64a833d53565a3579e12ab1d026 (diff) | |
drm/nouveau: testing the wrong variable
memtimings is a valid pointer here, the intent was to test for
kcalloc() failure.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Ben Skeggs <bskeggs@redhat.com>
Diffstat (limited to 'drivers/gpu')
| -rw-r--r-- | drivers/gpu/drm/nouveau/nouveau_perf.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpu/drm/nouveau/nouveau_perf.c b/drivers/gpu/drm/nouveau/nouveau_perf.c index 9f178aa94162..33d03fbf00df 100644 --- a/drivers/gpu/drm/nouveau/nouveau_perf.c +++ b/drivers/gpu/drm/nouveau/nouveau_perf.c | |||
| @@ -239,7 +239,7 @@ nouveau_perf_init(struct drm_device *dev) | |||
| 239 | if(version == 0x15) { | 239 | if(version == 0x15) { |
| 240 | memtimings->timing = | 240 | memtimings->timing = |
| 241 | kcalloc(entries, sizeof(*memtimings->timing), GFP_KERNEL); | 241 | kcalloc(entries, sizeof(*memtimings->timing), GFP_KERNEL); |
| 242 | if(!memtimings) { | 242 | if (!memtimings->timing) { |
| 243 | NV_WARN(dev,"Could not allocate memtiming table\n"); | 243 | NV_WARN(dev,"Could not allocate memtiming table\n"); |
| 244 | return; | 244 | return; |
| 245 | } | 245 | } |
