xiaoxiang781216 commented on a change in pull request #5256:
URL: https://github.com/apache/incubator-nuttx/pull/5256#discussion_r787666136



##########
File path: include/nuttx/board.h
##########
@@ -429,6 +429,23 @@ int board_composite_initialize(int port);
 FAR void *board_composite_connect(int port, int configid);
 #endif
 
+/****************************************************************************
+ * Name:  board_usbdev_serialstr
+ *
+ * Description:
+ *   Use board unique serial number string to iSerialNumber field in the
+ *   device descriptor. This is for determining the board when multiple
+ *   boards on the same host.
+ *
+ * Returned Value:
+ *   The board unique serial number string.
+ *
+ ****************************************************************************/
+
+#if defined(CONFIG_BOARD_USBDEV_SERIALSTR)
+FAR const char *board_usbdev_serialstr(void);

Review comment:
       But should we give a more general name here? It's too bad to couple 
serial string to usbdev. I could prefer that:
   ```
   #if defined(CONFIG_BOARD_UNIQUESTR)
   FAR const char *board_uniquestr(void);
   #endif
   ```
   and provide a defined implementation on top of board_uniqueid if no special 
need.




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to