We don't need to duplicate if (res) checks if we're always running
one or the other.
Signed-off-by: Kyle McMartin <kyle@redhat.com>
Signed-off-by: Guenter Roeck <guenter.roeck@ericsson.com>
- if (resource->caps.configurable_cap) {
+ if (resource->caps.configurable_cap)
res = register_attrs(resource, rw_cap_attrs);
res = register_attrs(resource, rw_cap_attrs);
- if (res)
- goto error;
- } else {
res = register_attrs(resource, ro_cap_attrs);
res = register_attrs(resource, ro_cap_attrs);
- if (res)
- goto error;
- }
+
+ if (res)
+ goto error;
+
res = register_attrs(resource, misc_cap_attrs);
if (res)
goto error;
}
res = register_attrs(resource, misc_cap_attrs);
if (res)
goto error;
}
if (resource->caps.flags & POWER_METER_CAN_TRIP) {
res = register_attrs(resource, trip_attrs);
if (res)
if (resource->caps.flags & POWER_METER_CAN_TRIP) {
res = register_attrs(resource, trip_attrs);
if (res)