linux-kernel - Re: [PATCH v2 0/2] Device managed platform_profile

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <687daa3a-af4e-f959-aeb9-43fccd1b8989@linux.intel.com>
Date: Tue, 7 Jan 2025 17:48:19 +0200 (EET)
From: Ilpo Järvinen <ilpo.jarvinen@...ux.intel.com>
To: Kurt Borja <kuurtb@...il.com>
cc: platform-driver-x86@...r.kernel.org, W_Armin@....de, 
    Hans de Goede <hdegoede@...hat.com>, lenb@...nel.org, 
    linux-acpi@...r.kernel.org, LKML <linux-kernel@...r.kernel.org>, 
    mario.limonciello@....com, Mark Pearson <mpearson-lenovo@...ebb.ca>, 
    "Rafael J. Wysocki" <rafael@...nel.org>, soyer@....hu
Subject: Re: [PATCH v2 0/2] Device managed platform_profile_register()

On Tue, 31 Dec 2024, Kurt Borja wrote:
> On Mon, Dec 30, 2024 at 08:31:28PM +0200, Ilpo Järvinen wrote:
> > On Tue, 24 Dec 2024, Kurt Borja wrote:
> > 
> > > Hi :)
> > > 
> > > This is meant to be merged on the pdx86 tree.
> > > 
> > > ~ Kurt
> > > 
> > > v2:
> > >  - Replaced convoluted cast with intermediate variable (1/2)
> > >  - Restored dropped empty line (1/2)
> > >  - Couldn't incorporate Armin's second comment. I probably didn't
> > >    understand it (1/2)
> > > v1: 
> > >  - https://lore.kernel.org/platform-driver-x86/20241221070817.3764-2-kuurtb@gmail.com
> > > 
> > > Kurt Borja (2):
> > >   ACPI: platform_profile: Add devm_platform_profile_register()
> > >   alienware-wmi: Use devm_platform_profile_register()
> > > 
> > >  drivers/acpi/platform_profile.c           | 29 +++++++++++++++++++++++
> > >  drivers/platform/x86/dell/alienware-wmi.c | 10 +-------
> > >  include/linux/platform_profile.h          |  1 +
> > >  3 files changed, 31 insertions(+), 9 deletions(-)
> > 
> > Thanks. I've now applied these.
> > 
> > The first patch is already in the for-next branch and the second is 
> > currently in the review-ilpo-next branch (as I wanted to retain ability to 
> > easily separate changes into platform_profile.c from the rest, they go to 
> > their own branch first).
> 
> Thanks Ilpo!
> 
> Should I rebase the alienware-wmi rework patch series on top of
> review-ilpo-next in v3? Currently it's on top of for-next branch.

Hi Kurt,

You've probably seen it by now but in general, the content in those 
review-ilpo-* branches is just a staging area to what will become the 
for-next or fixes branch once LKP has been able to build test the changes. 
If no issues appear, they'll become content of the for-next or fixes 
branch as is.

In case of problems, I will rebase/edit/drop patches in the 
review-ilpo-* branches with a relatively low bar so it might not always 
be fast-forwardable but other than that minor annoyance, I see no issue in 
basing your work on top of those branches (in particular, if you know 
there are going to be conflicts).

I might rebase also fixes and for-next at times, but I try to avoid 
having to do that.

-- 
 i.