board: stm32mp2: fix fwu_mdata node updating#5
Open
passgat wants to merge 7 commits intoSTMicroelectronics:v2023.10-stm32mpfrom
Open
board: stm32mp2: fix fwu_mdata node updating#5passgat wants to merge 7 commits intoSTMicroelectronics:v2023.10-stm32mpfrom
passgat wants to merge 7 commits intoSTMicroelectronics:v2023.10-stm32mpfrom
Conversation
Synchronize DT with kernel v6.6-stm32mp-r2-rc6-preint5 Signed-off-by: Patrice Chotard <[email protected]> Change-Id: I2c8ee8eaf1bd6094447a5f03b6e607155002c6a6
When this property is set, internal RCC clock is used for the RGMII 125MHz clock selection. Signed-off-by: Gatien Chevallier <[email protected]> Change-Id: Ibb48a3403696f17c1331ab4c119e78aa07b1ceb1 Reviewed-on: https://gerrit.st.com/c/mpu/oe/st/u-boot/+/460024 Reviewed-by: Christophe ROULLIER <[email protected]> ACI: CIBUILD <[email protected]> Reviewed-by: Gatien CHEVALLIER <[email protected]> ACI: CITOOLS <[email protected]> Tested-by: Gatien CHEVALLIER <[email protected]> Reviewed-by: Patrice CHOTARD <[email protected]> Tested-by: Christophe ROULLIER <[email protected]> Domain-Review: Christophe ROULLIER <[email protected]>
Some configs were missing in the stm32mp23_defconfig, preventing the platform to boot on M33TDCID flavor, mainly due to the missing mailbox configs. Change-Id: Ia82e839078bb1a04643085fc30f9bd5a738c7df2 Signed-off-by: Yann Gautier <[email protected]> Reviewed-on: https://gerrit.st.com/c/mpu/oe/st/u-boot/+/462648 ACI: CIBUILD <[email protected]> ACI: CITOOLS <[email protected]> Domain-Review: Patrice CHOTARD <[email protected]> Reviewed-by: Patrice CHOTARD <[email protected]>
Add the BOOT alias1 section for internal memories for STM32MP23. Change-Id: I75f4545b5ea56fa3a8b7ca39fa0c4f98b97ba28c Signed-off-by: Yann Gautier <[email protected]> Reviewed-on: https://gerrit.st.com/c/mpu/oe/st/u-boot/+/462649 Reviewed-by: Patrice CHOTARD <[email protected]> ACI: CIBUILD <[email protected]> Domain-Review: Patrice CHOTARD <[email protected]>
Add the display of STM32MP23 SoC revision Y or X. Change-Id: I0c6985bac72a13a9024c4de9bcb11ec4bf391c5f Signed-off-by: Yann Gautier <[email protected]> Reviewed-on: https://gerrit.st.com/c/mpu/oe/st/u-boot/+/462914 ACI: CITOOLS <[email protected]> Reviewed-by: Patrice CHOTARD <[email protected]> Domain-Review: Patrice CHOTARD <[email protected]> ACI: CIBUILD <[email protected]>
Update version in Makefile to prepare the label v2023.10-stm32mp-r2 for OpenSTLinux V6.1.0 Signed-off-by: Patrice Chotard <[email protected]> Change-Id: I475ea11aee2b76580a415bd703f09af0d52767e3
Comparing the SD card boot with the eMMC one, the DTS differs as follows: - fwu-mdata-store = <0x00000085>; + fwu-mdata-store = "/soc@0/bus@42080000/mmc@48230000"; In both cases, the U-Boot DTS loaded by tf-a-stm32mp at address 0x84400000 was obtained with the following commands: STM32MP> fdt addr 0x84400000 Working FDT set to 84400000 STM32MP> fdt print As shown in the diff, the working case (SD card) uses a numeric value, i.e. the phandle of the sdmmc1 node, while the eMMC case uses a path. This patch updates the fwu-mdata-store property to use the phandle instead of the path. Signed-off-by: Dario Binacchi <[email protected]>
Collaborator
|
Hi Dario, Your issue has been entered in our internal bug database. It will be part of our next release v6.2.1 which will be released in june. Thanks for your patch. |
pchotard
force-pushed
the
v2023.10-stm32mp
branch
from
64beae9 to
7fbc677
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Comparing the SD card boot with the eMMC one, the DTS differs as follows:
In both cases, the U-Boot DTS loaded by tf-a-stm32mp at address 0x84400000 was obtained with the following commands:
STM32MP> fdt addr 0x84400000
Working FDT set to 84400000
STM32MP> fdt print
As shown in the diff, the working case (SD card) uses a numeric value, i.e. the phandle of the sdmmc1 node, while the eMMC case uses a path. This patch updates the fwu-mdata-store property to use the phandle instead of the path.
Please do not submit a Pull Request via github. Our project makes use of
mailing lists for patch submission and review. For more details please
see https://u-boot.readthedocs.io/en/latest/develop/sending_patches.html
The only exception to this is in order to trigger a CI loop on Azure prior
to posting of patches.