Skip to content

Conversation

@h-oussama
Copy link

This PR supersedes #4392 .

Signed-off-by: Oussama HOUIDAR houidar.ousama@gmail.com

Copy link
Collaborator

@JamesH1978 JamesH1978 left a 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
Copy link
Collaborator

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
Copy link
Collaborator

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?

@github-actions github-actions bot added the Stale label Aug 12, 2023
@github-actions github-actions bot closed this Aug 19, 2023
@KevinOConnor
Copy link
Collaborator

This PR was inadvertently closed due to a regression introduced by #6293.

-Kevin

@KevinOConnor
Copy link
Collaborator

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

@KevinOConnor KevinOConnor added the pending feedback Topic is pending feedback from submitter label Oct 20, 2023
@github-actions github-actions bot added the inactive Not currently being worked on label Nov 11, 2023
@github-actions
Copy link

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,
~ Your friendly GitIssueBot

PS: I'm just an automated script, not a human being.

@github-actions github-actions bot closed this Nov 11, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Nov 10, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

inactive Not currently being worked on pending feedback Topic is pending feedback from submitter

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants