All of lore.kernel.org
 help / color / mirror / Atom feed
From: Stefano Babic <sbabic@denx.de>
To: Frieder Schrempf <frieder.schrempf@kontron.de>,
	"Peng Fan (OSS)" <peng.fan@oss.nxp.com>,
	Stefano Babic <sbabic@denx.de>,
	Fabio Estevam <festevam@gmail.com>
Cc: u-boot <u-boot@lists.denx.de>
Subject: Re: [NXP-IMX] please pull nxp-imx-2021-4-6
Date: Thu, 27 May 2021 16:59:14 +0200	[thread overview]
Message-ID: <79e0acd8-d1f5-7005-dddf-c0e4c02f29bc@denx.de> (raw)
In-Reply-To: <baf08c1c-aa6d-0a82-3000-117f5e0846b9@kontron.de>

Hi Frieder,

On 27.05.21 16:47, Frieder Schrempf wrote:
> Hi,
> 
> On 06.04.21 11:34, Peng Fan (OSS) wrote:
>> Hi Stefano,
>>
>> Please pull nxp-imx-2021-4-6. This PR is based on Tom's master branch. If you need
>> me rebase on imx/master, I could redo this PR. This is quite a large PR, since lots
>> stuff in downstream tree.
>>
>> I ran local buildman test for all boards, no issue found.
>> ------------------------------------------
>> convert to use binman for i.MX8MN/M/P EVK board
> 
> It looks like these patches haven't been sent to the list. Is this common practice to have patches pulled via PR that were not sent to the list before?

Never. Patches *MUST* be sent to the ML. I am sorry if some patches were 
pulled by me and they were not in the ML: it is then a mistake. Anyway, 
my work-flow foresees to get patches from patchworks and update the 
status after merging, and it is weird if some patches were not posted.

For the patches reported here, they were posted and you get at:

http://patchwork.ozlabs.org/project/uboot/list/?series=235820&state=*

but again, if some patches were merged without beeing posted, it was 
just a big mistake. It should not happen !

> To me it feels wrong as there is no chance they will ever get reviewed/tested before merging,

You're right, and patches must be posted without exceptions.

> but I don't know the rules.

Your feeling is correct.

> 
> @Peng: Since using binman to build the images there is the following output:
> 
> WARNING 'mkimage.flash.mkimage' not found, resulting binary is not-functional
> make[2]: Nothing to be done for 'SPL'.
> 
> Why is it there? How to get rid of it?
> 

Best regards,
Stefano

