ArduinoCore-sam icon indicating copy to clipboard operation
ArduinoCore-sam copied to clipboard

enabling Tone for Due

Open agdl opened this issue 9 years ago • 18 comments

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 avatar Feb 08 '17 14:02 agdl

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 avatar Feb 08 '17 14:02 agdl

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 avatar Feb 08 '17 14:02 agdl

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 avatar Feb 08 '17 14:02 agdl

From @q2dg on August 23, 2015 16:44

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

agdl avatar Feb 08 '17 14:02 agdl

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 avatar Feb 08 '17 14:02 agdl

From @shiftleftplusone on December 6, 2015 11:22

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

agdl avatar Feb 08 '17 14:02 agdl

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: https://github.com/arduino/Arduino/issues/2362#issuecomment-162304569

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

agdl avatar Feb 08 '17 14:02 agdl

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 avatar Feb 08 '17 14:02 agdl

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: https://github.com/arduino/Arduino/issues/2362#issuecomment-162533061

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

agdl avatar Feb 08 '17 14:02 agdl

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 avatar Feb 08 '17 14:02 agdl

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 avatar Feb 08 '17 14:02 agdl

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 avatar Feb 08 '17 14:02 agdl

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

agdl avatar Feb 08 '17 14:02 agdl

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

thinkyhead avatar Feb 25 '18 16:02 thinkyhead

Anything new on using tone() with the DUE?

EKersting avatar Jun 18 '20 11:06 EKersting

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

mkhrenov avatar Jun 26 '20 02:06 mkhrenov

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

tigoe avatar Dec 23 '20 15:12 tigoe

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

aliphys avatar May 22 '24 16:05 aliphys