diff options
author | Dave Airlie <airlied@redhat.com> | 2009-04-03 01:10:33 +0200 |
---|---|---|
committer | Dave Airlie <airlied@redhat.com> | 2009-04-03 01:10:33 +0200 |
commit | 16456c872e4e3d674dde73f9bdf3f69c0d95e070 (patch) | |
tree | 6a2241babb9703a435f3462cea7a5f3486cf6f65 /drivers/gpu | |
parent | DRM: drm_crtc_helper.h doesn't actually need i2c.h (diff) | |
download | linux-16456c872e4e3d674dde73f9bdf3f69c0d95e070.tar.xz linux-16456c872e4e3d674dde73f9bdf3f69c0d95e070.zip |
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 c67400067b85..2c4a340e682f 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); } |