Comment 1 for bug 1263229

It looks like the logic for this SS is really messed up in some modems. Network-wise, what I see is:

mako:
test-ss "*31#" -> No line number shown in called phone. Shown status is "disabled".
test-ss "#31#" -> Line number shown in called phone. Shown status is "enabled".

maguro:
test-ss "*31#" -> Line number shown in called phone. Shown status is "enabled".
test-ss "#31#" -> No line number shown in called phone. Shown status is "enabled".

In both cases, RIL_REQUEST_SET_CLIR is the same for the same string (*31# or #31#). SIM card is the same.

RIL_REQUEST_GET_CLIR returns:

mako:
After (test-ss "*31#"): {2,4}
After (test-ss "#31#"): {1,4}

maguro:
After (test-ss "*31#"): {2,3}
After (test-ss "#31#"): {1,4}

The reason for the different status displayed is because of the interpretation that is done by the second integer in core ofono call_settings.c:

 case CLIR_STATUS_TEMPORARY_RESTRICTED: // 3
  if (override == OFONO_CLIR_OPTION_SUPPRESSION) // 2
   value = "enabled";
  else
   value = "disabled";
  break;

 case CLIR_STATUS_TEMPORARY_ALLOWED: // 4
  if (override == OFONO_CLIR_OPTION_INVOCATION) // 1
   value = "enabled";
  else
   value = "disabled";
  break;

The values returned by maguro do not really match this.

To complicate things more, ofono core uses inverse logic when activating/deactivating this SS. For control type SS_CONTROL_TYPE_ACTIVATION, it calls clir_set(OFONO_CLIR_OPTION_SUPPRESSION), and for SS_CONTROL_TYPE_DEACTIVATION it calls clir_set(OFONO_CLIR_OPTION_INVOCATION). I do not think this is right, but it certainly matches maguro's behaviour, but not mako's (that is the reason for having the # command activating the SS instead of the * command, which seems to be the right thing looking at the standards).

So, my conclusion is that not all modems behave the right way regarding this SS and somewhat the messy ofono core code proves this. To get this right in both mako and maguro, we need different implementations, or detect the modem type and behave accordingly.