2023-01-19 07:42:59

by Philip Prindeville

[permalink] [raw]
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 <[email protected]> wrote:
>
> Hi Philip,
>
> On 1/14/23 00:11, Philip Prindeville wrote:
>> From: Philip Prindeville <[email protected]>
>>
>> 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/[email protected]/
> https://lore.kernel.org/platform-driver-x86/[email protected]/
>
> 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(-)
>>
>


2023-01-19 10:58:05

by Hans de Goede

[permalink] [raw]
Subject: Re: [PATCH v4 0/2] Expand APU2 driver to 3/4/5/6 models

Hi,

On 1/19/23 08:17, Philip Prindeville wrote:
>
>
>> On Jan 17, 2023, at 3:25 AM, Hans de Goede <[email protected]> wrote:
>>
>> Hi Philip,
>>
>> On 1/14/23 00:11, Philip Prindeville wrote:
>>> From: Philip Prindeville <[email protected]>
>>>
>>> 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/[email protected]/
>> https://lore.kernel.org/platform-driver-x86/[email protected]/
>>
>> Regards,
>>
>> Hans
>
>
> I tried to copy Enrico on earlier revisions of this patch but they bounced so I stopped including him.

Weird I don't think I have received any bounces for Enrico.

Regards,

Hans




>>> 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(-)
>>>
>>
>