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

P4 compile error (DSP-143) #90

Open
4 tasks done
Jason2866 opened this issue Aug 12, 2024 · 4 comments
Open
4 tasks done

P4 compile error (DSP-143) #90

Jason2866 opened this issue Aug 12, 2024 · 4 comments
Labels

Comments

@Jason2866
Copy link

Jason2866 commented Aug 12, 2024

Answers checklist.

  • I have read the ESP-DSP documentation and the issue is not addressed there.
  • I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
  • I have updated my ESP-DSP branch (master or release) to the latest version and checked that the issue is present there.
  • I have searched the issue tracker for a similar issue and not found a similar issue.

IDF version.

IDF 5.2 latest master

Module or chip used.

Compile try for ESP32-P4

DSP version.

latest master

What is the expected behavior?

Code compiles

What is the actual behavior?

compile error with modules/dotprod/float/dsps_dotprode_f32_arp4.S:52: Error: unrecognized opcode esp.lp.setup 0,a3,.dotprod_loop'

Steps to reproduce.

Just tring to compile fails

Code to reproduce this issue.

Not needed

Debug logs.

See CI run of Arduino Lib Builder https://github.com/Jason2866/esp32-arduino-lib-builder/actions/runs/10341546001/job/28623471052

More Information.

No response

@Jason2866 Jason2866 added the Type: Bug Bugs in ESP-IDF label Aug 12, 2024
@github-actions github-actions bot changed the title P4 compile error P4 compile error (DSP-143) Aug 12, 2024
@dmitry1945
Copy link
Collaborator

Hi @Jason2866,
Thank you for feedback.

Could you try to switch to latest idf master branch?
Then ./install.sh and then . ./export.sh

Now only latest master has a toolchain to support esp32p4 instructions like esp.lp.setup

Please let me know the result.
Thank you.

@Jason2866
Copy link
Author

Jason2866 commented Aug 13, 2024

Understood, branch IDF 5.2 is not supported. Can't change to IDF master branch easily, many code changes needed. Will the needed changes backported to IDF 5.3.x and 5.2.x?
If not, no drama will exclude DSP lib when compiling for P4
Imho the idf_component.yml should be adopted to reflect the needed versions of IDF

@dmitry1945
Copy link
Collaborator

@Jason2866 I think it's a plans to make a backport.
Now, you can just take binutils from latest master, and use them.
Yes, I will add restriction to idf_component.yml
Thank you.

@dmitry1945
Copy link
Collaborator

dmitry1945 commented Aug 13, 2024

@Jason2866
Also, you can now disable optimization for esp-dsp in menuconfig, and you will be able to use the esp-dsp. For floating point operations, you will not lose a lot of performance. And in future will fix the problem.

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

No branches or pull requests

3 participants