Skip to content

Commit e20c6da

Browse files
committed
Return integer from enum rather than string
1 parent 485af79 commit e20c6da

File tree

1 file changed

+3
-15
lines changed

1 file changed

+3
-15
lines changed

src/mrb_esp32_system.c

Lines changed: 3 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -57,21 +57,9 @@ mrb_esp32_get_chip_model(mrb_state *mrb, mrb_value self) {
5757
esp_chip_info_t info;
5858
esp_chip_info(&info);
5959

60-
// The integer in info.model is one of an enum. Values taken from here:
61-
// https://github.com/espressif/esp-idf/blob/67552c31dac8cd94fb0d63192a538f4f984c5b6e/components/esp_hw_support/include/esp_chip_info.h#L22C7-L22C7
62-
switch (info.model) {
63-
// String format matches style used when installing ESP-IDF.
64-
case 1: return mrb_str_new_cstr(mrb, "esp32");
65-
case 2: return mrb_str_new_cstr(mrb, "esp32s2");
66-
case 9: return mrb_str_new_cstr(mrb, "esp32s3");
67-
case 5: return mrb_str_new_cstr(mrb, "esp32c3");
68-
case 12: return mrb_str_new_cstr(mrb, "esp32c2");
69-
case 13: return mrb_str_new_cstr(mrb, "esp32c6");
70-
case 16: return mrb_str_new_cstr(mrb, "esp32h2");
71-
case 18: return mrb_str_new_cstr(mrb, "esp32p4");
72-
case 999: return mrb_str_new_cstr(mrb, "simulator");
73-
default: return mrb_str_new_cstr(mrb, "unknown");
74-
}
60+
// The integer value in info.model is one of an enum, esp_chip_model_t, found here:
61+
// https://github.com/espressif/esp-idf/blob/master/components/esp_hw_support/include/esp_chip_info.h
62+
return mrb_fixnum_value(info.model);
7563
}
7664

7765
void

0 commit comments

Comments
 (0)