diff options
| author | Dave Airlie <airlied@redhat.com> | 2009-04-03 09:10:33 +1000 |
|---|---|---|
| committer | Dave Airlie <airlied@redhat.com> | 2009-04-03 09:10:33 +1000 |
| commit | 16456c872e4e3d674dde73f9bdf3f69c0d95e070 (patch) | |
| tree | 6a2241babb9703a435f3462cea7a5f3486cf6f65 /drivers/gpu | |
| parent | 3c6fc3521acbee33637e7db803ac3cf3b0e2ff04 (diff) | |
| download | op-kernel-dev-16456c872e4e3d674dde73f9bdf3f69c0d95e070.zip op-kernel-dev-16456c872e4e3d674dde73f9bdf3f69c0d95e070.tar.gz | |
drm: fix typo in edid vendor parsing.
Should be,
edid_vendor[2] = (edid->mfg_id[1] & 0x1f) + '@';
Since vendor ID has only two bytes only, I am somewhat surprised why gcc
doesn't complain this.
Reported-by: Guo, Chaohong <chaohong.guo@intel.com>
Signed-off-by: Dave Airlie <airlied@redhat.com>
Diffstat (limited to 'drivers/gpu')
| -rw-r--r-- | drivers/gpu/drm/drm_edid.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c index c674000..2c4a340 100644 --- a/drivers/gpu/drm/drm_edid.c +++ b/drivers/gpu/drm/drm_edid.c @@ -162,7 +162,7 @@ static bool edid_vendor(struct edid *edid, char *vendor) edid_vendor[0] = ((edid->mfg_id[0] & 0x7c) >> 2) + '@'; edid_vendor[1] = (((edid->mfg_id[0] & 0x3) << 3) | ((edid->mfg_id[1] & 0xe0) >> 5)) + '@'; - edid_vendor[2] = (edid->mfg_id[2] & 0x1f) + '@'; + edid_vendor[2] = (edid->mfg_id[1] & 0x1f) + '@'; return !strncmp(edid_vendor, vendor, 3); } |
