Convert more modules to use MP_REGISTER_MODULE

Convert adafruit_bus_device, adafruit_pixelbuf, analogio, atexit, audiobusio, audiocore, audioio, audiomixer, and audiomp3 modules to use MP_REGISTER_MODULE. Related to #5183.
2021-08-25 20:48:55 -04:00 · 2021-08-25 20:48:55 -04:00 · ad103ac6f1
parent 13cba8a17f
commit ad103ac6f1
10 changed files with 27 additions and 70 deletions
--- a/py/circuitpy_mpconfig.h
+++ b/py/circuitpy_mpconfig.h
@ -258,54 +258,13 @@ extern const struct _mp_obj_module_t alarm_module;
 #define ALARM_MODULE
 #endif

-#if CIRCUITPY_ANALOGIO
-#define ANALOGIO_MODULE        { MP_OBJ_NEW_QSTR(MP_QSTR_analogio), (mp_obj_t)&analogio_module },
-extern const struct _mp_obj_module_t analogio_module;
-#else
-#define ANALOGIO_MODULE
-#endif
-
-#if CIRCUITPY_ATEXIT
-extern const struct _mp_obj_module_t atexit_module;
-#define ATEXIT_MODULE           { MP_OBJ_NEW_QSTR(MP_QSTR_atexit), (mp_obj_t)&atexit_module },
-#else
-#define ATEXIT_MODULE
-#endif
-
-#if CIRCUITPY_AUDIOBUSIO
-#define AUDIOBUSIO_MODULE        { MP_OBJ_NEW_QSTR(MP_QSTR_audiobusio), (mp_obj_t)&audiobusio_module },
-extern const struct _mp_obj_module_t audiobusio_module;
-#else
-#define AUDIOBUSIO_MODULE
-#endif
-
-#if CIRCUITPY_AUDIOCORE
-#define AUDIOCORE_MODULE         { MP_OBJ_NEW_QSTR(MP_QSTR_audiocore), (mp_obj_t)&audiocore_module },
-extern const struct _mp_obj_module_t audiocore_module;
-#else
-#define AUDIOCORE_MODULE
-#endif
-
-#if CIRCUITPY_AUDIOIO
-#define AUDIOIO_MODULE         { MP_OBJ_NEW_QSTR(MP_QSTR_audioio), (mp_obj_t)&audioio_module },
-extern const struct _mp_obj_module_t audioio_module;
-#else
-#define AUDIOIO_MODULE
-#endif
-
-#if CIRCUITPY_AUDIOMIXER
-#define AUDIOMIXER_MODULE         { MP_OBJ_NEW_QSTR(MP_QSTR_audiomixer), (mp_obj_t)&audiomixer_module },
-extern const struct _mp_obj_module_t audiomixer_module;
-#else
-#define AUDIOMIXER_MODULE
-#endif
-
-#if CIRCUITPY_AUDIOMP3
-#define AUDIOMP3_MODULE         { MP_OBJ_NEW_QSTR(MP_QSTR_audiomp3), (mp_obj_t)&audiomp3_module },
-extern const struct _mp_obj_module_t audiomp3_module;
-#else
-#define AUDIOMP3_MODULE
-#endif
+// CIRCUITPY_ANALOGIO uses MP_REGISTER_MODULE
+// CIRCUITPY_ATEXIT uses MP_REGISTER_MODULE
+// CIRCUITPY_AUDIOBUSIO uses MP_REGISTER_MODULE
+// CIRCUITPY_AUDIOCORE uses MP_REGISTER_MODULE
+// CIRCUITPY_AUDIOIO uses MP_REGISTER_MODULE
+// CIRCUITPY_AUDIOMIXER uses MP_REGISTER_MODULE
+// CIRCUITPY_AUDIOMP3 uses MP_REGISTER_MODULE

 #if CIRCUITPY_AUDIOPWMIO
 #define AUDIOPWMIO_MODULE         { MP_OBJ_NEW_QSTR(MP_QSTR_audiopwmio), (mp_obj_t)&audiopwmio_module },
@ -370,12 +329,7 @@ extern const struct _mp_obj_module_t board_module;
 #define BOARD_UART_ROOT_POINTER
 #endif

-#if CIRCUITPY_BUSDEVICE
-extern const struct _mp_obj_module_t adafruit_bus_device_module;
-#define BUSDEVICE_MODULE           { MP_OBJ_NEW_QSTR(MP_QSTR_adafruit_bus_device), (mp_obj_t)&adafruit_bus_device_module },
-#else
-#define BUSDEVICE_MODULE
-#endif
+// CIRCUITPY_BUSDEVICE (adafruit_bus_device_module) uses MP_REGISTER_MODULE

 #if CIRCUITPY_BUSIO
 extern const struct _mp_obj_module_t busio_module;
@ -614,13 +568,7 @@ extern const struct _mp_obj_module_t pew_module;
 #define PEW_MODULE
 #endif

-#if CIRCUITPY_PIXELBUF
-extern const struct _mp_obj_module_t pixelbuf_module;
-#define PIXELBUF_MODULE   { MP_OBJ_NEW_QSTR(MP_QSTR_adafruit_pixelbuf),(mp_obj_t)&pixelbuf_module }, \
-    { MP_OBJ_NEW_QSTR(MP_QSTR__pixelbuf),(mp_obj_t)&pixelbuf_module },
-#else
-#define PIXELBUF_MODULE
-#endif
+// CIRCUITPY_PIXELBUF (pixelbuf_module) uses MP_REGISTER_MODULE

 #if CIRCUITPY_PS2IO
 extern const struct _mp_obj_module_t ps2io_module;
