[patch] drm/amd/powerplay: precedence bug in init_non_clock_fields()

2016-01-04 Thread Dan Carpenter
The cast to uint8_t happens before the right shift so this always sets .m3arb to zero. The cast is actually a no-op so we can remove it. Fixes: 3bace3591493 ('drm/amd/powerplay: add hardware manager sub-component') Signed-off-by: Dan Carpenter --- Static analysis stuff. Not tested. diff --git

[patch] drm/amd/powerplay: precedence bug in init_non_clock_fields()

2016-01-04 Thread Alex Deucher
On Mon, Jan 4, 2016 at 3:44 PM, Dan Carpenter wrote: > The cast to uint8_t happens before the right shift so this always sets > .m3arb to zero. The cast is actually a no-op so we can remove it. > > Fixes: 3bace3591493 ('drm/amd/powerplay: add hardware manager sub-component') > Signed-off-by: Dan