チョコさんが以前に指摘されていたRL78コード生成機能によるCSI連続送信の終了の割り込みコードの問題を調べています

チョコさん、こんにちは。NoMaYです。

すみません、掲題の件、下記のスレッドでの話題ですが、コード生成されたソースの以下のコードのことでしょうか?

r_cg_serial_user.c

static void __near r_csi00_interrupt(void)
{
    if (g_csi00_tx_count > 0U)
    {
        SIO00 = *gp_csi00_tx_address;
        gp_csi00_tx_address++;
        g_csi00_tx_count--;
    }
    else
    {
        if ((SSR00 & _0040_SAU_UNDER_EXECUTE) == 0U)
        {
            r_csi00_callback_sendend();    /* complete send */
        }
        else
        {
            SMR00 &= ~_0001_SAU_BUFFER_EMPTY;
        }
    }
}

RL78/G14のEEPROMのI2C通信について
japan.renesasrulz.com/cafe_rene/f/002-2095199602/5962/rl78-g14-eeprom-i2c/33068#33068

チョコです。
鈴木さん,MAX7219をCSIで制御してLED表示を行っていて,問題を見つけました。
CSIの連続送信で最後のデータを書き込んだ後で,割り込みタイミングを書き換えてますが,転送速度とプログラムの実行速度の関係で,最後の割り込みが発生しない可能性があります。


RL78/G13 CSIのマスタ連続送受信でオーバーランエラーが発生する
japan.renesasrulz.com/cafe_rene/f/002-2095199602/5972/rl78-g13-csi/33137#33137

チョコです。
この現象については,「RL78/G14のEEPROMのI2C通信について」で少し触れていますが,次のような動作になっていると考えられます。
①最後のデータを書き込むための割り込みが発生します。
②CSIは最後のデータを送信します。
③CPUは割り込みを受け付けると,最後のデータの送信中かチェックして,最後のデータなら割り込みタイミングを転送完了に書き換えます。
ところが,③で割り込みタイミングを書き換える前に送信が完了してしまっていると,書き換えた時点では,送信完了割り込みの発生タイミングを終了しているので,期待した最後の割り込みが発生しません。
そのため,通信処理を終えることができなくなってしまいます。


なお、以下のCS+ V8.03の画面コピーは先程のソースを生成させた時のコード生成機能の設定内容です。



 

