* [PATCH -next] drm/amd/display: use swap() in sort()
@ 2024-07-24 7:37 Jiapeng Chong
2024-07-24 16:55 ` Alex Deucher
2024-07-25 6:55 ` Christian König
0 siblings, 2 replies; 3+ messages in thread
From: Jiapeng Chong @ 2024-07-24 7:37 UTC (permalink / raw)
To: harry.wentland
Cc: sunpeng.li, Rodrigo.Siqueira, alexander.deucher,
christian.koenig, Xinhui.Pan, airlied, daniel, amd-gfx,
dri-devel, linux-kernel, Jiapeng Chong, Abaci Robot
Use existing swap() function rather than duplicating its implementation.
./drivers/gpu/drm/amd/display/dc/dml2/dml21/src/dml2_pmo/dml2_pmo_dcn3.c:17:29-30: WARNING opportunity for swap().
Reported-by: Abaci Robot <abaci@linux.alibaba.com>
Closes: https://bugzilla.openanolis.cn/show_bug.cgi?id=9573
Signed-off-by: Jiapeng Chong <jiapeng.chong@linux.alibaba.com>
---
.../display/dc/dml2/dml21/src/dml2_pmo/dml2_pmo_dcn3.c | 8 ++------
1 file changed, 2 insertions(+), 6 deletions(-)
diff --git a/drivers/gpu/drm/amd/display/dc/dml2/dml21/src/dml2_pmo/dml2_pmo_dcn3.c b/drivers/gpu/drm/amd/display/dc/dml2/dml21/src/dml2_pmo/dml2_pmo_dcn3.c
index 717536d7bb30..8e68a8094658 100644
--- a/drivers/gpu/drm/amd/display/dc/dml2/dml21/src/dml2_pmo/dml2_pmo_dcn3.c
+++ b/drivers/gpu/drm/amd/display/dc/dml2/dml21/src/dml2_pmo/dml2_pmo_dcn3.c
@@ -7,16 +7,12 @@
static void sort(double *list_a, int list_a_size)
{
- double temp;
// For all elements b[i] in list_b[]
for (int i = 0; i < list_a_size - 1; i++) {
// Find the first element of list_a that's larger than b[i]
for (int j = i; j < list_a_size - 1; j++) {
- if (list_a[j] > list_a[j + 1]) {
- temp = list_a[j];
- list_a[j] = list_a[j + 1];
- list_a[j + 1] = temp;
- }
+ if (list_a[j] > list_a[j + 1])
+ swap(list_a[j], list_a[j + 1]);
}
}
}
--
2.32.0.3.g01195cf9f
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH -next] drm/amd/display: use swap() in sort()
2024-07-24 7:37 [PATCH -next] drm/amd/display: use swap() in sort() Jiapeng Chong
@ 2024-07-24 16:55 ` Alex Deucher
2024-07-25 6:55 ` Christian König
1 sibling, 0 replies; 3+ messages in thread
From: Alex Deucher @ 2024-07-24 16:55 UTC (permalink / raw)
To: Jiapeng Chong
Cc: harry.wentland, sunpeng.li, Rodrigo.Siqueira, alexander.deucher,
christian.koenig, Xinhui.Pan, airlied, daniel, amd-gfx,
dri-devel, linux-kernel, Abaci Robot
Applied. Thanks!
Alex
On Wed, Jul 24, 2024 at 3:38 AM Jiapeng Chong
<jiapeng.chong@linux.alibaba.com> wrote:
>
> Use existing swap() function rather than duplicating its implementation.
>
> ./drivers/gpu/drm/amd/display/dc/dml2/dml21/src/dml2_pmo/dml2_pmo_dcn3.c:17:29-30: WARNING opportunity for swap().
>
> Reported-by: Abaci Robot <abaci@linux.alibaba.com>
> Closes: https://bugzilla.openanolis.cn/show_bug.cgi?id=9573
> Signed-off-by: Jiapeng Chong <jiapeng.chong@linux.alibaba.com>
> ---
> .../display/dc/dml2/dml21/src/dml2_pmo/dml2_pmo_dcn3.c | 8 ++------
> 1 file changed, 2 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/display/dc/dml2/dml21/src/dml2_pmo/dml2_pmo_dcn3.c b/drivers/gpu/drm/amd/display/dc/dml2/dml21/src/dml2_pmo/dml2_pmo_dcn3.c
> index 717536d7bb30..8e68a8094658 100644
> --- a/drivers/gpu/drm/amd/display/dc/dml2/dml21/src/dml2_pmo/dml2_pmo_dcn3.c
> +++ b/drivers/gpu/drm/amd/display/dc/dml2/dml21/src/dml2_pmo/dml2_pmo_dcn3.c
> @@ -7,16 +7,12 @@
>
> static void sort(double *list_a, int list_a_size)
> {
> - double temp;
> // For all elements b[i] in list_b[]
> for (int i = 0; i < list_a_size - 1; i++) {
> // Find the first element of list_a that's larger than b[i]
> for (int j = i; j < list_a_size - 1; j++) {
> - if (list_a[j] > list_a[j + 1]) {
> - temp = list_a[j];
> - list_a[j] = list_a[j + 1];
> - list_a[j + 1] = temp;
> - }
> + if (list_a[j] > list_a[j + 1])
> + swap(list_a[j], list_a[j + 1]);
> }
> }
> }
> --
> 2.32.0.3.g01195cf9f
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH -next] drm/amd/display: use swap() in sort()
2024-07-24 7:37 [PATCH -next] drm/amd/display: use swap() in sort() Jiapeng Chong
2024-07-24 16:55 ` Alex Deucher
@ 2024-07-25 6:55 ` Christian König
1 sibling, 0 replies; 3+ messages in thread
From: Christian König @ 2024-07-25 6:55 UTC (permalink / raw)
To: Jiapeng Chong, harry.wentland
Cc: sunpeng.li, Rodrigo.Siqueira, alexander.deucher,
christian.koenig, Xinhui.Pan, airlied, daniel, amd-gfx,
dri-devel, linux-kernel, Abaci Robot
Am 24.07.24 um 09:37 schrieb Jiapeng Chong:
> Use existing swap() function rather than duplicating its implementation.
>
> ./drivers/gpu/drm/amd/display/dc/dml2/dml21/src/dml2_pmo/dml2_pmo_dcn3.c:17:29-30: WARNING opportunity for swap().
>
> Reported-by: Abaci Robot <abaci@linux.alibaba.com>
> Closes: https://bugzilla.openanolis.cn/show_bug.cgi?id=9573
> Signed-off-by: Jiapeng Chong <jiapeng.chong@linux.alibaba.com>
> ---
> .../display/dc/dml2/dml21/src/dml2_pmo/dml2_pmo_dcn3.c | 8 ++------
> 1 file changed, 2 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/display/dc/dml2/dml21/src/dml2_pmo/dml2_pmo_dcn3.c b/drivers/gpu/drm/amd/display/dc/dml2/dml21/src/dml2_pmo/dml2_pmo_dcn3.c
> index 717536d7bb30..8e68a8094658 100644
> --- a/drivers/gpu/drm/amd/display/dc/dml2/dml21/src/dml2_pmo/dml2_pmo_dcn3.c
> +++ b/drivers/gpu/drm/amd/display/dc/dml2/dml21/src/dml2_pmo/dml2_pmo_dcn3.c
> @@ -7,16 +7,12 @@
>
> static void sort(double *list_a, int list_a_size)
> {
> - double temp;
> // For all elements b[i] in list_b[]
> for (int i = 0; i < list_a_size - 1; i++) {
> // Find the first element of list_a that's larger than b[i]
While at it please also replace all // comments by using /* */.
Apart from that looks good to me.
Regards,
Christian.
> for (int j = i; j < list_a_size - 1; j++) {
> - if (list_a[j] > list_a[j + 1]) {
> - temp = list_a[j];
> - list_a[j] = list_a[j + 1];
> - list_a[j + 1] = temp;
> - }
> + if (list_a[j] > list_a[j + 1])
> + swap(list_a[j], list_a[j + 1]);
> }
> }
> }
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2024-07-25 6:55 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-07-24 7:37 [PATCH -next] drm/amd/display: use swap() in sort() Jiapeng Chong
2024-07-24 16:55 ` Alex Deucher
2024-07-25 6:55 ` Christian König
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®