@ -883,13 +831,6 @@ extern const struct _mp_obj_module_t msgpack_module;
 #define MICROPY_PORT_BUILTIN_MODULES_STRONG_LINKS \
    AESIO_MODULE \
    ALARM_MODULE \
-    ANALOGIO_MODULE \
-    ATEXIT_MODULE \
-    AUDIOBUSIO_MODULE \
-    AUDIOCORE_MODULE \
-    AUDIOIO_MODULE \
-    AUDIOMIXER_MODULE \
-    AUDIOMP3_MODULE \
    AUDIOPWMIO_MODULE \
    BINASCII_MODULE \
    BITBANGIO_MODULE \
@ -897,7 +838,6 @@ extern const struct _mp_obj_module_t msgpack_module;
    BITOPS_MODULE \
    BLEIO_MODULE \
    BOARD_MODULE \
-    BUSDEVICE_MODULE \
    BUSIO_MODULE \
    CAMERA_MODULE \
    CANIO_MODULE \
@ -925,7 +865,6 @@ extern const struct _mp_obj_module_t msgpack_module;
    NEOPIXEL_WRITE_MODULE \
    ONEWIREIO_MODULE \
    PEW_MODULE \
-    PIXELBUF_MODULE \
    PS2IO_MODULE \
    PULSEIO_MODULE \
    PWMIO_MODULE \
--- a/shared-bindings/adafruit_bus_device/init.c
+++ b/shared-bindings/adafruit_bus_device/init.c
@ -76,3 +76,5 @@ const mp_obj_module_t adafruit_bus_device_module = {
    .base = { &mp_type_module },
    .globals = (mp_obj_dict_t *)&adafruit_bus_device_module_globals,
 };
+
+MP_REGISTER_MODULE(MP_QSTR_adafruit_bus_device, adafruit_bus_device_module, CIRCUITPY_BUSDEVICE);
--- a/shared-bindings/adafruit_pixelbuf/init.c
+++ b/shared-bindings/adafruit_pixelbuf/init.c
@ -54,3 +54,5 @@ const mp_obj_module_t pixelbuf_module = {
    .base = { &mp_type_module },
    .globals = (mp_obj_dict_t *)&pixelbuf_module_globals,
 };
+
+MP_REGISTER_MODULE(MP_QSTR_adafruit_pixelbuf, pixelbuf_module, CIRCUITPY_PIXELBUF);
--- a/shared-bindings/analogio/init.c
+++ b/shared-bindings/analogio/init.c
@ -72,3 +72,5 @@ const mp_obj_module_t analogio_module = {
    .base = { &mp_type_module },
    .globals = (mp_obj_dict_t *)&analogio_module_globals,
 };
+
+MP_REGISTER_MODULE(MP_QSTR_analogio, analogio_module, CIRCUITPY_ANALOGIO);
--- a/shared-bindings/atexit/init.c
+++ b/shared-bindings/atexit/init.c
@ -91,3 +91,5 @@ const mp_obj_module_t atexit_module = {
    .base = { &mp_type_module },
    .globals = (mp_obj_dict_t *)&atexit_module_globals,
 };
+
+MP_REGISTER_MODULE(MP_QSTR_atexit, atexit_module, CIRCUITPY_ATEXIT);
--- a/shared-bindings/audiobusio/init.c
+++ b/shared-bindings/audiobusio/init.c
@ -58,3 +58,5 @@ const mp_obj_module_t audiobusio_module = {
    .base = { &mp_type_module },
    .globals = (mp_obj_dict_t *)&audiobusio_module_globals,
 };
+
+MP_REGISTER_MODULE(MP_QSTR_audiobusio, audiobusio_module, CIRCUITPY_AUDIOBUSIO);
--- a/shared-bindings/audiocore/init.c
+++ b/shared-bindings/audiocore/init.c
@ -50,3 +50,5 @@ const mp_obj_module_t audiocore_module = {
    .base = { &mp_type_module },
    .globals = (mp_obj_dict_t *)&audiocore_module_globals,
 };
+
+MP_REGISTER_MODULE(MP_QSTR_audiocore, audiocore_module, CIRCUITPY_AUDIOCORE);
--- a/shared-bindings/audioio/init.c
+++ b/shared-bindings/audioio/init.c
@ -61,3 +61,5 @@ const mp_obj_module_t audioio_module = {
    .base = { &mp_type_module },
    .globals = (mp_obj_dict_t *)&audioio_module_globals,
 };
+
+MP_REGISTER_MODULE(MP_QSTR_audioio, audioio_module, CIRCUITPY_AUDIOIO);
--- a/shared-bindings/audiomixer/init.c
+++ b/shared-bindings/audiomixer/init.c
@ -46,3 +46,5 @@ const mp_obj_module_t audiomixer_module = {
    .base = { &mp_type_module },
    .globals = (mp_obj_dict_t *)&audiomixer_module_globals,
 };
+
+MP_REGISTER_MODULE(MP_QSTR_audiomixer, audiomixer_module, CIRCUITPY_AUDIOMIXER);
--- a/shared-bindings/audiomp3/init.c
+++ b/shared-bindings/audiomp3/init.c
@ -45,3 +45,5 @@ const mp_obj_module_t audiomp3_module = {
    .base = { &mp_type_module },
    .globals = (mp_obj_dict_t *)&audiomp3_module_globals,
 };
+
+MP_REGISTER_MODULE(MP_QSTR_audiomp3, audiomp3_module, CIRCUITPY_AUDIOMP3);