Show
Ignore:
Timestamp:
11/02/10 13:59:31 (3 years ago)
Author:
khali
Message:

Work with the feature type instead of the shifted subfeature type where
possible. This makes the code more readable.

Files:
1 modified

Legend:

Unmodified
Added
Removed

  • lm-sensors/trunk/lib/sysfs.c

    r5874 r5875  
    383383                if (sftype == SENSORS_SUBFEATURE_UNKNOWN) 
    384384                        continue; 
     385                ftype = sftype >> 8; 
    385386 
    386387                /* Adjust the channel number */ 
    387                 switch (sftype & 0xFF00) { 
    388                 case SENSORS_SUBFEATURE_FAN_INPUT: 
    389                 case SENSORS_SUBFEATURE_TEMP_INPUT: 
    390                 case SENSORS_SUBFEATURE_POWER_AVERAGE: 
    391                 case SENSORS_SUBFEATURE_ENERGY_INPUT: 
    392                 case SENSORS_SUBFEATURE_CURR_INPUT: 
     388                switch (ftype) { 
     389                case SENSORS_FEATURE_FAN: 
     390                case SENSORS_FEATURE_TEMP: 
     391                case SENSORS_FEATURE_POWER: 
     392                case SENSORS_FEATURE_ENERGY: 
     393                case SENSORS_FEATURE_CURR: 
    393394                        nr--; 
     395                        break; 
     396                default: 
    394397                        break; 
    395398                } 
    … …  
    407410                /* "calculate" a place to store the subfeature in our sparse, 
    408411                   sorted table */ 
    409                 switch (sftype) { 
    410                 case SENSORS_SUBFEATURE_VID: 
     412                switch (ftype) { 
     413                case SENSORS_FEATURE_VID: 
    411414                        i = SUB_OFFSET_OTHER + 
    412                             ((sftype >> 8) - SENSORS_FEATURE_VID) * 
    413                             FEATURE_TYPE_SIZE + 
     415                            (ftype - SENSORS_FEATURE_VID) * FEATURE_TYPE_SIZE + 
    414416                            nr * FEATURE_SIZE + (sftype & 0xFF); 
    415417                        break; 
    416                 case SENSORS_SUBFEATURE_BEEP_ENABLE: 
     418                case SENSORS_FEATURE_BEEP_ENABLE: 
    417419                        i = SUB_OFFSET_MISC + 
    418                             ((sftype >> 8) - SENSORS_FEATURE_BEEP_ENABLE); 
     420                            (ftype - SENSORS_FEATURE_BEEP_ENABLE); 
    419421                        break; 
    420422                default: 
    421                         i = (sftype >> 8) * FEATURE_TYPE_SIZE + 
     423                        i = ftype * FEATURE_TYPE_SIZE + 
    422424                            nr * FEATURE_SIZE + 
    423425                            ((sftype & 0x80) >> 7) * MAX_SUBFEATURES +