[PATCH v3 0/2] hwmon: (lenovo-ec-sensors): Fix EC signature validation and I/O resource management

Kean Ren posted 2 patches 3 days, 16 hours ago
drivers/hwmon/lenovo-ec-sensors.c | 20 +++++++-------------
1 file changed, 7 insertions(+), 13 deletions(-)
[PATCH v3 0/2] hwmon: (lenovo-ec-sensors): Fix EC signature validation and I/O resource management
Posted by Kean Ren 3 days, 16 hours ago
Hi Guenter,

Thank you for your patient review.
This is v3 of the lenovo-ec-sensors fix series.

Changes from v2:

- Corrected bug description and Remove unnecessary {}.
- Reproduce patch 0002 based 0001 changed.

Patch 1 converts manual request_region()/release_region() calls to
devm_request_region(), eliminating a double-release on the probe error
path and a release-after-use window on module exit.

Patch 2 fixes the EC "MCHP" signature check that used && instead of ||,
which caused the validation to accept a non-Microchip EC if any single
byte of the expected 4-byte signature happened to match.

Both patches apply cleanly against v7.1-rc3 and pass checkpatch with
0 errors, 0 warnings.

Kean Ren (2):
  hwmon: (lenovo-ec-sensors): Convert to  devm_request_region()
  hwmon: (lenovo-ec-sensors): Fix EC "MCHP" signature  validation logic

 drivers/hwmon/lenovo-ec-sensors.c | 20 +++++++-------------
 1 file changed, 7 insertions(+), 13 deletions(-)

-- 
2.53.0