A few more bugs in the XMEGA ASF routines...

Go To Last Post
11 posts / 0 new
Author
Message
#1
  • 1
  • 2
  • 3
  • 4
  • 5
Total votes: 0

For reference, here are a few more ASF bugs that I'm not sure have been previously reported:

twim.c, twim_idle():

static inline bool twim_idle (const TWI_t * twi)
{
  return (twi->MASTER.STATUS & TWI_MASTER_BUSSTATE_IDLE_gc);
}

This function supposedly indicates if the TWI master status is "IDLE", right? The problem is that "IDLE" is "0x01" (bitwise "01"), but "BUSY" is "0x03" (bitwise "11"), so this code returns "TRUE" for the bus being idle if the status is either "IDLE" or "BUSY". I don't think this is by design, but of course, I could be wrong.

tc.h, tc_clear_cc_interrupt():

case TC_CCA:
  ((TC1_t *)tc)->INTFLAGS = TC1_CCAIF_bm;
  break;
case TC_CCB:
  ((TC1_t *)tc)->INTFLAGS = TC1_CCAIF_bm;
  break;

This is a snippet from the bottom of this function, after the code has determined that the timer is a "TC1" timer. Notice how both "TC_CCA" and "TC_CCB" use the same "TC1_CCAIF_bm" bitmask for their interrupt? To correctly clear the TC_CCB interrupt, this needs to use "TC1_CCBIF_bm" instead.

  • 1
  • 2
  • 3
  • 4
  • 5
Total votes: 0

Could you add a bug report on the ASF Bugzilla please?

http://asf.atmel.com/bugzilla/

That way it'll remain on the developer's issue list until resolved.

- Dean :twisted:

Make Atmel Studio better with my free extensions. Open source and feedback welcome!

  • 1
  • 2
  • 3
  • 4
  • 5
Total votes: 0

It's quite annoying to sign up for their bug tracker, since they broadcast your email address to the world, meaning you can't just use the one you've got. So you have to go make a new email address.

I reported 3 bugs using a yahoo account, and I've received 41 emails from the bug tracker, with the changes yet to be released to the public.

Jeff Nichols

  • 1
  • 2
  • 3
  • 4
  • 5
Total votes: 0

In the Bugzilla->preferences->Email: you can configure for which steps you want to receive a mail (comments, status change etc..)

-sma

  • 1
  • 2
  • 3
  • 4
  • 5
Total votes: 0

Sorry, I'm not willing to create an account on the Bugzilla just to log these. If Atmel wants to continue to engage the community, they should be actively pursuing all bug reports (including those posted on the forum) and ensuring that these are accurately reflected in their database.

In addition to bugs, there are also places where the ASF would simply benefit from more comments. For example, programmers should be instructed that turning a timer off via a call to tc_disable() does more than just stopping the timer, i.e. it also causes the timer count and/or period to be read incorrectly when subsequently retrieved.

  • 1
  • 2
  • 3
  • 4
  • 5
Total votes: 0

Slordak wrote:
Sorry, I'm not willing to create an account on the Bugzilla just to log these. If Atmel wants to continue to engage the community, they should be actively pursuing all bug reports (including those posted on the forum) and ensuring that these are accurately reflected in their database.

Ok, you have no real interest that this problem gets fixed, so no big deal. But then please don't complain about bugs in the ASF.

  • 1
  • 2
  • 3
  • 4
  • 5
Total votes: 0

By that logic, you don't have any interest either. I think it's a legitimate gripe.

Jeff Nichols

  • 1
  • 2
  • 3
  • 4
  • 5
Total votes: 0

Submitted for you:

http://asf.atmel.com/bugzilla/sh...

I agree the bug database is a tad chatty what with the new steps to perform integrating testing/verification/review, but you can turn off emails as "sma" says. If no one reports the bugs in the proper channels, there's a much lower chance of it being fixed.

- Dean :twisted:

Make Atmel Studio better with my free extensions. Open source and feedback welcome!

  • 1
  • 2
  • 3
  • 4
  • 5
Total votes: 0

pixel2001n wrote:
By that logic, you don't have any interest either. I think it's a legitimate gripe.

I have submitted several ASF issues to Bugzilla. I think the developer already have a hard time improving AS5, fixing and extending ASF features almost every month. There is a working email support. Why making it harder for them when not supporting bugzilla? And for most of all, it's all free. I don't understand what some people have expectations how perfect this all has to work.

We have licensed the Codewarrior Eclipse suite from Freescale for about $2200 every year. It's out for 11 month now, and there are more bugs in it than in AS5. The email support sometimes needs more than 2 weeks for an answer, even that we pay the subscription. Thats only for comparison.

  • 1
  • 2
  • 3
  • 4
  • 5
Total votes: 0

Quote:

I have submitted several ASF issues to Bugzilla. I think the developer already have a hard time improving AS5, fixing and extending ASF features almost every month.

The AVRStudio product is developed by Tools, while the ASF product is handled by Applications - they're different departments. Please do submit bugs on both, as the developers really do want to make things better. Even if it's not fixed tomorrow, having it in the database means it will be fixed eventually.

- Dean :twisted:

Make Atmel Studio better with my free extensions. Open source and feedback welcome!

  • 1
  • 2
  • 3
  • 4
  • 5
Total votes: 0

There is an atmel rep out here at the moment in Melbourne. I have a meeting with him on Friday. I intend to ask a few questions. Does anyone want to create a list of questions?