Re: [U-Boot] [PATCH 00/22] i.MX8MM support

2019-08-14 Thread Peng Fan
Hi Frieder,

> Subject: Re: [PATCH 00/22] i.MX8MM support
> 
> Hi Peng,
> 
> On 09.08.19 06:14, Peng Fan wrote:
> > This is a splitted and updated patch from
> > https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpatc
> >
> hwork.ozlabs.org%2Fcover%2F1128799%2Fdata=02%7C01%7Cpeng.fa
> n%40nxp.com%7Ca565754f3e0c4feaa73a08d720d55a57%7C686ea1d3bc2b4
> c6fa92cd99c5c301635%7C0%7C0%7C637013973037743092sdata=q7
> CqyQDxkK3%2BPy8sIIfAcnhptOMBLWD59waEEqh%2BgUw%3Dreserv
> ed=0 which is to support both i.MX8MM and i.MX8MN.
> 
> I have sent you feedback for some of the patches. My own board is working,
> but there are still a few things to fix. 

Thanks for your feedback. I'll check and fix my patch.

I will be out of office for a few days and
> will continue to work on this afterwards.
> For reference you can also look at my current working tree:
> https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.
> com%2Ffschrempf%2Fu-boot%2Fcommits%2Fv2019.10-ktn-mx8mmda
> ta=02%7C01%7Cpeng.fan%40nxp.com%7Ca565754f3e0c4feaa73a08d720d5
> 5a57%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C63701397303
> 7743092sdata=h6OR0YzoZCoT%2BJD1V4lB88bCGty08lnjxJcMuSUmqS
> 0%3Dreserved=0.

Thanks.
Peng.

> 
> Thanks,
> Frieder
> 
> >
> > This patchset only covers i.MX8MM and depends on the following patch
> > to work,
> > https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpatc
> >
> hwork.ozlabs.org%2Fcover%2F1144317%2Fdata=02%7C01%7Cpeng.fa
> n%40nx
> >
> p.com%7Ca565754f3e0c4feaa73a08d720d55a57%7C686ea1d3bc2b4c6fa92c
> d99c5c3
> >
> 01635%7C0%7C0%7C637013973037743092sdata=MlWolli%2BnhtNm
> oA72Sp6O1t
> > PrhOcmaXu8tcQjoh6XSg%3Dreserved=0
> > https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpatc
> >
> hwork.ozlabs.org%2Fpatch%2F1142686%2Fdata=02%7C01%7Cpeng.fa
> n%40nx
> >
> p.com%7Ca565754f3e0c4feaa73a08d720d55a57%7C686ea1d3bc2b4c6fa92c
> d99c5c3
> >
> 01635%7C0%7C0%7C637013973037743092sdata=iTqnmA8mjQbbxwR
> gNgqI3IM6b
> > iHdSk3DOZukVZpsqi0%3Dreserved=0
> > https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpatc
> >
> hwork.ozlabs.org%2Fpatch%2F1142690%2Fdata=02%7C01%7Cpeng.fa
> n%40nx
> >
> p.com%7Ca565754f3e0c4feaa73a08d720d55a57%7C686ea1d3bc2b4c6fa92c
> d99c5c3
> >
> 01635%7C0%7C0%7C637013973037743092sdata=eDmRT3B42gew6t
> DohedSUaJpd
> > uSc30Uaj13CvdCLLuI%3Dreserved=0
> > https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpatc
> >
> hwork.ozlabs.org%2Fpatch%2F1142691%2Fdata=02%7C01%7Cpeng.fa
> n%40nx
> >
> p.com%7Ca565754f3e0c4feaa73a08d720d55a57%7C686ea1d3bc2b4c6fa92c
> d99c5c3
> >
> 01635%7C0%7C0%7C637013973037743092sdata=2DFGP9Nu71xH1t9
> 40%2B%2BaM
> > 5AkAi5b6Gd%2Bq5GZEJghApE%3Dreserved=0
> > https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpatc
> >
> hwork.ozlabs.org%2Fpatch%2F1143291%2Fdata=02%7C01%7Cpeng.fa
> n%40nx
> >
> p.com%7Ca565754f3e0c4feaa73a08d720d55a57%7C686ea1d3bc2b4c6fa92c
> d99c5c3
> >
> 01635%7C0%7C0%7C637013973037743092sdata=iHyfUD3%2FB2rjzN
> BvqwlGm8r
> > feO3zYGk7DOnWWx%2B04RQ%3Dreserved=0
> > https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpatc
> >
> hwork.ozlabs.org%2Fpatch%2F1143765%2Fdata=02%7C01%7Cpeng.fa
> n%40nx
> >
> p.com%7Ca565754f3e0c4feaa73a08d720d55a57%7C686ea1d3bc2b4c6fa92c
> d99c5c3
> >
> 01635%7C0%7C0%7C637013973037743092sdata=C%2BWdEsBGt0eI
> M85LZap4fCq
> > yGhr5w9vCo27mSIMwX%2Bc%3Dreserved=0
> >
> > There is a README added, following that to test if you would like to.
> >
> > Peng Fan (21):
> >tools: imx8m_image: align spl bin image size
> >ddr: imx8m: fix ddr firmware location when enable SPL OF
> >imx8m: add image cfg for i.MX8MM lpddr4
> >imx: add IMX8MQ kconfig entry
> >imx: add IMX8MM kconfig entry
> >imx: imx8mm: add clock bindings header
> >imx: add i.MX8MM cpu type
> >imx: spl: add spl_board_boot_device for i.MX8MM
> >imx8m: update imx-regs for i.MX8MM
> >imx: add get_cpu_rev support for i.MX8MM
> >imx8m: add pin header for i.MX8MM
> >imx: add i.MX8MM PE property
> >imx8m: Fix MMU table issue for OPTEE memory
> >imx8m: set BYPASS ID SWAP to avoid AXI bus errors
> >imx8m: soc: enable SCTR clock before timer init
> >imx8m: rename clock to clock_imx8mq
> >imx8m: restructure clock.h
> >imx8m: add clk support for i.MX8MM
> >arm: dts: import i.MX8MM dtsi
> >arm: dts: add i.MX8MM pin func
> >imx: Add i.MX8MM EVK board support.
> >
> > Ye Li (1):
> >imx8m: Configure trustzone region 0 for non-secure access
> >
> >   arch/arm/dts/Makefile  |3 +-
> >   arch/arm/dts/imx8mm-evk-u-boot.dtsi|   92 +
> >   arch/arm/dts/imx8mm-evk.dts|  235 +++
> >   arch/arm/dts/imx8mm-pinfunc.h  |  629
> +++
> >   arch/arm/dts/imx8mm.dtsi   |  733
> 
> >   arch/arm/include/asm/arch-imx/cpu.h|6 +
> >   arch/arm/include/asm/arch-imx8m/clock.h   

