Re: [PATCH v2 0/2] Add driver support for ESWIN EIC7700 PVT controller

[Huan He]

> 
> Add support for the ESWIN EIC7700 PVT (Process, Voltage, Temperature)
> sensor
> 
> Features:
> The driver supports monitoring of process, voltage and temperature
> parameters through the hardware monitoring subsystem. It provides an
> access to the sampled Temperature and Voltage.
> 
> Test:
> Tested this patch on the SiFive HiFive Premier P550 (which uses the ESWIN
> EIC7700 SoC).
> 
> Updates:
> 
>   Changes in v2:
>   - Update eswin,eic7700-pvt.yaml
>     - Reference the hwmon-common.yaml file
>     - Remove the clock-names and reset-names properties
>     - Move additionalProperties: false after the required block
>     - Remove one example node to avoid redundancy
>   - Update eic7700-pvt.c and eic7700-pvt.h
>     - Remove unused sensor macros (PVT_SENSOR_FIRST, PVT_SENSOR_LAST,
>       PVT_SENSORS_NUM)
>     - Drop the unnecessary hwmon-sysfs.h header
>     - Replace dynamic sensor info allocation with a static array and unify
>       sensor labels
>     - Remove unused hwmon_temp_type attribute
>     - Eliminate redundant validation checks
>     - Remove mutex and related locking, relying on hwmon core
>       serialization
>     - Replace per-sensor caches and completions with a single data cache
>       and completion object
>     - Remove pvt->sensor tracking. ISR no longer depends on the currently
>       selected sensor
>     - Move devm_add_action() registration after init_completion() for
>       safer cleanup, and update cleanup function (pvt_clear_data)
>     - Replace devm_reset_control_get_optional_exclusive() with
>       devm_reset_control_get_exclusive_deasserted()
>     - Replace eic7700_pvt_remove() with eic7700_pvt_disable_pm_runtime()
>       and move it after PM runtime enable to avoid resource leaks on probe
>       failure and remove clock disable and reset assert from
>       eic7700_pvt_disable_pm_runtime() as it is already handled by devm_*
>       framework
>     - Remove redundant clock presence check in runtime_resume
> 

Hi Krzysztof, Guenter, all,

The v1 patch received reviews and comments from Krzysztof and Guenter.
The v2 patch incorporates changes based on those comments and has received
feedback from Conor.

I'm not sure if there has been time to review the updated v2 yet. Should I
wait for further feedback, or is any action needed from my side?

Best regards,
Huan He

>   - Link to v1: https://lore.kernel.org/all/20260109090718.442-1-hehuan1@xxxxxxxxxxxxxxxxxx/
>