linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Fa-Iz Faadhillah Ibrahim <faiz.faadhillah@gmail.com>
To: Armin Wolf <W_Armin@gmx.de>
Cc: basak.sb2006@gmail.com, hdegoede@redhat.com,
	ilpo.jarvinen@linux.intel.com, jlee@suse.com, kuurtb@gmail.com,
	linux-kernel@vger.kernel.org,
	platform-driver-x86@vger.kernel.org, rayanmargham4@gmail.com
Subject: Re: [RFC PATCH v2 0/3] platform/x86: acer-wmi: Add fan control support
Date: Tue, 02 Sep 2025 11:39:09 +0700	[thread overview]
Message-ID: <5007139.GXAFRqVoOG@cacher> (raw)
In-Reply-To: <5b7f6b40-8d4f-470e-ae84-b51d4177af39@gmx.de>

On Monday, September 1, 2025 2:33:53 AM Western Indonesia Time Armin Wolf 
wrote:
> Am 23.08.25 um 03:27 schrieb Fa-Iz Faadhillah Ibrahim:
> > On 8/22/25 7:02 PM, Armin Wolf wrote:
> >> Am 15.08.25 um 08:17 schrieb Fa-Iz Faadhillah Ibrahim:
> >>>> > This experimental patch series aims to add fan control support to
> >>>> 
> >>>> the > acer-wmi driver. The patches are compile-tested only and need
> >>>> to be > tested on real hardware to verify that they actually work.
> >>>> 
> >>>> > > I CCed two users who requested support for this feature. I
> >>>> 
> >>>> would be > very happy if both of you could test those patches and
> >>>> report back. > > I am ready to help you both with compiling a
> >>>> custom linux kernel for > testing this series.
> >>>> Any updates from the two people with Acer hardware?
> >>>> 
> >>>> Thanks,
> >>>> Armin Wolf
> >>> 
> >>> Hello, I've tried your patch on my Predator Helios Neo 16 (PHN16-72)
> >>> using quirks from PH16-72 and it looks like fan control works just
> >>> fine.
> >>> On other note, can you please add PHN16-72 to the quirk table?
> >>> if you need anything to test please do notify.
> >>> 
> >>> Thanks,
> >>> Fa-Iz Faadhillah Ibrahim
> >> 
> >> Hi,
> >> 
> >> sorry for taking a bit long to respond, i am currently on vacation.
> >> Nice to know that the experimental patches work :).
> >> Can you send me the output of "acpidump" on your machine?
> > 
> > Hello,
> > 
> > Sorry for disturbing your vacation, hope you're having a great time :).
> > Output of "acpidump" attached below.
> > 
> > P.S. Sorry for the messed up formatting before, first time here so I'm
> > still learning how this works, hopefully it'll be okay now.
> > 
> > Thanks,
> > Fa-Iz Faadhillah Ibrahim
> 
> Thanks.
> 
> Would it be OK for you if i send the non-RFC patches once i am back home
> (around 15.09.)? The reason for this is that i made some improvements to
> the RFC patches, but those improved patches are only stored on my develop
> machine at home.
Of course, I'll try it once you send the patches.
I'm wondering, where should I send a feature request and a bug report? it 
would be nice to have the battery limit function upstreamed, and I discovered 
some bugs here and there as well. 

Thanks,
Fa-Iz Faadhillah Ibrahim
> 
> Thanks,
> Armin Wolf
> 
> >> Thanks,
> >> Armin Wolf
> >> 
> >>>> > Changes since v2: > - remove duplicate include and replace
> >>>> 
> >>>> hwmon_pwm_mode with > hwmon_pwm_enable in second patch > > Armin
> >>>> Wolf (3): > platform/x86: acer-wmi: Fix setting of fan behavior >
> >>>> platform/x86: acer-wmi: Add fan control support > platform/x86:
> >>>> acer-wmi: Enable fan control for PH16-72 and PT14-51 > >
> >>>> drivers/platform/x86/acer-wmi.c | 298
> >>>> +++++++++++++++++++++++++++++--- > 1 file changed, 273
> >>>> insertions(+), 25 deletions(-) > > -- > 2.39.5 > >
> >>>> -----------------------------------------------------------------------
> >>>> -





      reply	other threads:[~2025-09-02  4:39 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-02-15 17:45 [RFC PATCH v2 0/3] platform/x86: acer-wmi: Add fan control support Armin Wolf
2025-02-15 17:45 ` [RFC PATCH v2 1/3] platform/x86: acer-wmi: Fix setting of fan behavior Armin Wolf
2025-02-15 18:51   ` Kurt Borja
2025-02-15 19:35     ` Armin Wolf
2025-02-15 17:45 ` [RFC PATCH v2 2/3] platform/x86: acer-wmi: Add fan control support Armin Wolf
2025-02-15 17:45 ` [RFC PATCH v2 3/3] platform/x86: acer-wmi: Enable fan control for PH16-72 and PT14-51 Armin Wolf
2025-03-05  0:23 ` [RFC PATCH v2 0/3] platform/x86: acer-wmi: Add fan control support Armin Wolf
2025-03-09 12:51   ` Rayan Margham
2025-03-11 21:53     ` Armin Wolf
2025-03-13 18:21       ` Rayan Margham
2025-03-17  0:18         ` Armin Wolf
2025-08-15  6:17   ` Fa-Iz Faadhillah Ibrahim
2025-08-22 12:02     ` Armin Wolf
     [not found]       ` <1bb3edd6-405f-4615-831f-301dba525bc3@gmail.com>
2025-08-31 19:33         ` Armin Wolf
2025-09-02  4:39           ` Fa-Iz Faadhillah Ibrahim [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=5007139.GXAFRqVoOG@cacher \
    --to=faiz.faadhillah@gmail.com \
    --cc=W_Armin@gmx.de \
    --cc=basak.sb2006@gmail.com \
    --cc=hdegoede@redhat.com \
    --cc=ilpo.jarvinen@linux.intel.com \
    --cc=jlee@suse.com \
    --cc=kuurtb@gmail.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=platform-driver-x86@vger.kernel.org \
    --cc=rayanmargham4@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).