On Wed, Oct 6, 2021 at 4:45 PM Tim Harvey <thar...@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 <thar...@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
>
