Commit b36d8d6b authored Jan 21, 2021 by Alex Deucher

drm/amdgpu: only set restart on first cmd of the smu i2c transaction



Not sure how the firmware interprets these.

Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Reviewed-by: Luben Tuikov <luben.tuikov@amd.com>

parent 73a5784a

drivers/gpu/drm/amd/pm/swsmu/smu11/arcturus_ppt.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -1984,7 +1984,7 @@ static int arcturus_i2c_xfer(struct i2c_adapter *i2c_adap,
		if ((msg[i].flags & I2C_M_STOP) \|\|
		(!remaining_bytes))
		cmd->CmdConfig \|= CMDCONFIG_STOP_MASK;
		if ((i > 0) && !(msg[i].flags & I2C_M_NOSTART))
		if ((i > 0) && (j == 0) && !(msg[i].flags & I2C_M_NOSTART))
		cmd->CmdConfig \|= CMDCONFIG_RESTART_BIT;
		}
		}

drivers/gpu/drm/amd/pm/swsmu/smu11/navi10_ppt.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -2783,7 +2783,7 @@ static int navi10_i2c_xfer(struct i2c_adapter *i2c_adap,
		if ((msg[i].flags & I2C_M_STOP) \|\|
		(!remaining_bytes))
		cmd->CmdConfig \|= CMDCONFIG_STOP_MASK;
		if ((i > 0) && !(msg[i].flags & I2C_M_NOSTART))
		if ((i > 0) && (j == 0) && !(msg[i].flags & I2C_M_NOSTART))
		cmd->CmdConfig \|= CMDCONFIG_RESTART_BIT;
		}
		}

drivers/gpu/drm/amd/pm/swsmu/smu11/sienna_cichlid_ppt.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -3490,7 +3490,7 @@ static int sienna_cichlid_i2c_xfer(struct i2c_adapter *i2c_adap,
		if ((msg[i].flags & I2C_M_STOP) \|\|
		(!remaining_bytes))
		cmd->CmdConfig \|= CMDCONFIG_STOP_MASK;
		if ((i > 0) && !(msg[i].flags & I2C_M_NOSTART))
		if ((i > 0) && (j == 0) && !(msg[i].flags & I2C_M_NOSTART))
		cmd->CmdConfig \|= CMDCONFIG_RESTART_BIT;
		}
		}