> +static void amd_stop_sensor_v2(struct amd_mp2_dev *privdata, u16 sensor_idx)
> +{
> + union sfh_cmd_base cmd_base;
> +
> + cmd_base.ul = 0;
> + cmd_base.cmd_v2.cmd_id = DISABLE_SENSOR;
> + cmd_base.cmd_v2.period = 0;
> + cmd_base.cmd_v2.sensor_id = sensor_idx;
> + cmd_base.cmd_v2.length = 16;
> +
> + writeq(0x0, privdata->mmio + AMD_C2P_MSG2);
That should be AMD_C2P_MSG1, no? The new hardware has no param register.
