Re: [PATCH] platform/x86: oxpec: add OneXPlayer X2 Mini (G3E) DMI entry

[Antheas Kapenekakis]

On Sun, 1 Nov 2026 at 02:35, Antheas Kapenekakis <lkml@xxxxxxxxxxx> wrote:
>
> On Sun, 1 Nov 2026 at 02:26, Jiajun Leng <adolphpi@xxxxxxxxxxx> wrote:
> >
> > The OneXPlayer X2 Mini (G3E) is the Intel variant and reports DMI board
> > name "ONEXPLAYER X2Mini".
>
> > It is not covered by the existing "ONEXPLAYER
> > X2Mini PRO" entry, which is the AMD board using oxp_fly.
>
> Obviously. Remove.
>
> > Map it to oxp_g1_i, the profile used by the ONEXPLAYER 3, which has the
> > same Intel EC layout.
>
> Keep. Move up to the first paragraph.
>
> > Verified on Bazzite 44 with kernel 7.2.11-ogc4.1.fc44.x86_64:
>
> Drop. Broadly speaking we don't care, and I could ask you to test a
> mainline kernel now. But it is not necessary.
>
> > reading, manual PWM and charging control work.
>
> Move to the initial paragraph
>
> > AI assistance: Claude (Anthropic) was used to draft this changelog and the
> > diff. The submitter reviewed the change, built it as an out-of-tree module
> > and tested it on the hardware.
>
> Also drop. if you want to mention you used AI, do it after the
> trailer. The assisted-by tag is not used anymore.
>
> > Signed-off-by: Jiajun Leng <adolphpi@xxxxxxxxxxx>
> > ---
>
> /\ this is the trailer
>
> Best,
> Antheas
>
> >  drivers/platform/x86/oxpec.c | 7 +++++++
> >  1 file changed, 7 insertions(+)
> >
> > diff --git a/drivers/platform/x86/oxpec.c b/drivers/platform/x86/oxpec.c
> > index c4c4babe8e30..50d169302fc3 100644
> > --- a/drivers/platform/x86/oxpec.c
> > +++ b/drivers/platform/x86/oxpec.c
> > @@ -230,6 +230,13 @@ static const struct dmi_system_id dmi_table[] = {
> >                 },
> >                 .driver_data = (void *)oxp_g1_i,
> >         },
> > +       {
> > +               .matches = {
> > +                       DMI_MATCH(DMI_BOARD_VENDOR, "ONE-NETBOOK"),
> > +                       DMI_EXACT_MATCH(DMI_BOARD_NAME, "ONEXPLAYER X2Mini"),
> > +               },
> > +               .driver_data = (void *)oxp_g1_i,
> > +       },

Also, move above X2Mini Pro to adhere to alphabetical sorting.

> >         {
> >                 .matches = {
> >                         DMI_MATCH(DMI_BOARD_VENDOR, "ONE-NETBOOK"),
> > --
> > 2.53.0
> >
> >