Klipper3d / klipper

Klipper is a 3d-printer firmware
GNU General Public License v3.0
8.95k stars 5.16k forks source link

stm32: Fix USART3 ALT pinout on STM32G0 #6523

Closed rakosi2 closed 3 months ago

rakosi2 commented 3 months ago

I have a BTT E3EZ (STM32G0B1) and the UART on the TFT did not work. I have worked out that the ALT mux was not set correctly. This is the only board that I can verify with so I am not sure if this is a typo or if another chip needs it as AF7. I have not looked at the other USARTs since I can not verify it.

This will resolve this issue: https://github.com/bigtreetech/Manta-E3EZ/issues/8

image

Attila Rakosi rattila5@hotmail.com

JamesH1978 commented 3 months ago

Thank you for submitting a PR, pleas refer to point 3 in "What to expect in a review" in https://github.com/Klipper3d/klipper/blob/master/docs/CONTRIBUTING.md and provide a signed off by line.

Thanks James

JamesH1978 commented 3 months ago

As mentioned over in the discord a lot of other stm32 chips use AF7 for this UART, Especially ones used in the 3D printing world like the F4 and the H7 series. You will need to rewrite your code to be specifically for the G0 series using AF0

Thanks James

rakosi2 commented 3 months ago

Set it to only change for STM32G0 series to AF0 and all other STM32 will use AF7 I have also fix the Signed-off-by Thank you JamesH1978

KevinOConnor commented 3 months ago

Thanks. The change looks fine to me, but I don't see a singed-off-by line though.

-Kevin

rakosi2 commented 3 months ago

I was unaware of it on the first commit, put it on the second commit and edited the first message with the signed-off at the end.

KevinOConnor commented 3 months ago

Sorry - I'm still not seeing it. I need to see a line that looks something like Signed-off-by: Some Name <some@email.addr>. The line can be either in a comment here or in a commit message - I don't see a line like that in either location.

Cheers, -Kevin

EDIT: I do see your name and email address in the second commit message, but I don't see the leading "Signed-off-by:" signature indicator.

rakosi2 commented 3 months ago

Signed-off-by: Attila Rakosi rattila5@hotmail.com

KevinOConnor commented 3 months ago

Thanks.

-Kevin