Re: [PATCH 1/2] drm/amd/powerplay: add prefix for all powerplay pr_* prints

On 12/23/2016 08:54 PM, Huang Rui wrote: > On Fri, Dec 23, 2016 at 05:32:58PM +0800, Edward O'Callaghan wrote: >> Hi, >> >> I would say drop all the header relocation churn, it distracts away from >> the functional changes in this commit. Also see inline comments. >> > > Yes, if double check

Re: [PATCH 1/2] drm/amd/powerplay: add prefix for all powerplay pr_* prints

Hi, I would say drop all the header relocation churn, it distracts away from the functional changes in this commit. Also see inline comments. With those fixes, Acked-by: Edward O'Callaghan Kindest Regards, Edward. On 12/23/2016 01:45 PM, Huang Rui wrote: >

[PATCH 1/2] drm/amd/powerplay: add prefix for all powerplay pr_* prints

Powerplay will be used them instead of raw printk, and we can dynamic change the debug level with it. The prefix is like below: [ 197.755167] [powerplay] amdgpu: powerplay initialized Suggested-by: Grazvydas Ignotas Signed-off-by: Huang Rui Cc: Arindam