Skip to content
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

Custom device #214

Closed
wants to merge 55 commits into from
Closed

Custom device #214

wants to merge 55 commits into from

Conversation

elral
Copy link
Collaborator

@elral elral commented Oct 21, 2022

This is a proposal for adding a custom device to the firmware.
It requires also changes in the connector which have not been done so far.
The intention is to have a base for further discussion.

The pre condition for the custom device is that it will be set up from the connector like:
"kTypeCustomDevice , x.thDevice, Pin1, Pin2, Pin3, Pin4, Pin5, Pin6, initString"
This gives the possibility to use up to 6 Pins from the custom device and to get an init parameter which could differ if more than one custom device from the same type are defined within the connector.

We can also think about not to define 6 fixed Pins but to handle this in a way that only required pins will be defined. In this case Pin1 would be replaced by the number of following Pins.

Data to the custom device should be transferred like for the LCD display. For now a haven't foreseen a different way. I guess this gives the maximum flexibilty to send informations like variables or fixed test or whatever.

A lot of comments are added in the source files to support a user to set up his custom device.

I am pretty sure there will be some optimisation possible in this pull request.

There will come a second pull request with additional changes which are sensefull in my eyes to have maximum flexibility for the custom device.

@github-actions
Copy link

Firmware for this pull request:
Firmware.zip

@neilenns
Copy link
Contributor

Looks reasonable, it's essentially what I did when I had custom devices. I don't understand the init parameter though. What would it get used for?

@elral
Copy link
Collaborator Author

elral commented Oct 22, 2022

Like other devices this custom device can also be initialized multiple times. There could be cases where each/some device must be initialized in a different way or needs different parameters to work. For this the init string would be used.
One example is my motorized axis where I need some parameters to calculate a manual movement while in operation. These are depending on the hardware setup like how many turns, speed... And even if I would define them in the firmware, I would need a different firmware for each hardware variant.
Aother one could be setting up a 16 segment display. Defining the number of digits could be also an init parameter.

@github-actions
Copy link

Firmware for this pull request:
Firmware.zip

@github-actions
Copy link

Firmware for this pull request:
Firmware.zip

@github-actions
Copy link

github-actions bot commented Jan 6, 2023

Firmware for this pull request:
Firmware.zip

@github-actions
Copy link

Firmware for this pull request:
Firmware.zip

@github-actions
Copy link

Firmware for this pull request:
Firmware.zip

@github-actions
Copy link

github-actions bot commented Apr 2, 2023

Firmware for this pull request:
Firmware.zip

@github-actions
Copy link

Firmware for this pull request:
Firmware.zip

@github-actions
Copy link

Firmware for this pull request:
Firmware.zip

@github-actions
Copy link

Firmware for this pull request:
Firmware.zip

@github-actions
Copy link

Firmware for this pull request:
Firmware.zip

@github-actions
Copy link

Firmware for this pull request:
Firmware.zip

@github-actions
Copy link

Firmware for this pull request:
Firmware.zip

@elral elral self-assigned this Sep 8, 2023
@github-actions
Copy link

github-actions bot commented Sep 8, 2023

Firmware for this pull request:
Firmware.zip

@elral elral marked this pull request as ready for review September 8, 2023 17:08
@github-actions
Copy link

github-actions bot commented Sep 9, 2023

Firmware for this pull request:
Firmware.zip

@github-actions
Copy link

Firmware for this pull request:
Firmware.zip

@elral elral closed this Sep 19, 2023
@elral
Copy link
Collaborator Author

elral commented Sep 19, 2023

This PR is closed as major changes are implemented in a new branch.
I will issue a new one.

@elral elral mentioned this pull request Sep 19, 2023
@elral elral deleted the Custom_Device branch September 19, 2023 09:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants