diff options
| author |   Marcin Slusarz <marcin.slusarz@gmail.com> | 2012-11-17 21:33:15 +0100 |
|---|---|---|
| committer |   Ben Skeggs <bskeggs@redhat.com> | 2012-11-19 08:54:20 +1000 |
| commit | 3bb076af2ae571a48465972d5747175cec3564cd (patch) | |
| tree | e12b91324bd928305d2a113faa569cfdcaa6c2dd /drivers/gpu/drm/nouveau/core/subdev/bios/dcb.c | |
| parent | drm/nouveau: add missing pll_calc calls (diff) | |
| download | linux-dev-3bb076af2ae571a48465972d5747175cec3564cd.tar.xz linux-dev-3bb076af2ae571a48465972d5747175cec3564cd.zip | |
drm/nouveau/bios: fix DCB v1.5 parsing
memcmp->nv_strncmp conversion, in addition to name change, should have
inverted the return value.
But nv_strncmp does not act like strncmp - it does not check for string
terminator, returns true/false instead of -1/0/1 and has different
parameters order.
Let's rename it to nv_memcmp and let it act like memcmp.
Signed-off-by: Marcin Slusarz <marcin.slusarz@gmail.com>
Signed-off-by: Ben Skeggs <bskeggs@redhat.com>
Diffstat (limited to '')
| -rw-r--r-- | drivers/gpu/drm/nouveau/core/subdev/bios/dcb.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpu/drm/nouveau/core/subdev/bios/dcb.c b/drivers/gpu/drm/nouveau/core/subdev/bios/dcb.c index 7d750382a833..c51197157749 100644 --- a/drivers/gpu/drm/nouveau/core/subdev/bios/dcb.c +++ b/drivers/gpu/drm/nouveau/core/subdev/bios/dcb.c @@ -64,7 +64,7 @@ dcb_table(struct nouveau_bios *bios, u8 *ver, u8 *hdr, u8 *cnt, u8 *len) } } else if (*ver >= 0x15) { - if (!nv_strncmp(bios, dcb - 7, 7, "DEV_REC")) { + if (!nv_memcmp(bios, dcb - 7, "DEV_REC", 7)) { u16 i2c = nv_ro16(bios, dcb + 2); *hdr = 4; *cnt = (i2c - dcb) / 10; |