Skip to content

qmi-framework: Unify API naming and error handling conventions - #10

Merged
Deepak Kumar Singh (quic-deesin) merged 2 commits into
mainfrom
development
Dec 10, 2025
Merged

Deepak Kumar Singh (quic-deesin) merged 2 commits into
mainfrom
development

Conversation

@quic-pphansal

Copy link
Copy Markdown
Contributor

Standardize the QMI Common Client Interface (CCI) and QMI Common Service
Interface (CSI) APIs to follow consistent naming patterns and error handling
approaches. This change improves code readability, maintainability, and
developer experience by:

  1. Renaming files to follow consistent naming patterns:

    • qmi_client.h → qmi_cci.h
    • qcsi.h → qmi_csi.h
  2. Standardizing API function names with consistent prefixes:

    • Client APIs use qmi_cci_* prefix (e.g., qmi_cci_init, qmi_cci_send_msg)
    • Service APIs use qmi_csi_* prefix (e.g., qmi_csi_register, qmi_csi_send_resp)
  3. Converting error code definitions from preprocessor macros to enums in the
    client interface for improved type safety and debugging, matching the
    approach already used in the service interface.

Standardize the QMI Common Client Interface (CCI) and QMI Common Service
Interface (CSI) APIs to follow consistent naming patterns and error handling
approaches. This change improves code readability, maintainability, and
developer experience by:

1. Renaming files to follow consistent naming patterns:
   - qmi_client.h → qmi_cci.h
   - qcsi.h → qmi_csi.h

2. Standardizing API function names with consistent prefixes:
   - Client APIs use qmi_cci_* prefix (e.g., qmi_cci_init, qmi_cci_send_msg)
   - Service APIs use qmi_csi_* prefix (e.g., qmi_csi_register, qmi_csi_send_resp)

3. Converting error code definitions from preprocessor macros to enums in the
   client interface for improved type safety and debugging, matching the
   approach already used in the service interface.

Signed-off-by: Pranav Mahesh Phansalkar <pranav.phansalkar@oss.qualcomm.com>
qmi-framework: Unify API naming and error handling conventions
@quic-deesin
Deepak Kumar Singh (quic-deesin) merged commit b3b2dc0 into main Dec 10, 2025
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants