return ret_val;
}
-Device *Factory::create_w1_device(int family_code_param,
+Device *Factory::create_device(int family_code_param,
string device_id_param,
dirent *direntry_param) {
Device *ret_val;
ret_val = NULL;
type = get_device_type_by_family_code(family_code_param);
if (type.empty() == false) {
- ret_val = create_w1_device(family_code_param,
+ ret_val = create_device(family_code_param,
type,
device_id_param,
direntry_param);
return ret_val;
}
-Device *Factory::create_w1_device(int family_code_param,
+Device *Factory::create_device(int device_w1_family_code_param,
+ string device_type_param,
string device_id_param,
- string type_param,
dirent *direntry_param) {
Device *ret_val;
DeviceConfig *config;
+ bool succ;
ret_val = NULL;
- if (type_param.empty() == false) {
- switch(family_code_param) {
+ if (device_type_param.empty() == false) {
+ switch(device_w1_family_code_param) {
case 0x10:
case 0x28:
- ret_val = new W1TemperatureSensor(device_id_param, type_param, direntry_param);
+ ret_val = new W1TemperatureSensor(device_id_param, device_type_param, direntry_param);
break;
case 0x1d:
- ret_val = new W1CounterDevice(device_id_param, type_param, direntry_param);
+ ret_val = new W1CounterDevice(device_id_param, device_type_param, direntry_param);
break;
case 0x81:
// 0x81 is the 1-wire USB dongle... No need to create device for it.
break;
default:
- log_debug("Unsupported 1-wire-family code: %#x, device not created: %s\n", family_code_param, device_id_param.c_str());
+ log_debug("Unsupported 1-wire-family code: %#x, device not created: %s\n", device_w1_family_code_param, device_id_param.c_str());
break;
}
if (ret_val != NULL) {
config = DeviceConfig::get_device_config(device_id_param);
if (config != NULL) {
// if not, create default device config
- type_param = config->get_config_value(DEVICE_CONFIG_VALUE_KEY__TYPE);
- if (type_param.empty() == true) {
- type_param = ret_val->get_type();
- config->set_config_value(DEVICE_CONFIG_VALUE_KEY__TYPE, type_param);
- config->set_config_value(DEVICE_CONFIG_VALUE_KEY__ID, ret_val->get_id());
+ succ = config->get_config_value(DEVICE_CONFIG_VALUE_KEY__TYPE, device_type_param);
+ if (succ == false) {
+ device_type_param = ret_val->get_type();
+ config->set_config_value(DEVICE_CONFIG_VALUE_KEY__TYPE, device_type_param, false);
+ config->set_config_value(DEVICE_CONFIG_VALUE_KEY__ID, ret_val->get_id(), true);
}
delete(config);
}
return ret_val;
}
-Device *Factory::create_w1_device(string device_type_param,
+Device *Factory::create_device(string device_type_param,
string device_id_param) {
int family_code;
Device *ret_val;
- ret_val = NULL;
+ ret_val = NULL;
if (device_type_param.empty() == false) {
family_code = get_family_code_by_device_type(device_type_param);
if (family_code != -1) {
- ret_val = create_w1_device(family_code, device_type_param, device_id_param, NULL);
+ ret_val = create_device(family_code, device_type_param, device_id_param, NULL);
}
}
return ret_val;
}
-Device *Factory::create_w1_device(dirent *direntry_param,
+Device *Factory::create_device(dirent *direntry_param,
int *err_code_param) {
string folder_name;
string tmp_str;
- string device_name;
+ string device_id;
int pos;
int family_code;
bool suc_flg;
if (family_code != 0x81) {
if (suc_flg == true) {
log_debug("1-wire device family code: %#x\n", family_code);
- device_name = folder_name.substr(pos + 1, folder_name.length() - pos);
- ret_val = Factory::create_w1_device(family_code,
- device_name,
+ device_id = folder_name.substr(pos + 1, folder_name.length() - pos);
+ ret_val = Factory::create_device(family_code,
+ device_id,
direntry_param);
if ((ret_val == NULL) &&
(family_code != 0x81)) {
is_subdir = FileUtil::is_subdirectory(W1_SCAN_ROOTDIR, direntry);
if (is_subdir == true) {
err_flg = 0;
- device = create_w1_device(direntry, &err_flg);
+ device = create_device(direntry, &err_flg);
if (device != NULL) {
ret_val.push_back(device);
}
}
if (found == false) {
// reader device is not in the list of active devices. create and add it to list as in-active one...
- type = reader->get_device_type();
- if (type.empty() == false) {
- device = create_w1_device(type, id1);
+ found = reader->get_device_type(type);
+ if (found == true) {
+ device = create_device(type, id1);
if (device != NULL) {
ret_val.push_back(device);
}