Commit b48cc753 authored by Javier Carrasco's avatar Javier Carrasco Committed by Alexandre Belloni

rtc: pcf8523: fix coding-style issues

Minor modifications for coding-style correctness (tabs, spaces and blank
lines before and after brackets). In total 7 errors, 3 warnings and 1
check where removed from the checkpatch output without damaging code
readability.
Signed-off-by: default avatarJavier Carrasco <javier.carrasco@wolfvision.net>
Link: https://lore.kernel.org/r/20230315082021.2104452-2-javier.carrasco@wolfvision.netSigned-off-by: default avatarAlexandre Belloni <alexandre.belloni@bootlin.com>
parent cca1e2f7
...@@ -21,7 +21,7 @@ ...@@ -21,7 +21,7 @@
#define PCF8523_CONTROL2_AF BIT(3) #define PCF8523_CONTROL2_AF BIT(3)
#define PCF8523_REG_CONTROL3 0x02 #define PCF8523_REG_CONTROL3 0x02
#define PCF8523_CONTROL3_PM GENMASK(7,5) #define PCF8523_CONTROL3_PM GENMASK(7, 5)
#define PCF8523_PM_STANDBY 0x7 #define PCF8523_PM_STANDBY 0x7
#define PCF8523_CONTROL3_BLF BIT(2) /* battery low bit, read-only */ #define PCF8523_CONTROL3_BLF BIT(2) /* battery low bit, read-only */
#define PCF8523_CONTROL3_BSF BIT(3) #define PCF8523_CONTROL3_BSF BIT(3)
...@@ -234,8 +234,7 @@ static int pcf8523_param_get(struct device *dev, struct rtc_param *param) ...@@ -234,8 +234,7 @@ static int pcf8523_param_get(struct device *dev, struct rtc_param *param)
int ret; int ret;
u32 value; u32 value;
switch(param->param) { switch (param->param) {
case RTC_PARAM_BACKUP_SWITCH_MODE: case RTC_PARAM_BACKUP_SWITCH_MODE:
ret = regmap_read(pcf8523->regmap, PCF8523_REG_CONTROL3, &value); ret = regmap_read(pcf8523->regmap, PCF8523_REG_CONTROL3, &value);
if (ret < 0) if (ret < 0)
...@@ -243,7 +242,7 @@ static int pcf8523_param_get(struct device *dev, struct rtc_param *param) ...@@ -243,7 +242,7 @@ static int pcf8523_param_get(struct device *dev, struct rtc_param *param)
value = FIELD_GET(PCF8523_CONTROL3_PM, value); value = FIELD_GET(PCF8523_CONTROL3_PM, value);
switch(value) { switch (value) {
case 0x0: case 0x0:
case 0x4: case 0x4:
param->uvalue = RTC_BSM_LEVEL; param->uvalue = RTC_BSM_LEVEL;
...@@ -273,7 +272,7 @@ static int pcf8523_param_set(struct device *dev, struct rtc_param *param) ...@@ -273,7 +272,7 @@ static int pcf8523_param_set(struct device *dev, struct rtc_param *param)
struct pcf8523 *pcf8523 = dev_get_drvdata(dev); struct pcf8523 *pcf8523 = dev_get_drvdata(dev);
u8 mode; u8 mode;
switch(param->param) { switch (param->param) {
case RTC_PARAM_BACKUP_SWITCH_MODE: case RTC_PARAM_BACKUP_SWITCH_MODE:
switch (param->uvalue) { switch (param->uvalue) {
case RTC_BSM_DISABLED: case RTC_BSM_DISABLED:
...@@ -385,9 +384,9 @@ static const struct rtc_class_ops pcf8523_rtc_ops = { ...@@ -385,9 +384,9 @@ static const struct rtc_class_ops pcf8523_rtc_ops = {
}; };
static const struct regmap_config regmap_config = { static const struct regmap_config regmap_config = {
.reg_bits = 8, .reg_bits = 8,
.val_bits = 8, .val_bits = 8,
.max_register = 0x13, .max_register = 0x13,
}; };
static int pcf8523_probe(struct i2c_client *client) static int pcf8523_probe(struct i2c_client *client)
......
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