 |
 |
| View previous topic :: View next topic |
| Author |
Message |
suspiciousdeveloper
Joined: 02 Mar 2025
Posts: 1
|
| Having issues while trying to use ADC |
 Posted: Sun Mar 02, 2025 3:52 pm |
 |
|
Compiler Version: V5.118
MCU: 33CH128MP202-I/SS
Hey there, while trying to use ADC the code seems to be getting stuck whenever the read_adc() function is called. Tried multiple methods but no luck, appreciate any help provided. Code is attached below:
| Code: | #include <33CH128MP202.h>
#device adc=12 // Ensure 12-bit ADC resolution
#fuses NOWDT, HS // Disable watchdog, high-speed oscillator
#use delay(crystal=10mhz, clock=180000000)
// UART1 Configuration
#pin_select U1TX = PIN_B6
#pin_select U1RX = PIN_B5
#use rs232(UART1, STREAM=UART1, baud=9600, xmit=PIN_B6, rcv=PIN_B5, ERRORS)
// ADC Configuration
#define ADC_CHANNEL 0 // **Try AN0 Instead of AN4**
#define VREF 3.3 // Reference Voltage
#define ADC_MAX 4095 // 12-bit ADC max value
#define PIN_LED PIN_B7 // **LED for debugging**
void blink_led() {
int i;
for (i = 0; i < 5; i++) { // **Blink LED 5 times before ADC setup**
output_toggle(PIN_LED);
delay_ms(500);
}
}
// **Initialize ADC (With Debugging and Reset)**
void init_adc() {
fprintf(UART1, "Initializing ADC...\r\n");
setup_adc_ports(sAN0); // Ensure AN0 is analog
delay_ms(5);
setup_adc(ADC_CLOCK_DIV_64); // **Try a more stable clock setting**
set_adc_channel(ADC_CHANNEL); // Select AN0
delay_ms(10); // Allow ADC to stabilize
// **Check if ADC Module is Enabled**
if (!adc_done()) {
fprintf(UART1, "ERROR: ADC is NOT READY! ADC module may be OFF.\r\n");
} else {
fprintf(UART1, "ADC Initialized Successfully.\r\n");
}
}
// **Read ADC Value with Debugging**
unsigned int16 read_adc_value() {
unsigned int16 adc_result;
int timeout = 5000;
fprintf(UART1, "Starting ADC Conversion...\r\n");
set_adc_channel(ADC_CHANNEL); // Select correct channel
delay_us(10); // Small delay before conversion
// **Use ADC_START_ONLY to Prevent Hanging**
read_adc(ADC_START_ONLY);
// **Timeout Protection**
while (!adc_done() && timeout > 0) {
delay_us(10);
timeout--;
}
if (timeout == 0) {
fprintf(UART1, "ERROR: ADC Conversion Timed Out!\r\n");
return 0;
}
adc_result = read_adc(ADC_READ_ONLY);
fprintf(UART1, "DEBUG: read_adc() returned: %u\r\n", adc_result);
return adc_result;
}
// **Main Function**
void main() {
unsigned int16 temp_adc;
float voltage;
delay_ms(100); // Allow MCU to stabilize
fprintf(UART1, "\r\nBOOTING SYSTEM...\r\n");
blink_led(); // **Confirm MCU is running**
init_adc(); // **Initialize ADC**
while (TRUE) {
fprintf(UART1, "Trying to read ADC...\r\n");
temp_adc = read_adc_value(); // Get ADC reading
voltage = (temp_adc / (float)ADC_MAX) * VREF; // Convert ADC value to voltage
fprintf(UART1, "ADC Read: %u | Voltage: %.2fV\r\n", temp_adc, voltage);
delay_ms(1000); // Read every second
}
}
|
|
|
 |
temtronic
Joined: 01 Jul 2010
Posts: 9415
Location: Greensville,Ontario
|
|
| Posted: Sun Mar 02, 2025 5:50 pm |
|
|
ok ,curious like my cat...FYI I've never used it but...
THAT PIC appears to be 'dual core' . If so I assume YOU have to configure what pins are ADC and where they go --> PIC #1, --> PIC #2
first though , I'd get the clock set to something reasonable, say 10MHz for test purposes AND run the '1Hz LED' program to confirm it runs....
when 'happy' have main()
do....
read ADC
send data to PC
delay 500ms
...forever
CONFIRM this works, put pot on input,vary, result should 'follow' wiper.
bottom line..go back to basics, build on code that works.
forget about 'floating point' too !!
query... is that PIC rated for 180 MHz operation at 3 volt VDD ? |
|
|
Ttelmah
Joined: 11 Mar 2010
Posts: 19706
|
|
| Posted: Mon Mar 03, 2025 6:13 am |
|
|
He is rather pushing the core. It is rated for 180MHz, but only over a limited
supply and temperature range.
There are lots of extra complexities with these chips:
First the ADC clock source is not being specified. By default will be 'SYSTEM'.
Then, he needs to specify which ADC channel to use. On this ADC, there
are multiple SAR's feeding the single dedicated core on the master chip.
By default it will read all enabled channels, but he has none enabled. |
|
|
temtronic
Joined: 01 Jul 2010
Posts: 9415
Location: Greensville,Ontario
|
|
| Posted: Mon Mar 03, 2025 6:43 am |
|
|
yeesh.... sounds like a 'fun' PIC to deal with !
Kinda wondered why the 'warp speed' for the clock, especially for testing .
I KNOW board layout,wiring,etc. is critical to get more than 10 bits of good data from a 12 bit ADC. |
|
|
|
|
You cannot post new topics in this forum
You cannot reply to topics in this forum
You cannot edit your posts in this forum
You cannot delete your posts in this forum
You cannot vote in polls in this forum
|
Powered by phpBB © 2001, 2005 phpBB Group
|
FAQ 
Search
Register
Profile 
Log in to check your private messages 
Log in
