Commit 2e20c9dd authored by Jérémy Lefaure's avatar Jérémy Lefaure Committed by Daniel Vetter

drm/via: use ARRAY_SIZE

Using the ARRAY_SIZE macro improves the readability of the code.

Found with Coccinelle with the following semantic patch:
@r depends on (org || report)@
type T;
T[] E;
position p;
@@
(
 (sizeof(E)@p /sizeof(*E))
|
 (sizeof(E)@p /sizeof(E[...]))
|
 (sizeof(E)@p /sizeof(T))
)
Reviewed-by: default avatarThierry Reding <treding@nvidia.com>
Signed-off-by: default avatarJérémy Lefaure <jeremy.lefaure@lse.epita.fr>
Signed-off-by: default avatarDaniel Vetter <daniel.vetter@ffwll.ch>
Link: https://patchwork.freedesktop.org/patch/msgid/20171016023357.20174-1-jeremy.lefaure@lse.epita.fr
parent c7d234dc
...@@ -34,6 +34,7 @@ ...@@ -34,6 +34,7 @@
#include <drm/drm_legacy.h> #include <drm/drm_legacy.h>
#include "via_verifier.h" #include "via_verifier.h"
#include "via_drv.h" #include "via_drv.h"
#include <linux/kernel.h>
typedef enum { typedef enum {
state_command, state_command,
...@@ -1102,10 +1103,7 @@ setup_hazard_table(hz_init_t init_table[], hazard_t table[], int size) ...@@ -1102,10 +1103,7 @@ setup_hazard_table(hz_init_t init_table[], hazard_t table[], int size)
void via_init_command_verifier(void) void via_init_command_verifier(void)
{ {
setup_hazard_table(init_table1, table1, setup_hazard_table(init_table1, table1, ARRAY_SIZE(init_table1));
sizeof(init_table1) / sizeof(hz_init_t)); setup_hazard_table(init_table2, table2, ARRAY_SIZE(init_table2));
setup_hazard_table(init_table2, table2, setup_hazard_table(init_table3, table3, ARRAY_SIZE(init_table3));
sizeof(init_table2) / sizeof(hz_init_t));
setup_hazard_table(init_table3, table3,
sizeof(init_table3) / sizeof(hz_init_t));
} }
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment