On Wed, Oct 6, 2021 at 4:45 PM Tim Harvey tharvey@gateworks.com wrote:
The GW7902 is based on the i.MX 8M Mini / Nano SoC featuring:
- LPDDR4 DRAM
- eMMC FLASH
- Gateworks System Controller
- LTE CAT M1 modem
- USB 2.0 HUB
- M.2 Socket with USB2.0, PCIe, and dual-SIM
- IMX8M FEC
- PCIe based GbE
- RS232/RS485/RS422 serial transceiver
- GPS
- CAN bus
- WiFi / Bluetooth
- MIPI header (DSI/CSI/GPIO/PWM/I2S)
- PMIC
To add support for the i.MX8M Nano GW7902:
- Add imx8mn-venice dts/defconfig/include
- Add imx8mn-gw7902 dts
- Add imx8mn-2gb lpddr4 dram configs
- Add misc support for IMX8M Nano SoC
- rename imx8mm-venice.c to venice.c as it is no longer imx8mm specific
- update README with differences for IMX8MN vs IMX8MM
Signed-off-by: Tim Harvey tharvey@gateworks.com
v2:
- rebase on origin/master
- remove unused fdt_pack_reg function
arch/arm/dts/Makefile | 2 + arch/arm/dts/imx8mn-venice-gw7902-u-boot.dtsi | 29 + arch/arm/dts/imx8mn-venice-gw7902.dts | 888 ++++++++++ arch/arm/dts/imx8mn-venice-u-boot.dtsi | 222 +++ arch/arm/dts/imx8mn-venice.dts | 152 ++ arch/arm/mach-imx/imx8m/Kconfig | 7 + board/gateworks/venice/Kconfig | 15 + board/gateworks/venice/MAINTAINERS | 6 +- board/gateworks/venice/Makefile | 10 +- board/gateworks/venice/README | 3 +- board/gateworks/venice/gsc.c | 13 + .../gateworks/venice/imximage-8mn-lpddr4.cfg | 10 + board/gateworks/venice/lpddr4_timing.h | 5 + ...lpddr4_timing.c => lpddr4_timing_imx8mm.c} | 0 .../lpddr4_timing_imx8mn_2gb_dual_die.c | 1444 ++++++++++++++++ .../lpddr4_timing_imx8mn_2gb_single_die.c | 1445 +++++++++++++++++ board/gateworks/venice/spl.c | 31 +- .../venice/{imx8mm_venice.c => venice.c} | 0 configs/imx8mn_venice_defconfig | 114 ++ include/configs/imx8mn_venice.h | 114 ++ 20 files changed, 4503 insertions(+), 7 deletions(-) create mode 100644 arch/arm/dts/imx8mn-venice-gw7902-u-boot.dtsi create mode 100644 arch/arm/dts/imx8mn-venice-gw7902.dts create mode 100644 arch/arm/dts/imx8mn-venice-u-boot.dtsi create mode 100644 arch/arm/dts/imx8mn-venice.dts create mode 100644 board/gateworks/venice/imximage-8mn-lpddr4.cfg rename board/gateworks/venice/{lpddr4_timing.c => lpddr4_timing_imx8mm.c} (100%) create mode 100644 board/gateworks/venice/lpddr4_timing_imx8mn_2gb_dual_die.c create mode 100644 board/gateworks/venice/lpddr4_timing_imx8mn_2gb_single_die.c rename board/gateworks/venice/{imx8mm_venice.c => venice.c} (100%) create mode 100644 configs/imx8mn_venice_defconfig create mode 100644 include/configs/imx8mn_venice.h
Stefano,
I noticed you have applied my first version of this patch into your imx master-next tree which is now broken because of some other patches that required it to be rebased: 7abf178bb815 power: Tidy up #undef of CONFIG_DM_PMIC 2a7360666871 serial: Rename SERIAL_SUPPORT to SERIAL 103c5f180694 mmc: Rename MMC_SUPPORT to MMC 7cfbba36e9f8 Convert CONFIG_SYS_MALLOC_LEN to Kconfig 148b8bb4b6bd imx: Finish migration of IMX_CONFIG to Kconfig 49c8ef0e45a9 Convert CONFIG_SYS_LOAD_ADDR to Kconfig 72d81360aabd global: Convert CONFIG_LOADADDR to CONFIG_SYS_LOADADDR 15e7b7682474 Convert CONFIG_SYS_I2C_MXC et al to Kconfig
This v2 patch here was rebased on top of origin/master to take care of this.
A grep for SERIAL_SUPPORT (which was renamed to SERIAL) shows all the boards in your master-next that got merged like mine that currently likely need fixups: $ git grep SERIAL_SUPPORT configs/ configs/imx8mm-cl-iot-gate-optee_defconfig:CONFIG_SPL_SERIAL_SUPPORT=y configs/imx8mn_venice_defconfig:CONFIG_SPL_SERIAL_SUPPORT=y configs/iot2050_defconfig:CONFIG_SPL_SERIAL_SUPPORT=y configs/kontron-sl-mx6ul_defconfig:CONFIG_SPL_SERIAL_SUPPORT=y configs/kontron-sl-mx8mm_defconfig:CONFIG_SPL_SERIAL_SUPPORT=y
A quick test shows all the above with the exception of perhaps iot2050_defconfig are not buildable.
How do you wish us to proceed? I have a set of patches that I can submit to resolve this but I'm not clear if I should squash them and if you should perhaps squash them into the original commit before requesting a git pull to master? imx8mn_venice: fix SERIAL imx8mn_venice: fix MMC_SUPPORT imx8mn_venice: fix SYS_MALLOC_LEN imx8mn-venice: fix IMX_CONFIG imx8mn_venice: fix loadaddr imx8mn-venice: fix I2C
Or should you revert and instead apply this v2 version that takes care of these things?
What are your plans for requesting a git pull to master?
Best regards,
Tim [1] https://source.denx.de/u-boot/custodians/u-boot-imx/-/tree/master-next
-- 2.17.1