On Sun, Oct 15, 2017 at 10:33:56PM -0400, Jérémy Lefaure wrote:
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: Thierry Reding treding@nvidia.com Signed-off-by: Jérémy Lefaure jeremy.lefaure@lse.epita.fr
This patch was part of a bigger patch [1] reviewed by Thierry Reding before it was split in several patches.
drivers/gpu/drm/via/via_verifier.c | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-)
Also applied to drm-misc-next.
Thanks, Daniel
diff --git a/drivers/gpu/drm/via/via_verifier.c b/drivers/gpu/drm/via/via_verifier.c index 0677bbf4ec7e..fb2609434df7 100644 --- a/drivers/gpu/drm/via/via_verifier.c +++ b/drivers/gpu/drm/via/via_verifier.c @@ -34,6 +34,7 @@ #include <drm/drm_legacy.h> #include "via_verifier.h" #include "via_drv.h" +#include <linux/kernel.h>
typedef enum { state_command, @@ -1102,10 +1103,7 @@ setup_hazard_table(hz_init_t init_table[], hazard_t table[], int size)
void via_init_command_verifier(void) {
- setup_hazard_table(init_table1, table1,
sizeof(init_table1) / sizeof(hz_init_t));- setup_hazard_table(init_table2, table2,
sizeof(init_table2) / sizeof(hz_init_t));- setup_hazard_table(init_table3, table3,
sizeof(init_table3) / sizeof(hz_init_t));
- setup_hazard_table(init_table1, table1, ARRAY_SIZE(init_table1));
- setup_hazard_table(init_table2, table2, ARRAY_SIZE(init_table2));
- setup_hazard_table(init_table3, table3, ARRAY_SIZE(init_table3));
}
2.14.2
dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel