Hello,

I'm writing to you after discus= sing a fancontrol issue on the lm-sensors issue tracker. I have a AMD Ryzen= 5 2400G APU (with integrated graphics). The amdgpu driver is loaded and al= so creates a fan1_input attribute. However there is no separate GPU fan, th= ere is only the CPU (APU) fan which is controlled by the motherboard. The f= an1_input also seems to give some invalid readouts. I have not yet reached = the bottom of this investigation, but so far my conclusion is that this cau= ses fancontrol to choke up.

Spec= ifically the comment by Guenter Roeck here https://github.com/l= m-sensors/lm-sensors/issues/144#issuecomment-442165451 describes the fi= ndings regarding the amdgpu driver.

I'm writing to notify you about this problem, perhaps there can be = some improvement in the amdgpu driver. I'm not really knowledgeable to = discuss the workings of these drivers but i can provide debug output if nee= ded.

Best regards,

Philippe

--000000000000069a7b057bbbf360-- From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pg1-f195.google.com ([209.85.215.195]:37972 "EHLO mail-pg1-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727307AbeK2HDX (ORCPT ); Thu, 29 Nov 2018 02:03:23 -0500 Received: by mail-pg1-f195.google.com with SMTP id g189so9980564pgc.5 for ; Wed, 28 Nov 2018 12:00:35 -0800 (PST) Subject: Re: amdgpu creating fan attribute for integrated graphics To: "Deucher, Alexander" , Philippe L , "Koenig, Christian" , "Zhou, David(ChunMing)" Cc: "linux-hwmon@vger.kernel.org" References: From: Guenter Roeck Message-ID: <4fac9995-cca2-48ad-0e65-badad14e0cde@roeck-us.net> Date: Wed, 28 Nov 2018 12:00:31 -0800 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=windows-1252; format=flowed Content-Language: en-US Content-Transfer-Encoding: base64 Sender: linux-hwmon-owner@vger.kernel.org List-Id: linux-hwmon@vger.kernel.org T24gMTEvMjgvMTggMTA6NTMgQU0sIERldWNoZXIsIEFsZXhhbmRlciB3cm90ZToNCj4gSSBi ZWxpZXZlIHRoZSBhdHRhY2hlZCBwYXRjaCBzaG91bGQgZml4IHRoZSBpc3N1ZS4NCj4gDQoN ClRoZXJlIGlzIGFsc28gYSBwb3dlciBhdHRyaWJ1dGUgd2hpY2ggcmV0dXJucyAtRUlOVkFM LCBwbHVzIGlmIEkgcmVtZW1iZXINCmNvcnJlY3RseSBzb21lIHZvbHRhZ2UgYXR0cmlidXRl cy4NCg0KR3VlbnRlcg0KDQo+IA0KPiBBbGV4DQo+IA0KPiAtLS0tLS0tLS0tLS0tLS0tLS0t LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0N Cj4gKkZyb206KiBQaGlsaXBwZSBMIDxmbGlwMTAxQGdtYWlsLmNvbT4NCj4gKlNlbnQ6KiBX ZWRuZXNkYXksIE5vdmVtYmVyIDI4LCAyMDE4IDExOjIwOjM4IEFNDQo+ICpUbzoqIERldWNo ZXIsIEFsZXhhbmRlcjsgS29lbmlnLCBDaHJpc3RpYW47IFpob3UsIERhdmlkKENodW5NaW5n KQ0KPiAqQ2M6KiBHdWVudGVyIFJvZWNrOyBsaW51eC1od21vbkB2Z2VyLmtlcm5lbC5vcmcN Cj4gKlN1YmplY3Q6KiBhbWRncHUgY3JlYXRpbmcgZmFuIGF0dHJpYnV0ZSBmb3IgaW50ZWdy YXRlZCBncmFwaGljcw0KPiBIZWxsbywNCj4gDQo+IEknbSB3cml0aW5nIHRvIHlvdSBhZnRl ciBkaXNjdXNzaW5nIGEgZmFuY29udHJvbCBpc3N1ZSBvbiB0aGUgbG0tc2Vuc29ycyBpc3N1 ZSB0cmFja2VyLiBJIGhhdmUgYSBBTUQgUnl6ZW4gNSAyNDAwRyBBUFUgKHdpdGggaW50ZWdy YXRlZCBncmFwaGljcykuIFRoZSBhbWRncHUgZHJpdmVyIGlzIGxvYWRlZCBhbmQgYWxzbyBj cmVhdGVzIGEgZmFuMV9pbnB1dCBhdHRyaWJ1dGUuIEhvd2V2ZXIgdGhlcmUgaXMgbm8gc2Vw YXJhdGUgR1BVIGZhbiwgdGhlcmUgaXMgb25seSB0aGUgQ1BVIChBUFUpIGZhbiB3aGljaCBp cyBjb250cm9sbGVkIGJ5IHRoZSBtb3RoZXJib2FyZC4gVGhlIGZhbjFfaW5wdXQgYWxzbyBz ZWVtcyB0byBnaXZlIHNvbWUgaW52YWxpZCByZWFkb3V0cy4gSSBoYXZlIG5vdCB5ZXQgcmVh Y2hlZCB0aGUgYm90dG9tIG9mIHRoaXMgaW52ZXN0aWdhdGlvbiwgYnV0IHNvIGZhciBteSBj b25jbHVzaW9uIGlzIHRoYXQgdGhpcyBjYXVzZXMgZmFuY29udHJvbCB0byBjaG9rZSB1cC4N Cj4gDQo+IFNwZWNpZmljYWxseSB0aGUgY29tbWVudCBieSBHdWVudGVyIFJvZWNrIGhlcmUg aHR0cHM6Ly9naXRodWIuY29tL2xtLXNlbnNvcnMvbG0tc2Vuc29ycy9pc3N1ZXMvMTQ0I2lz c3VlY29tbWVudC00NDIxNjU0NTEgZGVzY3JpYmVzIHRoZSBmaW5kaW5ncyByZWdhcmRpbmcg dGhlIGFtZGdwdSBkcml2ZXIuDQo+IA0KPiBJJ20gd3JpdGluZyB0byBub3RpZnkgeW91IGFi b3V0IHRoaXMgcHJvYmxlbSwgcGVyaGFwcyB0aGVyZSBjYW4gYmUgc29tZSBpbXByb3ZlbWVu dCBpbiB0aGUgYW1kZ3B1IGRyaXZlci4gSSdtIG5vdCByZWFsbHkga25vd2xlZGdlYWJsZSB0 byBkaXNjdXNzIHRoZSB3b3JraW5ncyBvZiB0aGVzZSBkcml2ZXJzIGJ1dCBpIGNhbiBwcm92 aWRlIGRlYnVnIG91dHB1dCBpZiBuZWVkZWQuDQo+IA0KPiBCZXN0IHJlZ2FyZHMsDQo+IFBo aWxpcHBlDQoNCg==