Le mardi 19 septembre 2023 à 07:08 +1000, John Watts a écrit : > On Mon, Sep 18, 2023 at 11:01:15PM +0200, Paul Cercueil wrote: > > The datasheet does say a 5ms sleep time is necesary after a reset. > > I > > assume the 120ms delay you quote is when a *software* reset is > > performed in Sleep-out mode. The code here does a hard-reset. > > > > Cheers, > > -Paul > > Hello Paul, > > Section 7.3 of the data sheet (AC characteristic) says that the reset > can take > up to 120ms to complete if the reset is applied during sleep out > mode. > > John. The driver is guaranteed to always reset the panel in sleep-in mode - as long as the panel was off when the driver started. What I'd suggest if you really need to support a case where the panel was enabled by the bootloader, is to read the 0x0a register after enabling the regulator to read the mode, and sleep 120ms if it was in sleep-out mode. But that's only if it's a case that you can test with. I won't accept a patch that makes sense on the surface if it addresses a corner case that nobody ever tested for. For what I know, this patch just adds a huge delay to panel boot-up for all existing users for no valid reason. Cheers, -Paul
