Attention is currently required from: fixeria, dexter.

Patch set 2:Code-Review +1

View Change

1 comment:

• File src/osmo-bsc/abis_om2000.c:

  • Patch Set #1, Line 1016: enum abis_nm_op_state oml_opstate_from_om2k_opstate(uint8_t op_state)

    I am not sure about the naming scheme. "abis" is one layer below. […]

    Why do you say abis is one layer below? AFAIU OM2K is (part of) an implementation of the Abis interface, which deviates from what is specified in TS 12.21.

    abis_nm_op_sate_from_om2k_op_state()
    om2k_to_abis_nm_op_state()

To view, visit change 28914. To unsubscribe, or for help writing mail filters, visit settings.