- 07 Sep, 2020 9 commits
-
-
Sebastien Pasdeloup authored
Signed-off-by:Sebastien Pasdeloup <sebastien.pasdeloup-ext@st.com> Change-Id: I5367d8b35af703d3647f9953aa8398870523a69c
-
Yann Gautier authored
The clock enable and disable was previously used to register IWDG clock in the shared resources framework. This is no more needed, remove the code. Change-Id: Iaeb70ac82018931747e51e3b0ab155ff9dddebc0 Signed-off-by:
Yann Gautier <yann.gautier@st.com> Reviewed-on: https://gerrit.st.com/c/mpu/oe/st/tf-a/+/169804 Reviewed-by:
CITOOLS <smet-aci-reviews@lists.codex.cro.st.com> Reviewed-by:
CIBUILD <smet-aci-builds@lists.codex.cro.st.com> Reviewed-by:
Etienne CARRIERE <etienne.carriere@st.com> Reviewed-by:
Lionel DEBIEVE <lionel.debieve@st.com>
-
Yann Gautier authored
The ICR static flags are cleared before sending a command. The SDMMC_DCTRLR register is set to 0 if no data is expected on a given command or on the next command in case of CMD55. Change-Id: I5ae172a484218f53160e98b3684967c6960475a6 Signed-off-by:
-
Yann Gautier authored
The retries number should be 3. A warning message is added in mmc_block_read(), and the code is refactored. Change-Id: I577c7dd91c451c7580b1660042cb5fe26ee3fa12 Signed-off-by:
-
Yann Gautier authored
Depending on compiler, the issue about bool or uint*_t not defined can appear. Correct this by adding stdbool.h and stdint.h includes in etzpc.h. Change-Id: If1419dc511efbe682459fa4a776481fa52a38aa3 Signed-off-by:
-
Yann Gautier authored
The call to fdt_get_status(node) has to be done after the DT is found to be valid. Fixes: 1fc2130c stm32mp1: update device tree and gpio functions Change-Id: I70f803aae3dde128a9e740f54c8837b64cb1a244 Signed-off-by:
-
Yann Gautier authored
The previous correction was incomplete, and could lead to compilation errors. Fixes:4c638aa8abe3072ab31917c637e35ca1df2a03d4 Change-Id: If5d9519accb85cfa9274943c13b7213f16f1f964 Signed-off-by:
-
Yann Gautier authored
Remove the IMAGE_BL2 flag that was restricting the GPIO banks clock gating in BL2 stage. The GPIO banks clocks can now be accessed from BL32. Change-Id: Ia9f5261e2f267ec0e7aec30ae9f8ff2244251486 Signed-off-by:
-
Lionel Debieve authored
nand_wait_ready is called with a millisecond delay but the timeout used a micro second. Fixing the conversion in the timeout call. Signed-off-by:
Christophe KERELLO <christophe.kerello@st.com> Reviewed-by:
-
- 22 Jul, 2020 2 commits
-
-
Sebastien Pasdeloup authored
Signed-off-by: -
Lionel Debieve authored
In case of uart programmation, the header length could be equals to boot_api_image_header_t. In this case, the header must be saved to avoid assert when retrieving header for integrity check. Signed-off-by:
-
- 03 Jul, 2020 1 commit
-
-
Sebastien Pasdeloup authored
Signed-off-by:
-
- 02 Jul, 2020 1 commit
-
-
Lionel Debieve authored
Calibration uses the ARM counter which is 32 bit signed value. Depending on the clock source, some required timing value cannot be reached and must be limited to the INT32 max value. Signed-off-by:
-
- 01 Jul, 2020 4 commits
-
-
Lionel Debieve authored
The stack size is not large enough to manage the calibration interrupt and data abort occurred when exiting from FIQ handler. Increase the stack size fix the crash. Signed-off-by:
-
Lionel Debieve authored
Avoid possible lock when the capture is not stable. Base the capture value to a define threshold. Signed-off-by:
-
Yann Gautier authored
When coming back from STANDBY, nothing has to be loaded from storage devices, as the DDR was in Self-Refresh, and all the code is already there. BL32 image load is also explicitly skipped. Change-Id: I960a757c95f216f0325ddbd7842b54d0b80f5848 Signed-off-by:
-
Nicolas Le Bayon authored
Describe PLL1 as optional with the introduction of the OPP table. Change-Id: I6373354464a931f2329427a1c812e56d75985588 Signed-off-by:
Nicolas Le Bayon <nicolas.le.bayon@st.com> Reviewed-on: https://gerrit.st.com/c/mpu/oe/st/tf-a/+/168993 Reviewed-by:
-
- 28 May, 2020 1 commit
-
-
Lionel Debieve authored
Increase delay from 10µs to 50µs to avoid panic or DDR corruption after self refresh exit sequence. Signed-off-by:
-
- 27 May, 2020 2 commits
-
-
Lionel Debieve authored
When clock is disabled in device tree, the parent will not be found. It must not be a reason to panic. Signed-off-by:
-
Lionel Debieve authored
Remove the initial loop that forces the secure state to non-secure for all shared resources. They must remain secure and must be only unsecured if specified in device tree. Signed-off-by:
-
- 25 May, 2020 20 commits
-
-
Lionel Debieve authored
In case Self-Refresh function fails, the cstop_enter() function must panic and not return, as some parts of the STOP mode entry sequence has already been done. Signed-off-by:
-
Benjamin Gaignard authored
Define macros for some STGEN related bit fields in RCC registers. Signed-off-by:
Benjamin Gaignard <benjamin.gaignard@st.com> Change-Id: I67bc8a58a804a354d5ba82f09150f39ae534ec45 Reviewed-on: https://gerrit.st.com/c/mpu/oe/st/tf-a/+/166139 Reviewed-by:
-
Sebastien Pasdeloup authored
Move "WARNING: Cannot get CPU version" to INFO Move "Use default chip ID" to INFO These messages are more an INFO messages rather than an issues. Change-Id: I5ca25d852b6785cc903963964592ebb65bf15f58 Signed-off-by:
-
Lionel Debieve authored
Change the detection message from WARN to INFO when macronix NOR is detected. Signed-off-by:
-
Lionel Debieve authored
Some crash may occur if the stack is not properly aligned. By switching to uint32_t, we ensure this. Signed-off-by:
-
Yann Gautier authored
SYSCFG configuration is not mandatory for UART and console. Initializing it after console allows traces to be displayed. Change-Id: I51f45d464c512d2a67c9fec4ce4b22e4c6a64fad Signed-off-by:
-
Yann Gautier authored
Since commit [1], there is no more subnode pwr-regulators in the pwr node, hence the vdd-supply was not correct, giving a 0V voltage. Remove this subnode check in dt_get_pwr_vdd_voltage(). [1]: 634bbb42 ("fdts: stm32mp1: device tree alignment kernel v5.4") Change-Id: Id4cf9767b189a639a756aff27c9ecabf85dff9c9 Signed-off-by: -
Yann Gautier authored
Do not try to set a regulator voltage if the previous value is not correct. Change-Id: I0a89600ca34b4509bf9c059f4fb23471ba4b96eb Signed-off-by:
-
Yann Gautier authored
Use errno values, or the return of called functions, instead of -1. Change-Id: If7b53de5cbfb4d2c9979bce0e594dd92bf07a77a Signed-off-by:
-
Yann Gautier authored
This issue was found with cppcheck in our downstream code: [drivers/st/io/io_stm32image.c:234] -> [drivers/st/io/io_stm32image.c:244]: (warning) Either the condition 'buffer!=0U' is redundant or there is possible null pointer dereference: local_buffer. Change-Id: Ieb615b7e485dc93bbeeed4cd8bf845eb84c14ac9 Signed-off-by:
-
Yann Gautier authored
This warning was issued by cppcheck in our downstream code: [plat/st/common/stm32mp_dt.c:629] -> [plat/st/common/stm32mp_dt.c:634]: (warning) Identical condition 'node<0', second condition is always false The second test has to check variable pwr_regulators_node. Change-Id: I4a20c4a3ac0ef0639c2df36309d90a61c02b511f Signed-off-by:
-
Yann Gautier authored
The variable is wrongly set to 0L, whereas it is an unsigned int, it should then be 0U. Change-Id: I0b164c0ea598ec8a503f1693da2f3789f59da238 Signed-off-by:
-
Yann Gautier authored
Correct the following sparse warnings: plat/st/common/stm32mp_dt.c:103:5: warning: symbol 'fdt_get_node_parent_address_cells' was not declared. Should it be static? plat/st/common/stm32mp_dt.c:123:5: warning: symbol 'fdt_get_node_parent_size_cells' was not declared. Should it be static? Change-Id: Iad721f12128df83a3de3f53e7920a9c1dce64c56 Signed-off-by:
-
Yann Gautier authored
Correct the following warning given by sparse tool: include/drivers/raw_nand.h:158:3: warning: symbol '__packed' was not declared. Should it be static? Change-Id: I03bd9a8aee5cdc5212ce5225be8033f1a6e92bd9 Signed-off-by:
-
Yann Gautier authored
Sparse issue: drivers/st/spi/stm32_qspi.c:445:5: warning: symbol 'stm32_qspi_init' was not declared. Should it be static? Cppcheck issue: [drivers/st/spi/stm32_qspi.c:175] -> [drivers/st/spi/stm32_qspi.c:187]: (style) Variable 'len' is reassigned a value before the old one has been used. [drivers/st/spi/stm32_qspi.c:178]: (style) The scope of the variable 'timeout' can be reduced. Change-Id: I575fb50766355a6717cbd193fc4a80ff1923014c Signed-off-by:
-
Benjamin Gaignard authored
Make sure that i2c->i2c_state is correctly initialized with I2C_STATE_RESET value this avoid hi2c->lock to not be set to 0 when calling stm32_i2c_init during platform suspend/resume operations. Signed-off-by:
-
Lionel Debieve authored
Device tree alignment with Linux kernel v5.4-stm32mp-r1 Signed-off-by:
-
Lionel Debieve authored
The backup partition will be checked only by the default load process. It is removed from io_stm32image.c to clarify the load processing. Signed-off-by:
-
Lionel Debieve authored
Use plat_try_next_boot_source() to look for a backup partition if available. Signed-off-by:
-
Lionel Debieve authored
In case of backup partition usage, it is easier to pass the image ID as argument to plat_try_next_boot_source() to properly identify and update the corrupted partition to be read. Signed-off-by:
-
