[amd-staging-drm-next] regression - no fan info (sensors) on RX580

[Dieter Nützel]

Sorry Rex,

after return from our vacation,
I've tested latest amd-staging-drm-next (e5f6a57e350a)
but it is NOT solved on my RX580.
I'll try bisecting if I find some more time in the coming days.

amdgpu-pci-0100
Adapter: PCI adapter
temp1:        +27.0°C  (crit =  +0.0°C, hyst =  +0.0°C)

01:00.0 VGA compatible controller: Advanced Micro Devices, Inc. 
[AMD/ATI] Ellesmere [Radeon RX 470/480/570/580] (rev e7) (prog-if 00 
[VGA controller])
         Subsystem: Sapphire Technology Limited Radeon RX 570

[    36.740] (--) AMDGPU(0): Chipset: "Radeon RX 580 Series" (ChipID = 
0x67df)

Thanks,
Dieter

Am 30.09.2017 05:09, schrieb Zhu, Rex:
> Yes, caused by the commit e37a7b4088da
> ("drm/amd/powerplay: tidy up ret checks in amd_powerplay.c")
> 
> Replace error when split patches.
> 
> Have sent the fix patch.
> Please review.
> 
> Best Regards
> Rex
> 
> 
> -----Original Message-----
> From: Alex Deucher [mailto:alexdeucher at gmail.com]
> Sent: Friday, September 29, 2017 10:11 PM
> To: Dieter Nützel; Zhu, Rex
> Cc: amd-devel; DRI Devel; Wentland, Harry; Michel Dänzer
> Subject: Re: [amd-staging-drm-next] regression - no fan info (sensors) 
> on RX580
> 
> Rex, probably related to the recent cleanups in powerplay.
> 
> On Fri, Sep 29, 2017 at 10:09 AM, Dieter Nützel <Dieter at nuetzel-hh.de> 
> wrote:
>> Hello all,
>> 
>> since latest update
>> 
>> 1d7da702e70d3c27408a3bb312c71d6be9f7bebe
>> drm/amd/powerplay: fix spelling mistake: "dividable" -> "divisible"
>> 
>> I didn't get fan info with my RX580 (Polaris21) any longer.
>> 
>> Worked with this commit:
>> 
>> 786df0b89fe5a0b405d4de0a1ce03003c0743ec3
>> drm/amd/display: fix pflip irq registor for raven
>> 
>> Sorry, I do not have full time for bisect, because we are on way to
>> our vacation.
>> 
>> Maybe in the evening (only a few commits).
>> 
>> Greetings,
>> Dieter