mbox series

[v6,0/2] Enhancements for tmp513 driver

Message ID 20230907071404.24334-1-biju.das.jz@bp.renesas.com (mailing list archive)
Headers show
Series Enhancements for tmp513 driver | expand

Message

Biju Das Sept. 7, 2023, 7:14 a.m. UTC
This patch series aims to add enhancements for tmp513 driver.
 * patch#1 Adds max_channels variable to struct tmp51x_data for HW
           differences.
 * Patch#2 Simplify tmp51x_read_properties()

This patch series is only compile tested.

v5->v6:
 * Updated TMP51X_TEMP_CONFIG_CONV_RATE macro by removing FIELD_PREP macro
   and just use GENMASK(9, 7).
 * Dropped bitfield.h as there is no user.
v4->v5:
 * Updated the TMP51X_TEMP_CHANNEL_MASK macro.
 * Updated the indentation for TMP51X_TEMP_CONFIG_DEFAULT macro.
v3->v4:
 * Updated commit description for patch#1
 * Updated macros TMP51X_TEMP_CONFIG_DEFAULT and TMP51X_TEMP_CHANNEL_MASK.
 * Removed trailing comma in the terminator entry for OF table.
 * Added Rb tag from Geert and Andy for patch#2.
v2->v3:
 * Dropped patch#1 and patch#2 as it is accepted
 * Split patch#3 into 3.
 * Avoided Yoda style logic in tmp51x_is_visible().
 * Replaced OF/ID data from tmp51x_ids->max_channels
 * Updated the macro TMP51X_TEMP_CONFIG_DEFAULT by adding bit definitions
   for Configuration register 2.
 * Dropped unused macros TMP51{2,3}_TEMP_CONFIG_DEFAULT.
v1->v2:
 * Created 2 new patches. Patch#1 for fixing channel number in
   tmp51x_is_visible() and patch#3 avoids using enum chip id for HW
   differences and improves the code by avoiding checks.

Biju Das (2):
  hwmon: tmp513: Add max_channels variable to struct tmp51x_data
  hwmon: tmp513: Simplify tmp51x_read_properties()

 drivers/hwmon/tmp513.c | 52 +++++++++++++++++++-----------------------
 1 file changed, 23 insertions(+), 29 deletions(-)