Re: [U-Boot] [PATCH 00/22] i.MX8MM support

2019-08-14 Thread Schrempf Frieder
Hi Peng,

On 09.08.19 06:14, Peng Fan wrote:
> This is a splitted and updated patch from
> https://patchwork.ozlabs.org/cover/1128799/ which is to support both
> i.MX8MM and i.MX8MN.

I have sent you feedback for some of the patches. My own board is 
working, but there are still a few things to fix. I will be out of 
office for a few days and will continue to work on this afterwards.
For reference you can also look at my current working tree: 
https://github.com/fschrempf/u-boot/commits/v2019.10-ktn-mx8mm.

Thanks,
Frieder

> 
> This patchset only covers i.MX8MM and depends on the following patch
> to work,
> https://patchwork.ozlabs.org/cover/1144317/
> https://patchwork.ozlabs.org/patch/1142686/
> https://patchwork.ozlabs.org/patch/1142690/
> https://patchwork.ozlabs.org/patch/1142691/
> https://patchwork.ozlabs.org/patch/1143291/
> https://patchwork.ozlabs.org/patch/1143765/
> 
> There is a README added, following that to test if you would like to.
> 
> Peng Fan (21):
>tools: imx8m_image: align spl bin image size
>ddr: imx8m: fix ddr firmware location when enable SPL OF
>imx8m: add image cfg for i.MX8MM lpddr4
>imx: add IMX8MQ kconfig entry
>imx: add IMX8MM kconfig entry
>imx: imx8mm: add clock bindings header
>imx: add i.MX8MM cpu type
>imx: spl: add spl_board_boot_device for i.MX8MM
>imx8m: update imx-regs for i.MX8MM
>imx: add get_cpu_rev support for i.MX8MM
>imx8m: add pin header for i.MX8MM
>imx: add i.MX8MM PE property
>imx8m: Fix MMU table issue for OPTEE memory
>imx8m: set BYPASS ID SWAP to avoid AXI bus errors
>imx8m: soc: enable SCTR clock before timer init
>imx8m: rename clock to clock_imx8mq
>imx8m: restructure clock.h
>imx8m: add clk support for i.MX8MM
>arm: dts: import i.MX8MM dtsi
>arm: dts: add i.MX8MM pin func
>imx: Add i.MX8MM EVK board support.
> 
> Ye Li (1):
>imx8m: Configure trustzone region 0 for non-secure access
> 
>   arch/arm/dts/Makefile  |3 +-
>   arch/arm/dts/imx8mm-evk-u-boot.dtsi|   92 +
>   arch/arm/dts/imx8mm-evk.dts|  235 +++
>   arch/arm/dts/imx8mm-pinfunc.h  |  629 +++
>   arch/arm/dts/imx8mm.dtsi   |  733 
>   arch/arm/include/asm/arch-imx/cpu.h|6 +
>   arch/arm/include/asm/arch-imx8m/clock.h|  493 +
>   arch/arm/include/asm/arch-imx8m/clock_imx8mm.h |  387 
>   arch/arm/include/asm/arch-imx8m/clock_imx8mq.h |  424 +
>   arch/arm/include/asm/arch-imx8m/imx-regs.h |   75 +-
>   arch/arm/include/asm/arch-imx8m/imx8mm_pins.h  |  691 +++
>   arch/arm/include/asm/mach-imx/iomux-v3.h   |4 +
>   arch/arm/include/asm/mach-imx/sys_proto.h  |8 +
>   arch/arm/mach-imx/cpu.c|   12 +
>   arch/arm/mach-imx/imx8m/Kconfig|   17 +-
>   arch/arm/mach-imx/imx8m/Makefile   |4 +-
>   arch/arm/mach-imx/imx8m/clock_imx8mm.c |  306 +++
>   .../arm/mach-imx/imx8m/{clock.c => clock_imx8mq.c} |5 +-
>   arch/arm/mach-imx/imx8m/clock_slice.c  |   63 +
>   arch/arm/mach-imx/imx8m/imximage-8mm-lpddr4.cfg|   16 +
>   arch/arm/mach-imx/imx8m/soc.c  |   91 +-
>   arch/arm/mach-imx/spl.c|8 +
>   board/freescale/imx8mm_evk/Kconfig |   12 +
>   board/freescale/imx8mm_evk/MAINTAINERS |6 +
>   board/freescale/imx8mm_evk/Makefile|   12 +
>   board/freescale/imx8mm_evk/imx8mm_evk.c|   90 +
>   board/freescale/imx8mm_evk/lpddr4_timing.c | 1980 
> 
>   board/freescale/imx8mm_evk/spl.c   |  102 +
>   configs/imx8mm_evk_defconfig   |   70 +
>   drivers/ddr/imx/imx8m/helper.c |   12 +-
>   include/configs/imx8mm_evk.h   |  164 ++
>   include/dt-bindings/clock/imx8mm-clock.h   |  253 +++
>   tools/imx8m_image.sh   |5 +-
>   33 files changed, 6533 insertions(+), 475 deletions(-)
>   create mode 100644 arch/arm/dts/imx8mm-evk-u-boot.dtsi
>   create mode 100644 arch/arm/dts/imx8mm-evk.dts
>   create mode 100644 arch/arm/dts/imx8mm-pinfunc.h
>   create mode 100644 arch/arm/dts/imx8mm.dtsi
>   create mode 100644 arch/arm/include/asm/arch-imx8m/clock_imx8mm.h
>   create mode 100644 arch/arm/include/asm/arch-imx8m/clock_imx8mq.h
>   create mode 100644 arch/arm/include/asm/arch-imx8m/imx8mm_pins.h
>   create mode 100644 arch/arm/mach-imx/imx8m/clock_imx8mm.c
>   rename arch/arm/mach-imx/imx8m/{clock.c => clock_imx8mq.c} (99%)
>   create mode 100644 arch/arm/mach-imx/imx8m/imximage-8mm-lpddr4.cfg
>   create mode 100644 board/freescale/imx8mm_evk/Kconfig
>   create mode 100644 board/freescale/imx8mm_evk/MAINTAINERS
>   create mode 100644 

