PIC24をもちいた手動ADCサンプリング

Question

前提・実現したいこと

現在マイコンでのアナログ入力を試しているのですがうまく動かせておりません。
下記の事をしたいのですがお力をお貸しいただけますでしょうか。

最終的に複数のスライドボリューム（可変抵抗器）とマイコンのアナログ入力から
LEDをPWM調光する。

ここに質問の内容を詳しく書いてください。

使用マイコン
pic24fj64ga002

接続
スライドボリューム　ー＞（アナログ入力）マイコン（アウトプット）ー＞抵抗ー＞LED

上記接続時に手動サンプリングのプログラムがうまく動かせない

発生している問題・エラーメッセージ

MCCでEnable Auto Samplingにチェックを入れた場合はAN0からの値を読み取れて
LEDの調光ができるのですが
手動でサンプリングを行おうとしたときに値をAN0から読み取ろうとしない。
ソースコード自体はMCC v4のadc1.hのサンプルコードを参照しております。

エラーメッセージ
無し

該当のソースコード

xc16 + mcc

ソースコード

#include "mcc_generated_files/system.h"
#include "mcc_generated_files/adc1.h"

//  32000000UL/2　system.hにて#define _XTAL_FREQ定義
#define FCY 16000000UL
#include <libpic30.h>


int main(void)
{

    unsigned int conversion;
    
    // initialize the device
    SYSTEM_Initialize();
    ADC1_Initialize();
    while (1)
    {
        ADC1_ChannelSelect(channel_AN0);
        ADC1_Start();
        //Provide Delay充電時間？
        __delay_us(20);
        ADC1_Stop();
        while(!ADC1_IsConversionComplete())
        {
            ADC1_Tasks();   
        }
        conversion = ADC1_ConversionResultGet();       
       
        
 //       conversion= (ADC1_GetConversion(0x00));
      //PWM分解能8bit(10bit/4)
        OC1RS=(conversion/4);
        __delay_us(20);

        // Add your application code

    }

    return 1;
}

試したこと

他のチャンネルでも読み取れなかったAN1,AN9～AN12
充電時間120usでも同様
Enable ADC Interruptのチェック

void ADC1_ChannelSelect( ADC1_CHANNEL channel )
{
    AD1CHS = channel;
//ここを直接値を入れても同様 0x0000 や0b00000000
}

補足情報（FW/ツールのバージョンなど）

PICKIT3
PIC24F-GA-GB_DEP 1.4.141
xc16 v1.6

Answer 1

MCC ADC1 "Enable Auto Sampling" のチェックを外す

ADC データ読み出し

unsigned int i;
i = ADC1_GetConversion( channel_AN0 );

MCC adc1.c 自動生成

uint16_t ADC1_GetConversion(ADC1_CHANNEL channel)
{
    ADC1_ChannelSelect(channel);
    
    ADC1_Start();
    ADC1_Stop();
    
    while (!ADC1_IsConversionComplete())
    {
    }
       
    return ADC1_ConversionResultGet();
}

追加

テストプログラム
 printf("PIC24FJ64GA002 adc test \n\r");
 printf("ADC channel_AN0 = %04X \n\r", ADC1_GetConversion( channel_AN0 ) );
 printf("ADC channel_AN1 = %04X \n\r", ADC1_GetConversion( channel_AN1 ) );

シリアル出力
PIC24FJ64GA002 adc test
ADC channel_AN0 = 0000
ADC channel_AN1 = 03FF

Comments

[masa1124]

ご回答ありがとうございます。
チェックを外し
初期設定のAD1CHS を両方ともAN9にして
ADC1_GetConversion( channel)のチャンネルをAN1にしても
AN0の値が呼び出されてしまいます。
（AN0ボリューム、AN1に3.3V、AN9に０V印加）

[koujikuu]

当方と違う点は、ADC Clock : RC clock 指定
Scan Enable のチェックがありません

[masa1124]

RC Clock指定
及びScan Enableのチェックを外してPICを新しいものに取り換えたら無事動作しました。
こちらの情報提供不足、技術レベル不足にもかかわらず
設定画面・動作ソースの貼り付け、出力結果の書き出しまでしていただいたおかげで
プログラムとそれ以外にも問題があることがわかり問題解決できましたので
koujikuu様のご回答をベストアンサーとさせていただきます。
皆様本当にありがとうございました。

Answer 2

そのPICマイコンの資料見てないんで、一般的な話となりますが、
A/Dデバイスの変換時間というのは一般的に数μsecというオーダなので、コードで終了判定する必要はないです。
また、その都度変換開始トリガするだけでいいので、おそらくADC1_Stop();は不要かと思われます
ということで、

        ADC1_ChannelSelect(channel_AN0);
        ADC1_Start();
        //Provide Delay充電時間？
        __delay_us(20);
        conversion = ADC1_ConversionResultGet();      

としといて、この変換結果を見れるようにシリアルポートに出力するなりしときましょう。
その数値をモニタして、ボリウムに応じた値になってるでしょうか

Comments

[masa1124]

ご回答ありがとうございます。

ADC1_Stop();はコメントアウトしてもしなくても動作に変わりはありませんでした。
入力ピンを一つだけにするとそのピンでの入力が可能になりました。
ADC1_ChannelSelect()がうまくいっていないみたいです。

PCとシリアルでつなぐのに９Vへのレベル変換基盤を持っていない為確認できません。
代わりにPWMの出力ピンに対してオシロスコープで波形を見ている限りでは
単独ピンで動作しているときは値が変化しているみたいです。