Timing diagram for MSGEQ7 spectrum analyzer chip, is it correct?

[HenrikOlsson]

As usual, have YOU tried it yourself?

Random comments:
* Don't use HIGH/LOW, they're wasting time and program space.
* After STROBE goes LOW (active) there's a 36us (minimum) settling time so I'd wait that amount of time before sampling the output.
* The ADCIN statement is likely to take longer than the needed 18us so there shouldn't be a need to have that delay in there.
* What's being done in the display subroutine? Does it take longer than 18us? If so move things around and eliminate the other delay as well

FOR Band = 0 to 6
  STROBE = 0
  PAUSEUS 36   ' Wait specified settling time
  ADCIN 1, Value   ' Take a reading, this will take tens of uS depending on how the ADC is setup.
  STROBE = 1
  GOSUB Display  ' This will also take time so we'll most likely meet the 72us minimum strobe to strobe time - needs verifying.
NEXT

Or, sample all bands and then display them

Spectrum VAR BYTE[7]
Band VAR BYTE
FOR Band = 0 to 6
  STROBE = 0
  PAUSEUS 36   ' Wait specified settling time
  ADCIN 1, Spectrum[Band]
  STROBE = 1
  PAUSEUS 36   ' Can probably be removed altogether since ADCIN will provide the needed "delay".
NEXT

The above has obviously not been tested, it's being provided as food for thought.

/Henrik.

[CuriousOne]

No, I haven't tested yet by myself, because I need to assemble circuit, etc. So instead I decided to check software first.

This code was just an example how I understand timings, in real code I won't use HIGH and LOW statements.
Display subroutine will either drive MAX7219 with 8x8 led matrix connected, or will multiplex 5x7 discrete leds.

[OldMarty]

* Don't use HIGH/LOW, they're wasting time and program space.
/Henrik.

Excuse my ignorance, but what's the alternative to setting bits without using high/low?
Is it preferable just to use "PORTB.1 = 1" etc?

Regards,
Marty.

[richard]

Excuse my ignorance, but what's the alternative to setting bits without using high/low?
Is it preferable just to use "PORTB.1 = 1" etc?

yes or EVEN BETTER LATB.1 = 1 if chip has LAT regs

high/low always resets TRIS bit wether it needs to or not (ie already set to output)
which takes a

BANKSEL TRISX
BCF TRISX,PIN
BANKSEL PORTX
BSF/BCF PORTX,PIN
BANKSEL 0

HEAPS OF STUFF

[OldMarty]

high/low always resets TRIS bit wether it needs to or not (ie already set to output)
which takes a

BANKSEL TRISX
BCF TRISX,PIN
BANKSEL PORTX
BSF/BCF PORTX,PIN
BANKSEL 0

HEAPS OF STUFF

wow, that's a lot of junk going on ;-)
thanks.