最新の fsbl で Linux が起動しない理由を調べる のバックアップ(No.2)

fsbl/src/main.c †

+ * 15.00a gan 07/21/16   Fix for CR# 953654 -(2016.3)FSBL -
+ * 			In pcap.c/pcap.h/main.c,
+ * 			Fabric Initialization sequence
+ * 			is modified to check the PL power
+ * 			before sequence starts and checking
+ * 			INIT_B reset status twice in case
+ * 			of failure.
  ...
  #ifdef STDOUT_BASEADDRESS
+ #ifdef XPAR_XUARTPS_0_BASEADDR
  #include "xuartps_hw.h"
+ #endif
  #endif
  ...

-			FabricInit();
+ 			Status = FabricInit();
+			if(Status != XST_SUCCESS){
+				ClearFSBLIn();
+				FsblHookFallback();
+			}
  ...

  void OutputStatus(u32 State)
  {
  #ifdef STDOUT_BASEADDRESS
+ #ifdef XPAR_XUARTPS_0_BASEADDR
  	u32 UartReg = 0;
+ #endif
  
  	fsbl_printf(DEBUG_GENERAL,"FSBL Status = 0x%.4lx\r\n", State);
  	/*
  	 * The TX buffer needs to be flushed out
  	 * If this is not done some of the prints will not appear on the
  	 * serial output
  	 */
+ #ifdef XPAR_XUARTPS_0_BASEADDR
  	UartReg = Xil_In32(STDOUT_BASEADDRESS + XUARTPS_SR_OFFSET);
  	while ((UartReg & XUARTPS_SR_TXEMPTY) != XUARTPS_SR_TXEMPTY) {
  		UartReg = Xil_In32(STDOUT_BASEADDRESS + XUARTPS_SR_OFFSET);
  	}
+ #endif
  #endif
  }

どうやら #ifdef XPAR_XUARTPS_0_BASEADDR があちこちに追加されているのだけれど、 今の場合は fsbl_bsp\ps7_cortexa9_0\include\xparameters.h で、

/* Canonical definitions for peripheral PS7_UART_0 */
#define XPAR_XUARTPS_0_DEVICE_ID XPAR_PS7_UART_0_DEVICE_ID
#define XPAR_XUARTPS_0_BASEADDR 0xE0000000
#define XPAR_XUARTPS_0_HIGHADDR 0xE0000FFF
#define XPAR_XUARTPS_0_UART_CLK_FREQ_HZ 100000000
#define XPAR_XUARTPS_0_HAS_MODEM 0

/* Canonical definitions for peripheral PS7_UART_1 */
#define XPAR_XUARTPS_1_DEVICE_ID XPAR_PS7_UART_1_DEVICE_ID
#define XPAR_XUARTPS_1_BASEADDR 0xE0001000
#define XPAR_XUARTPS_1_HIGHADDR 0xE0001FFF
#define XPAR_XUARTPS_1_UART_CLK_FREQ_HZ 100000000
#define XPAR_XUARTPS_1_HAS_MODEM 0

と定義されていて、事前に main.c で読み込まれているため、問題にはならなさそう。

FabricInit() の戻り値がチェックされるようになったのだけれど、 これはすでに FsblFallback の中なので、ここに到達する時点でエラーが生じているはず。 これもあまり意味が無い。

main.c の流れを追うと、

• main() の中で、
  • ps7_init(); // PCW initialization for MIO,PLL,CLK and DDR
  • // この時点で fsbl_printf を利用可能
  • SlcrUnlock(); // Unlock SLCR for SLCR register write
  • Xil_DCacheFlush(); // Flush the Caches
  • Xil_DCacheDisable(); // Disable Data Cache
  • RegisterHandlers(); // Register the Exception handlers
  • fsbl_printf(DEBUG_GENERAL,"\n\rXilinx First Stage Boot Loader \n\r");

なので、ここまで順調に来ていれば初期メッセージが表示されるはず。

fsbl_bsp\ps7_cortexa9_0\include\xparameters.h †

- #define STDIN_BASEADDRESS 0xE0001000
- #define STDOUT_BASEADDRESS 0xE0001000
+ #define STDIN_BASEADDRESS 0xE0000000
+ #define STDOUT_BASEADDRESS 0xE0000000

これ、ダメだ。

直前で見たとおり、
0xE0001000 は XUARTPS_1 のアドレス、
0xE0000000 は XUARTPS_0 のアドレス。

これを 0xE0001000 に書き換えて試すと、、、
やっぱりうんともすんとも言わないか。

main.c の ps7_init() の直後でメッセージを表示 †

何も表示されない

fsbl/src/fsbl_debug.h †

- #define	FSBL_DEBUG_INFO	1
  #define DEBUG_GENERAL	0x00000001    /* general debug  messages */
  #define DEBUG_INFO	0x00000002    /* More debug information */
  
  #if defined (FSBL_DEBUG_INFO)
  #define fsbl_dbg_current_types ((DEBUG_INFO) | (DEBUG_GENERAL))
  #elif defined (FSBL_DEBUG)
  #define fsbl_dbg_current_types (DEBUG_GENERAL)
  #else
  #define fsbl_dbg_current_types 0
  #endif
  
  #ifdef STDOUT_BASEADDRESS
  #define fsbl_printf(type,...) \
  		if (((type) & fsbl_dbg_current_types))  {xil_printf (__VA_ARGS__); }
  #else
  #define fsbl_printf(type, ...)
  #endif

さしあたってはこれが原因では無さそうだけれど、

#define FSBL_DEBUG_INFO 1 は復活させておく

ps7_init() †

• init_data にある初期化手順を実行する形になっている
• ps7GetSiliconVersion() により初期化手順を選択する

