Commit e281d594 authored by Alex Deucher's avatar Alex Deucher
Browse files

drm/amdgpu/swsmu/i2c: return an error if the SMU is not running 



Return an error if someone tries to use the i2c bus when the
SMU is not running.  Otherwise we can end up sending commands
to the SMU which will either get ignored or could cause other
issues depending on what state the GPU and SMU are in.

Cc: Luben.Tuikov@amd.com
Reviewed-by: default avatarLuben Tuikov <luben.tuikov@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 3ed89339

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

+3 −0
Original line number Diff line number Diff line
@@ -2071,6 +2071,9 @@ static int arcturus_i2c_xfer(struct i2c_adapter *i2c_adap, 
	int i, j, r, c;

	u16 dir;



	if (!adev->pm.dpm_enabled)

		return -EBUSY;



	req = kzalloc(sizeof(*req), GFP_KERNEL);

	if (!req)

		return -ENOMEM;

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

+3 −0
Original line number Diff line number Diff line
@@ -2754,6 +2754,9 @@ static int navi10_i2c_xfer(struct i2c_adapter *i2c_adap, 
	int i, j, r, c;

	u16 dir;



	if (!adev->pm.dpm_enabled)

		return -EBUSY;



	req = kzalloc(sizeof(*req), GFP_KERNEL);

	if (!req)

		return -ENOMEM;

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

+3 −0
Original line number Diff line number Diff line
@@ -3449,6 +3449,9 @@ static int sienna_cichlid_i2c_xfer(struct i2c_adapter *i2c_adap, 
	int i, j, r, c;

	u16 dir;



	if (!adev->pm.dpm_enabled)

		return -EBUSY;



	req = kzalloc(sizeof(*req), GFP_KERNEL);

	if (!req)

		return -ENOMEM;

drivers/gpu/drm/amd/pm/swsmu/smu13/aldebaran_ppt.c

+3 −0
Original line number Diff line number Diff line
@@ -1473,6 +1473,9 @@ static int aldebaran_i2c_xfer(struct i2c_adapter *i2c_adap, 
	int i, j, r, c;

	u16 dir;



	if (!adev->pm.dpm_enabled)

		return -EBUSY;



	req = kzalloc(sizeof(*req), GFP_KERNEL);

	if (!req)

		return -ENOMEM;