[U-Boot] [PATCH 00/22] i.MX8MM support

2019-08-08 Thread Peng Fan
This is a splitted and updated patch from
https://patchwork.ozlabs.org/cover/1128799/ which is to support both
i.MX8MM and i.MX8MN.

This patchset only covers i.MX8MM and depends on the following patch
to work,
https://patchwork.ozlabs.org/cover/1144317/
https://patchwork.ozlabs.org/patch/1142686/
https://patchwork.ozlabs.org/patch/1142690/
https://patchwork.ozlabs.org/patch/1142691/
https://patchwork.ozlabs.org/patch/1143291/
https://patchwork.ozlabs.org/patch/1143765/

There is a README added, following that to test if you would like to.

Peng Fan (21):
  tools: imx8m_image: align spl bin image size
  ddr: imx8m: fix ddr firmware location when enable SPL OF
  imx8m: add image cfg for i.MX8MM lpddr4
  imx: add IMX8MQ kconfig entry
  imx: add IMX8MM kconfig entry
  imx: imx8mm: add clock bindings header
  imx: add i.MX8MM cpu type
  imx: spl: add spl_board_boot_device for i.MX8MM
  imx8m: update imx-regs for i.MX8MM
  imx: add get_cpu_rev support for i.MX8MM
  imx8m: add pin header for i.MX8MM
  imx: add i.MX8MM PE property
  imx8m: Fix MMU table issue for OPTEE memory
  imx8m: set BYPASS ID SWAP to avoid AXI bus errors
  imx8m: soc: enable SCTR clock before timer init
  imx8m: rename clock to clock_imx8mq
  imx8m: restructure clock.h
  imx8m: add clk support for i.MX8MM
  arm: dts: import i.MX8MM dtsi
  arm: dts: add i.MX8MM pin func
  imx: Add i.MX8MM EVK board support.

