Re: PATCH v4 ASUS EC Sensors

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello,

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

Regards,
Eugene

On Tue, 11 Jan 2022 at 19:17, Barnabás Pőcze <pobrn@xxxxxxxxxxxxxx> wrote:
>
> 2022. január 11., kedd 19:03 keltezéssel, Eugene Shalygin írta:
>
> > 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_probe
> >    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, `git format-patch`
> has the `--cover-letter` option, which will generate a cover letter template
> with the right patch version, etc.
>
>
> Regards,
> Barnabás Pőcze




[Index of Archives]     [LM Sensors]     [Linux Sound]     [ALSA Users]     [ALSA Devel]     [Linux Audio Users]     [Linux Media]     [Kernel]     [Gimp]     [Yosemite News]     [Linux Media]

  Powered by Linux