stmhal: Rename mp_const_vcp_interrupt to mp_kbd_exception.

dpgeorge · dpgeorge · commit f254cfd3c464 · 2016-12-15T12:32:00.000+11:00
mp_kbd_exception is now considered the standard variable name to hold the
singleton KeyboardInterrupt exception.

This patch also moves the creation of this object from pyb_usb_init() to
main().
diff --git a/stmhal/main.c b/stmhal/main.c
@@ -438,6 +438,7 @@ int main(void) {
     mp_obj_list_append(mp_sys_path, MP_OBJ_NEW_QSTR(MP_QSTR__slash_flash));
     mp_obj_list_append(mp_sys_path, MP_OBJ_NEW_QSTR(MP_QSTR__slash_flash_slash_lib));
     mp_obj_list_init(mp_sys_argv, 0);
+    MP_STATE_PORT(mp_kbd_exception) = mp_obj_new_exception(&mp_type_KeyboardInterrupt);
 
     // Initialise low-level sub-systems.  Here we need to very basic things like
     // zeroing out memory and resetting any of the sub-systems.  Following this
diff --git a/stmhal/mpconfigport.h b/stmhal/mpconfigport.h
@@ -208,7 +208,7 @@ extern const struct _mp_obj_module_t mp_module_network;
 #define MICROPY_PORT_ROOT_POINTERS \
     const char *readline_hist[8]; \
     \
-    mp_obj_t mp_const_vcp_interrupt; \
+    mp_obj_t mp_kbd_exception; \
     mp_obj_t pyb_hid_report_desc; \
     \
     mp_obj_t pyb_config_main; \
diff --git a/stmhal/pendsv.c b/stmhal/pendsv.c
@@ -36,7 +36,7 @@
 // PENDSV call that actually raises the exception.  It must be non-static
 // otherwise gcc-5 optimises it away.  It can point to the heap but is not
 // traced by GC.  This is okay because we only ever set it to
-// mp_const_vcp_interrupt which is in the root-pointer set.
+// mp_kbd_exception which is in the root-pointer set.
 void *pendsv_object;
 
 void pendsv_init(void) {
diff --git a/stmhal/usb.c b/stmhal/usb.c
@@ -96,9 +96,7 @@ const mp_obj_tuple_t pyb_usb_hid_keyboard_obj = {
 };
 
 void pyb_usb_init0(void) {
-    // create an exception object for interrupting by VCP
-    MP_STATE_PORT(mp_const_vcp_interrupt) = mp_obj_new_exception(&mp_type_KeyboardInterrupt);
-    USBD_CDC_SetInterrupt(-1, MP_STATE_PORT(mp_const_vcp_interrupt));
+    USBD_CDC_SetInterrupt(-1, MP_STATE_PORT(mp_kbd_exception));
     MP_STATE_PORT(pyb_hid_report_desc) = MP_OBJ_NULL;
 }
 
@@ -146,9 +144,9 @@ bool usb_vcp_is_enabled(void) {
 void usb_vcp_set_interrupt_char(int c) {
     if (pyb_usb_flags & PYB_USB_FLAG_DEV_ENABLED) {
         if (c != -1) {
-            mp_obj_exception_clear_traceback(MP_STATE_PORT(mp_const_vcp_interrupt));
+            mp_obj_exception_clear_traceback(MP_STATE_PORT(mp_kbd_exception));
         }
-        USBD_CDC_SetInterrupt(c, MP_STATE_PORT(mp_const_vcp_interrupt));
+        USBD_CDC_SetInterrupt(c, MP_STATE_PORT(mp_kbd_exception));
     }
 }
 

Original file line number	Diff line number	Diff line change
`@@ -208,7 +208,7 @@ extern const struct _mp_obj_module_t mp_module_network;`
`208`	`208`	`#define MICROPY_PORT_ROOT_POINTERS \`
`209`	`209`	`const char *readline_hist[8]; \`
`210`	`210`	`\`
`211`		`- mp_obj_t mp_const_vcp_interrupt; \`
	`211`	`+ mp_obj_t mp_kbd_exception; \`
`212`	`212`	`mp_obj_t pyb_hid_report_desc; \`
`213`	`213`	`\`
`214`	`214`	`mp_obj_t pyb_config_main; \`
Original file line number	Diff line number	Diff line change
`@@ -96,9 +96,7 @@ const mp_obj_tuple_t pyb_usb_hid_keyboard_obj = {`
`96`	`96`	`};`
`97`	`97`
`98`	`98`	`void pyb_usb_init0(void) {`
`99`		`- // create an exception object for interrupting by VCP`
`100`		`- MP_STATE_PORT(mp_const_vcp_interrupt) = mp_obj_new_exception(&mp_type_KeyboardInterrupt);`
`101`		`- USBD_CDC_SetInterrupt(-1, MP_STATE_PORT(mp_const_vcp_interrupt));`
	`99`	`+ USBD_CDC_SetInterrupt(-1, MP_STATE_PORT(mp_kbd_exception));`
`102`	`100`	`MP_STATE_PORT(pyb_hid_report_desc) = MP_OBJ_NULL;`
`103`	`101`	`}`
`104`	`102`
`@@ -146,9 +144,9 @@ bool usb_vcp_is_enabled(void) {`
`146`	`144`	`void usb_vcp_set_interrupt_char(int c) {`
`147`	`145`	`if (pyb_usb_flags & PYB_USB_FLAG_DEV_ENABLED) {`
`148`	`146`	`if (c != -1) {`
`149`		`- mp_obj_exception_clear_traceback(MP_STATE_PORT(mp_const_vcp_interrupt));`
	`147`	`+ mp_obj_exception_clear_traceback(MP_STATE_PORT(mp_kbd_exception));`
`150`	`148`	`}`
`151`		`- USBD_CDC_SetInterrupt(c, MP_STATE_PORT(mp_const_vcp_interrupt));`
	`149`	`+ USBD_CDC_SetInterrupt(c, MP_STATE_PORT(mp_kbd_exception));`
`152`	`150`	`}`
`153`	`151`	`}`
`154`	`152`