diff options
| author | Luben Tuikov <luben.tuikov@amd.com> | 2021-02-05 18:48:01 -0500 | 
|---|---|---|
| committer | Alex Deucher <alexander.deucher@amd.com> | 2021-07-01 00:24:40 -0400 | 
| commit | a43996573ad531ad1def11f0ecf5fdad361348a6 (patch) | |
| tree | 6a2b5c4875cc4fcc29589ea742c596433361bf9b /drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c | |
| parent | c28aa44de813d29d84bb6375eb92ff3cf92dab97 (diff) | |
drm/amdgpu: Rename misspelled function
Instead of fixing the spelling in
  amdgpu_ras_eeprom_process_recods(),
rename it to,
  amdgpu_ras_eeprom_xfer(),
to look similar to other I2C and protocol
transfer (read/write) functions.
Also to keep the column span to within reason by
using a shorter name.
Change the "num" function parameter from "int" to
"const u32" since it is the number of items
(records) to xfer, i.e. their count, which cannot
be a negative number.
Also swap the order of parameters, keeping the
pointer to records and their number next to each
other, while the direction now becomes the last
parameter.
Cc: Jean Delvare <jdelvare@suse.de>
Cc: Alexander Deucher <Alexander.Deucher@amd.com>
Cc: Andrey Grodzovsky <Andrey.Grodzovsky@amd.com>
Cc: Lijo Lazar <Lijo.Lazar@amd.com>
Cc: Stanley Yang <Stanley.Yang@amd.com>
Cc: Hawking Zhang <Hawking.Zhang@amd.com>
Signed-off-by: Luben Tuikov <luben.tuikov@amd.com>
Acked-by: Alexander Deucher <Alexander.Deucher@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c')
| -rw-r--r-- | drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c | 11 | 
1 files changed, 5 insertions, 6 deletions
| diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c index c13b02caf8c3..ba7eea90b36f 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c @@ -1817,10 +1817,10 @@ int amdgpu_ras_save_bad_pages(struct amdgpu_device *adev)  	save_count = data->count - control->num_recs;  	/* only new entries are saved */  	if (save_count > 0) { -		if (amdgpu_ras_eeprom_process_recods(control, -							&data->bps[control->num_recs], -							true, -							save_count)) { +		if (amdgpu_ras_eeprom_xfer(control, +					   &data->bps[control->num_recs], +					   save_count, +					   true)) {  			dev_err(adev->dev, "Failed to save EEPROM table data!");  			return -EIO;  		} @@ -1850,8 +1850,7 @@ static int amdgpu_ras_load_bad_pages(struct amdgpu_device *adev)  	if (!bps)  		return -ENOMEM; -	if (amdgpu_ras_eeprom_process_recods(control, bps, false, -		control->num_recs)) { +	if (amdgpu_ras_eeprom_xfer(control, bps, control->num_recs, false)) {  		dev_err(adev->dev, "Failed to load EEPROM table records!");  		ret = -EIO;  		goto out; | 
