linux-kernel - Re: [PATCH v4 0/2] Expand APU2 driver to 3/4/5/6 models

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-next>] [day] [month] [year] [list]

Message-Id: <343F820D-69E1-4120-89DA-980FC78E3656@redfish-solutions.com>
Date:   Thu, 19 Jan 2023 00:17:21 -0700
From:   Philip Prindeville <philipp@...fish-solutions.com>
To:     Hans de Goede <hdegoede@...hat.com>
Cc:     platform-driver-x86@...r.kernel.org, linux-x86_64@...r.kernel.org,
        "Enrico Weigelt, metux IT consult" <info@...ux.net>,
        Ed Wildgoose <lists@...dgooses.com>,
        Andres Salomon <dilinger@...ued.net>,
        Andreas Eberlein <foodeas@...erlein.de>,
        Paul Spooren <paul@...oren.de>, linux-kernel@...r.kernel.org,
        x86@...nel.org
Subject: Re: [PATCH v4 0/2] Expand APU2 driver to 3/4/5/6 models



> On Jan 17, 2023, at 3:25 AM, Hans de Goede <hdegoede@...hat.com> wrote:
> 
> Hi Philip,
> 
> On 1/14/23 00:11, Philip Prindeville wrote:
>> From: Philip Prindeville <philipp@...fish-solutions.com>
>> 
>> Extend PCEngines APU2 driver to derivative models APU3, 4, 5
>> and 6.
>> 
>> Of the five, the APU5 is the most dissimilar.  The APU2 is not
>> derivative of the APU1, and we call that out in the leds-apu
>> driver.  We also clarify the expanded compatibility of the
>> pcengines-apuv2 driver in the Kconfig menu.
>> 
>> We correct some line labeling for GPIO lines 4-6, and we add
>> appropriate definitions to the GPIO lines for the APU5 which
>> is mapped out differently.
>> 
>> The majority of the work was done by Ed Wildgoose.  Philip
>> Prindeville took over the effort to upstream the fixes and
>> made some minor cleanup.
> 
> Thank you both for your work on this.
> 
> In would really like to hear back from Enrico what he thinks
> about these patches. FWIW I'm fine with merging these as is.
> 
> Also note that Enrico is listed as maintainer for the 
> pcengines-apuv2.c file in MAINTAINERS, so you really should
> have Cc-ed him on this.
> 
> Enrico, can you take a look at this series please? You can find
> the 2 actual patches here:
> 
> https://lore.kernel.org/platform-driver-x86/20230113231139.436956-1-philipp@redfish-solutions.com/
> https://lore.kernel.org/platform-driver-x86/20230113231140.437040-1-philipp@redfish-solutions.com/
> 
> Regards,
> 
> Hans


I tried to copy Enrico on earlier revisions of this patch but they bounced so I stopped including him.

-Philip


> 
> 
>> 
>> Philip Prindeville (2):
>>  x86: Support APU5 & APU6 in PCEngines platform driver
>>  x86: Further PCEngines APU4/5/6 cleanup
>> 
>> drivers/leds/leds-apu.c                |   2 +-
>> drivers/platform/x86/Kconfig           |   4 +-
>> drivers/platform/x86/pcengines-apuv2.c | 118 ++++++++++++++++++++++---
>> 3 files changed, 107 insertions(+), 17 deletions(-)
>> 
>