Running PCIe in CM7 IMX8MP

[mbahmani90]

Hi,

Hope you are all well.

I would like to use PCIe through CM7 in IMX8MP. I found a PCIe driver in zephyr/drivers/pcie/pcie.h

I included this in my project #include <zephyr/drivers/pcie/pcie.h>.

And I defined pcie node in device tree nxp/nxp_imx8ml_m7.dtsi:

pcie0: pcie@33800000 {
	compatible = "pcie-controller";
	reg = <0x33800000 0x400000>, <0x1ff00000 0x80000>;
	#address-cells = <3>;
	#size-cells = <2>;
	ranges =  <0x81000000 0 0x00000000 0x1ff80000 0 0x00010000>, /* downstream I/O 64KB */
				<0x82000000 0 0x18000000 0x18000000 0 0x07f00000>; /* non-prefetchable memory */
	status = "okay";	  
};

In imx8mp_evk_mimx8ml8_m7.dts:

	chosen {
		...
		zephyr_pcie_controller = &pcie0;
	};

In prj.conf:

CONFIG_PCIE=y
CONFIG_PCIE_CONTROLLER=y

However, I get error:

-- Zephyr version: 3.6.99 (/home/xxx/nxp/Repositories/DSP_Zephyr/zephyr/zephyr), build: v3.6.0-2050-g3c42daee9c1b
[73/128] Building C object zephyr/drivers/pcie/host/CMakeFiles/drivers__pcie__host.dir/controller.c.obj
/home/xxx/nxp/Repositories/DSP_Zephyr/zephyr/zephyr/drivers/pcie/host/controller.c: In function 'pcie_generic_ctrl_enumerate_bars':
/home/xxx/nxp/Repositories/DSP_Zephyr/zephyr/zephyr/drivers/pcie/host/controller.c:108:63: warning: right shift count >= width of type [-Wshift-count-overflow]
  108 |                         pcie_conf_write(bdf, reg + 1, scratch >> 32);
      |                                                               ^~
In file included from /home/xxx/nxp/Repositories/DSP_Zephyr/zephyr/zephyr/include/zephyr/logging/log.h:11,
                 from /home/xxx/nxp/Repositories/DSP_Zephyr/zephyr/zephyr/drivers/pcie/host/controller.c:7:
