MIME-Version: 1.0
References: <20220111180347.1245774-1-eugene.shalygin@gmail.com> <0uakBuv1oBXoq7hUQxghKrFabqWebbG437WBNMuBVpYRL4vCE-IxwsbH16I82L-hq4xLtuSvBM1UaPfUy1HxmoFJYFHkDuJPxh3dZ_P8LuM=@protonmail.com>
In-Reply-To: <0uakBuv1oBXoq7hUQxghKrFabqWebbG437WBNMuBVpYRL4vCE-IxwsbH16I82L-hq4xLtuSvBM1UaPfUy1HxmoFJYFHkDuJPxh3dZ_P8LuM=@protonmail.com>
From:   Eugene Shalygin <eugene.shalygin@gmail.com>
Date:   Tue, 11 Jan 2022 19:34:41 +0100
Message-ID: <CAB95QARvshfae_wwECr-5wEHxgyoc-2QWHUsK6N7LLE1n_DbZQ@mail.gmail.com>
Subject: Re: PATCH v4 ASUS EC Sensors
To:     =?UTF-8?B?QmFybmFiw6FzIFDFkWN6ZQ==?= <pobrn@protonmail.com>
Cc:     Andy Shevchenko <andy.shevchenko@gmail.com>,
        Denis Pauk <pauk.denis@gmail.com>,
        Jean Delvare <jdelvare@suse.com>,
        Guenter Roeck <linux@roeck-us.net>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        linux-hwmon@vger.kernel.org
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

Hello,

Yes, my mistake, and thank you for the hint!

Regards,
Eugene

On Tue, 11 Jan 2022 at 19:17, Barnab=C3=A1s P=C5=91cze <pobrn@protonmail.co=
m> wrote:
>
> 2022. janu=C3=A1r 11., kedd 19:03 keltez=C3=A9ssel, Eugene Shalygin =C3=
=ADrta:
>
> > This patchset replaces the HWMON asus_wmi_ec_sensors driver with
> > an implementation that does not use WMI but queries the embedded
> > controller directly.
> >
> > That provides two enhancements: sensor reading became quicker (on some
> > systems or kernel configuration it took almost a full second to read
> > all the sensors, that transfers less than 15 bytes of data), the driver
> > became more fexible. The driver now relies on ACPI mutex to lock access
> > to the EC, in the same way as the WMI DSDT code does.
> >
> > Chenges in v5:
> >  - Place the sensors bitset directly into the driver_data field of the
> >        dmi_system_id struct.
> >  - Replace doc comments with regular ones.
> >
> > Changes in v4:
> >  - Deprecate the wmi driver rather than removing it.
> >
> > Changes in v3:
> >  - Remove BIOS version checks and BIOS version dependent mutex path.
> >
> > Changes in v2:
> >  - Replace sensor flags enum with bitset
> >  - Replace module init/probe functions with module_platform_driver_prob=
e
> >    and ask the platform drivers framework to load the driver when ACPI
> >    EC is found (ACPI ID "PNP0C09").
> >  - Extend board data with BIOS version attribute for the mutex path to =
be
> >    BIOS version dependent.
> >  - Add module parameter to override the mutex path.
> >
>
> Hi
>
>
> I believe the subject should say "v5" and not "v4". In any case, FYI, `gi=
t format-patch`
> has the `--cover-letter` option, which will generate a cover letter templ=
ate
> with the right patch version, etc.
>
>
> Regards,
> Barnab=C3=A1s P=C5=91cze