From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753229AbcFOGA6 (ORCPT ); Wed, 15 Jun 2016 02:00:58 -0400 Received: from mail-pf0-f195.google.com ([209.85.192.195]:34338 "EHLO mail-pf0-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752736AbcFOGAS (ORCPT ); Wed, 15 Jun 2016 02:00:18 -0400 From: Andrey Smirnov To: rtc-linux@googlegroups.com Cc: Alessandro Zummo , Alexandre Belloni , linux-kernel@vger.kernel.org, Andrey Smirnov Subject: [PATCH 11/13] RTC: ds1307: Redefine RX8025_REG_* to minimize extra code Date: Tue, 14 Jun 2016 22:59:37 -0700 Message-Id: <1465970379-14703-12-git-send-email-andrew.smirnov@gmail.com> X-Mailer: git-send-email 2.5.5 In-Reply-To: <1465970379-14703-1-git-send-email-andrew.smirnov@gmail.com> References: <1465970379-14703-1-git-send-email-andrew.smirnov@gmail.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org The only place in the driver where RX8025_REG_* are used they are always shifted and ORed the same way, so instead of repeating that idiom make it a part of symbolic constant. Signed-off-by: Andrey Smirnov --- drivers/rtc/rtc-ds1307.c | 18 ++++++++++-------- 1 file changed, 10 insertions(+), 8 deletions(-) diff --git a/drivers/rtc/rtc-ds1307.c b/drivers/rtc/rtc-ds1307.c index 8ccfe5b..76e66a3 100644 --- a/drivers/rtc/rtc-ds1307.c +++ b/drivers/rtc/rtc-ds1307.c @@ -103,9 +103,11 @@ enum ds_type { #define DS13XX_TRICKLE_CHARGER_MAGIC 0xa0 -#define RX8025_REG_CTRL1 0x0e +#define RX8025_REG_CTRL1_ 0x0e +#define RX8025_REG_CTRL1 ((RX8025_REG_CTRL1_ << 4) | 0x08) # define RX8025_BIT_2412 0x20 -#define RX8025_REG_CTRL2 0x0f +#define RX8025_REG_CTRL2_ 0x0f +#define RX8025_REG_CTRL2 ((RX8025_REG_CTRL2_ << 4) | 0x08) # define RX8025_BIT_PON 0x10 # define RX8025_BIT_VDET 0x40 # define RX8025_BIT_XST 0x20 @@ -1353,7 +1355,7 @@ static int ds1307_chip_configure(const struct ds1307 *ds1307) } case rx_8025: tmp = i2c_smbus_read_i2c_block_data(client, - RX8025_REG_CTRL1 << 4 | 0x08, + RX8025_REG_CTRL1, 2, regs); if (tmp != 2) { dev_dbg(&client->dev, "read error %d\n", tmp); @@ -1364,7 +1366,7 @@ static int ds1307_chip_configure(const struct ds1307 *ds1307) if (!(regs[1] & RX8025_BIT_XST)) { regs[1] |= RX8025_BIT_XST; i2c_smbus_write_byte_data(client, - RX8025_REG_CTRL2 << 4 | 0x08, + RX8025_REG_CTRL2, regs[1]); dev_warn(&client->dev, "oscillator stop detected - SET TIME!\n"); @@ -1373,7 +1375,7 @@ static int ds1307_chip_configure(const struct ds1307 *ds1307) if (regs[1] & RX8025_BIT_PON) { regs[1] &= ~RX8025_BIT_PON; i2c_smbus_write_byte_data(client, - RX8025_REG_CTRL2 << 4 | 0x08, + RX8025_REG_CTRL2, regs[1]); dev_warn(&client->dev, "power-on detected\n"); } @@ -1381,7 +1383,7 @@ static int ds1307_chip_configure(const struct ds1307 *ds1307) if (regs[1] & RX8025_BIT_VDET) { regs[1] &= ~RX8025_BIT_VDET; i2c_smbus_write_byte_data(client, - RX8025_REG_CTRL2 << 4 | 0x08, + RX8025_REG_CTRL2, regs[1]); dev_warn(&client->dev, "voltage drop detected\n"); } @@ -1392,11 +1394,11 @@ static int ds1307_chip_configure(const struct ds1307 *ds1307) /* switch to 24 hour mode */ i2c_smbus_write_byte_data(client, - RX8025_REG_CTRL1 << 4 | 0x08, + RX8025_REG_CTRL1, regs[0] | RX8025_BIT_2412); tmp = i2c_smbus_read_i2c_block_data(client, - RX8025_REG_CTRL1 << 4 | 0x08, + RX8025_REG_CTRL1, 2, regs); if (tmp != 2) { dev_dbg(&client->dev, "read error %d\n", tmp); -- 2.5.5 From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Sender: rtc-linux@googlegroups.com MIME-Version: 1.0 Received: from mail-pf0-x241.google.com (mail-pf0-x241.google.com. [2607:f8b0:400e:c00::241]) by gmr-mx.google.com with ESMTPS id 62si3602201pfx.1.2016.06.14.23.00.13 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Tue, 14 Jun 2016 23:00:13 -0700 (PDT) Received: by mail-pf0-x241.google.com with SMTP id 62so986725pfd.3 for ; Tue, 14 Jun 2016 23:00:13 -0700 (PDT) From: Andrey Smirnov To: rtc-linux@googlegroups.com Cc: Alessandro Zummo , Alexandre Belloni , linux-kernel@vger.kernel.org, Andrey Smirnov Subject: [rtc-linux] [PATCH 11/13] RTC: ds1307: Redefine RX8025_REG_* to minimize extra code Date: Tue, 14 Jun 2016 22:59:37 -0700 Message-Id: <1465970379-14703-12-git-send-email-andrew.smirnov@gmail.com> In-Reply-To: <1465970379-14703-1-git-send-email-andrew.smirnov@gmail.com> References: <1465970379-14703-1-git-send-email-andrew.smirnov@gmail.com> Reply-To: rtc-linux@googlegroups.com Content-Type: text/plain; charset=UTF-8 List-ID: List-Post: , List-Help: , List-Archive: , List-Unsubscribe: , The only place in the driver where RX8025_REG_* are used they are always shifted and ORed the same way, so instead of repeating that idiom make it a part of symbolic constant. Signed-off-by: Andrey Smirnov --- drivers/rtc/rtc-ds1307.c | 18 ++++++++++-------- 1 file changed, 10 insertions(+), 8 deletions(-) diff --git a/drivers/rtc/rtc-ds1307.c b/drivers/rtc/rtc-ds1307.c index 8ccfe5b..76e66a3 100644 --- a/drivers/rtc/rtc-ds1307.c +++ b/drivers/rtc/rtc-ds1307.c @@ -103,9 +103,11 @@ enum ds_type { #define DS13XX_TRICKLE_CHARGER_MAGIC 0xa0 -#define RX8025_REG_CTRL1 0x0e +#define RX8025_REG_CTRL1_ 0x0e +#define RX8025_REG_CTRL1 ((RX8025_REG_CTRL1_ << 4) | 0x08) # define RX8025_BIT_2412 0x20 -#define RX8025_REG_CTRL2 0x0f +#define RX8025_REG_CTRL2_ 0x0f +#define RX8025_REG_CTRL2 ((RX8025_REG_CTRL2_ << 4) | 0x08) # define RX8025_BIT_PON 0x10 # define RX8025_BIT_VDET 0x40 # define RX8025_BIT_XST 0x20 @@ -1353,7 +1355,7 @@ static int ds1307_chip_configure(const struct ds1307 *ds1307) } case rx_8025: tmp = i2c_smbus_read_i2c_block_data(client, - RX8025_REG_CTRL1 << 4 | 0x08, + RX8025_REG_CTRL1, 2, regs); if (tmp != 2) { dev_dbg(&client->dev, "read error %d\n", tmp); @@ -1364,7 +1366,7 @@ static int ds1307_chip_configure(const struct ds1307 *ds1307) if (!(regs[1] & RX8025_BIT_XST)) { regs[1] |= RX8025_BIT_XST; i2c_smbus_write_byte_data(client, - RX8025_REG_CTRL2 << 4 | 0x08, + RX8025_REG_CTRL2, regs[1]); dev_warn(&client->dev, "oscillator stop detected - SET TIME!\n"); @@ -1373,7 +1375,7 @@ static int ds1307_chip_configure(const struct ds1307 *ds1307) if (regs[1] & RX8025_BIT_PON) { regs[1] &= ~RX8025_BIT_PON; i2c_smbus_write_byte_data(client, - RX8025_REG_CTRL2 << 4 | 0x08, + RX8025_REG_CTRL2, regs[1]); dev_warn(&client->dev, "power-on detected\n"); } @@ -1381,7 +1383,7 @@ static int ds1307_chip_configure(const struct ds1307 *ds1307) if (regs[1] & RX8025_BIT_VDET) { regs[1] &= ~RX8025_BIT_VDET; i2c_smbus_write_byte_data(client, - RX8025_REG_CTRL2 << 4 | 0x08, + RX8025_REG_CTRL2, regs[1]); dev_warn(&client->dev, "voltage drop detected\n"); } @@ -1392,11 +1394,11 @@ static int ds1307_chip_configure(const struct ds1307 *ds1307) /* switch to 24 hour mode */ i2c_smbus_write_byte_data(client, - RX8025_REG_CTRL1 << 4 | 0x08, + RX8025_REG_CTRL1, regs[0] | RX8025_BIT_2412); tmp = i2c_smbus_read_i2c_block_data(client, - RX8025_REG_CTRL1 << 4 | 0x08, + RX8025_REG_CTRL1, 2, regs); if (tmp != 2) { dev_dbg(&client->dev, "read error %d\n", tmp); -- 2.5.5 -- 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.