stm32/pyb_can: Include requested CAN baudrate in matching error.

projectgus · dpgeorge · commit 221a4ecf300d · 2025-01-17T12:13:17.000+11:00
This is redundant for bxCAN, but for CAN-FD with BRS it's otherwise unclear
which set of parameters (baudrate &amp; sample_point or brs_baudrate &amp;
brs_sample_point) failed to match. This makes finding a valid combination
extra annoying.

This work was funded through GitHub Sponsors.

Signed-off-by: Angus Gratton &lt;angus@redyak.com.au&gt;
diff --git a/ports/stm32/pyb_can.c b/ports/stm32/pyb_can.c
@@ -222,7 +222,7 @@ static void pyb_can_get_bit_timing(mp_uint_t baudrate, mp_uint_t sample_point,
         }
     }
 
-    mp_raise_msg(&mp_type_ValueError, MP_ERROR_TEXT("couldn't match baudrate and sample point"));
+    mp_raise_msg_varg(&mp_type_ValueError, MP_ERROR_TEXT("couldn't match baudrate %u and sample point %u"), baudrate, sample_point / 10);
 }
 
 // init(mode, prescaler=100, *, sjw=1, bs1=6, bs2=8)

Original file line number	Diff line number	Diff line change
`@@ -222,7 +222,7 @@ static void pyb_can_get_bit_timing(mp_uint_t baudrate, mp_uint_t sample_point,`
`222`	`222`	`}`
`223`	`223`	`}`
`224`	`224`
`225`		`- mp_raise_msg(&mp_type_ValueError, MP_ERROR_TEXT("couldn't match baudrate and sample point"));`
	`225`	`+ mp_raise_msg_varg(&mp_type_ValueError, MP_ERROR_TEXT("couldn't match baudrate %u and sample point %u"), baudrate, sample_point / 10);`
`226`	`226`	`}`
`227`	`227`
`228`	`228`	`// init(mode, prescaler=100, *, sjw=1, bs1=6, bs2=8)`