gustavonihei commented on code in PR #6304:
URL: https://github.com/apache/incubator-nuttx/pull/6304#discussion_r878131916


##########
arch/risc-v/src/mpfs/mpfs_ihc.c:
##########
@@ -115,22 +114,30 @@ struct mpfs_rptun_dev_s
   char                       shmemname[RPMSG_NAME_SIZE + 1];
 };
 
-typedef enum
-{
-  IHC_CHANNEL_TO_HART0    = 0x00, /* Your hart to hart 0 */
-  IHC_CHANNEL_TO_HART1    = 0x01, /* Your hart to hart 1 */
-  IHC_CHANNEL_TO_HART2    = 0x02, /* Your hart to hart 2 */
-  IHC_CHANNEL_TO_HART3    = 0x03, /* Your hart to hart 3 */
-  IHC_CHANNEL_TO_HART4    = 0x04, /* Your hart to hart 4 */
-  IHC_CHANNEL_TO_CONTEXTA = 0x05, /* Your hart to context A */
-  IHC_CHANNEL_TO_CONTEXTB = 0x06, /* Your hart to context B */
-} ihc_channel_t;
-
 struct mpfs_queue_table
 {
   void *data;
 };
 
+enum
+{
+  MP_IRQ  = 0x0,
+  ACK_IRQ = 0x1,
+};
+
+struct mpfs_ihc_msg

Review Comment:
   ```suggestion
   struct mpfs_ihc_msg_s
   ```
   Structs names should take the `_s` suffix according to the coding standard.



##########
arch/risc-v/src/mpfs/mpfs_ihc.c:
##########
@@ -115,22 +114,30 @@ struct mpfs_rptun_dev_s
   char                       shmemname[RPMSG_NAME_SIZE + 1];
 };
 
-typedef enum
-{
-  IHC_CHANNEL_TO_HART0    = 0x00, /* Your hart to hart 0 */
-  IHC_CHANNEL_TO_HART1    = 0x01, /* Your hart to hart 1 */
-  IHC_CHANNEL_TO_HART2    = 0x02, /* Your hart to hart 2 */
-  IHC_CHANNEL_TO_HART3    = 0x03, /* Your hart to hart 3 */
-  IHC_CHANNEL_TO_HART4    = 0x04, /* Your hart to hart 4 */
-  IHC_CHANNEL_TO_CONTEXTA = 0x05, /* Your hart to context A */
-  IHC_CHANNEL_TO_CONTEXTB = 0x06, /* Your hart to context B */
-} ihc_channel_t;
-
 struct mpfs_queue_table
 {
   void *data;
 };
 
+enum
+{
+  MP_IRQ  = 0x0,
+  ACK_IRQ = 0x1,
+};
+
+struct mpfs_ihc_msg
+{
+  uint32_t msg[IHC_MAX_MESSAGE_SIZE];
+};
+
+/* Used to store information for the remote via ecall (eg. Linux) */
+
+struct ihc_sbi_rx_msg

Review Comment:
   ```suggestion
   struct ihc_sbi_rx_msg_s
   ```
   Structs names should take the `_s` suffix according to the coding standard.



-- 
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: commits-unsubscr...@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to