Ye Li (1):
  imx8m: Configure trustzone region 0 for non-secure access

 arch/arm/dts/Makefile  |3 +-
 arch/arm/dts/imx8mm-evk-u-boot.dtsi|   92 +
 arch/arm/dts/imx8mm-evk.dts|  235 +++
 arch/arm/dts/imx8mm-pinfunc.h  |  629 +++
 arch/arm/dts/imx8mm.dtsi   |  733 
 arch/arm/include/asm/arch-imx/cpu.h|6 +
 arch/arm/include/asm/arch-imx8m/clock.h|  493 +
 arch/arm/include/asm/arch-imx8m/clock_imx8mm.h |  387 
 arch/arm/include/asm/arch-imx8m/clock_imx8mq.h |  424 +
 arch/arm/include/asm/arch-imx8m/imx-regs.h |   75 +-
 arch/arm/include/asm/arch-imx8m/imx8mm_pins.h  |  691 +++
 arch/arm/include/asm/mach-imx/iomux-v3.h   |4 +
 arch/arm/include/asm/mach-imx/sys_proto.h  |8 +
 arch/arm/mach-imx/cpu.c|   12 +
 arch/arm/mach-imx/imx8m/Kconfig|   17 +-
 arch/arm/mach-imx/imx8m/Makefile   |4 +-
 arch/arm/mach-imx/imx8m/clock_imx8mm.c |  306 +++
 .../arm/mach-imx/imx8m/{clock.c => clock_imx8mq.c} |5 +-
 arch/arm/mach-imx/imx8m/clock_slice.c  |   63 +
 arch/arm/mach-imx/imx8m/imximage-8mm-lpddr4.cfg|   16 +
 arch/arm/mach-imx/imx8m/soc.c  |   91 +-
 arch/arm/mach-imx/spl.c|8 +
 board/freescale/imx8mm_evk/Kconfig |   12 +
 board/freescale/imx8mm_evk/MAINTAINERS |6 +
 board/freescale/imx8mm_evk/Makefile|   12 +
 board/freescale/imx8mm_evk/imx8mm_evk.c|   90 +
 board/freescale/imx8mm_evk/lpddr4_timing.c | 1980 
 board/freescale/imx8mm_evk/spl.c   |  102 +
 configs/imx8mm_evk_defconfig   |   70 +
 drivers/ddr/imx/imx8m/helper.c |   12 +-
 include/configs/imx8mm_evk.h   |  164 ++
 include/dt-bindings/clock/imx8mm-clock.h   |  253 +++
 tools/imx8m_image.sh   |5 +-
 33 files changed, 6533 insertions(+), 475 deletions(-)
 create mode 100644 arch/arm/dts/imx8mm-evk-u-boot.dtsi
 create mode 100644 arch/arm/dts/imx8mm-evk.dts
 create mode 100644 arch/arm/dts/imx8mm-pinfunc.h
 create mode 100644 arch/arm/dts/imx8mm.dtsi
 create mode 100644 arch/arm/include/asm/arch-imx8m/clock_imx8mm.h
 create mode 100644 arch/arm/include/asm/arch-imx8m/clock_imx8mq.h
 create mode 100644 arch/arm/include/asm/arch-imx8m/imx8mm_pins.h
 create mode 100644 arch/arm/mach-imx/imx8m/clock_imx8mm.c
 rename arch/arm/mach-imx/imx8m/{clock.c => clock_imx8mq.c} (99%)
 create mode 100644 arch/arm/mach-imx/imx8m/imximage-8mm-lpddr4.cfg
 create mode 100644 board/freescale/imx8mm_evk/Kconfig
 create mode 100644 board/freescale/imx8mm_evk/MAINTAINERS
 create mode 100644 board/freescale/imx8mm_evk/Makefile
 create mode 100644 board/freescale/imx8mm_evk/imx8mm_evk.c
 create mode 100644 board/freescale/imx8mm_evk/lpddr4_timing.c
 create mode 100644 board/freescale/imx8mm_evk/spl.c
 create mode 100644 configs/imx8mm_evk_defconfig
 create mode 100644 include/configs/imx8mm_evk.h
 create mode 100644 include/dt-bindings/clock/imx8mm-clock.h

-- 
2.16.4

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot