Hi Ilpo,

I managed to get my hands on acpidumps for these models so this is
verified against those.

Thanks for all your latest reviews!

Signed-off-by: Kurt Borja <kuurtb@gmail.com>
---
Kurt Borja (3):
      platform/x86: alienware-wmi-wmax: Add support for new Area-51 laptops
      platform/x86: alienware-wmi-wmax: Add AWCC support for Alienware x16
      platform/x86: alienware-wmi-wmax: Add support for Alienware 16X Aurora

 drivers/platform/x86/dell/alienware-wmi-wmax.c | 32 ++++++++++++++++++++++++++
 1 file changed, 32 insertions(+)
---
base-commit: 9b9c0adbc3f8a524d291baccc9d0c04097fb4869
change-id: 20251111-area-51-7e6c2501e4eb

-- 
 ~ Kurt

On Fri, 05 Dec 2025 13:50:09 -0500, Kurt Borja wrote:

> I managed to get my hands on acpidumps for these models so this is
> verified against those.
> 
> Thanks for all your latest reviews!
> 
> 


Thank you for your contribution, it has been applied to my local
review-ilpo-fixes branch. Note it will show up in the public
platform-drivers-x86/review-ilpo-fixes branch only once I've pushed my
local branch there, which might take a while.

The list of commits applied:
[1/3] platform/x86: alienware-wmi-wmax: Add support for new Area-51 laptops
      commit: 433f7744cb302ac22800dc0cd50494319ce64ba0
[2/3] platform/x86: alienware-wmi-wmax: Add AWCC support for Alienware x16
      commit: a584644a490d276907e56817694859eaac2a4199
[3/3] platform/x86: alienware-wmi-wmax: Add support for Alienware 16X Aurora
      commit: 7f3c2499da24551968640528fee9aed3bb4f0c3f

--
 i.

On Fri, 5 Dec 2025, Kurt Borja wrote:

> I managed to get my hands on acpidumps for these models so this is
> verified against those.
> 
> Thanks for all your latest reviews!
> 
> Signed-off-by: Kurt Borja <kuurtb@gmail.com>

You don't need to signoff the coverletter. :-) (Hopefully it won't 
confuse any tools but I guess they should handle duplicate tags sensibly 
so likely no problem in this case).

For the series,

Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>

> ---
> Kurt Borja (3):
>       platform/x86: alienware-wmi-wmax: Add support for new Area-51 laptops
>       platform/x86: alienware-wmi-wmax: Add AWCC support for Alienware x16
>       platform/x86: alienware-wmi-wmax: Add support for Alienware 16X Aurora
> 
>  drivers/platform/x86/dell/alienware-wmi-wmax.c | 32 ++++++++++++++++++++++++++
>  1 file changed, 32 insertions(+)
> ---
> base-commit: 9b9c0adbc3f8a524d291baccc9d0c04097fb4869
> change-id: 20251111-area-51-7e6c2501e4eb
> 
> 

-- 
 i.

On Fri Dec 5, 2025 at 2:04 PM -05, Ilpo Järvinen wrote:
> On Fri, 5 Dec 2025, Kurt Borja wrote:
>
>> I managed to get my hands on acpidumps for these models so this is
>> verified against those.
>> 
>> Thanks for all your latest reviews!
>> 
>> Signed-off-by: Kurt Borja <kuurtb@gmail.com>
>
> You don't need to signoff the coverletter. :-) (Hopefully it won't 
> confuse any tools but I guess they should handle duplicate tags sensibly 
> so likely no problem in this case).

Actually, unless I messed up something, this is b4's default settings
:-). I'll take a look.

>
> For the series,
>
> Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>

Thanks!


-- 
 ~ Kurt

On Fri, Dec 05, 2025 at 02:08:52PM -0500, Kurt Borja wrote:
> >> Thanks for all your latest reviews!
> >> 
> >> Signed-off-by: Kurt Borja <kuurtb@gmail.com>
> >
> > You don't need to signoff the coverletter. :-) (Hopefully it won't 
> > confuse any tools but I guess they should handle duplicate tags sensibly 
> > so likely no problem in this case).
> 
> Actually, unless I messed up something, this is b4's default settings
> :-). I'll take a look.

This is intentional, because some subsystems use the cover letter as the
content of the merge commit.

-K

On Fri Dec 5, 2025 at 3:57 PM -05, Konstantin Ryabitsev wrote:
> On Fri, Dec 05, 2025 at 02:08:52PM -0500, Kurt Borja wrote:
>> >> Thanks for all your latest reviews!
>> >> 
>> >> Signed-off-by: Kurt Borja <kuurtb@gmail.com>
>> >
>> > You don't need to signoff the coverletter. :-) (Hopefully it won't 
>> > confuse any tools but I guess they should handle duplicate tags sensibly 
>> > so likely no problem in this case).
>> 
>> Actually, unless I messed up something, this is b4's default settings
>> :-). I'll take a look.
>
> This is intentional, because some subsystems use the cover letter as the
> content of the merge commit.
>
> -K

Makes sense. Thanks for clarifying!


-- 
 ~ Kurt