All of lore.kernel.org
 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: 16+ 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
2025-08-23  1:27       ` Fa-Iz Faadhillah Ibrahim
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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.