On 1/17/20 3:33 AM, Peng Fan wrote: > Hi Marek, Hi,
>> Subject: Re: [RFC PATCH] imx: imx8mm-evk/imx8mn-evk: enable sysreset >> driver for SPL >> >> On 1/16/20 3:21 AM, Peng Fan wrote: >> >> Hello Peng, >> >>>> Subject: [RFC PATCH] imx: imx8mm-evk/imx8mn-evk: enable sysreset >>>> driver for SPL >>>> >>> >>> NAK, this will not work on i.MX8MM/N. >>> >>> Currently sysreset psci is enabled, however psci will not work for >>> SPL, because SPL boots before BL31. >> >> Thank you for the constructive feedback. >> >> So basically , what we need is a real sysreset driver for iMX8MM , which can >> work with bare-bones hardware interface, that is, poking some register, >> correct ? > > Yes. > >> >> So, either you or Claudius needs to implement a driver in drivers/sysreset, >> which will bind in SPL and if needed, do some writel() into some registers to >> reset the board, correct ? > > Yes. > >> >> Because, I believe we can both agree that dumping such ad-hoc code which >> implements reset in board code is not the right way, esp. nowadays with all >> the DM/DT stuff in. > > Alought we still have ocram space, but our SPL is huge now, 100KB+ How so ? How much does DM sysreset add to that ?