From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752856AbcA0BUk (ORCPT ); Tue, 26 Jan 2016 20:20:40 -0500 Received: from mailout3.samsung.com ([203.254.224.33]:46297 "EHLO mailout3.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750999AbcA0BUh (ORCPT ); Tue, 26 Jan 2016 20:20:37 -0500 X-AuditID: cbfee690-f79646d000001316-03-56a81b630095 Date: Wed, 27 Jan 2016 10:20:34 +0900 From: Andi Shyti To: Javier Martinez Canillas Cc: linux-kernel@vger.kernel.org, Kukjin Kim , rtc-linux@googlegroups.com, Chanwoo Choi , Alexandre Belloni , Krzysztof Kozlowski , Laxman Dewangan , linux-samsung-soc@vger.kernel.org Subject: Re: [PATCH v3 02/10] rtc: max77686: Use ARRAY_SIZE() instead of current array length Message-id: <20160127012034.GB14296@samsunx.samsung> References: <1453836020-29579-1-git-send-email-javier@osg.samsung.com> <1453836020-29579-3-git-send-email-javier@osg.samsung.com> MIME-version: 1.0 Content-type: text/plain; charset=us-ascii Content-disposition: inline In-reply-to: <1453836020-29579-3-git-send-email-javier@osg.samsung.com> User-Agent: Mutt/1.5.24 (2015-08-30) X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFtrGIsWRmVeSWpSXmKPExsWyRsSkWDdZekWYQctOXouOa4uZLK5/ec5q 8ebtGiaL1y8MLfofv2a2WLpvNYvF5V1z2CxmnN/HZLG/s4PRgdPjyaaLjB57Jp5k89i0qpPN o7f5HZvHlv677B59W1YxenzeJBfAHsVlk5Kak1mWWqRvl8CVcf//apaCfq2KadvPsjYwvlDs YuTkkBAwkbjatpYVwhaTuHBvPVsXIxeHkMAKRolLl88wwhTd2N3CDJGYxSixqW09C4TzkVHi TOcnsHYWAVWJB/92M4PYbAKaEk23f7CB2CICphK/1r5gBWlgFtjLJNH46g4LSEJYIF5i4oZl YM28QCv+butnArGFBJoZJQ7P1YeIC0r8mHwPrJ5ZQEti/c7jTBC2tMSjvzPYQWxOAXeJ2933 gWwODlEBFYlXB+shrv7ILrFzGSfEbQIS3yYfYgEpkRCQldh0gBmiRFLi4IobLBMYxWYhWTYL ybJZSJYtYGRexSiaWpBcUJyUXmSiV5yYW1yal66XnJ+7iREYnaf/PZuwg/HeAetDjAIcjEo8 vDf2Lw8TYk0sK67MPcRoCnTFRGYp0eR8YArIK4k3NDYzsjA1MTU2Mrc0UxLnfS31M1hIID2x JDU7NbUgtSi+qDQntfgQIxMHp1QDI3/n/IXdK3x9JyQGG+5Rzstdlbfsu7hx9vZ/WYVbVaRW 3Az+s6qbme3+XDfn2beFJ/DoVyolvH/qutzWKDxW4Ur5h43P7IXmcn+r4ZpkfHqXbvHnyVPf /WP9/uj3zc9sClzM3k0WzFoV650OqDEFCXEeYO3zFLEwXVGiw5Ux9y5rSkKKdFO2EktxRqKh FnNRcSIAyFdDD8kCAAA= X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFmphleLIzCtJLcpLzFFi42I5/e+xoG6y9Iowg0db2S06ri1msrj+5Tmr xZu3a5gsXr8wtOh//JrZYum+1SwWl3fNYbOYcX4fk8X+zg5GB06PJ5suMnrsmXiSzWPTqk42 j97md2weW/rvsnv0bVnF6PF5k1wAe1QDo01GamJKapFCal5yfkpmXrqtkndwvHO8qZmBoa6h pYW5kkJeYm6qrZKLT4CuW2YO0GlKCmWJOaVAoYDE4mIlfTtME0JD3HQtYBojdH1DguB6jAzQ QMIaxoz7/1ezFPRrVUzbfpa1gfGFYhcjJ4eEgInEjd0tzBC2mMSFe+vZuhi5OIQEZjFKbGpb zwLhfGSUONP5iRWkikVAVeLBv91gHWwCmhJNt3+wgdgiAqYSv9a+YAVpYBbYyyTR+OoOC0hC WCBeYuKGZWDNvEDr/m7rZwKxhQSaGSUOz9WHiAtK/Jh8D6yeWUBLYv3O40wQtrTEo78z2EFs TgF3idvd94FsDg5RARWJVwfrJzACnYnQPQtJ9ywk3QsYmVcxSqQWJBcUJ6XnGuWllusVJ+YW l+al6yXn525iBKeAZ9I7GA/vcj/EKMDBqMTDe2P/8jAh1sSy4srcQ4wSHMxKIrzJDCvChHhT EiurUovy44tKc1KLDzGaAgNkIrOUaHI+MD3llcQbGpuYGVkamRtaGBmbK4nz7rsUGSYkkJ5Y kpqdmlqQWgTTx8TBKdXAyPzVdkWc3sNLylnJfgoHd4nqbZWKdRCe0bXs7u2lvhMVI5nWJu2x Dt0olidXdOTXmn32lxcYxacufPrvjGV145eXxn7bfrru7buw5+m0JpaE8qKkYv37ay9wLpxz wdFFNmbz98+3zLzubb/2hlUgNDb24+l/nos0cprs2x+YBSbe09Vlrw2JVWIpzkg01GIuKk4E AMgd9QUXAwAA DLP-Filter: Pass X-MTR: 20000000000000000@CPGS X-CFilter-Loop: Reflected Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Jan 26, 2016 at 04:20:12PM -0300, Javier Martinez Canillas wrote: > It is better to use the ARRAY_SIZE() macro instead of the array length > to avoid bugs if the array is later changed and the length not updated. > > Signed-off-by: Javier Martinez Canillas > Reviewed-by: Krzysztof Kozlowski > Tested-by: Krzysztof Kozlowski Reviewed-by: Andi Shyti > --- > > Changes in v3: > - Add Krzysztof Kozlowski's Tested-by tag to patch #2. > - Add Laxman Dewangan's Acked-by tag to patch #2. > - Use ARRAY_SIZE() in all places. Suggested by Laxman Dewangan. > > Changes in v2: > - Add Krzysztof Kozlowski's Reviewed-by tag to patch #2. > > drivers/rtc/rtc-max77686.c | 23 ++++++++++++----------- > 1 file changed, 12 insertions(+), 11 deletions(-) > > diff --git a/drivers/rtc/rtc-max77686.c b/drivers/rtc/rtc-max77686.c > index 6653c3d11b66..d84a50c9f7f7 100644 > --- a/drivers/rtc/rtc-max77686.c > +++ b/drivers/rtc/rtc-max77686.c > @@ -149,7 +149,7 @@ static int max77686_rtc_read_time(struct device *dev, struct rtc_time *tm) > goto out; > > ret = regmap_bulk_read(info->max77686->rtc_regmap, > - MAX77686_RTC_SEC, data, RTC_NR_TIME); > + MAX77686_RTC_SEC, data, ARRAY_SIZE(data)); > if (ret < 0) { > dev_err(info->dev, "%s: fail to read time reg(%d)\n", __func__, ret); > goto out; > @@ -177,7 +177,7 @@ static int max77686_rtc_set_time(struct device *dev, struct rtc_time *tm) > mutex_lock(&info->lock); > > ret = regmap_bulk_write(info->max77686->rtc_regmap, > - MAX77686_RTC_SEC, data, RTC_NR_TIME); > + MAX77686_RTC_SEC, data, ARRAY_SIZE(data)); > if (ret < 0) { > dev_err(info->dev, "%s: fail to write time reg(%d)\n", __func__, > ret); > @@ -205,7 +205,7 @@ static int max77686_rtc_read_alarm(struct device *dev, struct rtc_wkalrm *alrm) > goto out; > > ret = regmap_bulk_read(info->max77686->rtc_regmap, > - MAX77686_ALARM1_SEC, data, RTC_NR_TIME); > + MAX77686_ALARM1_SEC, data, ARRAY_SIZE(data)); > if (ret < 0) { > dev_err(info->dev, "%s:%d fail to read alarm reg(%d)\n", > __func__, __LINE__, ret); > @@ -215,7 +215,7 @@ static int max77686_rtc_read_alarm(struct device *dev, struct rtc_wkalrm *alrm) > max77686_rtc_data_to_tm(data, &alrm->time, info->rtc_24hr_mode); > > alrm->enabled = 0; > - for (i = 0; i < RTC_NR_TIME; i++) { > + for (i = 0; i < ARRAY_SIZE(data); i++) { > if (data[i] & ALARM_ENABLE_MASK) { > alrm->enabled = 1; > break; > @@ -252,7 +252,7 @@ static int max77686_rtc_stop_alarm(struct max77686_rtc_info *info) > goto out; > > ret = regmap_bulk_read(info->max77686->rtc_regmap, > - MAX77686_ALARM1_SEC, data, RTC_NR_TIME); > + MAX77686_ALARM1_SEC, data, ARRAY_SIZE(data)); > if (ret < 0) { > dev_err(info->dev, "%s: fail to read alarm reg(%d)\n", > __func__, ret); > @@ -261,11 +261,11 @@ static int max77686_rtc_stop_alarm(struct max77686_rtc_info *info) > > max77686_rtc_data_to_tm(data, &tm, info->rtc_24hr_mode); > > - for (i = 0; i < RTC_NR_TIME; i++) > + for (i = 0; i < ARRAY_SIZE(data); i++) > data[i] &= ~ALARM_ENABLE_MASK; > > ret = regmap_bulk_write(info->max77686->rtc_regmap, > - MAX77686_ALARM1_SEC, data, RTC_NR_TIME); > + MAX77686_ALARM1_SEC, data, ARRAY_SIZE(data)); > if (ret < 0) { > dev_err(info->dev, "%s: fail to write alarm reg(%d)\n", > __func__, ret); > @@ -291,7 +291,7 @@ static int max77686_rtc_start_alarm(struct max77686_rtc_info *info) > goto out; > > ret = regmap_bulk_read(info->max77686->rtc_regmap, > - MAX77686_ALARM1_SEC, data, RTC_NR_TIME); > + MAX77686_ALARM1_SEC, data, ARRAY_SIZE(data)); > if (ret < 0) { > dev_err(info->dev, "%s: fail to read alarm reg(%d)\n", > __func__, ret); > @@ -312,7 +312,7 @@ static int max77686_rtc_start_alarm(struct max77686_rtc_info *info) > data[RTC_DATE] |= (1 << ALARM_ENABLE_SHIFT); > > ret = regmap_bulk_write(info->max77686->rtc_regmap, > - MAX77686_ALARM1_SEC, data, RTC_NR_TIME); > + MAX77686_ALARM1_SEC, data, ARRAY_SIZE(data)); > if (ret < 0) { > dev_err(info->dev, "%s: fail to write alarm reg(%d)\n", > __func__, ret); > @@ -341,7 +341,7 @@ static int max77686_rtc_set_alarm(struct device *dev, struct rtc_wkalrm *alrm) > goto out; > > ret = regmap_bulk_write(info->max77686->rtc_regmap, > - MAX77686_ALARM1_SEC, data, RTC_NR_TIME); > + MAX77686_ALARM1_SEC, data, ARRAY_SIZE(data)); > > if (ret < 0) { > dev_err(info->dev, "%s: fail to write alarm reg(%d)\n", > @@ -406,7 +406,8 @@ static int max77686_rtc_init_reg(struct max77686_rtc_info *info) > > info->rtc_24hr_mode = 1; > > - ret = regmap_bulk_write(info->max77686->rtc_regmap, MAX77686_RTC_CONTROLM, data, 2); > + ret = regmap_bulk_write(info->max77686->rtc_regmap, > + MAX77686_RTC_CONTROLM, data, ARRAY_SIZE(data)); > if (ret < 0) { > dev_err(info->dev, "%s: fail to write controlm reg(%d)\n", > __func__, ret); > -- > 2.5.0 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mailout3.samsung.com (mailout3.samsung.com. [203.254.224.33]) by gmr-mx.google.com with ESMTPS id 190si416401pfb.1.2016.01.26.17.20.36 for (version=TLS1 cipher=AES128-SHA bits=128/128); Tue, 26 Jan 2016 17:20:36 -0800 (PST) Received: from epcpsbgr4.samsung.com (u144.gpu120.samsung.co.kr [203.254.230.144]) by mailout3.samsung.com (Oracle Communications Messaging Server 7.0.5.31.0 64bit (built May 5 2014)) with ESMTP id <0O1L02ZFK7QB3LA0@mailout3.samsung.com> for rtc-linux@googlegroups.com; Wed, 27 Jan 2016 10:20:35 +0900 (KST) Date: Wed, 27 Jan 2016 10:20:34 +0900 From: Andi Shyti To: Javier Martinez Canillas Cc: linux-kernel@vger.kernel.org, Kukjin Kim , rtc-linux@googlegroups.com, Chanwoo Choi , Alexandre Belloni , Krzysztof Kozlowski , Laxman Dewangan , linux-samsung-soc@vger.kernel.org Subject: [rtc-linux] Re: [PATCH v3 02/10] rtc: max77686: Use ARRAY_SIZE() instead of current array length Message-id: <20160127012034.GB14296@samsunx.samsung> References: <1453836020-29579-1-git-send-email-javier@osg.samsung.com> <1453836020-29579-3-git-send-email-javier@osg.samsung.com> MIME-version: 1.0 Content-type: text/plain; charset=UTF-8 In-reply-to: <1453836020-29579-3-git-send-email-javier@osg.samsung.com> Reply-To: rtc-linux@googlegroups.com List-ID: List-Post: , List-Help: , List-Archive: , List-Unsubscribe: , On Tue, Jan 26, 2016 at 04:20:12PM -0300, Javier Martinez Canillas wrote: > It is better to use the ARRAY_SIZE() macro instead of the array length > to avoid bugs if the array is later changed and the length not updated. > > Signed-off-by: Javier Martinez Canillas > Reviewed-by: Krzysztof Kozlowski > Tested-by: Krzysztof Kozlowski Reviewed-by: Andi Shyti > --- > > Changes in v3: > - Add Krzysztof Kozlowski's Tested-by tag to patch #2. > - Add Laxman Dewangan's Acked-by tag to patch #2. > - Use ARRAY_SIZE() in all places. Suggested by Laxman Dewangan. > > Changes in v2: > - Add Krzysztof Kozlowski's Reviewed-by tag to patch #2. > > drivers/rtc/rtc-max77686.c | 23 ++++++++++++----------- > 1 file changed, 12 insertions(+), 11 deletions(-) > > diff --git a/drivers/rtc/rtc-max77686.c b/drivers/rtc/rtc-max77686.c > index 6653c3d11b66..d84a50c9f7f7 100644 > --- a/drivers/rtc/rtc-max77686.c > +++ b/drivers/rtc/rtc-max77686.c > @@ -149,7 +149,7 @@ static int max77686_rtc_read_time(struct device *dev, struct rtc_time *tm) > goto out; > > ret = regmap_bulk_read(info->max77686->rtc_regmap, > - MAX77686_RTC_SEC, data, RTC_NR_TIME); > + MAX77686_RTC_SEC, data, ARRAY_SIZE(data)); > if (ret < 0) { > dev_err(info->dev, "%s: fail to read time reg(%d)\n", __func__, ret); > goto out; > @@ -177,7 +177,7 @@ static int max77686_rtc_set_time(struct device *dev, struct rtc_time *tm) > mutex_lock(&info->lock); > > ret = regmap_bulk_write(info->max77686->rtc_regmap, > - MAX77686_RTC_SEC, data, RTC_NR_TIME); > + MAX77686_RTC_SEC, data, ARRAY_SIZE(data)); > if (ret < 0) { > dev_err(info->dev, "%s: fail to write time reg(%d)\n", __func__, > ret); > @@ -205,7 +205,7 @@ static int max77686_rtc_read_alarm(struct device *dev, struct rtc_wkalrm *alrm) > goto out; > > ret = regmap_bulk_read(info->max77686->rtc_regmap, > - MAX77686_ALARM1_SEC, data, RTC_NR_TIME); > + MAX77686_ALARM1_SEC, data, ARRAY_SIZE(data)); > if (ret < 0) { > dev_err(info->dev, "%s:%d fail to read alarm reg(%d)\n", > __func__, __LINE__, ret); > @@ -215,7 +215,7 @@ static int max77686_rtc_read_alarm(struct device *dev, struct rtc_wkalrm *alrm) > max77686_rtc_data_to_tm(data, &alrm->time, info->rtc_24hr_mode); > > alrm->enabled = 0; > - for (i = 0; i < RTC_NR_TIME; i++) { > + for (i = 0; i < ARRAY_SIZE(data); i++) { > if (data[i] & ALARM_ENABLE_MASK) { > alrm->enabled = 1; > break; > @@ -252,7 +252,7 @@ static int max77686_rtc_stop_alarm(struct max77686_rtc_info *info) > goto out; > > ret = regmap_bulk_read(info->max77686->rtc_regmap, > - MAX77686_ALARM1_SEC, data, RTC_NR_TIME); > + MAX77686_ALARM1_SEC, data, ARRAY_SIZE(data)); > if (ret < 0) { > dev_err(info->dev, "%s: fail to read alarm reg(%d)\n", > __func__, ret); > @@ -261,11 +261,11 @@ static int max77686_rtc_stop_alarm(struct max77686_rtc_info *info) > > max77686_rtc_data_to_tm(data, &tm, info->rtc_24hr_mode); > > - for (i = 0; i < RTC_NR_TIME; i++) > + for (i = 0; i < ARRAY_SIZE(data); i++) > data[i] &= ~ALARM_ENABLE_MASK; > > ret = regmap_bulk_write(info->max77686->rtc_regmap, > - MAX77686_ALARM1_SEC, data, RTC_NR_TIME); > + MAX77686_ALARM1_SEC, data, ARRAY_SIZE(data)); > if (ret < 0) { > dev_err(info->dev, "%s: fail to write alarm reg(%d)\n", > __func__, ret); > @@ -291,7 +291,7 @@ static int max77686_rtc_start_alarm(struct max77686_rtc_info *info) > goto out; > > ret = regmap_bulk_read(info->max77686->rtc_regmap, > - MAX77686_ALARM1_SEC, data, RTC_NR_TIME); > + MAX77686_ALARM1_SEC, data, ARRAY_SIZE(data)); > if (ret < 0) { > dev_err(info->dev, "%s: fail to read alarm reg(%d)\n", > __func__, ret); > @@ -312,7 +312,7 @@ static int max77686_rtc_start_alarm(struct max77686_rtc_info *info) > data[RTC_DATE] |= (1 << ALARM_ENABLE_SHIFT); > > ret = regmap_bulk_write(info->max77686->rtc_regmap, > - MAX77686_ALARM1_SEC, data, RTC_NR_TIME); > + MAX77686_ALARM1_SEC, data, ARRAY_SIZE(data)); > if (ret < 0) { > dev_err(info->dev, "%s: fail to write alarm reg(%d)\n", > __func__, ret); > @@ -341,7 +341,7 @@ static int max77686_rtc_set_alarm(struct device *dev, struct rtc_wkalrm *alrm) > goto out; > > ret = regmap_bulk_write(info->max77686->rtc_regmap, > - MAX77686_ALARM1_SEC, data, RTC_NR_TIME); > + MAX77686_ALARM1_SEC, data, ARRAY_SIZE(data)); > > if (ret < 0) { > dev_err(info->dev, "%s: fail to write alarm reg(%d)\n", > @@ -406,7 +406,8 @@ static int max77686_rtc_init_reg(struct max77686_rtc_info *info) > > info->rtc_24hr_mode = 1; > > - ret = regmap_bulk_write(info->max77686->rtc_regmap, MAX77686_RTC_CONTROLM, data, 2); > + ret = regmap_bulk_write(info->max77686->rtc_regmap, > + MAX77686_RTC_CONTROLM, data, ARRAY_SIZE(data)); > if (ret < 0) { > dev_err(info->dev, "%s: fail to write controlm reg(%d)\n", > __func__, ret); > -- > 2.5.0 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- -- You received this message because you are subscribed to "rtc-linux". Membership options at http://groups.google.com/group/rtc-linux . Please read http://groups.google.com/group/rtc-linux/web/checklist before submitting a driver. --- You received this message because you are subscribed to the Google Groups "rtc-linux" group. To unsubscribe from this group and stop receiving emails from it, send an email to rtc-linux+unsubscribe@googlegroups.com. For more options, visit https://groups.google.com/d/optout.