Re: [ASUS EC Sensors v7 1/3] hwmon: (asus-ec-sensors) add driver for ASUS EC

From: Eugene Shalygin
Date: Tue Jan 18 2022 - 22:25:58 EST


I did not read carefully what I committed and now two small
corrections are required (see inline). Should I resend again?

On Tue, 18 Jan 2022 at 17:53, Eugene Shalygin <eugene.shalygin@xxxxxxxxx> wrote:
> + DMI_EXACT_MATCH_BOARD(VENDOR_ASUS_UPPER_CASE, "ROG STRIX X570-F GAMING",
> + SENSOR_SET_TEMP_CHIPSET_CPU_MB |
> + SENSOR_TEMP_T_SENSOR | SENSOR_FAN_CHIPSET
Redundant line break.
> + ),
> + DMI_EXACT_MATCH_BOARD(VENDOR_ASUS_UPPER_CASE, "ROG STRIX X570-I GAMING",
> + SENSOR_TEMP_T_SENSOR | SENSOR_FAN_VRM_HS |
> + SENSOR_FAN_CHIPSET | SENSOR_CURR_CPU),


> +struct ec_sensors_data {
> + unsigned long board_sensors;
> + struct ec_sensor *sensors;
> + /** EC registers to read from */
> + u16 *registers;
> + u8 *read_buffer;
> + /** sorted list of unique register banks */
> + u8 banks[ASUS_EC_MAX_BANK + 1];
> + /** in jiffies */
> + unsigned long last_updated;
> + acpi_handle aml_mutex;
> + /** number of board EC sensors */
> + u8 nr_sensors;
> + /** number of EC registers to read (sensor might span more than 1 register) */
> + u8 nr_registers;
> + /** number of unique register banks */
> + u8 nr_banks;
> +};

Forgot to remove doc-comments.

Thanks,
Eugene