Re: [PATCH] drm/amd/powerplay: fix a signedness bugs

2017-05-18 Thread Eric Huang
Reviewed-by: Eric Huang On 2017-05-16 10:42 AM, Dan Carpenter wrote: Smatch complains about a signedness bug here: vega10_hwmgr.c:4202 vega10_force_clock_level() warn: always true condition '(i >= 0) => (0-u32max >= 0)' Fixes: 7b52db39a4c2 ("drm/amd

drm/amd/powerplay: add dpm force multiple levels on cz/tonga/fiji/polaris (v2)

2016-05-10 Thread Eric Huang
Hi Dan, It is for avoiding the last char of newline. I will change it. Thanks, Eric On 16-05-10 06:51 AM, Dan Carpenter wrote: > Hello Eric Huang, > > The patch 5632708f4452: "drm/amd/powerplay: add dpm force multiple > levels on cz/tonga/fiji/polaris (v2)" from Apr

[PATCH 3/6] drm/amdgpu: add powerplay sclk OD support through sysfs

2016-05-16 Thread Eric Huang
Thanks for your catch, Nils. On 16-05-14 02:27 AM, Nils Wallménius wrote: > Hi Eric, > > A little nitpick below. > > Regards > Nils > > On Fri, May 13, 2016 at 8:48 PM, Alex Deucher > wrote: > >> From: Eric Huang >> >> Add a new sys