Changeset 4854

Show
Ignore:
Timestamp:
09/23/07 15:53:11 (7 years ago)
Author:
khali
Message:

Delete unused SENSORS_ERR_ACCESS.
Rename SENSORS_ERR_PROC to the more neutral SENSORS_ERR_KERNEL.
Introduce SENSORS_ERR_NO_DEVS for finer-grained error reporting.
Use SENSORS_ERR_KERNEL and SENSORS_ERR_NO_DEVS where appropriate.
No error message for invalid error codes.

Location:
lm-sensors/branches/lm-sensors-3.0.0
Files:
5 modified

Legend:

Unmodified
Added
Removed
  • lm-sensors/branches/lm-sensors-3.0.0/lib/error.c

    r4836 r4854  
    22    error.c - Part of libsensors, a Linux library for reading sensor data. 
    33    Copyright (c) 1998, 1999  Frodo Looijaard <frodol@dds.nl> 
     4    Copyright (C) 2007        Jean Delvare <khali@linux-fr.org> 
    45 
    56    This program is free software; you can redistribute it and/or modify 
     
    3233 
    3334static const char *errorlist[] = { 
    34         /* Unknown error         */ "sensors_strerror: Unknown error!", 
     35        /* Invalid error code    */ NULL, 
    3536        /* SENSORS_ERR_WILDCARDS */ "Wildcard found in chip name", 
    3637        /* SENSORS_ERR_NO_ENTRY  */ "No such subfeature known", 
    37         /* SENSORS_ERR_ACCESS    */ "Can't read or write", 
    38         /* SENSORS_ERR_PROC      */ "Can't access sysfs file", 
     38        /* SENSORS_ERR_NO_DEVS   */ "No devices found", 
     39        /* SENSORS_ERR_KERNEL    */ "Kernel interface error", 
    3940        /* SENSORS_ERR_DIV_ZERO  */ "Divide by zero", 
    4041        /* SENSORS_ERR_CHIP_NAME */ "Can't parse chip name", 
  • lm-sensors/branches/lm-sensors-3.0.0/lib/error.h

    r4852 r4854  
    22    error.h - Part of libsensors, a Linux library for reading sensor data. 
    33    Copyright (c) 1998, 1999  Frodo Looijaard <frodol@dds.nl> 
     4    Copyright (C) 2007        Jean Delvare <khali@linux-fr.org> 
    45 
    56    This program is free software; you can redistribute it and/or modify 
     
    2324#define SENSORS_ERR_WILDCARDS   1 /* Wildcard found in chip name */ 
    2425#define SENSORS_ERR_NO_ENTRY    2 /* No such subfeature known */ 
    25 #define SENSORS_ERR_ACCESS      3 /* Can't read or write */ 
    26 #define SENSORS_ERR_PROC        4 /* Can't access /proc file */ 
     26#define SENSORS_ERR_NO_DEVS     3 /* No devices found */ 
     27#define SENSORS_ERR_KERNEL      4 /* Kernel interface error */ 
    2728#define SENSORS_ERR_DIV_ZERO    5 /* Divide by zero */ 
    2829#define SENSORS_ERR_CHIP_NAME   6 /* Can't parse chip name */ 
  • lm-sensors/branches/lm-sensors-3.0.0/lib/init.c

    r4834 r4854  
    3434 
    3535        if (!sensors_init_sysfs()) 
    36                 return -SENSORS_ERR_PROC; 
     36                return -SENSORS_ERR_KERNEL; 
    3737        if ((res = sensors_read_sysfs_bus()) || 
    3838            (res = sensors_read_sysfs_chips())) 
  • lm-sensors/branches/lm-sensors-3.0.0/lib/sysfs.c

    r4850 r4854  
    345345{ 
    346346        int domain, bus, slot, fn; 
    347         int err = -SENSORS_ERR_PARSE; 
     347        int err = -SENSORS_ERR_KERNEL; 
    348348        struct sysfs_attribute *attr, *bus_attr; 
    349349        char bus_path[SYSFS_PATH_MAX]; 
     
    431431        if (!(bus = sysfs_open_bus("i2c"))) { 
    432432                if (errno && errno != ENOENT) 
    433                         ret = -SENSORS_ERR_PROC; 
     433                        ret = -SENSORS_ERR_NO_DEVS; 
    434434                goto exit0; 
    435435        } 
     
    437437        if (!(devs = sysfs_get_bus_devices(bus))) { 
    438438                if (errno && errno != ENOENT) 
    439                         ret = -SENSORS_ERR_PROC; 
     439                        ret = -SENSORS_ERR_NO_DEVS; 
    440440                goto exit1; 
    441441        } 
     
    468468        if (!(clsdevs = sysfs_get_class_devices(cls))) { 
    469469                if (errno && errno != ENOENT) 
    470                         ret = -SENSORS_ERR_PROC; 
     470                        ret = -SENSORS_ERR_NO_DEVS; 
    471471                goto exit; 
    472472        } 
     
    475475                struct sysfs_device *dev; 
    476476                if (!(dev = sysfs_get_classdev_device(clsdev))) { 
    477                         ret = -SENSORS_ERR_PROC; 
     477                        ret = -SENSORS_ERR_NO_DEVS; 
    478478                        goto exit; 
    479479                } 
     
    499499        if (!(cls = sysfs_open_class("i2c-adapter"))) { 
    500500                if (errno && errno != ENOENT) 
    501                         ret = -SENSORS_ERR_PROC; 
     501                        ret = -SENSORS_ERR_KERNEL; 
    502502                goto exit0; 
    503503        } 
     
    505505        if (!(clsdevs = sysfs_get_class_devices(cls))) { 
    506506                if (errno && errno != ENOENT) 
    507                         ret = -SENSORS_ERR_PROC; 
     507                        ret = -SENSORS_ERR_KERNEL; 
    508508                goto exit1; 
    509509        } 
     
    554554                fclose(f); 
    555555                if (res != 1) 
    556                         return -SENSORS_ERR_PROC; 
     556                        return -SENSORS_ERR_KERNEL; 
    557557                *value /= get_type_scaling(subfeature->type); 
    558558        } else 
    559                 return -SENSORS_ERR_PROC; 
     559                return -SENSORS_ERR_KERNEL; 
    560560 
    561561        return 0; 
     
    575575                fclose(f); 
    576576        } else 
    577                 return -SENSORS_ERR_PROC; 
     577                return -SENSORS_ERR_KERNEL; 
    578578 
    579579        return 0; 
  • lm-sensors/branches/lm-sensors-3.0.0/prog/sensors/main.c

    r4781 r4854  
    172172 
    173173        if ((res = sensors_do_chip_sets(name))) { 
    174                 if (res == -SENSORS_ERR_PROC) { 
    175                         fprintf(stderr, "%s: %s for writing;\n", 
     174                if (res == -SENSORS_ERR_KERNEL) { 
     175                        fprintf(stderr, "%s: %s;\n", 
    176176                                sprintf_chip_name(name), 
    177177                                sensors_strerror(res));