Parents
  • こんにちは。NoMaYです。#スレッドのタイトルを少し変えました。

    今度は以下により現象を再現させるプログラムを作成しました。(RL78/G14 Fast Prototyping Boardの単体とRenesas RL78 Simulatorの両方で現象を再現させることが出来ました。)

    ・ CPU動作周波数は32MHz
    ・ CSIボーレートを2Mbps
    ・ 通常処理側で、DI+狙った数のNOP+EIにより、くだんの割り込みの受付を遅延させて、まさに危険な区間で送信終了する(シフトレジスタから送出終了する)ようにした

    以下にプロジェクトのファイル一式を添付します。(当方特有の事情で、コード生成はCS+ V8.03、シミュレータはCS+ V8.01、ビルドはCC-RL V1.02、を使用しています。)上記のCPU動作周波数とCSIボーレートの変更の他に、P1のb2, b1, b0を出力ポートに変更しています。今回は通常処理側でのDI/EIによる割り込み禁止区間が分かるようにb0(のみ)を0→1→0と変化させています。(なお、P1のb2, b1, b0はRL78/G14 Fast Prototyping BoardのArduinoソケットからでも観測可能です。)

    プロジェクトのファイル一式:
    issue_20200228_0302.zip

    今回、通常処理ルーチン側を以下の通りとしました。

    r_main.c

    void main(void)
    {
        R_MAIN_UserInit();
        /* Start user code. Do not edit comment generated here */
        {
            uint8_t tx[5];
            EI();
            R_CSI00_Start();
            tx[0] = 0;
            tx[1] = 1;
            tx[2] = 2;
            tx[3] = 3;
            tx[4] = 4;
            R_CSI00_Send(tx, 5);

    #if 0 /* Renesas RL78 Simulator */
    #define SIM if(1)
    #define _x_
    #define FPB else if(0)
    #elif 1 /* RL78/G14 Fast Prototyping Board */
    #define SIM if(0)
    #define _x_
    #define FPB else if(1)
    #endif

            /* Note that total clocks of below line are clocks of NOPs + clocks in ISR. */
            SIM{ NOP300();NOP5(); } _x_ FPB{ NOP300();NOP10();NOP5(); }
    #if 0
            /* LED0 == ON,  LED1 == ON     --> OK */
            /* nothing */
    #elif 0
            /* LED0 == ON,  LED1 == ON     --> OK */
            DI();
            P1 = 1;
            SIM{ NOP80();NOP3(); } _x_ FPB{ NOP70();NOP9(); }
            P1 = 0;
            EI();
    #elif 1
            /* LED0 == ON,  LED1 == OFF!!! --> NG!!! */
            DI();
            P1 = 1;
            SIM{ NOP80();NOP4(); } _x_ FPB{ NOP80();        }
            P1 = 0;
            EI();
    #elif 0
            /* LED0 == ON,  LED1 == OFF!!! --> NG!!! */
            DI();
            P1 = 1;
            SIM{ NOP90();NOP5(); } _x_ FPB{ NOP90();NOP3(); }
            P1 = 0;
            EI();
    #elif 0
            /* LED0 == OFF, LED1 == ON     --> OK */
            DI();
            P1 = 1;
            SIM{ NOP90();NOP6(); } _x_ FPB{ NOP90();NOP4(); }
            P1 = 0;
            EI();
    #endif
            NOP();

            while (false == g_csi00_tx_ready_flag)
            {
                NOP();
            }
        }
        END();

        while (1U)
        {
            NOP();
        }
        /* End user code. Do not edit comment generated here */
    }

    他方、割り込み処理ルーチン側は以下の通りです。

    r_cg_serial_user.c

    static void __near r_csi00_interrupt(void)
    {
        if (g_csi00_tx_count > 0U)
        {
            SIO00 = *gp_csi00_tx_address;
            gp_csi00_tx_address++;
            g_csi00_tx_count--;
        }
        else
        {

            if ((SSR00 & _0040_SAU_UNDER_EXECUTE) == 0U)
            {
                r_csi00_callback_sendend();    /* complete send */
                LED1 = LED_ON;
            }
            else
            {
                SMR00 &= ~_0001_SAU_BUFFER_EMPTY;
                LED0 = LED_ON;
            }
        }
    }

    なお、上記の割り込み処理ルーチンのリストファイルは以下の通りです。(デフォルト最適化でコンパイルしています。) SSR00からの読み出し箇所とSMR00への書き込み箇所を赤文字にしています。

        。。。途中省略。。。

        ;***       85 :         if ((SSR00 & _0040_SAU_UNDER_EXECUTE) == 0U)
        movw ax, !0x0100
        mov a, x
        bt a.6, $.BB@LABEL@1_3
    .BB@LABEL@1_2:  ; if_then_bb14
        ;***       86 :         {
        ;***       87 :             r_csi00_callback_sendend();    /* complete send */
        call !!_r_csi00_callback_sendend@1
        ;***       88 :             LED1 = LED_ON;
        clr1 0xFFF04.4
        br $.BB@LABEL@1_4
    .BB@LABEL@1_3:  ; if_else_bb18
        movw de, #0x0110
        ;***       89 :         }
        ;***       90 :         else
        ;***       91 :         {
        ;***       92 :             SMR00 &= ~_0001_SAU_BUFFER_EMPTY;
        movw ax, [de]
        xch a, x
        and a, #0xFE
        xch a, x
        movw [de], ax
        ;***       93 :             LED0 = LED_ON;
        clr1 0xFFF04.3

        。。。以後省略。。。

    以下、Renesas RL78 Simulatorでの実行結果です。

    /* LED0 == ON,  LED1 == ON     --> OK */
    /* nothing */
    /* LED0 == ON,  LED1 == ON     --> OK */
    DI();
    P1 = 1;
    SIM{ NOP80();NOP3(); } _x_ FPB{ NOP70();NOP9(); }
    P1 = 0;
    EI();
    /* LED0 == ON,  LED1 == OFF!!! --> NG!!! */
    DI();
    P1 = 1;
    SIM{ NOP80();NOP4(); } _x_ FPB{ NOP80();        }
    P1 = 0;
    EI();
    /* LED0 == OFF, LED1 == ON     --> OK */
    DI();
    P1 = 1;
    SIM{ NOP90();NOP6(); } _x_ FPB{ NOP90();NOP4(); }
    P1 = 0;
    EI();

     

Reply
  • こんにちは。NoMaYです。#スレッドのタイトルを少し変えました。

    今度は以下により現象を再現させるプログラムを作成しました。(RL78/G14 Fast Prototyping Boardの単体とRenesas RL78 Simulatorの両方で現象を再現させることが出来ました。)

    ・ CPU動作周波数は32MHz
    ・ CSIボーレートを2Mbps
    ・ 通常処理側で、DI+狙った数のNOP+EIにより、くだんの割り込みの受付を遅延させて、まさに危険な区間で送信終了する(シフトレジスタから送出終了する)ようにした

    以下にプロジェクトのファイル一式を添付します。(当方特有の事情で、コード生成はCS+ V8.03、シミュレータはCS+ V8.01、ビルドはCC-RL V1.02、を使用しています。)上記のCPU動作周波数とCSIボーレートの変更の他に、P1のb2, b1, b0を出力ポートに変更しています。今回は通常処理側でのDI/EIによる割り込み禁止区間が分かるようにb0(のみ)を0→1→0と変化させています。(なお、P1のb2, b1, b0はRL78/G14 Fast Prototyping BoardのArduinoソケットからでも観測可能です。)

    プロジェクトのファイル一式:
    issue_20200228_0302.zip

    今回、通常処理ルーチン側を以下の通りとしました。

    r_main.c

    void main(void)
    {
        R_MAIN_UserInit();
        /* Start user code. Do not edit comment generated here */
        {
            uint8_t tx[5];
            EI();
            R_CSI00_Start();
            tx[0] = 0;
            tx[1] = 1;
            tx[2] = 2;
            tx[3] = 3;
            tx[4] = 4;
            R_CSI00_Send(tx, 5);

    #if 0 /* Renesas RL78 Simulator */
    #define SIM if(1)
    #define _x_
    #define FPB else if(0)
    #elif 1 /* RL78/G14 Fast Prototyping Board */
    #define SIM if(0)
    #define _x_
    #define FPB else if(1)
    #endif

            /* Note that total clocks of below line are clocks of NOPs + clocks in ISR. */
            SIM{ NOP300();NOP5(); } _x_ FPB{ NOP300();NOP10();NOP5(); }
    #if 0
            /* LED0 == ON,  LED1 == ON     --> OK */
            /* nothing */
    #elif 0
            /* LED0 == ON,  LED1 == ON     --> OK */
            DI();
            P1 = 1;
            SIM{ NOP80();NOP3(); } _x_ FPB{ NOP70();NOP9(); }
            P1 = 0;
            EI();
    #elif 1
            /* LED0 == ON,  LED1 == OFF!!! --> NG!!! */
            DI();
            P1 = 1;
            SIM{ NOP80();NOP4(); } _x_ FPB{ NOP80();        }
            P1 = 0;
            EI();
    #elif 0
            /* LED0 == ON,  LED1 == OFF!!! --> NG!!! */
            DI();
            P1 = 1;
            SIM{ NOP90();NOP5(); } _x_ FPB{ NOP90();NOP3(); }
            P1 = 0;
            EI();
    #elif 0
            /* LED0 == OFF, LED1 == ON     --> OK */
            DI();
            P1 = 1;
            SIM{ NOP90();NOP6(); } _x_ FPB{ NOP90();NOP4(); }
            P1 = 0;
            EI();
    #endif
            NOP();

            while (false == g_csi00_tx_ready_flag)
            {
                NOP();
            }
        }
        END();

        while (1U)
        {
            NOP();
        }
        /* End user code. Do not edit comment generated here */
    }

    他方、割り込み処理ルーチン側は以下の通りです。

    r_cg_serial_user.c

    static void __near r_csi00_interrupt(void)
    {
        if (g_csi00_tx_count > 0U)
        {
            SIO00 = *gp_csi00_tx_address;
            gp_csi00_tx_address++;
            g_csi00_tx_count--;
        }
        else
        {

            if ((SSR00 & _0040_SAU_UNDER_EXECUTE) == 0U)
            {
                r_csi00_callback_sendend();    /* complete send */
                LED1 = LED_ON;
            }
            else
            {
                SMR00 &= ~_0001_SAU_BUFFER_EMPTY;
                LED0 = LED_ON;
            }
        }
    }

    なお、上記の割り込み処理ルーチンのリストファイルは以下の通りです。(デフォルト最適化でコンパイルしています。) SSR00からの読み出し箇所とSMR00への書き込み箇所を赤文字にしています。

        。。。途中省略。。。

        ;***       85 :         if ((SSR00 & _0040_SAU_UNDER_EXECUTE) == 0U)
        movw ax, !0x0100
        mov a, x
        bt a.6, $.BB@LABEL@1_3
    .BB@LABEL@1_2:  ; if_then_bb14
        ;***       86 :         {
        ;***       87 :             r_csi00_callback_sendend();    /* complete send */
        call !!_r_csi00_callback_sendend@1
        ;***       88 :             LED1 = LED_ON;
        clr1 0xFFF04.4
        br $.BB@LABEL@1_4
    .BB@LABEL@1_3:  ; if_else_bb18
        movw de, #0x0110
        ;***       89 :         }
        ;***       90 :         else
        ;***       91 :         {
        ;***       92 :             SMR00 &= ~_0001_SAU_BUFFER_EMPTY;
        movw ax, [de]
        xch a, x
        and a, #0xFE
        xch a, x
        movw [de], ax
        ;***       93 :             LED0 = LED_ON;
        clr1 0xFFF04.3

        。。。以後省略。。。

    以下、Renesas RL78 Simulatorでの実行結果です。

    /* LED0 == ON,  LED1 == ON     --> OK */
    /* nothing */
    /* LED0 == ON,  LED1 == ON     --> OK */
    DI();
    P1 = 1;
    SIM{ NOP80();NOP3(); } _x_ FPB{ NOP70();NOP9(); }
    P1 = 0;
    EI();
    /* LED0 == ON,  LED1 == OFF!!! --> NG!!! */
    DI();
    P1 = 1;
    SIM{ NOP80();NOP4(); } _x_ FPB{ NOP80();        }
    P1 = 0;
    EI();
    /* LED0 == OFF, LED1 == ON     --> OK */
    DI();
    P1 = 1;
    SIM{ NOP90();NOP6(); } _x_ FPB{ NOP90();NOP4(); }
    P1 = 0;
    EI();

     

Children
No Data