-
-
Notifications
You must be signed in to change notification settings - Fork 5.8k
Adds support for the STEVAL-3DP001V1 board from STmicroelectronics #4835
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…ipper into STEVAL3DP001V1_work
JamesH1978
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for submitting your config.
Please make sure that you follow the steps in https://github.com/Klipper3d/klipper/blob/master/docs/Example_Configs.md especially the point on regression tests.
Also please see my added comments.
Whereas I can review the configuration @KevinOConnor or another reviewer will have to review your extra code for the driver additions etc.
Thanks
James
| @@ -0,0 +1,327 @@ | |||
| # This file contains common pin mappings for STEVAL-3DP001V1 board. | |||
| # to use this config, the firmware should be compiled for the STM32F4 | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This will need to be expanded on, it has no mention of how you should set the make menuconfig, clocks, speeds, bootloader offsets etc. and then the method of flashing this file.
|
|
||
| [mcu] | ||
| serial: /dev/serial/by-id/usb-STMicroelectronics_STM32_STLink_0674FF514852897267245621-if02 | ||
| baud: 115200 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You have stated a custom speed, we generally do not set this unless their is a very good reason to do so, 250000 default works on the majority of known boards. Have you tested that it will not work at the default speed?
|
This PR was inadvertently closed due to a regression introduced by #6293. -Kevin |
|
It seems this PR "slipped through the cracks" on my side. Can you provide an update on the current status? Is this something that should still be merged? -Kevin |
|
It looks like this GitHub Pull Request has become inactive. If there are any further updates, you can add a comment here or open a new ticket. Best regards, PS: I'm just an automated script, not a human being. |
This PR supersedes #4392 .
Signed-off-by: Oussama HOUIDAR houidar.ousama@gmail.com