All of lore.kernel.org
 help / color / mirror / Atom feed
From: Wilken Gottwalt <wilken.gottwalt@posteo.net>
To: Shantanu Tushar <shantanu@kde.org>
Cc: jdelvare@suse.com, linux@roeck-us.net, linux-hwmon@vger.kernel.org
Subject: Re: [PATCH] hwmon: (corsair-psu) add support for HX1200i Series 2025
Date: Fri,  4 Jul 2025 06:40:03 +0000	[thread overview]
Message-ID: <20250704084003.21256f66@posteo.net> (raw)
In-Reply-To: <CABQ4Km_Ot1097OjNvvKNyYhxjqu5cD4JTZ0NoaQv9SOcWyE8=w@mail.gmail.com>

Hi Shantanu,

don't worry. That was only a comment/mental note for me. Looks like Corsair does
a lot of refreshs of that hardware. Hmm, but Guenter seems to be a bit slow on
this one.

greetings

On Fri, 4 Jul 2025 08:23:45 +0200
Shantanu Tushar <shantanu@kde.org> wrote:

> Hi Wilken,
> 
> Thanks for the review. Unfortunately I only have access to a HXi 1200
> PSU to test. Maybe support for the new HXi 1500 can come later?
> 
> Regards,
> Shantanu Tushar
> 
> On Tue, 1 Jul 2025 at 05:53, Wilken Gottwalt <wilken.gottwalt@posteo.net> wrote:
> >
> > Yeah, looks fine to me. And it looks like there is also a HXi 1500 Series
> > 2025 now.
> >
> > Reviewed-by: Wilken Gottwalt <wilken.gottwalt@posteo.net>
> >
> > On Mon, 30 Jun 2025 22:14:44 +0200
> > Shantanu Tushar <shantanu@kde.org> wrote:
> >
> > > Add the USB ID of the Corsair HXi Series 2025 HX1200i PSU (CP-9020307).
> > > Update the documentation to mention this.
> > >
> > > Signed-off-by: Shantanu Tushar <shantanu@kde.org>
> > > ---
> > >  Documentation/hwmon/corsair-psu.rst | 2 +-
> > >  drivers/hwmon/corsair-psu.c         | 1 +
> > >  2 files changed, 2 insertions(+), 1 deletion(-)
> > >
> > > diff --git a/Documentation/hwmon/corsair-psu.rst b/Documentation/hwmon/corsair-psu.rst
> > > index 7ed794087f84..2e99cfd556a0 100644
> > > --- a/Documentation/hwmon/corsair-psu.rst
> > > +++ b/Documentation/hwmon/corsair-psu.rst
> > > @@ -17,7 +17,7 @@ Supported devices:
> > >
> > >    Corsair HX1000i (Legacy and Series 2023)
> > >
> > > -  Corsair HX1200i (Legacy and Series 2023)
> > > +  Corsair HX1200i (Legacy, Series 2023 and Series 2025)
> > >
> > >    Corsair HX1500i (Legacy and Series 2023)
> > >
> > > diff --git a/drivers/hwmon/corsair-psu.c b/drivers/hwmon/corsair-psu.c
> > > index f8f22b8a67cd..6b5c8f200780 100644
> > > --- a/drivers/hwmon/corsair-psu.c
> > > +++ b/drivers/hwmon/corsair-psu.c
> > > @@ -885,6 +885,7 @@ static const struct hid_device_id corsairpsu_idtable[] = {
> > >       { HID_USB_DEVICE(0x1b1c, 0x1c1e) }, /* Corsair HX1000i Series 2023 */
> > >       { HID_USB_DEVICE(0x1b1c, 0x1c1f) }, /* Corsair HX1500i Legacy and Series 2023 */
> > >       { HID_USB_DEVICE(0x1b1c, 0x1c23) }, /* Corsair HX1200i Series 2023 */
> > > +     { HID_USB_DEVICE(0x1b1c, 0x1c27) }, /* Corsair HX1200i Series 2025 */
> > >       { },
> > >  };
> > >  MODULE_DEVICE_TABLE(hid, corsairpsu_idtable);
> >


  reply	other threads:[~2025-07-04  6:47 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-06-30 20:14 [PATCH] hwmon: (corsair-psu) add support for HX1200i Series 2025 Shantanu Tushar
2025-07-01  3:53 ` Wilken Gottwalt
2025-07-04  6:23   ` Shantanu Tushar
2025-07-04  6:40     ` Wilken Gottwalt [this message]
2025-07-05 19:19       ` Guenter Roeck
2025-07-06 12:08         ` Wilken Gottwalt
2025-07-16 14:45 ` Guenter Roeck

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20250704084003.21256f66@posteo.net \
    --to=wilken.gottwalt@posteo.net \
    --cc=jdelvare@suse.com \
    --cc=linux-hwmon@vger.kernel.org \
    --cc=linux@roeck-us.net \
    --cc=shantanu@kde.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.