[PATCH] configs: stm32mp*: reset via CONFIG_RESET_SCMI

[Patrick DELAUNAY]

Hi,

On 9/5/22 19:01, Oleksandr Suvorov wrote:
> Jorge,
>
> I think, renaming the patch to "fix" and adding a field "Fixes:"
> should help accept it faster.
>
> On Mon, Sep 5, 2022 at 7:32 PM Jorge Ramirez-Ortiz, Foundries
> <jorge at foundries.io> wrote:
>> On 30/08/22, Jorge Ramirez-Ortiz wrote:
>>> Enabling CONFIG_SYSRESET_PSCI prevents CONFIG_RESET_SCMI
>>> from executing.
>>>
>>> The side effect observed are I2C devices no longer being
>>> accessible from U-boot after a soft reset.
>> I think this PR should get a bit more of attention.
>>
>> The current reset configuration is broken, this is a fix.
>> Do I need to rename the PR?
>>
>> TIA
>> jorge
>>
>>
>>> Signed-off-by: Jorge Ramirez-Ortiz <jorge at foundries.io>
>>> ---
>>>   configs/stm32mp13_defconfig         | 1 -
>>>   configs/stm32mp15_defconfig         | 1 -
>>>   configs/stm32mp15_trusted_defconfig | 1 -
>>>   3 files changed, 3 deletions(-)
>>>
>>> diff --git a/configs/stm32mp13_defconfig b/configs/stm32mp13_defconfig
>>> index 673b468d31..44cee2e656 100644
>>> --- a/configs/stm32mp13_defconfig
>>> +++ b/configs/stm32mp13_defconfig
>>> @@ -69,7 +69,6 @@ CONFIG_RNG_OPTEE=y
>>>   CONFIG_DM_RTC=y
>>>   CONFIG_RTC_STM32=y
>>>   CONFIG_SERIAL_RX_BUFFER=y
>>> -CONFIG_SYSRESET_PSCI=y
>>>   CONFIG_TEE=y
>>>   CONFIG_OPTEE=y
>>>   # CONFIG_OPTEE_TA_AVB is not set
>>> diff --git a/configs/stm32mp15_defconfig b/configs/stm32mp15_defconfig
>>> index e5a2996c2c..2ad02f3652 100644
>>> --- a/configs/stm32mp15_defconfig
>>> +++ b/configs/stm32mp15_defconfig
>>> @@ -133,7 +133,6 @@ CONFIG_SPI=y
>>>   CONFIG_DM_SPI=y
>>>   CONFIG_STM32_QSPI=y
>>>   CONFIG_STM32_SPI=y
>>> -CONFIG_SYSRESET_PSCI=y
>>>   CONFIG_TEE=y
>>>   CONFIG_OPTEE=y
>>>   # CONFIG_OPTEE_TA_AVB is not set
>>> diff --git a/configs/stm32mp15_trusted_defconfig b/configs/stm32mp15_trusted_defconfig
>>> index e14668042f..9e24e82920 100644
>>> --- a/configs/stm32mp15_trusted_defconfig
>>> +++ b/configs/stm32mp15_trusted_defconfig
>>> @@ -134,7 +134,6 @@ CONFIG_SPI=y
>>>   CONFIG_DM_SPI=y
>>>   CONFIG_STM32_QSPI=y
>>>   CONFIG_STM32_SPI=y
>>> -CONFIG_SYSRESET_PSCI=y
>>>   CONFIG_TEE=y
>>>   CONFIG_OPTEE=y
>>>   # CONFIG_OPTEE_TA_AVB is not set
>>> --
>>> 2.34.1
>>>
>
>

This patch it is superseded by "configs: stm32mp*: fix system reset"

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

http://patchwork.ozlabs.org/project/uboot/patch/20220905173357.2231466-1-jorge@foundries.io/


with the added "Fixes:"


Regards


Patrick