質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

87.80%

PICマイコン タイマ割込み ループ処理中

解決済

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 1,521

score 106

やりたいこと

PICマイコン(PIC16F18xx、XC8コンパイラ)を使用して、タイマ割込み処理を行っています。
割込みフラグを用意して、フラグの値によって、①と②の動作を切り替えた動作をしています。
①通常動作
3色のLEDを順番に点灯させています。
②タイマ割込み動作
SW(スイッチ)が押されたら、スイッチが押されている間は(ずっと)すべてのLEDを点灯させます。 

いまできていること

↓のコードで、割込みフラグによって、①と②の動作を切り替えるところまでできています。

困っていること

①の動作の内容を変更したところ、タイマ割込み動作が行われなくなってしまいました。

変更した内容は、次の通りです。
①' 3色LEDの順番点灯を『100回繰り返し、100回に1度別の動作を行う。(ブザーを鳴らす)』
100回繰り返すところで、for文によるループ処理を入れています。

コード

繰り返し処理の部分は、コメントアウトしてあります。

    while(1){       
        if(flag ==0){
            LED();
        }
        else{
            SWON();
            flag=0;
        }
    }
}

//割り込み関数
void interrupt isr(){
    GIE = 0;
    if(TMR1IF == 1){
    //TMR1H = (63036 >>8); //タイマー1の初期化(65536-2500=63036);
    //TMR1L = (63036 & 0x00ff);
    flag=1;
    TMR1IF = 0; //割り込みフラグを落とす
    }
    GIE = 1;
}

void LED(void)
{
    int i;
    //LED初期設定
    LED1=0;
    LED2=0;
    LED3=0;

    //カウントごとに行う処理(省略)

    //for  (i = 0; i < 100; i++) {
        //SW = PORTCbits.RC3;
        LED1=1;
        __delay_ms(100);     //100msec
        LED1=0;
        __delay_ms(100);     //100msec
        LED2=1;
        __delay_ms(100);     //100msec
        LED2=0;
        __delay_ms(100);     //100msec
        LED3=1;
        __delay_ms(100);     //100msec
        LED3=0;
    //}

}

void SWON(void)
{
    while(SW == 0){
        LED1=1;
        LED2=1;
        LED3=1;
    }

}
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

0

forを入れるとタイマ割り込みが行われなくなるんではなくて、タイマ割り込みでフラグがセットされても、そのforループが終わらないとつけっぱの処理が走らないだけ、です。

で、提示のコードではタイマ割り込みはなんの処理もしてませんね

void interrupt isr(){
    if(TMR1IF == 1){
      flag=(SW!=0);
      TMR1IF = 0;
    }
    GIE = 1;     // この行いらないかも
}

   // メインループ
    while(1){       
        if(flag ==0){
            LED();
        }
        else{
          LED1=1;
          LED2=1;
          LED3=1;
        }
    }


こんなもんですかねー

#割り込みルーチン省略しすぎたので修正

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/08/27 10:51

    回答ありがとうございました。
    1つ教えてください。
    今回は、SWの状態に応じてフラグの値を設定し、フラグの値に応じた処理を実行しています。
    フラグを宣言する際に、「volatile」を使用するようにアドバイスをいただきましたが、
    なぜ、「volatile」を使用するのかが理解できていません。
    教えていただけないでしょうか。

    キャンセル

  • 2019/08/27 13:07

    コンパイル時に、関数の中の変数の値は代入しない限り変更されない、という前提で最適化されます。
    ふつーの変数の場合はこれで正しいのできちんと動きますが、割り込みで変更されるような変数の場合、それでは動作がおかしくなる場合があります
    volatile というのは、対象の変数の最適化はしないようにというにコンパイラの指示です
    これで、上記のような不具合は起こらなくなります

    キャンセル

  • 2019/08/27 14:03

    とても分かりやすい説明ありがとうございました。
    助かりました。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 87.80%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る