Skip to content

Commit 8d3a878

Browse files
authored
Merge pull request #3681 from adafruit/6.0.x
Merge protomatter update back into `main`
2 parents c995279 + 42ca57f commit 8d3a878

File tree

3 files changed

+12
-2
lines changed

3 files changed

+12
-2
lines changed

locale/circuitpython.pot

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -429,6 +429,11 @@ msgstr ""
429429
msgid "Bit clock and word select must share a clock unit"
430430
msgstr ""
431431

432+
#: shared-bindings/rgbmatrix/RGBMatrix.c
433+
#, c-format
434+
msgid "Bit depth must be from 1 to 6 inclusive, not %d"
435+
msgstr ""
436+
432437
#: shared-bindings/audiobusio/PDMIn.c
433438
msgid "Bit depth must be multiple of 8."
434439
msgstr ""

shared-bindings/rgbmatrix/RGBMatrix.c

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -197,6 +197,11 @@ STATIC mp_obj_t rgbmatrix_rgbmatrix_make_new(const mp_obj_type_t *type, size_t n
197197
uint8_t clock_pin = validate_pin(args[ARG_clock_pin].u_obj);
198198
uint8_t latch_pin = validate_pin(args[ARG_latch_pin].u_obj);
199199
uint8_t output_enable_pin = validate_pin(args[ARG_output_enable_pin].u_obj);
200+
int bit_depth = args[ARG_bit_depth].u_int;
201+
202+
if (bit_depth <= 0 || bit_depth > 6) {
203+
mp_raise_ValueError_varg(translate("Bit depth must be from 1 to 6 inclusive, not %d"), bit_depth);
204+
}
200205

201206
validate_pins(MP_QSTR_rgb_pins, rgb_pins, MP_ARRAY_SIZE(self->rgb_pins), args[ARG_rgb_list].u_obj, &rgb_count);
202207
validate_pins(MP_QSTR_addr_pins, addr_pins, MP_ARRAY_SIZE(self->addr_pins), args[ARG_addr_list].u_obj, &addr_count);
@@ -229,7 +234,7 @@ STATIC mp_obj_t rgbmatrix_rgbmatrix_make_new(const mp_obj_type_t *type, size_t n
229234

230235
common_hal_rgbmatrix_rgbmatrix_construct(self,
231236
args[ARG_width].u_int,
232-
args[ARG_bit_depth].u_int,
237+
bit_depth,
233238
rgb_count, rgb_pins,
234239
addr_count, addr_pins,
235240
clock_pin, latch_pin, output_enable_pin,

0 commit comments

Comments
 (0)