ROBIN PRO z low with fix

[shtaXveed]

Did you test the latest bugfix-2.0.x code?

Yes, and the problem still exists.

Bug Description

the lcd reset pin is assigned to PF6 which is a z tx pin. it should be PC6. it causes low on Z while compiling for uart

Bug Timeline

No response

Expected behavior

working printer in uart

Actual behavior

tmc connectivity error and motor stutter on z

Steps to Reproduce

compile uart on robin pro and install

Version of Marlin Firmware

2.0.bugfix 10.10.21

Printer model

artist d pro

Electronics

stock plus bltouch minus some jumpers

Add-ons

bl

Bed Leveling

ABL Bilinear mesh

Your Slicer

No response

Host Software

No response

Additional information & file uploads

found on a makerbase github

[thisiskeithb]

found on a makerbase github

Please open an issue in their repository. It has already been fixed here.

[shtaXveed]

i pulled last nights bugfix and it was a PF6 instead of PC6 and threw errors. only reason im bringing it up. i believe that one may have been missed.

…

On Mon, Oct 11, 2021, 11:56 AM Keith Bennett ***@***.***> wrote: Closed #22931 <#22931>. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#22931 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AVSX24VUIRXCPRKTBCMAMPLUGMJNNANCNFSM5FX6SQCA> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.

[shtaXveed]

[thisiskeithb]

I was looking at the Robin Pro 2 pins file which was correct, but the regular Pro is incorrect and has PF6 assigned multiple times as you noted:

Marlin/Marlin/src/pins/stm32f1/pins_MKS_ROBIN_PRO.h

Line 84 in 40cb7cf

#define Z_CS_PIN PF6

Marlin/Marlin/src/pins/stm32f1/pins_MKS_ROBIN_PRO.h

Line 143 in 40cb7cf

#define Z_SERIAL_RX_PIN PF6

Marlin/Marlin/src/pins/stm32f1/pins_MKS_ROBIN_PRO.h

Line 240 in 40cb7cf

#define LCD_RESET_PIN PF6

I'll post a PR shortly.

[thisiskeithb]

See #22937. Thanks for the report!

[ellensp]

This has been merged, closing

[github-actions]

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.