/home/xxx/nxp/Repositories/DSP_Zephyr/zephyr/zephyr/drivers/pcie/host/controller.c:135:33: warning: format '%lx' expects argument of type 'long unsigned int', but argument 6 has type 'size_t' {aka 'unsigned int'} [-Wformat=]
  135 |                         LOG_INF("[%02x:%02x.%x] BAR%d size 0x%lx "
      |                                 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
......
  138 |                                 bar, bar_size,
      |                                      ~~~~~~~~
      |                                      |
      |                                      size_t {aka unsigned int}
/home/xxx/nxp/Repositories/DSP_Zephyr/zephyr/zephyr/include/zephyr/logging/log_core.h:275:42: note: in definition of macro 'Z_LOG2'
  275 |                 z_log_printf_arg_checker(__VA_ARGS__); \
      |                                          ^~~~~~~~~~~
/home/xxx/nxp/Repositories/DSP_Zephyr/zephyr/zephyr/include/zephyr/logging/log.h:65:24: note: in expansion of macro 'Z_LOG'
   65 | #define LOG_INF(...)   Z_LOG(LOG_LEVEL_INF, __VA_ARGS__)
      |                        ^~~~~
/home/xxx/nxp/Repositories/DSP_Zephyr/zephyr/zephyr/drivers/pcie/host/controller.c:135:25: note: in expansion of macro 'LOG_INF'
  135 |                         LOG_INF("[%02x:%02x.%x] BAR%d size 0x%lx "
      |                         ^~~~~~~
/home/xxx/nxp/Repositories/DSP_Zephyr/zephyr/zephyr/drivers/pcie/host/controller.c:135:64: note: format string is defined here
  135 |                         LOG_INF("[%02x:%02x.%x] BAR%d size 0x%lx "
      |                                                              ~~^
      |                                                                |
      |                                                                long unsigned int
      |                                                              %x
/home/xxx/nxp/Repositories/DSP_Zephyr/zephyr/zephyr/drivers/pcie/host/controller.c:145:76: warning: right shift count >= width of type [-Wshift-count-overflow]
  145 |                                 pcie_conf_write(bdf, reg + 1, bar_bus_addr >> 32);
      |                                                                            ^~
/home/xxx/nxp/Repositories/DSP_Zephyr/zephyr/zephyr/drivers/pcie/host/controller.c:148:33: warning: format '%lx' expects argument of type 'long unsigned int', but argument 6 has type 'size_t' {aka 'unsigned int'} [-Wformat=]
  148 |                         LOG_INF("[%02x:%02x.%x] BAR%d size 0x%lx Failed memory allocation.",
      |                                 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
  149 |                                 PCIE_BDF_TO_BUS(bdf), PCIE_BDF_TO_DEV(bdf), PCIE_BDF_TO_FUNC(bdf),
  150 |                                 bar, bar_size);
      |                                      ~~~~~~~~
      |                                      |
      |                                      size_t {aka unsigned int}
/home/xxx/nxp/Repositories/DSP_Zephyr/zephyr/zephyr/include/zephyr/logging/log_core.h:275:42: note: in definition of macro 'Z_LOG2'
  275 |                 z_log_printf_arg_checker(__VA_ARGS__); \
      |                                          ^~~~~~~~~~~
/home/xxx/nxp/Repositories/DSP_Zephyr/zephyr/zephyr/include/zephyr/logging/log.h:65:24: note: in expansion of macro 'Z_LOG'
   65 | #define LOG_INF(...)   Z_LOG(LOG_LEVEL_INF, __VA_ARGS__)
      |                        ^~~~~
/home/xxx/nxp/Repositories/DSP_Zephyr/zephyr/zephyr/drivers/pcie/host/controller.c:148:25: note: in expansion of macro 'LOG_INF'
  148 |                         LOG_INF("[%02x:%02x.%x] BAR%d size 0x%lx Failed memory allocation.",
      |                         ^~~~~~~
/home/xxx/nxp/Repositories/DSP_Zephyr/zephyr/zephyr/drivers/pcie/host/controller.c:148:64: note: format string is defined here
  148 |                         LOG_INF("[%02x:%02x.%x] BAR%d size 0x%lx Failed memory allocation.",
      |                                                              ~~^
      |                                                                |
      |                                                                long unsigned int
      |                                                              %x
[123/128] Linking C executable zephyr/zephyr_pre0.elf
FAILED: zephyr/zephyr_pre0.elf zephyr/zephyr_pre0.map /home/xxx/nxp/Repositories/DSP_Zephyr/zephyr/build/zephyr/zephyr_pre0.map 
: && /home/xxx/zephyr-sdk-0.16.5-1/arm-zephyr-eabi/bin/arm-zephyr-eabi-gcc  -gdwarf-4 zephyr/CMakeFiles/zephyr_pre0.dir/misc/empty_file.c.obj -o zephyr/zephyr_pre0.elf  zephyr/CMakeFiles/offsets.dir/./arch/arm/core/offsets/offsets.c.obj  -fuse-ld=bfd  -T  zephyr/linker_zephyr_pre0.cmd  -Wl,-Map=/home/xxx/nxp/Repositories/DSP_Zephyr/zephyr/build/zephyr/zephyr_pre0.map  -Wl,--whole-archive  app/libapp.a  zephyr/libzephyr.a  zephyr/arch/common/libarch__common.a  zephyr/arch/arch/arm/core/libarch__arm__core.a  zephyr/arch/arch/arm/core/cortex_m/libarch__arm__core__cortex_m.a  zephyr/arch/arch/arm/core/mpu/libarch__arm__core__mpu.a  zephyr/lib/libc/picolibc/liblib__libc__picolibc.a  zephyr/lib/libc/common/liblib__libc__common.a  zephyr/drivers/pcie/host/libdrivers__pcie__host.a  zephyr/drivers/clock_control/libdrivers__clock_control.a  zephyr/drivers/console/libdrivers__console.a  zephyr/drivers/gpio/libdrivers__gpio.a  zephyr/drivers/pinctrl/libdrivers__pinctrl.a  zephyr/drivers/serial/libdrivers__serial.a  zephyr/drivers/timer/libdrivers__timer.a  modules/hal_nxp/hal_nxp/lib..__modules__hal__nxp.a  -Wl,--no-whole-archive  zephyr/kernel/libkernel.a  -L"/home/xxx/zephyr-sdk-0.16.5-1/arm-zephyr-eabi/bin/../lib/gcc/arm-zephyr-eabi/12.2.0/thumb/v7e-m/nofp"  -L/home/xxx/nxp/Repositories/DSP_Zephyr/zephyr/build/zephyr  -lgcc  zephyr/arch/common/libisr_tables.a  -mcpu=cortex-m7  -mthumb  -mabi=aapcs  -mfp16-format=ieee  -mtp=soft  -Wl,--gc-sections  -Wl,--build-id=none  -Wl,--sort-common=descending  -Wl,--sort-section=alignment  -Wl,-u,_OffsetAbsSyms  -Wl,-u,_ConfigAbsSyms  -nostdlib  -static  -Wl,-X  -Wl,-N  -Wl,--orphan-handling=warn  -Wl,-no-pie  -DPICOLIBC_LONG_LONG_PRINTF_SCANF  --specs=picolibc.specs  -lc  -lgcc && cd /home/xxx/nxp/Repositories/DSP_Zephyr/zephyr/build/zephyr && /usr/bin/cmake -E true
/home/xxx/zephyr-sdk-0.16.5-1/arm-zephyr-eabi/bin/../lib/gcc/arm-zephyr-eabi/12.2.0/../../../../arm-zephyr-eabi/bin/ld.bfd: zephyr/drivers/pcie/host/libdrivers__pcie__host.a(controller.c.obj): in function `pcie_ctrl_conf_read':
/home/xxx/nxp/Repositories/DSP_Zephyr/zephyr/zephyr/include/zephyr/drivers/pcie/controller.h:202: undefined reference to `__device_dts_ord_168'
collect2: error: ld returned 1 exit status
ninja: build stopped: subcommand failed.
FATAL ERROR: command exited with status 1: /usr/bin/cmake --build /home/xxx/nxp/Repositories/DSP_Zephyr/zephyr/build

Can you help me to find out the issue?

Thanks.

[JiafeiPan]

Hi, @mbahmani90 currently we don't enable and verify PCIe driver on Zephyr of IMX8MP.