Skip to content

enabling Tone for Due #24

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

Open
agdl opened this issue Feb 8, 2017 · 18 comments
Open

enabling Tone for Due #24

agdl opened this issue Feb 8, 2017 · 18 comments

Comments

@agdl
Copy link
Member

agdl commented Feb 8, 2017

From @HanaJin on October 13, 2014 21:17

Hi Arduino Team,

Is there a plan or timeline in mind to enable Tone for Due board in future releases?

Copied from original issue: arduino/Arduino#2362

@agdl
Copy link
Member Author

agdl commented Feb 8, 2017

From @q2dg on August 8, 2015 13:20

Well, a solution seems to be here (http://forum.arduino.cc/index.php?topic=136500.0) .
Could you add that piece of code to official tone() function, plis??

...And, on the other hand, what about tone() implementation in Zero, Gemma (#3519) ...??? API must be consistent between boards!!!

@agdl
Copy link
Member Author

agdl commented Feb 8, 2017

From @aethaniel on August 10, 2015 13:11

@q2dg, tone() is implemented on Zero: https://github.com/arduino/ArduinoCore-samd/blob/master/cores/arduino/Tone.cpp

@agdl
Copy link
Member Author

agdl commented Feb 8, 2017

From @q2dg on August 10, 2015 16:52

@aethaniel Ah, oook, thanks! (and sorry!).
Anyway, clarificating the documentation to specify the diferent API's functionality scope among the diferent boards (if there is any) is fundamental. But that's for another issue.

@agdl
Copy link
Member Author

agdl commented Feb 8, 2017

From @q2dg on August 23, 2015 16:44

Well, I only wanted to point that problem with tone() and Due still remains...

@agdl
Copy link
Member Author

agdl commented Feb 8, 2017

From @shiftleftplusone on October 21, 2015 7:37

I don't see how it now should work for the Due - can you provide a example source code please?

@agdl
Copy link
Member Author

agdl commented Feb 8, 2017

From @shiftleftplusone on December 6, 2015 11:22

OMG, the TOP dated from October 2014 !!
why hadn't it been fixed yet ?!?

@agdl
Copy link
Member Author

agdl commented Feb 8, 2017

From @aethaniel on December 6, 2015 23:23

I have a question to answer yours as you seem to be very demanding on various projects:
Why don't you provide any patch to help on missing features, Helmutt?

On 6 December 2015 12:22:45 CET, VogonJeltz [email protected] wrote:

OMG, the TOP dated from October 2014 !!
why hadn't it been fixed yet ?!?


Reply to this email directly or view it on GitHub:
arduino/Arduino#2362 (comment)

Envoyé de mon téléphone Android avec K-9 Mail. Excusez la brièveté.

@agdl
Copy link
Member Author

agdl commented Feb 8, 2017

From @shiftleftplusone on December 7, 2015 14:6

Perhaps you miss something decisive:
I am a user and a customer, not a manufacturer and not a developer!

Buying a 60 EUR Arduino I simply expect it to work, like I expect it when buying a washing machine or a TV.

Or do you buy either one and then when realizing that there is no water pump inside or no tone on TV
and thus complaining about those issues at your seller, getting to hear:

"I have a question to answer yours as you seem to be very demanding on various projects: Why don't you provide any patch to help on missing features, aethaniel " ?!?

@agdl
Copy link
Member Author

agdl commented Feb 8, 2017

From @aethaniel on December 7, 2015 14:30

Hi,

Like many people, I personally think there is a delta between finished manufactured products and DIY Community boards, even very well branded.

Cheers,

T.

On 7 December 2015 15:07:00 CET, VogonJeltz [email protected] wrote:

I am a user, not a constructer.
Buying a Arduino I seemply expect it to work, like I expect buying a
washing machine or a TV.
Or do you buy either one and then having no water pump or no tone on
TV,
and when complaining those issues at your seller getting to hear
"I have a question to answer yours as you seem to be very demanding on
various projects: Why don't you provide any patch to help on missing
features,aethaniel " ?!?


Reply to this email directly or view it on GitHub:
arduino/Arduino#2362 (comment)

Envoyé de mon téléphone Android avec K-9 Mail. Excusez la brièveté.

@agdl
Copy link
Member Author

agdl commented Feb 8, 2017

From @shiftleftplusone on December 7, 2015 14:36

you may personally think what you wish, but to me a Arduino Due is a finished manufactured product which is simply expected to work by it's libs!

@agdl
Copy link
Member Author

agdl commented Feb 8, 2017

From @NicoHood on December 7, 2015 16:36

He is partly right.

Not everyone is able to patch those things (especially timer related stuff) hisself without reading up on this for hours. Even I do not know where to start, not even on avr. So he is probably right. And tone() is a basic API feature that should be added.

DYI is always incomplete and not perfect. Arduino is the best example, even though it tries to make stuff simple. But time passes, version changes, new boards come and not enough people are working on this. And the Arduino community which just uses the boards will never attempt to fix this. This is hard core.

So yeah, I'd vote for @vogonjeltz that someone fixes this, as its a basic feature like digitalWrite() etc. However I think @vogonjeltz should try to post a bit less aggressive here. Things wont change if you force them to do.

@agdl
Copy link
Member Author

agdl commented Feb 8, 2017

From @shiftleftplusone on December 7, 2015 18:12

I agree, perhaps my posts partially might appear to be sort of harsh - if one didn't know te story before.

I am always used to ask friendly and politely the first time, but if repetively there comes no answer at all or something like "help yourself, don't expect others to do your homework" and the topic immediately gets closed then the limit of my politeness has been quickly reached.

Finally this tone issue is pending since the Due has been released, and even in this repository this issue has been successfully ignored since more than one year, nothing just silence in the forest. When would it be time then to slowly getting impatient?

@agdl
Copy link
Member Author

agdl commented Feb 8, 2017

From @shiftleftplusone on December 7, 2015 18:18

BTW, if it's really timer-related stuff (no idea, I don't understand enough about it):
why note set it up to DueTImer by attaching a free interrupt?

Timer.getAvailable().attachInterrupt(duetone).start();

https://github.com/ivanseidel/DueTimer/blob/master/examples/AvaliableTimer/AvaliableTimer.pde

@thinkyhead
Copy link

We just did an ad hoc implementation for MarlinFirmware. It's quite small and –dare I say– cute.

@EKersting
Copy link

Anything new on using tone() with the DUE?

@mkhrenov
Copy link

I've written an implementation and just submitted a pull request, would love to see it be incorporated so folks can finally use this feature. #108

@tigoe
Copy link
Member

tigoe commented Dec 23, 2020

Had a report from one of my former students about this issue too, would love to see it completed and closed.

@aliphys
Copy link

aliphys commented May 22, 2024

I noticed that the compile-sketches.yml workflow in the arduino/arduino-examples repository mentions this issue at line 184. Any updates on this issue?

          - fqbn: arduino:sam:arduino_due_x
            usb: true
            serial1: true
            starter-kit: false
            # Bug report: https://github.com/arduino/ArduinoCore-sam/issues/24
            # Change the value to true once it is fixed.
            tone: false
            a5: true

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

7 participants