{"id":809264,"url":"http://patchwork.ozlabs.org/api/patches/809264/?format=json","web_url":"http://patchwork.ozlabs.org/project/netdev/patch/1504439110050-1887263060-8-diffsplit-thomas@m3y3r.de/","project":{"id":7,"url":"http://patchwork.ozlabs.org/api/projects/7/?format=json","name":"Linux network development","link_name":"netdev","list_id":"netdev.vger.kernel.org","list_email":"netdev@vger.kernel.org","web_url":null,"scm_url":null,"webscm_url":null,"list_archive_url":"","list_archive_url_format":"","commit_url_format":""},"msgid":"<1504439110050-1887263060-8-diffsplit-thomas@m3y3r.de>","list_archive_url":null,"date":"2017-09-03T12:19:31","name":"[8/10] ath9k: Use ARRAY_SIZE macro","commit_ref":null,"pull_url":null,"state":"awaiting-upstream","archived":true,"hash":"97bcdc1939518d412c2ab3c3bedb1d1e0d169369","submitter":{"id":10277,"url":"http://patchwork.ozlabs.org/api/people/10277/?format=json","name":"Thomas Meyer","email":"thomas@m3y3r.de"},"delegate":{"id":34,"url":"http://patchwork.ozlabs.org/api/users/34/?format=json","username":"davem","first_name":"David","last_name":"Miller","email":"davem@davemloft.net"},"mbox":"http://patchwork.ozlabs.org/project/netdev/patch/1504439110050-1887263060-8-diffsplit-thomas@m3y3r.de/mbox/","series":[{"id":1221,"url":"http://patchwork.ozlabs.org/api/series/1221/?format=json","web_url":"http://patchwork.ozlabs.org/project/netdev/list/?series=1221","date":"2017-09-03T12:19:31","name":null,"version":1,"mbox":"http://patchwork.ozlabs.org/series/1221/mbox/"}],"comments":"http://patchwork.ozlabs.org/api/patches/809264/comments/","check":"pending","checks":"http://patchwork.ozlabs.org/api/patches/809264/checks/","tags":{},"related":[],"headers":{"Return-Path":"<netdev-owner@vger.kernel.org>","X-Original-To":"patchwork-incoming@ozlabs.org","Delivered-To":"patchwork-incoming@ozlabs.org","Authentication-Results":"ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=vger.kernel.org\n\t(client-ip=209.132.180.67; helo=vger.kernel.org;\n\tenvelope-from=netdev-owner@vger.kernel.org;\n\treceiver=<UNKNOWN>)","Received":["from vger.kernel.org (vger.kernel.org [209.132.180.67])\n\tby ozlabs.org (Postfix) with ESMTP id 3xlXPt09WHz9sP3\n\tfor <patchwork-incoming@ozlabs.org>;\n\tSun,  3 Sep 2017 22:30:41 +1000 (AEST)","(majordomo@vger.kernel.org) by vger.kernel.org via listexpand\n\tid S1752865AbdICMad (ORCPT <rfc822;patchwork-incoming@ozlabs.org>);\n\tSun, 3 Sep 2017 08:30:33 -0400","from www17.your-server.de ([213.133.104.17]:34267 \"EHLO\n\twww17.your-server.de\" rhost-flags-OK-OK-OK-OK) by vger.kernel.org\n\twith ESMTP id S1751611AbdICMab (ORCPT\n\t<rfc822;netdev@vger.kernel.org>); Sun, 3 Sep 2017 08:30:31 -0400","from [95.222.27.54] (helo=olymp)\n\tby www17.your-server.de with esmtpsa\n\t(TLSv1.2:DHE-RSA-AES256-GCM-SHA384:256) (Exim 4.85_2)\n\t(envelope-from <thomas@m3y3r.de>)\n\tid 1doU2o-0007P5-63; Sun, 03 Sep 2017 14:30:30 +0200"],"Subject":"[PATCH 8/10] ath9k: Use ARRAY_SIZE macro","From":"Thomas Meyer <thomas@m3y3r.de>","To":"kvalo@codeaurora.org, linux-wireless@vger.kernel.org,\n\tnetdev@vger.kernel.org, linux-kernel@vger.kernel.org","Content-Type":"text/plain; charset=\"UTF-8\"","Mime-Version":"1.0","Message-ID":"<1504439110050-1887263060-8-diffsplit-thomas@m3y3r.de>","References":"<1504439110050-939061377-0-diffsplit-thomas@m3y3r.de>","In-Reply-To":"<1504439110050-939061377-0-diffsplit-thomas@m3y3r.de>","Date":"Sun, 03 Sep 2017 14:19:31 +0200","X-Mailer":"Evolution 3.22.6-1 ","Content-Transfer-Encoding":"7bit","X-Authenticated-Sender":"thomas@m3y3r.de","X-Virus-Scanned":"Clear (ClamAV 0.99.2/23763/Sun Sep  3 10:37:41 2017)","Sender":"netdev-owner@vger.kernel.org","Precedence":"bulk","List-ID":"<netdev.vger.kernel.org>","X-Mailing-List":"netdev@vger.kernel.org"},"content":"Use ARRAY_SIZE macro, rather than explicitly coding some variant of it\nyourself.\nFound with: find -type f -name \"*.c\" -o -name \"*.h\" | xargs perl -p -i -e\n's/\\bsizeof\\s*\\(\\s*(\\w+)\\s*\\)\\s*\\ /\\s*sizeof\\s*\\(\\s*\\1\\s*\\[\\s*0\\s*\\]\\s*\\)\n/ARRAY_SIZE(\\1)/g' and manual check/verification.\n\nSigned-off-by: Thomas Meyer <thomas@m3y3r.de>\n---","diff":"diff --git a/drivers/net/wireless/ath/ath9k/ar9003_eeprom.c b/drivers/net/wireless/ath/ath9k/ar9003_eeprom.c\nindex 3dbfd86ebe36..c2e210c0a770 100644\n--- a/drivers/net/wireless/ath/ath9k/ar9003_eeprom.c\n+++ b/drivers/net/wireless/ath/ath9k/ar9003_eeprom.c\n@@ -15,6 +15,7 @@\n  */\n \n #include <asm/unaligned.h>\n+#include <linux/kernel.h>\n #include \"hw.h\"\n #include \"ar9003_phy.h\"\n #include \"ar9003_eeprom.h\"\n@@ -2946,14 +2947,12 @@ static const struct ar9300_eeprom *ar9300_eep_templates[] = {\n \n static const struct ar9300_eeprom *ar9003_eeprom_struct_find_by_id(int id)\n {\n-#define N_LOOP (sizeof(ar9300_eep_templates) / sizeof(ar9300_eep_templates[0]))\n \tint it;\n \n-\tfor (it = 0; it < N_LOOP; it++)\n+\tfor (it = 0; it < ARRAY_SIZE(ar9300_eep_templates); it++)\n \t\tif (ar9300_eep_templates[it]->templateVersion == id)\n \t\t\treturn ar9300_eep_templates[it];\n \treturn NULL;\n-#undef N_LOOP\n }\n \n static int ath9k_hw_ar9300_check_eeprom(struct ath_hw *ah)\n","prefixes":["8/10"]}