Re: [NXP-IMX] please pull nxp-imx-2021-4-6
On 5/27/21 11:40 AM, Stefano Babic wrote: Hi Sean, On 27.05.21 17:27, Sean Anderson wrote: On 5/27/21 10:59 AM, Stefano Babic wrote: > 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=* But none of the feedback was addressed. Ex: https://lore.kernel.org/u-boot/c8f2512d-d35a-748c-1c00-190849ffa...@nxp.com/ https://lore.kernel.org/u-boot/ce3f1e35-d215-6ace-4f38-1b36fc6ab...@denx.de/ And in fact, the related patches are not applied - or do you see the opposite ? There is not a second fsl CAAM driver... Ah, looks like there was a second PR which never got merged. I was confused because Frieder commented on the first one. --Sean Again, I am sorry if something was picked up without fixing the comments - but this should not be the case. This last one is even your own feedback, but here is the patch anyway. Agree with you, and again, raise your hand as soon as you find that something slip away and was merged. For i.MX, I will revert before asking Tom to merge my tree. I don't understand why there is a PR without addressing why we are adding a second caam driver, which was brought up by two people. PR was sent by Peng, but then it is my responsibility as i.MX maintainer to take it at whole or drop the patches (if orthogonal, as in this case) that have open comments. And of course, you're right and comments should be worked (or at least discussed) before merging into main tree. Best regards, Stefano --Sean > > 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.gitdata=04%7C01%7Cfrieder.schrempf%40kontron.de%7C2ed49d663d694871619d08d8f8df37e6%7C8c9d3c973fd941c8a2b1646f3942daf1%7C0%7C0%7C637532984941310093%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000sdata=QEMqCBw9qvbdQfELx%2BpMAMTjw5Z7kgpo5G%2FzwTaLULY%3Dreserved=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
Re: [NXP-IMX] please pull nxp-imx-2021-4-6
Hi Sean, On 27.05.21 17:27, Sean Anderson wrote: On 5/27/21 10:59 AM, Stefano Babic wrote: > 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=* But none of the feedback was addressed. Ex: https://lore.kernel.org/u-boot/c8f2512d-d35a-748c-1c00-190849ffa...@nxp.com/ https://lore.kernel.org/u-boot/ce3f1e35-d215-6ace-4f38-1b36fc6ab...@denx.de/ And in fact, the related patches are not applied - or do you see the opposite ? There is not a second fsl CAAM driver... Again, I am sorry if something was picked up without fixing the comments - but this should not be the case. This last one is even your own feedback, but here is the patch anyway. Agree with you, and again, raise your hand as soon as you find that something slip away and was merged. For i.MX, I will revert before asking Tom to merge my tree. I don't understand why there is a PR without addressing why we are adding a second caam driver, which was brought up by two people. PR was sent by Peng, but then it is my responsibility as i.MX maintainer to take it at whole or drop the patches (if orthogonal, as in this case) that have open comments. And of course, you're right and comments should be worked (or at least discussed) before merging into main tree. Best regards, Stefano --Sean > > 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.gitdata=04%7C01%7Cfrieder.schrempf%40kontron.de%7C2ed49d663d694871619d08d8f8df37e6%7C8c9d3c973fd941c8a2b1646f3942daf1%7C0%7C0%7C637532984941310093%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000sdata=QEMqCBw9qvbdQfELx%2BpMAMTjw5Z7kgpo5G%2FzwTaLULY%3Dreserved=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
Re: [NXP-IMX] please pull nxp-imx-2021-4-6
On 5/27/21 10:59 AM, Stefano Babic wrote: > 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=* But none of the feedback was addressed. Ex: https://lore.kernel.org/u-boot/c8f2512d-d35a-748c-1c00-190849ffa...@nxp.com/ https://lore.kernel.org/u-boot/ce3f1e35-d215-6ace-4f38-1b36fc6ab...@denx.de/ This last one is even your own feedback, but here is the patch anyway. I don't understand why there is a PR without addressing why we are adding a second caam driver, which was brought up by two people. --Sean > > 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.gitdata=04%7C01%7Cfrieder.schrempf%40kontron.de%7C2ed49d663d694871619d08d8f8df37e6%7C8c9d3c973fd941c8a2b1646f3942daf1%7C0%7C0%7C637532984941310093%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000sdata=QEMqCBw9qvbdQfELx%2BpMAMTjw5Z7kgpo5G%2FzwTaLULY%3Dreserved=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
Re: [NXP-IMX] please pull nxp-imx-2021-4-6
Hi Stefano, On 27.05.21 16:59, Stefano Babic wrote: > 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: > > https://eur04.safelinks.protection.outlook.com/?url=http%3A%2F%2Fpatchwork.ozlabs.org%2Fproject%2Fuboot%2Flist%2F%3Fseries%3D235820%26state%3D*data=04%7C01%7Cfrieder.schrempf%40kontron.de%7C6757587cd70a41af63f308d9211fff81%7C8c9d3c973fd941c8a2b1646f3942daf1%7C0%7C0%7C637577243581770096%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000sdata=AkXzKPgEtURhDXRY4tfdu4BjFCqOKBYYPyQ0JSXusn0%3Dreserved=0 > > 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. Thanks for clarifying and for pointing to patchwork. I should have been looking there before. Your link points to a different series, but I found the patches I was looking for here: http://patchwork.ozlabs.org/project/uboot/list/?series=237531=%2A=both. For some reason I can't find them in my mailbox, but that's probably a different story. Thanks a lot! Frieder
Re: [NXP-IMX] please pull nxp-imx-2021-4-6
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=* 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.gitdata=04%7C01%7Cfrieder.schrempf%40kontron.de%7C2ed49d663d694871619d08d8f8df37e6%7C8c9d3c973fd941c8a2b1646f3942daf1%7C0%7C0%7C637532984941310093%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000sdata=QEMqCBw9qvbdQfELx%2BpMAMTjw5Z7kgpo5G%2FzwTaLULY%3Dreserved=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:
Re: [NXP-IMX] please pull nxp-imx-2021-4-6
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? To me it feels wrong as there is no chance they will ever get reviewed/tested before merging, but I don't know the rules. @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? 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.gitdata=04%7C01%7Cfrieder.schrempf%40kontron.de%7C2ed49d663d694871619d08d8f8df37e6%7C8c9d3c973fd941c8a2b1646f3942daf1%7C0%7C0%7C637532984941310093%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000sdata=QEMqCBw9qvbdQfELx%2BpMAMTjw5Z7kgpo5G%2FzwTaLULY%3Dreserved=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 >
Re: [NXP-IMX] please pull nxp-imx-2021-4-6
Hi Stefano, On 2021/4/6 17:45, Stefano Babic wrote: Hi Peng, 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 i.MX8M caam/dek support hab update/fix for i.MX8M/7ULP env/distro cleanup for i.MX8M i.MX8M ddr script/driver update -- Can you help me to sync myself with then patches in patchwork ? IMHO you have two big series, I name them as "update on imx8m" (26 patches) and "imx: hab/caam new features and update" (37 patches). In PR there should be something more as the count of patches does not match ;-) Sorry for this. I have a V2 PR https://patchwork.ozlabs.org/project/uboot/patch/db6pr0402mb276fc5d80300aa73058e88...@db6pr0402mb2760.eurprd04.prod.outlook.com/ The last PR was wrongly did direclty from my local tree. V2 PR is done from downloading patches on patchwork. I have updated the patches with awaiting upstream status. Thanks, Peng. Regards, Stefano 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://github.com/MrVan/u-boot.git 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
RE: [NXP-IMX] please pull nxp-imx-2021-4-6
> Subject: [NXP-IMX] please pull nxp-imx-2021-4-6 Please ignore this PR, I'll resend a v2 PR Thanks, Peng. > > 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 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://github.com/MrVan/u-boot.git 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 >
Re: [NXP-IMX] please pull nxp-imx-2021-4-6
Hi Peng, 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 i.MX8M caam/dek support hab update/fix for i.MX8M/7ULP env/distro cleanup for i.MX8M i.MX8M ddr script/driver update -- Can you help me to sync myself with then patches in patchwork ? IMHO you have two big series, I name them as "update on imx8m" (26 patches) and "imx: hab/caam new features and update" (37 patches). In PR there should be something more as the count of patches does not match ;-) Regards, Stefano 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://github.com/MrVan/u-boot.git 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:
[NXP-IMX] please pull nxp-imx-2021-4-6
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 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://github.com/MrVan/u-boot.git 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 -