LANG:c
unsigned long *ps7_mio_init_data = ps7_mio_init_data_3_0;
unsigned long *ps7_pll_init_data = ps7_pll_init_data_3_0;
unsigned long *ps7_clock_init_data = ps7_clock_init_data_3_0;
unsigned long *ps7_ddr_init_data = ps7_ddr_init_data_3_0;
unsigned long *ps7_peripherals_init_data = ps7_peripherals_init_data_3_0;

int
ps7_init() 
{
  unsigned long si_ver = ps7GetSiliconVersion ();
  int ret;

  if (si_ver == PCW_SILICON_VERSION_1) {
    ps7_mio_init_data = ps7_mio_init_data_1_0;
    ps7_pll_init_data = ps7_pll_init_data_1_0;
    ps7_clock_init_data = ps7_clock_init_data_1_0;
    ps7_ddr_init_data = ps7_ddr_init_data_1_0;
    ps7_peripherals_init_data = ps7_peripherals_init_data_1_0;
  } else if (si_ver == PCW_SILICON_VERSION_2) {
    ps7_mio_init_data = ps7_mio_init_data_2_0;
    ps7_pll_init_data = ps7_pll_init_data_2_0;
    ps7_clock_init_data = ps7_clock_init_data_2_0;
    ps7_ddr_init_data = ps7_ddr_init_data_2_0;
    ps7_peripherals_init_data = ps7_peripherals_init_data_2_0;
  } else {
    ps7_mio_init_data = ps7_mio_init_data_3_0;
    ps7_pll_init_data = ps7_pll_init_data_3_0;
    ps7_clock_init_data = ps7_clock_init_data_3_0;
    ps7_ddr_init_data = ps7_ddr_init_data_3_0;
    ps7_peripherals_init_data = ps7_peripherals_init_data_3_0;
  }

  ret = ps7_config (ps7_mio_init_data);  
  if (ret != PS7_INIT_SUCCESS) return ret;

  ret = ps7_config (ps7_pll_init_data); 
  if (ret != PS7_INIT_SUCCESS) return ret;

  ret = ps7_config (ps7_clock_init_data);
  if (ret != PS7_INIT_SUCCESS) return ret;

  ret = ps7_config (ps7_ddr_init_data);
  if (ret != PS7_INIT_SUCCESS) return ret;

  ret = ps7_config (ps7_peripherals_init_data);
  if (ret != PS7_INIT_SUCCESS) return ret;
  return PS7_INIT_SUCCESS;
}

シリコンバージョンはメモリの値を読むだけなので失敗しようがないし、 そもそもこのあたりのコードはうまく行く方も行かない方も共通。

LANG:c
unsigned long
ps7GetSiliconVersion () {
  // Read PS version from MCTRL register [31:28]
  unsigned long mask = 0xF0000000;
  unsigned long *addr = (unsigned long*) 0XF8007080;    
  unsigned long ps_version = (*addr & mask) >> 28;
  return ps_version;
}

LANG:c
int
ps7_config(unsigned long * ps7_config_init) 
{
    unsigned long *ptr = ps7_config_init;

    unsigned long  opcode;            // current instruction ..
    unsigned long  args[16];           // no opcode has so many args ...
    int  numargs;           // number of arguments of this instruction
    int  j;                 // general purpose index

    volatile unsigned long *addr;         // some variable to make code readable
    unsigned long  val,mask;              // some variable to make code readable

    int finish = -1 ;           // loop while this is negative !
    int i = 0;                  // Timeout variable
    
    while( finish < 0 ) {
        numargs = ptr[0] & 0xF;
        opcode = ptr[0] >> 4;

        for( j = 0 ; j < numargs ; j ++ ) 
            args[j] = ptr[j+1];
        ptr += numargs + 1;
        
        
        switch ( opcode ) {
            
        case OPCODE_EXIT:
            finish = PS7_INIT_SUCCESS;
            break;
            
        case OPCODE_CLEAR:
            addr = (unsigned long*) args[0];
            *addr = 0;
            break;

        case OPCODE_WRITE:
            addr = (unsigned long*) args[0];
            val = args[1];
            *addr = val;
            break;

        case OPCODE_MASKWRITE:
            addr = (unsigned long*) args[0];
            mask = args[1];
            val = args[2];
            *addr = ( val & mask ) | ( *addr & ~mask);
            break;

        case OPCODE_MASKPOLL:
            addr = (unsigned long*) args[0];
            mask = args[1];
            i = 0;
            while (!(*addr & mask)) {
                if (i == PS7_MASK_POLL_TIME) {
                    finish = PS7_INIT_TIMEOUT;
                    break;
                }
                i++;
            }
            break;
        case OPCODE_MASKDELAY:
            addr = (unsigned long*) args[0];
            mask = args[1];
            int delay = get_number_of_cycles_for_delay(mask);
            perf_reset_and_start_timer(); 
            while ((*addr < delay)) {
            }
            break;
        default:
            finish = PS7_INIT_CORRUPT;
            break;
        }
    }
    return finish;
}

#define PS7_MASK_POLL_TIME 100000000

fsbl_bsp\ps7_cortexa9_0\libsrc\standalone_v?_?\src\xil_printf.c †

最終的には outbyte を呼び出す

fsbl_bsp\ps7_cortexa9_0\libsrc\standalone_v?_?\src\outbyte.c †

バージョン間で変更なし

LANG:c
void outbyte(char c) {
	 XUartPs_SendByte(STDOUT_BASEADDRESS, c);
}

fsbl_bsp\ps7_cortexa9_0\libsrc\uartps_v?_?\src\xuartps.c †

かなりいろいろ変更されていて厳しい、が、どうやらキャストを増やしてたりする感じ。