> Thanks
> Frieder
> 
>> i.MX8M caam/dek support
>> hab update/fix for i.MX8M/7ULP
>> env/distro cleanup for i.MX8M
>> i.MX8M ddr script/driver update
>> ------------------------------------------
>>
>> Thanks,
>> Peng.
>>
>> The following changes since commit 90eba245a66aa20589404ba537215faf2012c1a3:
>>
>>    Merge branch 'next' (2021-04-05 11:29:57 -0400)
>>
>> are available in the Git repository at:
>>
>>    https://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMrVan%2Fu-boot.git&amp;data=04%7C01%7Cfrieder.schrempf%40kontron.de%7C2ed49d663d694871619d08d8f8df37e6%7C8c9d3c973fd941c8a2b1646f3942daf1%7C0%7C0%7C637532984941310093%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&amp;sdata=QEMqCBw9qvbdQfELx%2BpMAMTjw5Z7kgpo5G%2FzwTaLULY%3D&amp;reserved=0 tags/nxp-imx-2021-4-6
>>
>> for you to fetch changes up to 95dfcc086b419aece6835fd7aaad704de661e246:
>>
>>    doc: imx8mp-evk: update after using binman (2021-04-06 17:19:45 +0800)
>>
>> ----------------------------------------------------------------
>> Aymen Sghaier (6):
>>        crypto: caam: Add CAAM support to i.MX8M platforms
>>        crypto: caam: Fix build warnings pointer casting
>>        crypto: Add blob command support for i.MX8M platforms
>>        crypto: caam: Fix pointer size to 32bit for i.MX8M
>>        crypto: caam: Add secure memory vid 3 support
>>        crypto: caam: RNG4 TRNG errata
>>
>> Breno Lima (13):
>>        imx: imx7 Support for Manufacturing Protection
>>        imx: Avoid hardcoded output ring size register offset (ORSR)
>>        imx: Ensure CAAM clock is enabled prior getting out_jr_size
>>        imx: Avoid hardcoded Job Ring Max size
>>        imx: hab: Enable hab.c to authenticate additional images in open configuration
>>        imx: hab: Check if IVT header is HABv4
>>        mx7ulp: hab: Add hab_status command for HABv4 M4 boot
>>        imx: hab: Fix build warnings in 32-bit targets
>>        crypto: fsl: blob: Flush dcache range for destination address
>>        mx6dq: hab: Fix chip version in hab.h code
>>        cmd: blob: Add IMX_HAB and CAAM supported SoCs as dependency
>>        cmd: blob: Instantiate RNG before running CMD_BLOB
>>        fsl_mfgprot: Fix typo in sign_mppubk()
>>
>> Clement Faure (2):
>>        imx8m: Add DEK blob encapsulation for imx8m
>>        imx8: Add DEK blob encapsulation
>>
>> Clement Le Marquis (1):
>>        imx: caam: new u-boot command to set PRIBLOB bitfield from CAAM SCFGR register to 0x3
>>
>> Franck LENORMAND (3):
>>        crypto: caam: change JR running loop
>>        caam: enable support for iMX7ULP
>>        imx7ulp: Enable support for cmd blob
>>
>> Jacky Bai (1):
>>        imx8mn: Update the DDR4 timing script on imx8mn ddr4 evk
>>
>> Peng Fan (21):
>>        tools: imx image: fix write warning
>>        imx8mm/p: remove boot.cmd
>>        imx8mm_evk: add/cleanup variable for distro
>>        imx8mp_evk: add/cleanup variable for distro
>>        imx8mp_evk: spl: clean up including headers
>>        imx8mp_evk: Increase VDD_ARM to 0.95v Overdrive voltage
>>        power: pca9450: add a new parameter for power_pca9450_init
>>        imx8mn_evk: drop duplicated code
>>        imx8mn: Add LPDDR4 EVK board support
>>        imx: logos: use NXP logo
>>        imx8m: soc: update fuse path
>>        arch: mach-imx: imx8m: fix unique_id read error for imx8mp
>>        imx: HAB: Update hab codes to support ARM64 and i.MX8M
>>        imx8m: add regs used by CAAM
>>        imx8mm_evk: switch to use binman to pack images
>>        doc: imx8mm_evk: update doc after using binman
>>        imx8mn-ddr4-evk: switch to use binman
>>        imx8mn-evk: switch to use binman
>>        doc: imx8mn_evk: update doc after using binman
>>        imx8mp-evk: switch to use binman
>>        doc: imx8mp-evk: update after using binman
>>
>> Sherry Sun (1):
>>        imx8mp: ddr: Add inline ECC feature support
>>
>> Utkarsh Gupta (2):
>>        imx: HAB: Validate IVT before authenticating image
>>        imx: hab: Display All HAB events via hab_status command
>>
>> Ye Li (18):
>>        imx8mm_evk: Update to latest LPDDR4 script
>>        imx8mm_evk: Switch to new imx8mm evk board
>>        imx8mp_evk: Update LPDDR4 timing for new FW 202006
>>        imx8mp_evk: Update LPDDR4 refresh time
>>        imx8mn: Add low drive mode support for DDR4/LPDDR4 EVK
>>        imx8mn: Add support for 11x11 UltraLite part number
>>        imx8m: Update thermal and PMU kernel nodes for dual/single cores
>>        imx8m: ddr: Disable CA VREF Training for LPDDR4
>>        iMX8MQ: Recognize the B2 revision
>>        misc: ocotp: Update OCOTP driver for iMX8MQ B2
>>        imx8mq_evk: Applying default LPDDR4 script for B2
>>        imx: hab: Add function to authenticate kernel image
>>        hab: Change calling to ROM API failsafe
>>        imx: HAB: Add support for iMX8MM
>>        iMX8M: Add support to enable CONFIG_IMX_HAB
>>        imx: cmd_dek: Enable DEK only for chips supporting CAAM
>>        crypto: caam: Add fsl caam driver
>>        crypto: fsl: refactor for 32 bit version CAAM support on ARM64
>>
>> haidong.zheng (1):
>>        imx8mp: refine power on imx8mp board
>>
>>   arch/arm/Kconfig                                   |    6 +
>>   arch/arm/dts/Makefile                              |    1 +
>>   arch/arm/dts/imx8mm-evk-u-boot.dtsi                |  117 ++++++++-
>>   arch/arm/dts/imx8mm-evk.dtsi                       |  127 +++++-----
>>   arch/arm/dts/imx8mn-ddr4-evk-u-boot.dtsi           |  115 +++++++++
>>   arch/arm/dts/imx8mn-evk-u-boot.dtsi                |  128 ++++++++++
>>   arch/arm/dts/imx8mn-evk.dts                        |  128 ++++++++++
>>   arch/arm/dts/imx8mp-evk-u-boot.dtsi                |  112 +++++++++
>>   arch/arm/include/asm/arch-imx/cpu.h                |   13 +-
>>   arch/arm/include/asm/arch-imx8/image.h             |   11 +
>>   arch/arm/include/asm/arch-imx8m/clock.h            |    1 +
>>   arch/arm/include/asm/arch-imx8m/imx-regs.h         |   21 ++
>>   arch/arm/include/asm/arch-mx7/crm_regs.h           |    8 +
>>   arch/arm/include/asm/arch-mx7ulp/imx-regs.h        |   12 +
>>   arch/arm/include/asm/mach-imx/hab.h                |   37 ++-
>>   arch/arm/include/asm/mach-imx/sys_proto.h          |    6 +-
>>   arch/arm/mach-imx/Kconfig                          |   44 +++-
>>   arch/arm/mach-imx/Makefile                         |    3 +
>>   arch/arm/mach-imx/cmd_dek.c                        |  262 ++++++++++++++++++--
>>   arch/arm/mach-imx/cmd_mfgprot.c                    |  150 ++++++++++++
>>   arch/arm/mach-imx/cpu.c                            |    8 +-
>>   arch/arm/mach-imx/hab.c                            |  417 ++++++++++++++++++++++++++++----
>>   arch/arm/mach-imx/imx8/Kconfig                     |    1 +
>>   arch/arm/mach-imx/imx8m/Kconfig                    |   11 +
>>   arch/arm/mach-imx/imx8m/clock_imx8mm.c             |    8 +
>>   arch/arm/mach-imx/imx8m/clock_imx8mq.c             |    7 +
>>   arch/arm/mach-imx/imx8m/soc.c                      |  183 ++++++++++++--
>>   arch/arm/mach-imx/mx7ulp/Kconfig                   |    1 +
>>   arch/arm/mach-imx/priblob.c                        |   33 +++
>>   board/freescale/imx8mm_evk/boot.cmd                |   35 ---
>>   board/freescale/imx8mm_evk/imximage-8mm-lpddr4.cfg |    9 +
>>   board/freescale/imx8mm_evk/lpddr4_timing.c         |  692 +++++++++++++++++++++-------------------------------
>>   board/freescale/imx8mm_evk/spl.c                   |   33 +--
>>   board/freescale/imx8mn_evk/Kconfig                 |    6 +-
>>   board/freescale/imx8mn_evk/Makefile                |    6 +
>>   board/freescale/imx8mn_evk/ddr4_timing.c           | 1057 ++++++++++++++++++++++++++++++++++----------------------------------------------
>>   board/freescale/imx8mn_evk/ddr4_timing_ld.c        | 1057 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
>>   board/freescale/imx8mn_evk/imximage-8mn-ddr4.cfg   |   10 +
>>   board/freescale/imx8mn_evk/lpddr4_timing.c         | 1587 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
>>   board/freescale/imx8mn_evk/lpddr4_timing_ld.c      | 1440 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
>>   board/freescale/imx8mn_evk/spl.c                   |   50 +++-
>>   board/freescale/imx8mp_evk/boot.cmd                |   25 --
>>   board/freescale/imx8mp_evk/imximage-8mp-lpddr4.cfg |   10 +
>>   board/freescale/imx8mp_evk/lpddr4_timing.c         |  372 +++++++++++++++++++++-------
>>   board/freescale/imx8mp_evk/spl.c                   |   38 +--
>>   board/freescale/imx8mq_evk/spl.c                   |    2 +-
>>   board/phytec/phycore_imx8mp/spl.c                  |    2 +-
>>   cmd/Kconfig                                        |    8 +
>>   cmd/Makefile                                       |    1 +
>>   cmd/blob.c                                         |   16 ++
>>   cmd/cmd_fsl_caam.c                                 |   88 +++++++
>>   configs/imx8mm_evk_defconfig                       |    6 +-
>>   configs/imx8mn_ddr4_evk_defconfig                  |    4 +-
>>   configs/imx8mn_evk_defconfig                       |   93 +++++++
>>   configs/imx8mp_evk_defconfig                       |    4 +-
>>   doc/board/freescale/imx8mm_evk.rst                 |    3 +-
>>   doc/board/freescale/imx8mn_evk.rst                 |    3 +-
>>   doc/board/freescale/imx8mp_evk.rst                 |   11 +-
>>   doc/imx/habv4/guides/mx6_mx7_secure_boot.txt       |   25 ++
>>   drivers/crypto/Makefile                            |    1 +
>>   drivers/crypto/fsl/Kconfig                         |    6 +
>>   drivers/crypto/fsl/Makefile                        |    4 +-
>>   drivers/crypto/fsl/desc.h                          |   49 ++--
>>   drivers/crypto/fsl/desc_constr.h                   |   28 +--
>>   drivers/crypto/fsl/fsl_blob.c                      |    6 +
>>   drivers/crypto/fsl/fsl_hash.c                      |    6 +-
>>   drivers/crypto/fsl/fsl_mfgprot.c                   |  160 ++++++++++++
>>   drivers/crypto/fsl/jobdesc.c                       |   16 +-
>>   drivers/crypto/fsl/jr.c                            |   53 ++--
>>   drivers/crypto/fsl/jr.h                            |   11 +-
>>   drivers/crypto/fsl/type.h                          |   16 ++
>>   drivers/crypto/fsl_caam.c                          |  720 ++++++++++++++++++++++++++++++++++++++++++++++++++++++
>>   drivers/crypto/fsl_caam_internal.h                 |  230 ++++++++++++++++++
>>   drivers/ddr/imx/imx8m/Kconfig                      |    8 +
>>   drivers/misc/mxc_ocotp.c                           |    2 +-
>>   drivers/power/pmic/pmic_pca9450.c                  |    4 +-
>>   include/configs/imx8mm_evk.h                       |    8 +-
>>   include/configs/imx8mp_evk.h                       |    8 +-
>>   include/fsl_caam.h                                 |   24 ++
>>   include/fsl_sec.h                                  |   54 +++--
>>   include/power/pca9450.h                            |    2 +-
>>   tools/imx8image.c                                  |    2 +-
>>   tools/imx8mimage.c                                 |    2 +-
>>   tools/logos/freescale.bmp                          |  Bin 46738 -> 47670 bytes
>>   84 files changed, 8600 insertions(+), 1484 deletions(-)
>>   create mode 100644 arch/arm/dts/imx8mn-evk-u-boot.dtsi
>>   create mode 100644 arch/arm/dts/imx8mn-evk.dts
>>   create mode 100644 arch/arm/mach-imx/cmd_mfgprot.c
>>   create mode 100644 arch/arm/mach-imx/priblob.c
>>   delete mode 100644 board/freescale/imx8mm_evk/boot.cmd
>>   create mode 100644 board/freescale/imx8mm_evk/imximage-8mm-lpddr4.cfg
>>   create mode 100644 board/freescale/imx8mn_evk/ddr4_timing_ld.c
>>   create mode 100644 board/freescale/imx8mn_evk/imximage-8mn-ddr4.cfg
>>   create mode 100644 board/freescale/imx8mn_evk/lpddr4_timing.c
>>   create mode 100644 board/freescale/imx8mn_evk/lpddr4_timing_ld.c
>>   delete mode 100644 board/freescale/imx8mp_evk/boot.cmd
>>   create mode 100644 board/freescale/imx8mp_evk/imximage-8mp-lpddr4.cfg
>>   mode change 100644 => 100755 board/freescale/imx8mp_evk/lpddr4_timing.c
>>   create mode 100644 cmd/cmd_fsl_caam.c
>>   create mode 100644 configs/imx8mn_evk_defconfig
>>   create mode 100644 drivers/crypto/fsl/fsl_mfgprot.c
>>   create mode 100644 drivers/crypto/fsl/type.h
>>   create mode 100644 drivers/crypto/fsl_caam.c
>>   create mode 100644 drivers/crypto/fsl_caam_internal.h
>>   create mode 100644 include/fsl_caam.h
>>


-- 
=====================================================================
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic@denx.de
=====================================================================

  reply	other threads:[~2021-05-27 14:59 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-04-06  9:34 [NXP-IMX] please pull nxp-imx-2021-4-6 Peng Fan
2021-04-06  9:45 ` Stefano Babic
2021-04-08  1:21   ` Peng Fan
2021-04-07  5:27 ` Peng Fan
2021-05-27 14:47 ` Frieder Schrempf
2021-05-27 14:59   ` Stefano Babic [this message]
2021-05-27 15:09     ` Frieder Schrempf
2021-05-27 15:27     ` Sean Anderson
2021-05-27 15:40       ` Stefano Babic
2021-05-27 15:42         ` Sean Anderson

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=79e0acd8-d1f5-7005-dddf-c0e4c02f29bc@denx.de \
    --to=sbabic@denx.de \
    --cc=festevam@gmail.com \
    --cc=frieder.schrempf@kontron.de \
    --cc=peng.fan@oss.nxp.com \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.