One tick KScan wait on RP2350 as for RP2040

[peterjc]

PR check-list

• Branch has a clean commit history
• Additional tests are included, if changing behaviors/core code that is testable.
• Proper Copyright + License headers added to applicable files (Generally, we stick to "The ZMK Contributors" for copyrights to help avoid churn when files get edited)
• Pre-commit used to check formatting of files, commit messages, etc.
• Includes any necessary documentation changes.

Based on an observation made on the ZMK Discord, I tested this change on my diode-free Slump52 keyboard using active low with open drain (a recent change which seems to be working fine with the RP2040).

RP2350 on Base ZMK - single key presses worked, but some like Qwerty M and L triggered a nearby key as well.

RP2350 With this change - all single key presses seemed to work

Test built here: peterjc/zmk-keyboard-graph-theory@630b77f

[petejohanson]

Is there a family we can key on for all of RP2350, so we catch this properly once the Hazard stuff is in, or folks use the B variants?

[peterjc]

I don't know - I did wonder about having the logical-or include the Hazard form too.

Maybe SOC_SERIES_RP2350 would do it? See https://github.com/zmkfirmware/zephyr/blob/v4.1.0%2Bzmk-fixes/soc/raspberrypi/rpi_pico/rp2350/Kconfig or https://github.com/zmkfirmware/zephyr/blob/v4.1.0%2Bzmk-fixes/soc/raspberrypi/rpi_pico/rp2350/Kconfig.soc

[petejohanson]

Yes, I'd use the series for this.

[peterjc]

Change applied to the branch and PR, not tested yet.