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

Does not work on platform-espressif32 4.0.0 or more. #49

Open
asukiaaa opened this issue May 24, 2022 · 4 comments
Open

Does not work on platform-espressif32 4.0.0 or more. #49

asukiaaa opened this issue May 24, 2022 · 4 comments

Comments

@asukiaaa
Copy link

This library does not work on PlatformIO with using platform-espressif32 4.0.0 (arduino v2.0.0) or more(4.1.0, 4.2.0 and 4.2.0) but works with 3.5.0.
Is there any way to work on 4.0.0 or more?

Thank you.

@julian-weinert
Copy link

Hi @asukiaaa! I've had to modify the CMakeList.txt for the ps3 component
There is also a fork and an open pull request for 4.0 support, you may simply use that.

I had connection trouble, so maybe it's some incompatibility thing, but I've not had any look searching / reaching out.

idf_component_register(SRCS "src/ps3.c"
                            "src/ps3_spp.c"
                            "src/ps3_parser.c"
                            "src/ps3_l2cap.c"
                        REQUIRES nvs_flash bt
                        PRIV_REQUIRES bt
                        INCLUDE_DIRS src/include
                        PRIV_INCLUDE_DIRS   ${IDF_PATH}/components/bt/common/include/
                                            ${IDF_PATH}/components/bt/host/bluedroid/common/include/
                                            ${IDF_PATH}/components/bt/host/bluedroid/stack/include)

@asukiaaa
Copy link
Author

Thank you for the response.

I've had to modify the CMakeList.txt for the ps3 component
There is also a fork and an open pull request for 4.0 support, you may simply use that.

Can you show some urls for them?

@julian-weinert
Copy link

Oh, sorry for the late response. You can go to the pull requests tab and then navigate to the fork.
Here is the direct link https://github.com/rrooding/esp32-ps3/tree/feature/esp-idf-4-support

@julian-weinert
Copy link

It might be worth mentioning that the fork of rrooding uses a pre-release bluetooth GAP API that is not supported by 4.0 anymore.
I would recommend using this original fork and add the CMakeList.txt as I have it above.
There is another change in rroodings fork, in the src/ps3_spp.c:150. It appears that ESP-IDF now supports the use of ESP_SPP_SEC_AUTHORIZE instead of ESP_SPP_SEC_NONE but I'm not sure how important it is.

No matter what I do I can't seem to be able to get it connecting. It might be necessary to go over the project again and maybe convert it to the new official gap API if it supports the needed functionality.

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

No branches or pull requests

2 participants