Fix the comment for RIL_Data_Call_Response as well.

[android-x86/hardware-ril.git] / include / telephony / ril.h

diff --git a/include/telephony/ril.h b/include/telephony/ril.h
index f96bcf8..d3c7569 100644 (file)
--- a/include/telephony/ril.h
+++ b/include/telephony/ril.h
@@ -160,8 +160,8 @@ typedef struct {
     char *          type;       /* One of the PDP_type values in TS 27.007 section 10.1.1.
                                    For example, "IP", "IPV6", "IPV4V6", or "PPP". */
     char *          apn;
-    char *          address;    /* A space-delimited list of addresses, e.g., "192.0.1.3" or
-                                   "192.0.1.11 2001:db8::1". */
+    char *          address;    /* The IPv4 or IPv6 address assigned to the call, e.g., "192.0.1.3"
+                                   or "2001:db8::1". */
 } RIL_Data_Call_Response;
 
 typedef struct {
@@ -871,6 +871,9 @@ typedef struct {
  *
  * Hang up a specific line (like AT+CHLD=1x)
  *
+ * After this HANGUP request returns, RIL should show the connection is NOT
+ * active anymore in next RIL_REQUEST_GET_CURRENT_CALLS query.
+ *
  * "data" is an int *
  * (int *)data)[0] contains Connection index (value of 'x' in CHLD above)
  *
@@ -889,6 +892,9 @@ typedef struct {
  *
  * Hang up waiting or held (like AT+CHLD=0)
  *
+ * After this HANGUP request returns, RIL should show the connection is NOT
+ * active anymore in next RIL_REQUEST_GET_CURRENT_CALLS query.
+ *
  * "data" is NULL
  * "response" is NULL
  *
@@ -905,6 +911,9 @@ typedef struct {
  *
  * Hang up waiting or held (like AT+CHLD=1)
  *
+ * After this HANGUP request returns, RIL should show the connection is NOT
+ * active anymore in next RIL_REQUEST_GET_CURRENT_CALLS query.
+ *
  * "data" is NULL
  * "response" is NULL
  *
@@ -1314,15 +1323,19 @@ typedef struct {
  *                          For example, "IP", "IPV6", "IPV4V6", or "PPP".
  *
  * "response" is a char **
- * ((char **)response)[0] indicating PDP CID, which is generated by RIL. This Connection ID is
- *                          used in GSM/UMTS and CDMA
- * ((char **)response)[1] indicating the network interface name for GSM/UMTS or CDMA
- * ((char **)response)[2] a space-separated list of IP addresses for this interface for GSM/UMTS
- *                          and NULL for CDMA
- *
- * FIXME may need way to configure QoS settings
+ * ((char **)response)[0] the Connection ID, CID, which is generated by RIL.
+ * ((char **)response)[1] the network interface name.
+ * ((char **)response)[2] a numeric IPv4 or IPv6 address that has been assigned to the interface.
+ * ((char **)response)[3] a space-separated list of numeric IPv4 or IPv6 DNS addresses.
+ *                        Ignored on Android platforms before 3.0 and instead two DNS IP addresses
+ *                        are retrieved from system properties "net.$IN.dns1" and "net.$IN.dns2".
+ * ((char **)response)[4] the numeric IPv4 or IPv6 address of the default gateway.
+ *                        Ignored on Android platforms before 3.0 and instead an IP address
+ *                        is retrieved from system property "net.$IN.gw".
  *
- * replaces  RIL_REQUEST_SETUP_DEFAULT_PDP
+ * Notes:
+ *   1) Numeric addresses must be in the Java InetAddress parsable representation.
+ *   2) $IN in the above comments is the interface name from response[1].
  *
  * Valid errors:
  *  SUCCESS
@@ -1617,6 +1630,9 @@ typedef struct {
  *
  * "data" is const char **
  * ((char**)data)[0] indicating CID
+ * ((char**)data)[1] indicating Disconnect Reason
+ *                   0 => No specific reason specified
+ *                   1 => Radio shutdown requested
  *
  * "response" is NULL
  *