[PATCH] mx7ulp: soc: s_init should only be executed once

2020-01-20 Thread sbabic
> On SPL enabled systems, the current s_init code (wdog, clock and ldo > init) is executed twice (by SPL and u-boot). This is not necessary and > might lead to boot issues (ie, starting PMC1 when it is already running). > Signed-off-by: Jorge Ramirez-Ortiz > Reviewed-by: Fabio Estevam Applied to

Re: [PATCH] mx7ulp: soc: s_init should only be executed once

2020-01-19 Thread Fabio Estevam
Hi Jorge, On Sun, Jan 19, 2020 at 3:52 PM Jorge Ramirez-Ortiz, Foundries wrote: > hi Favio/all > > plese could you confirm if this patch will be merged (just so we dont have to > carry it separately in our product branches) Yes, I think it makes sense to apply it. I have already given my

Re: [PATCH] mx7ulp: soc: s_init should only be executed once

2020-01-19 Thread Jorge Ramirez-Ortiz, Foundries
On 17/01/20 13:08:44, Jorge Ramirez-Ortiz, Gmail wrote: > On 17/01/20 08:53:03, Fabio Estevam wrote: > > Hi Jorge, > > > > On Fri, Jan 17, 2020 at 6:50 AM Jorge Ramirez-Ortiz > > wrote: > > > > > > On SPL enabled systems, the current s_init code (wdog, clock and ldo > > > init) is executed

Re: [PATCH] mx7ulp: soc: s_init should only be executed once

2020-01-17 Thread Jorge Ramirez-Ortiz, Gmail
On 17/01/20 08:53:03, Fabio Estevam wrote: > Hi Jorge, > > On Fri, Jan 17, 2020 at 6:50 AM Jorge Ramirez-Ortiz > wrote: > > > > On SPL enabled systems, the current s_init code (wdog, clock and ldo > > init) is executed twice (by SPL and u-boot). This is not necessary and > > might lead to boot

Re: [PATCH] mx7ulp: soc: s_init should only be executed once

2020-01-17 Thread Fabio Estevam
Hi Jorge, On Fri, Jan 17, 2020 at 6:50 AM Jorge Ramirez-Ortiz wrote: > > On SPL enabled systems, the current s_init code (wdog, clock and ldo > init) is executed twice (by SPL and u-boot). This is not necessary and > might lead to boot issues (ie, starting PMC1 when it is already running). > >

[PATCH] mx7ulp: soc: s_init should only be executed once

2020-01-17 Thread Jorge Ramirez-Ortiz
On SPL enabled systems, the current s_init code (wdog, clock and ldo init) is executed twice (by SPL and u-boot). This is not necessary and might lead to boot issues (ie, starting PMC1 when it is already running). Signed-off-by: Jorge Ramirez-Ortiz --- arch/arm/mach-imx/mx7ulp/soc.c | 2 ++ 1