diff options
author | Dave Airlie <airlied@redhat.com> | 2009-04-02 19:10:33 -0400 |
---|---|---|
committer | Dave Airlie <airlied@redhat.com> | 2009-04-02 19:10:33 -0400 |
commit | 16456c872e4e3d674dde73f9bdf3f69c0d95e070 (patch) | |
tree | 6a2241babb9703a435f3462cea7a5f3486cf6f65 | |
parent | 3c6fc3521acbee33637e7db803ac3cf3b0e2ff04 (diff) |
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>
-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) | |||
162 | edid_vendor[0] = ((edid->mfg_id[0] & 0x7c) >> 2) + '@'; | 162 | edid_vendor[0] = ((edid->mfg_id[0] & 0x7c) >> 2) + '@'; |
163 | edid_vendor[1] = (((edid->mfg_id[0] & 0x3) << 3) | | 163 | edid_vendor[1] = (((edid->mfg_id[0] & 0x3) << 3) | |
164 | ((edid->mfg_id[1] & 0xe0) >> 5)) + '@'; | 164 | ((edid->mfg_id[1] & 0xe0) >> 5)) + '@'; |
165 | edid_vendor[2] = (edid->mfg_id[2] & 0x1f) + '@'; | 165 | edid_vendor[2] = (edid->mfg_id[1] & 0x1f) + '@'; |
166 | 166 | ||
167 | return !strncmp(edid_vendor, vendor, 3); | 167 | return !strncmp(edid_vendor, vendor, 3); |
168 | } | 168 | } |