Skip to content

Commit c918d80

Browse files
committed
correct more error detection
1 parent 9a354a4 commit c918d80

File tree

1 file changed

+8
-8
lines changed

1 file changed

+8
-8
lines changed

ports/raspberrypi/common-hal/rp2pio/StateMachine.c

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -447,10 +447,10 @@ static void consider_instruction(introspect_t *state, uint16_t full_instruction,
447447
uint16_t bit_count = full_instruction & 0x001f;
448448
if (source == 0) {
449449
if (!state->has_in_pin) {
450-
mp_raise_ValueError_varg(translate("Missing first_in_pin. %q[%u] shifts in from pin(s)"), i);
450+
mp_raise_ValueError_varg(translate("Missing first_in_pin. %q[%u] shifts in from pin(s)"), what, i);
451451
}
452452
if (bit_count > state->in_pin_count) {
453-
mp_raise_ValueError_varg(translate("%q[%u] shifts in more bits than pin count"), i);
453+
mp_raise_ValueError_varg(translate("%q[%u] shifts in more bits than pin count"), what, i);
454454
}
455455
}
456456
if (state->auto_push) {
@@ -465,10 +465,10 @@ static void consider_instruction(introspect_t *state, uint16_t full_instruction,
465465
// Check for pins or pindirs destination.
466466
if (destination == 0x0 || destination == 0x4) {
467467
if (!state->has_out_pin) {
468-
mp_raise_ValueError_varg(translate("Missing first_out_pin. %q[%u] shifts out to pin(s)"), i);
468+
mp_raise_ValueError_varg(translate("Missing first_out_pin. %q[%u] shifts out to pin(s)"), what, i);
469469
}
470470
if (bit_count > state->out_pin_count) {
471-
mp_raise_ValueError_varg(translate("%q[%u] shifts out more bits than pin count"), i);
471+
mp_raise_ValueError_varg(translate("%q[%u] shifts out more bits than pin count"), what, i);
472472
}
473473
}
474474
if (state->auto_pull) {
@@ -481,18 +481,18 @@ static void consider_instruction(introspect_t *state, uint16_t full_instruction,
481481
uint16_t destination = (full_instruction & 0x00e0) >> 5;
482482
// Check for pins or pindirs destination.
483483
if ((destination == 0x00 || destination == 0x4) && !state->has_set_pin) {
484-
mp_raise_ValueError_varg(translate("Missing first_set_pin. %q[%u] sets pin(s)"), i);
484+
mp_raise_ValueError_varg(translate("Missing first_set_pin. %q[%u] sets pin(s)"), what, i);
485485
}
486486
}
487487
if (instruction == pio_instr_bits_mov) {
488488
uint16_t source = full_instruction & 0x0007;
489489
uint16_t destination = (full_instruction & 0x00e0) >> 5;
490490
// Check for pins or pindirs destination.
491491
if (destination == 0x0 && !state->has_out_pin) {
492-
mp_raise_ValueError_varg(translate("Missing first_out_pin. %q[%u] writes pin(s)"), i);
492+
mp_raise_ValueError_varg(translate("Missing first_out_pin. %q[%u] writes pin(s)"), what, i);
493493
}
494-
if (source == 0x0 && !state->has_out_pin) {
495-
mp_raise_ValueError_varg(translate("Missing first_in_pin. %q[%u] reads pin(s)"), i);
494+
if (source == 0x0 && !state->has_in_pin) {
495+
mp_raise_ValueError_varg(translate("Missing first_in_pin. %q[%u] reads pin(s)"), what, i);
496496
}
497497
if (destination == 0x6) {
498498
state->in_loaded = true;

0 commit comments

Comments
 (0)