Received-SPF: pass (google.com: domain of linux-kernel+bounces-83018-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1;
From: =?UTF-8?q?Ilpo=20J=C3=A4rvinen?= <ilpo.jarvinen@linux.intel.com>
Date: Tue, 27 Feb 2024 12:18:16 +0200 (EET)
To: Szilard Fabian <szfabian@bluemarch.art>
cc: LKML <linux-kernel@vger.kernel.org>, platform-driver-x86@vger.kernel.org, 
    jwoithe@just42.net, Hans de Goede <hdegoede@redhat.com>, W_Armin@gmx.de
Subject: Re: [PATCH v4] platform/x86/fujitsu-laptop: Add battery charge
 control support
In-Reply-To: <170834440647.4050.13047961348645894978.b4-ty@linux.intel.com>
Message-ID: <a4b35e55-1f29-6c98-745c-df6896e49270@linux.intel.com>
References: <20240129163502.161409-2-szfabian@bluemarch.art> <20240129175714.164326-2-szfabian@bluemarch.art> <20240207023031.56805-2-szfabian@bluemarch.art> <20240215203012.228758-2-szfabian@bluemarch.art>
 <170834440647.4050.13047961348645894978.b4-ty@linux.intel.com>
Precedence: bulk
MIME-Version: 1.0
Content-Type: multipart/mixed; BOUNDARY="8323328-957297465-1709028876=:1099"
Content-ID: <8b80bf93-d78a-388a-e88d-a911f9418556@linux.intel.com>

  This message is in MIME format.  The first part should be readable text,
  while the remaining parts are likely unreadable without MIME-aware tools.

--8323328-957297465-1709028876=:1099
Content-Type: text/plain; CHARSET=ISO-8859-15
Content-Transfer-Encoding: QUOTED-PRINTABLE
Content-ID: <20edec04-805c-ce6f-95c4-8d07e40e6e6f@linux.intel.com>

On Mon, 19 Feb 2024, Ilpo J=E4rvinen wrote:

> On Thu, 15 Feb 2024 20:31:43 +0000, Szilard Fabian wrote:
>=20
> > This patch adds battery charge control support on Fujitsu notebooks
> > via the S006 method of the FUJ02E3 ACPI device. With this method it's
> > possible to set charge_control_end_threshold between 50 and 100%.
> >=20
> > Tested on Lifebook E5411 and Lifebook U728. Sadly I can't test this
> > patch on a dual battery one, but I didn't find any clue about
> > independent battery charge control on dual battery Fujitsu notebooks
> > either. And by that I mean checking the DSDT table of various Lifebook
> > notebooks and reverse engineering FUJ02E3.dll.
> >=20
> > [...]
>=20
>=20
> Thank you for your contribution, it has been applied to my local
> review-ilpo branch. Note it will show up in the public
> platform-drivers-x86/review-ilpo branch only once I've pushed my
> local branch there, which might take a while.
>=20
> The list of commits applied:
> [1/1] platform/x86/fujitsu-laptop: Add battery charge control support
>       commit: 9b716ef48c90446b8d6d86726c6c3f0141e53091

Hi,

Can you please take a look at the two build failures LKP has brought up.=20
They looked like some depends on and/or select clauses missing from=20
Kconfig.

I'll be merging the patches that fix those into the original commit so=20
don't spend too much fine-tuning the commit message.

--=20
 i.
--8323328-957297465-1709028876=:1099--