Skip to content

New 3rd Party Boards - 4D Systems #8530

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

Merged
merged 2 commits into from
Aug 29, 2023
Merged

New 3rd Party Boards - 4D Systems #8530

merged 2 commits into from
Aug 29, 2023

Conversation

James-4DSystems
Copy link
Contributor

Description of Change

Adding new board range from 4D Systems
gen4-ESP32 Range, featuring ESP32-S3R8 SoC, 16MB Flash
Custom Partitions to suit our modules and 16MB Flash

New range of boards from 4D Systems
@P-R-O-C-H-Y
Copy link
Member

Hi @James-4DSystems, can you please provide a link for your new board?
Btw. is it intended to have all SPI pins declared as -1 in pins_arduino.h? Thanks

@James-4DSystems
Copy link
Contributor Author

Hi @James-4DSystems, can you please provide a link for your new board? Btw. is it intended to have all SPI pins declared as -1 in pins_arduino.h? Thanks

Hi @P-R-O-C-H-Y
The website link is not yet finished, should be ready in about a week I believe.
Can I email you the pre-release datasheets for now?

SPI pins we decided to set to -1 as the pins_arduino.h file is shared with all our ESP32 products, and we have different SPI requirements for each. The SDFAT library seems to be the only thing that uses these 'defaults' when first included (when using our library for our products), but when we call the library for actual use, the pins are set to the required pins for what the library is actually being used for, so this seems to have no negative effect. So yes, it is intended.
Our library which is used with our products will be releasing soon, deals with setting these up correctly.
We did have them all defined for one of the products, but it was 'wrong' for the others, but it had no negative effects as when we call our library it sets everything correctly anyway.
Setting them as -1 seemed to be the best compromise, so it was not incorrect for anything. -1 is there to define cases like this we thought.
We had thought about splitting them up, but its actually for no gain, as everything else in the pins_arduino.h and variant folder, other than those pins, are common.

If its a strict requirement we have to have these set, then we will have to split the products up, just to have these pins different, and more work will flow on for us for our library and IDE.

Please advise if this is your requirement.

If you want a copy of the pre-release datasheets, please flick me an email and I will happily provide them. Just dont want to make them public until the website is up and the range is announced. email me at james [at] 4dsystems.com.au

@P-R-O-C-H-Y
Copy link
Member

Hi @James-4DSystems, can you please provide a link for your new board? Btw. is it intended to have all SPI pins declared as -1 in pins_arduino.h? Thanks

Hi @P-R-O-C-H-Y The website link is not yet finished, should be ready in about a week I believe. Can I email you the pre-release datasheets for now?

SPI pins we decided to set to -1 as the pins_arduino.h file is shared with all our ESP32 products, and we have different SPI requirements for each. The SDFAT library seems to be the only thing that uses these 'defaults' when first included (when using our library for our products), but when we call the library for actual use, the pins are set to the required pins for what the library is actually being used for, so this seems to have no negative effect. So yes, it is intended. Our library which is used with our products will be releasing soon, deals with setting these up correctly. We did have them all defined for one of the products, but it was 'wrong' for the others, but it had no negative effects as when we call our library it sets everything correctly anyway. Setting them as -1 seemed to be the best compromise, so it was not incorrect for anything. -1 is there to define cases like this we thought. We had thought about splitting them up, but its actually for no gain, as everything else in the pins_arduino.h and variant folder, other than those pins, are common.

If its a strict requirement we have to have these set, then we will have to split the products up, just to have these pins different, and more work will flow on for us for our library and IDE.

Please advise if this is your requirement.

If you want a copy of the pre-release datasheets, please flick me an email and I will happily provide them. Just dont want to make them public until the website is up and the range is announced. email me at james [at] 4dsystems.com.au

Thank you @James-4DSystems for all the informations.
I don't need the datasheet as you don't declare many pins so there is no need to check them. But it's better that in the PR is a datasheet linked. So maybe you can edit the description later when the website is ready, even if the PR will be already merged :)

@SuGlider SuGlider added this to the 2.0.12 milestone Aug 23, 2023
@P-R-O-C-H-Y P-R-O-C-H-Y added the Status: Pending Merge Pull Request is ready to be merged label Aug 23, 2023
@SuGlider
Copy link
Collaborator

@P-R-O-C-H-Y / @James-4DSystems -
We want to launch it within the 2.0.12.

@me-no-dev me-no-dev merged commit de04fb2 into espressif:master Aug 29, 2023
@James-4DSystems James-4DSystems deleted the Board branch September 1, 2023 04:03
@James-4DSystems
Copy link
Contributor Author

@P-R-O-C-H-Y

Thank you @James-4DSystems for all the informations. I don't need the datasheet as you don't declare many pins so there is no need to check them. But it's better that in the PR is a datasheet linked. So maybe you can edit the description later when the website is ready, even if the PR will be already merged :)

Product line has been released:
https://4dsystems.com.au/product-category/intelligent-display-modules/gen4-esp32-display-modules/

2.4", 2.8", 3.2", 3.5", 4.3", 5.0" all released, and 7.0" and 9.0" coming soon. Non-Touch, Capacitive Touch, and Resistive Touch available for various models.

Datasheets available from the individual product pages, linked from the above link, along with the Github link for the Arduino library which goes along with these modules (Also available from the Library Manager in the Arduino IDE).

Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Pending Merge Pull Request is ready to be merged Type: 3rd party Boards
Projects
Development

Successfully merging this pull request may close these issues.

5 participants