[Bug 30718] New: evergreen.c bad cast/shift
bugzilla-daemon at freedesktop.org
bugzilla-daemon at freedesktop.org
Fri Oct 8 17:01:37 PDT 2010
https://bugs.freedesktop.org/show_bug.cgi?id=30718
Summary: evergreen.c bad cast/shift
Product: DRI
Version: unspecified
Platform: Other
OS/Version: Linux (All)
Status: NEW
Severity: normal
Priority: medium
Component: DRM/Radeon
AssignedTo: dri-devel at lists.freedesktop.org
ReportedBy: freedesktop at treblig.org
Hi,
Sparse spotted the following and it does look like the wrong order of cast
and shift:
drivers/gpu/drm/radeon/evergreen.c:1140:78: warning: right shift by bigger than
source value
efuse_box_bit_127_124 = (u8)(efuse_straps_3 & 0xF0000000) >>
28;
I think I agree with it that in that the value is truncated to u8 before being
shifted down by 28 and therefore always being 0. So easiest fix I guess is:
efuse_box_bit_127_124 = (u8)((efuse_straps_3 & 0xF0000000)
>>28);
Not sure what the effect of this would be, it seems to be in some very device
specific code.
Dave
--
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the dri-devel
mailing list