[PATCH 09/26] lib: fwts_battery: remove redundant framework parameter in helper functions
Keng-Yu Lin
kengyu at canonical.com
Wed Oct 17 07:42:02 UTC 2012
On Mon, Oct 15, 2012 at 4:32 AM, Colin King <colin.king at canonical.com> wrote:
> From: Colin Ian King <colin.king at canonical.com>
>
> We don't require the fwts_framework parameter in the following functions:
>
> fwts_battery_get_count_sys_fs
> fwts_battery_get_count_proc_fs
> fwts_battery_get_name_sys_fs
> fwts_battery_get_name_proc_fs
>
> ..so remove it.
>
> Signed-off-by: Colin Ian King <colin.king at canonical.com>
> ---
> src/lib/src/fwts_battery.c | 16 ++++++++--------
> 1 file changed, 8 insertions(+), 8 deletions(-)
>
> diff --git a/src/lib/src/fwts_battery.c b/src/lib/src/fwts_battery.c
> index 656cc33..b792d0b 100644
> --- a/src/lib/src/fwts_battery.c
> +++ b/src/lib/src/fwts_battery.c
> @@ -176,7 +176,7 @@ static int fwts_battery_get_capacity_proc_fs(fwts_framework *fw,
> return FWTS_OK;
> }
>
> -static int fwts_battery_get_count_sys_fs(fwts_framework *fw, DIR *dir, int *count)
> +static int fwts_battery_get_count_sys_fs(DIR *dir, int *count)
> {
> struct dirent *entry;
> char path[PATH_MAX];
> @@ -197,7 +197,7 @@ static int fwts_battery_get_count_sys_fs(fwts_framework *fw, DIR *dir, int *coun
> return FWTS_OK;
> }
>
> -static int fwts_battery_get_count_proc_fs(fwts_framework *fw, DIR *dir, int *count)
> +static int fwts_battery_get_count_proc_fs(DIR *dir, int *count)
> {
> struct dirent *entry;
> do {
> @@ -208,7 +208,7 @@ static int fwts_battery_get_count_proc_fs(fwts_framework *fw, DIR *dir, int *cou
> return FWTS_OK;
> }
>
> -static int fwts_battery_get_name_sys_fs(fwts_framework *fw, DIR *dir, int index, char *name)
> +static int fwts_battery_get_name_sys_fs(DIR *dir, int index, char *name)
> {
> struct dirent *entry;
> char path[PATH_MAX];
> @@ -242,7 +242,7 @@ static int fwts_battery_get_name_sys_fs(fwts_framework *fw, DIR *dir, int index,
> return FWTS_ERROR;
> }
>
> -static int fwts_battery_get_name_proc_fs(fwts_framework *fw, DIR *dir, int index, char *name)
> +static int fwts_battery_get_name_proc_fs(DIR *dir, int index, char *name)
> {
> struct dirent *entry;
> int i = 0;
> @@ -591,10 +591,10 @@ int fwts_battery_get_name(fwts_framework *fw, int index, char *name)
> DIR *dir;
>
> if ((dir = opendir(FWTS_SYS_CLASS_POWER_SUPPLY)) != NULL) {
> - ret = fwts_battery_get_name_sys_fs(fw, dir, index, name);
> + ret = fwts_battery_get_name_sys_fs(dir, index, name);
> closedir(dir);
> } else if ((dir = opendir(FWTS_PROC_ACPI_BATTERY)) != NULL) {
> - ret = fwts_battery_get_name_proc_fs(fw, dir, index, name);
> + ret = fwts_battery_get_name_proc_fs(dir, index, name);
> closedir(dir);
> } else {
> return FWTS_ERROR;
> @@ -609,10 +609,10 @@ int fwts_battery_get_count(fwts_framework *fw, int *count)
> DIR *dir;
>
> if ((dir = opendir(FWTS_SYS_CLASS_POWER_SUPPLY)) != NULL) {
> - ret = fwts_battery_get_count_sys_fs(fw, dir, count);
> + ret = fwts_battery_get_count_sys_fs(dir, count);
> closedir(dir);
> } else if ((dir = opendir(FWTS_PROC_ACPI_BATTERY)) != NULL) {
> - ret = fwts_battery_get_count_proc_fs(fw, dir, count);
> + ret = fwts_battery_get_count_proc_fs(dir, count);
> closedir(dir);
> } else {
> return FWTS_ERROR;
> --
> 1.7.10.4
>
Acked-by: Keng-Yu Lin <kengyu at canonical.com>
More information about the fwts-devel
mailing list