Small fixes and doxygen update

Author: Antti Kauppila

features/cellular/framework/AT/ATHandler.h

@@ -281,7 +281,7 @@ class ATHandler {
      * @param resp_buf_size Response buffer size
      * @param format Format string for variadic arguments to be added to AT command; No separator needed.
      *        Use %d for integer, %s for string and %b for byte string (requires 2 arguments: string and length)
-     * @return @return last error that happened when parsing AT responses
+     * @return last error that happened when parsing AT responses
      */
     nsapi_error_t at_cmd_str(const char *cmd, const char *cmd_chr, char *resp_buf, size_t resp_buf_size, const char *format = "", ...);
 
@@ -292,18 +292,17 @@ class ATHandler {
      * @param resp Integer to hold response
      * @param format Format string for variadic arguments to be added to AT command; No separator needed.
      *        Use %d for integer, %s for string and %b for byte string (requires 2 arguments: string and length)
-     * @return @return last error that happened when parsing AT responses
+     * @return last error that happened when parsing AT responses
      */
     nsapi_error_t at_cmd_int(const char *cmd, const char *cmd_chr, int &resp, const char *format = "", ...);
 
     /**
      * @brief at_cmd_discard Send an AT command and read and discard a response. Locks and unlocks ATHandler for operation
      * @param cmd AT command in form +<CMD> (will be used also in response reading, no extra chars allowed)
      * @param cmd_chr Char to be added to specific AT command: '?', '=' or ''. Will be used as such so '=1' is valid as well.
-     * @param resp Integer to hold response
      * @param format Format string for variadic arguments to be added to AT command; No separator needed.
      *        Use %d for integer, %s for string and %b for byte string (requires 2 arguments: string and length)
-     * @return @return last error that happened when parsing AT responses
+     * @return last error that happened when parsing AT responses
      */
     nsapi_error_t at_cmd_discard(const char *cmd, const char *cmd_chr, const char *format = "", ...);
 

features/cellular/framework/AT/AT_CellularNetwork.cpp

@@ -216,6 +216,7 @@ nsapi_error_t AT_CellularNetwork::set_registration(const char *plmn)
         if (mode != NWModeAutomatic) {
             return _at.at_cmd_discard("+COPS", "=0");
         }
+        return NSAPI_ERROR_OK;
     } else {
         tr_debug("Manual network registration to %s", plmn);
         if (_op_act != RAT_UNKNOWN) {
@@ -325,7 +326,7 @@ nsapi_error_t AT_CellularNetwork::scan_plmn(operList_t &operators, int &opsCount
 
     _at.lock();
 
-    _at.cmd_start_stop("+COPS", "?");
+    _at.cmd_start_stop("+COPS", "=?");
 
     _at.resp_start("+COPS:");