From 9bc0d06522286cbe59d7a32ea376ed94bc8b18a7 Mon Sep 17 00:00:00 2001 From: Alex Duchesne Date: Tue, 27 Aug 2024 10:49:38 -0400 Subject: [PATCH] rg_gui: Expose more LED configuration options --- components/retro-go/rg_gui.c | 27 +++++++++++++++++++++------ 1 file changed, 21 insertions(+), 6 deletions(-) diff --git a/components/retro-go/rg_gui.c b/components/retro-go/rg_gui.c index 31a39e65e..245f4ee8f 100644 --- a/components/retro-go/rg_gui.c +++ b/components/retro-go/rg_gui.c @@ -1225,13 +1225,28 @@ static rg_gui_event_t speedup_update_cb(rg_gui_option_t *option, rg_gui_event_t return RG_DIALOG_VOID; } -static rg_gui_event_t disk_activity_cb(rg_gui_option_t *option, rg_gui_event_t event) +static rg_gui_event_t led_indicator_opt_cb(rg_gui_option_t *option, rg_gui_event_t event) { if (event == RG_DIALOG_PREV || event == RG_DIALOG_NEXT) { - rg_system_set_indicator_mask(RG_INDICATOR_DISK_ACTIVITY, !rg_system_get_indicator_mask(RG_INDICATOR_DISK_ACTIVITY)); + rg_system_set_indicator_mask(option->arg, !rg_system_get_indicator_mask(option->arg)); + } + strcpy(option->value, rg_system_get_indicator_mask(option->arg) ? "On " : "Off"); + return RG_DIALOG_VOID; +} + +static rg_gui_event_t led_indicator_cb(rg_gui_option_t *option, rg_gui_event_t event) +{ + if (event == RG_DIALOG_ENTER) + { + const rg_gui_option_t options[] = { + {RG_INDICATOR_SYSTEM_ACTIVITY, "System activity", "-", RG_DIALOG_FLAG_NORMAL, &led_indicator_opt_cb}, + {RG_INDICATOR_DISK_ACTIVITY, "Disk activity", "-", RG_DIALOG_FLAG_NORMAL, &led_indicator_opt_cb}, + {RG_INDICATOR_LOW_BATTERY, "Low battery", "-", RG_DIALOG_FLAG_NORMAL, &led_indicator_opt_cb}, + RG_DIALOG_END, + }; + rg_gui_dialog("LED options", &options, 0); } - strcpy(option->value, rg_system_get_indicator_mask(RG_INDICATOR_DISK_ACTIVITY) ? "On " : "Off"); return RG_DIALOG_VOID; } @@ -1349,13 +1364,13 @@ void rg_gui_options_menu(void) // Global settings that aren't essential to show when inside a game if (app->isLauncher) { -#ifdef RG_GPIO_LED // Only show disk LED option if disk LED GPIO pin is defined - *opt++ = (rg_gui_option_t){0, "Disk LED ", "-", RG_DIALOG_FLAG_NORMAL, &disk_activity_cb}; -#endif *opt++ = (rg_gui_option_t){0, "Font type ", "-", RG_DIALOG_FLAG_NORMAL, &font_type_cb}; *opt++ = (rg_gui_option_t){0, "Theme ", "-", RG_DIALOG_FLAG_NORMAL, &theme_cb}; *opt++ = (rg_gui_option_t){0, "Show clock", "-", RG_DIALOG_FLAG_NORMAL, &show_clock_cb}; *opt++ = (rg_gui_option_t){0, "Timezone ", "-", RG_DIALOG_FLAG_NORMAL, &timezone_cb}; +#ifdef RG_GPIO_LED // Only show disk LED option if disk LED GPIO pin is defined + *opt++ = (rg_gui_option_t){0, "LED options", NULL, RG_DIALOG_FLAG_NORMAL, &led_indicator_cb}; +#endif } // App settings that are shown only inside a game else