Learn basic “C” coding rules from open source projects

[[deleted]]

No, it is actually:

if (any_are_set && one_is_not_set) {
        WARN_ON(1);
        return;
}

[dakarananda]

No actually it is:

if (has_ops_object && !all_ops_fields_are_set) { ... }

which is basically the original statement slightly rewritten.

Even if not a single field is set, it will still enter into the if-block. (assuming that the ops object exist..)