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

Infineon (Cypress) LBEE5KL1DX / CYW4343W (other ?) also need special SCO routing configuration #586

Open
ubicore opened this issue Aug 1, 2022 · 1 comment

Comments

@ubicore
Copy link

ubicore commented Aug 1, 2022

We are using a Infineon (Cypress) LBEE5KL1DX module that is based on the CYW4343W chipset that is based itself on a BCM4343W from Broadcom.
So it's chip.manufacturer is BT_COMPID_CYPRESS / 0x0131 not the BT_COMPID_BROADCOM / 0x000F one.

In sco.c, in sco_setup_connection_dispatcher() function the test to configure the SCO routing is only done with BT_COMPID_BROADCOM.

if (a->chip.manufacturer == BT_COMPID_BROADCOM) {

In our case we have a Broadcom chip, but it's manufacturer ID is BT_COMPID_CYPRESS but we still want to run the specific configuration.

I don't know whether to add the cypress ID in addition to the broadcom one ?

@arkq
Copy link
Owner

arkq commented Aug 1, 2022

Yes, I've got a branch with change like that:

-       if (a->chip.manufacturer == BT_COMPID_BROADCOM) {
+       if (a->chip.manufacturer == BT_COMPID_BROADCOM ||
+                       a->chip.manufacturer == BT_COMPID_CYPRESS) {

But it's not in master because I'm not sure whether it will not "break" some other cypress BT chips. If you are sure that cypress does not manufactures BT chips on its own (Broadcom was acquired by Cypress some time ago) maybe we could add that change to master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants