[PATCH v2 5/5] lib: utils/reset: Add generic da9063 reset driver

[Nikita Shubin]

On Wed, 3 Nov 2021 13:07:29 +0100
Heinrich Schuchardt <heinrich.schuchardt at canonical.com> wrote:

> > After rebooting with Linux reboot:  
> > => reset  
> > resetting ...
> > 
> > All okay and that's means that it's not a full reset, I2C was
> > initialised by linux, and not resetted.  
> 
> Doesn't this imply that the I2C initialization code should be added
> to OpenSBI?
> 

Without any doubt.

> Best regards
> 
> Heinrich
> 
> > 
> > However if we arm the clock and go shutdown:
> > 
> > sh -c "echo `date '+%s' -d '+ 1 minutes'` >
> > /sys/class/rtc/rtc0/wakealarm" shutdown
> > shutdown
> > 
> > We get:  
> > => reset  
> > resetting ...
> > da9063_system_reset: chip is not da9063 PMIC
> > 
> > So "reset" with RTC is indeed a full reset...
> > 
> > All above also means that "sacrificing" RTC means sacrificing user
> > setted wakeup ALARM's.
> > 
> > May be it's worth splitting reset into WARM_REBOOT, COLD_REBOOT with
> > WARM reboot is going through method proposed by Alexandre and COLD
> > reboot via RTC, and explicitly specify that COLD reboot WILL
> > overwrite ALARM setting ?
> > 
> > What do you think about this ?
> >   
> >>
> >> Best regards
> >>
> >> Heinrich  
> >   
>