diff options
author | Kim, Milo <Milo.Kim@ti.com> | 2013-03-14 20:19:29 -0400 |
---|---|---|
committer | Anton Vorontsov <anton@enomsg.org> | 2013-04-01 02:33:19 -0400 |
commit | 4b44a1ed82b4c2462d86fede459a802668b19d45 (patch) | |
tree | 60d966e6764df6ad3a450dd610756eb01b8ad043 /drivers/power | |
parent | feeb3a92b1f4e4b53bbad2b1c0d0eb1bcdd6d5a7 (diff) |
lp8788-charger: Use PAGE_SIZE for the sysfs read operation
The sysfs allocates PAGE_SIZE. It is used by each R/W operation method.
Use it instead of another buffer size.
Signed-off-by: Milo(Woogyom) Kim <milo.kim@ti.com>
Signed-off-by: Anton Vorontsov <anton@enomsg.org>
Diffstat (limited to 'drivers/power')
-rw-r--r-- | drivers/power/lp8788-charger.c | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/drivers/power/lp8788-charger.c b/drivers/power/lp8788-charger.c index 6d1f452810b8..ed49b50b220b 100644 --- a/drivers/power/lp8788-charger.c +++ b/drivers/power/lp8788-charger.c | |||
@@ -49,7 +49,6 @@ | |||
49 | #define LP8788_CHG_START 0x11 | 49 | #define LP8788_CHG_START 0x11 |
50 | #define LP8788_CHG_END 0x1C | 50 | #define LP8788_CHG_END 0x1C |
51 | 51 | ||
52 | #define LP8788_BUF_SIZE 40 | ||
53 | #define LP8788_ISEL_MAX 23 | 52 | #define LP8788_ISEL_MAX 23 |
54 | #define LP8788_ISEL_STEP 50 | 53 | #define LP8788_ISEL_STEP 50 |
55 | #define LP8788_VTERM_MIN 4100 | 54 | #define LP8788_VTERM_MIN 4100 |
@@ -633,7 +632,7 @@ static ssize_t lp8788_show_charger_status(struct device *dev, | |||
633 | lp8788_read_byte(pchg->lp, LP8788_CHG_STATUS, &data); | 632 | lp8788_read_byte(pchg->lp, LP8788_CHG_STATUS, &data); |
634 | state = (data & LP8788_CHG_STATE_M) >> LP8788_CHG_STATE_S; | 633 | state = (data & LP8788_CHG_STATE_M) >> LP8788_CHG_STATE_S; |
635 | 634 | ||
636 | return scnprintf(buf, LP8788_BUF_SIZE, "%s\n", desc[state]); | 635 | return scnprintf(buf, PAGE_SIZE, "%s\n", desc[state]); |
637 | } | 636 | } |
638 | 637 | ||
639 | static ssize_t lp8788_show_eoc_time(struct device *dev, | 638 | static ssize_t lp8788_show_eoc_time(struct device *dev, |
@@ -647,7 +646,7 @@ static ssize_t lp8788_show_eoc_time(struct device *dev, | |||
647 | lp8788_read_byte(pchg->lp, LP8788_CHG_EOC, &val); | 646 | lp8788_read_byte(pchg->lp, LP8788_CHG_EOC, &val); |
648 | val = (val & LP8788_CHG_EOC_TIME_M) >> LP8788_CHG_EOC_TIME_S; | 647 | val = (val & LP8788_CHG_EOC_TIME_M) >> LP8788_CHG_EOC_TIME_S; |
649 | 648 | ||
650 | return scnprintf(buf, LP8788_BUF_SIZE, "End Of Charge Time: %s\n", | 649 | return scnprintf(buf, PAGE_SIZE, "End Of Charge Time: %s\n", |
651 | stime[val]); | 650 | stime[val]); |
652 | } | 651 | } |
653 | 652 | ||
@@ -667,8 +666,7 @@ static ssize_t lp8788_show_eoc_level(struct device *dev, | |||
667 | val = (val & LP8788_CHG_EOC_LEVEL_M) >> LP8788_CHG_EOC_LEVEL_S; | 666 | val = (val & LP8788_CHG_EOC_LEVEL_M) >> LP8788_CHG_EOC_LEVEL_S; |
668 | level = mode ? abs_level[val] : relative_level[val]; | 667 | level = mode ? abs_level[val] : relative_level[val]; |
669 | 668 | ||
670 | return scnprintf(buf, LP8788_BUF_SIZE, "End Of Charge Level: %s\n", | 669 | return scnprintf(buf, PAGE_SIZE, "End Of Charge Level: %s\n", level); |
671 | level); | ||
672 | } | 670 | } |
673 | 671 | ||
674 | static DEVICE_ATTR(charger_status, S_IRUSR, lp8788_show_charger_status, NULL); | 672 | static DEVICE_ATTR(charger_status, S_IRUSR, lp8788_show_charger_status, NULL); |