* [PATCH] drm/amdgpu: Add missing '\n' in log messages
@ 2020-04-11 14:03 Christophe JAILLET
2020-04-13 2:50 ` Quan, Evan
0 siblings, 1 reply; 2+ messages in thread
From: Christophe JAILLET @ 2020-04-11 14:03 UTC (permalink / raw)
To: alexander.deucher, christian.koenig, David1.Zhou, airlied,
daniel, Hawking.Zhang, evan.quan, andrey.grodzovsky, Monk.Liu,
kent.russell, le.ma
Cc: Christophe JAILLET, kernel-janitors, dri-devel, amd-gfx, linux-kernel
Message logged by 'dev_xxx()' or 'pr_xxx()' should end with a '\n'.
While at it, split some long lines that where not that far.
Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
---
Most of them have been added in commit bd607166af7f ("drm/amdgpu: Enable reading FRU chip via I2C v3")
---
drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 16 +++++++++-------
1 file changed, 9 insertions(+), 7 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
index 87f7c129c8ce..3d0a50e8c36b 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
@@ -3249,25 +3249,25 @@ int amdgpu_device_init(struct amdgpu_device *adev,
r = device_create_file(adev->dev, &dev_attr_pcie_replay_count);
if (r) {
- dev_err(adev->dev, "Could not create pcie_replay_count");
+ dev_err(adev->dev, "Could not create pcie_replay_count\n");
return r;
}
r = device_create_file(adev->dev, &dev_attr_product_name);
if (r) {
- dev_err(adev->dev, "Could not create product_name");
+ dev_err(adev->dev, "Could not create product_name\n");
return r;
}
r = device_create_file(adev->dev, &dev_attr_product_number);
if (r) {
- dev_err(adev->dev, "Could not create product_number");
+ dev_err(adev->dev, "Could not create product_number\n");
return r;
}
r = device_create_file(adev->dev, &dev_attr_serial_number);
if (r) {
- dev_err(adev->dev, "Could not create serial_number");
+ dev_err(adev->dev, "Could not create serial_number\n");
return r;
}
@@ -4270,7 +4270,7 @@ int amdgpu_device_gpu_recover(struct amdgpu_device *adev,
job_signaled = true;
if (job_signaled) {
- dev_info(adev->dev, "Guilty job already signaled, skipping HW reset");
+ dev_info(adev->dev, "Guilty job already signaled, skipping HW reset\n");
goto skip_hw_reset;
}
@@ -4339,10 +4339,12 @@ int amdgpu_device_gpu_recover(struct amdgpu_device *adev,
if (r) {
/* bad news, how to tell it to userspace ? */
- dev_info(tmp_adev->dev, "GPU reset(%d) failed\n", atomic_read(&tmp_adev->gpu_reset_counter));
+ dev_info(tmp_adev->dev, "GPU reset(%d) failed\n",
+ atomic_read(&tmp_adev->gpu_reset_counter));
amdgpu_vf_error_put(tmp_adev, AMDGIM_ERROR_VF_GPU_RESET_FAIL, 0, r);
} else {
- dev_info(tmp_adev->dev, "GPU reset(%d) succeeded!\n", atomic_read(&tmp_adev->gpu_reset_counter));
+ dev_info(tmp_adev->dev, "GPU reset(%d) succeeded!\n",
+ atomic_read(&tmp_adev->gpu_reset_counter));
}
}
--
2.20.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* RE: [PATCH] drm/amdgpu: Add missing '\n' in log messages
2020-04-11 14:03 [PATCH] drm/amdgpu: Add missing '\n' in log messages Christophe JAILLET
@ 2020-04-13 2:50 ` Quan, Evan
0 siblings, 0 replies; 2+ messages in thread
From: Quan, Evan @ 2020-04-13 2:50 UTC (permalink / raw)
To: Christophe JAILLET, Deucher, Alexander, Koenig, Christian, Zhou,
David(ChunMing),
airlied, daniel, Zhang, Hawking, Grodzovsky, Andrey, Liu, Monk,
Russell, Kent, Ma, Le
Cc: kernel-janitors, dri-devel, amd-gfx, linux-kernel
Reviewed-by: Evan Quan <evan.quan@amd.com>
-----Original Message-----
From: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Sent: Saturday, April 11, 2020 10:04 PM
To: Deucher, Alexander <Alexander.Deucher@amd.com>; Koenig, Christian <Christian.Koenig@amd.com>; Zhou, David(ChunMing) <David1.Zhou@amd.com>; airlied@linux.ie; daniel@ffwll.ch; Zhang, Hawking <Hawking.Zhang@amd.com>; Quan, Evan <Evan.Quan@amd.com>; Grodzovsky, Andrey <Andrey.Grodzovsky@amd.com>; Liu, Monk <Monk.Liu@amd.com>; Russell, Kent <Kent.Russell@amd.com>; Ma, Le <Le.Ma@amd.com>
Cc: amd-gfx@lists.freedesktop.org; dri-devel@lists.freedesktop.org; linux-kernel@vger.kernel.org; kernel-janitors@vger.kernel.org; Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Subject: [PATCH] drm/amdgpu: Add missing '\n' in log messages
Message logged by 'dev_xxx()' or 'pr_xxx()' should end with a '\n'.
While at it, split some long lines that where not that far.
Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
---
Most of them have been added in commit bd607166af7f ("drm/amdgpu: Enable reading FRU chip via I2C v3")
---
drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 16 +++++++++-------
1 file changed, 9 insertions(+), 7 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
index 87f7c129c8ce..3d0a50e8c36b 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
@@ -3249,25 +3249,25 @@ int amdgpu_device_init(struct amdgpu_device *adev,
r = device_create_file(adev->dev, &dev_attr_pcie_replay_count);
if (r) {
- dev_err(adev->dev, "Could not create pcie_replay_count");
+ dev_err(adev->dev, "Could not create pcie_replay_count\n");
return r;
}
r = device_create_file(adev->dev, &dev_attr_product_name);
if (r) {
- dev_err(adev->dev, "Could not create product_name");
+ dev_err(adev->dev, "Could not create product_name\n");
return r;
}
r = device_create_file(adev->dev, &dev_attr_product_number);
if (r) {
- dev_err(adev->dev, "Could not create product_number");
+ dev_err(adev->dev, "Could not create product_number\n");
return r;
}
r = device_create_file(adev->dev, &dev_attr_serial_number);
if (r) {
- dev_err(adev->dev, "Could not create serial_number");
+ dev_err(adev->dev, "Could not create serial_number\n");
return r;
}
@@ -4270,7 +4270,7 @@ int amdgpu_device_gpu_recover(struct amdgpu_device *adev,
job_signaled = true;
if (job_signaled) {
- dev_info(adev->dev, "Guilty job already signaled, skipping HW reset");
+ dev_info(adev->dev, "Guilty job already signaled, skipping HW reset\n");
goto skip_hw_reset;
}
@@ -4339,10 +4339,12 @@ int amdgpu_device_gpu_recover(struct amdgpu_device *adev,
if (r) {
/* bad news, how to tell it to userspace ? */
- dev_info(tmp_adev->dev, "GPU reset(%d) failed\n", atomic_read(&tmp_adev->gpu_reset_counter));
+ dev_info(tmp_adev->dev, "GPU reset(%d) failed\n",
+ atomic_read(&tmp_adev->gpu_reset_counter));
amdgpu_vf_error_put(tmp_adev, AMDGIM_ERROR_VF_GPU_RESET_FAIL, 0, r);
} else {
- dev_info(tmp_adev->dev, "GPU reset(%d) succeeded!\n", atomic_read(&tmp_adev->gpu_reset_counter));
+ dev_info(tmp_adev->dev, "GPU reset(%d) succeeded!\n",
+ atomic_read(&tmp_adev->gpu_reset_counter));
}
}
--
2.20.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
end of thread, other threads:[~2020-04-13 2:50 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-04-11 14:03 [PATCH] drm/amdgpu: Add missing '\n' in log messages Christophe JAILLET
2020-04-13 2:50 ` Quan, Evan
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).