extra extra... assertion catches bug

kilograham · kilograham · commit b3552b8a3616 · 2024-11-08T18:55:30.000-06:00
diff --git a/src/rp2_common/hardware_clocks/clocks.c b/src/rp2_common/hardware_clocks/clocks.c
@@ -245,7 +245,9 @@ void clock_gpio_init_int_frac16(uint gpio, uint src, uint32_t div_int, uint16_t
         invalid_params_if(HARDWARE_CLOCKS, true);
     }
 
+#if !PICO_RP2040 // assert currently broken on RP2040, but we know that hardware has 16 bit integer part
     static_assert(CLOCKS_CLK_GPOUT0_DIV_INT_MSB - CLOCKS_CLK_GPOUT0_DIV_INT_LSB == 15, "");
+#endif
     invalid_params_if(HARDWARE_CLOCKS, div_int >> 16);
     // Set up the gpclk generator
     clocks_hw->clk[gpclk].ctrl = (src << CLOCKS_CLK_GPOUT0_CTRL_AUXSRC_LSB) |

Original file line number	Diff line number	Diff line change
`@@ -245,7 +245,9 @@ void clock_gpio_init_int_frac16(uint gpio, uint src, uint32_t div_int, uint16_t`
`245`	`245`	`invalid_params_if(HARDWARE_CLOCKS, true);`
`246`	`246`	`}`
`247`	`247`
	`248`	`+#if !PICO_RP2040 // assert currently broken on RP2040, but we know that hardware has 16 bit integer part`
`248`	`249`	`static_assert(CLOCKS_CLK_GPOUT0_DIV_INT_MSB - CLOCKS_CLK_GPOUT0_DIV_INT_LSB == 15, "");`
	`250`	`+#endif`
`249`	`251`	`invalid_params_if(HARDWARE_CLOCKS, div_int >> 16);`
`250`	`252`	`// Set up the gpclk generator`
`251`	`253`	`clocks_hw->clk[gpclk].ctrl = (src << CLOCKS_CLK_GPOUT0_CTRL_AUXSRC_LSB) \|`