Hi Neil, On Mon, 2 Mar 2020 10:30:31 +0100 Neil Armstrong narmstr...@baylibre.com wrote:
> Hi Analolij, > > On 17/02/2020 12:53, Anatolij Gustschin wrote: > > On Mon, 17 Feb 2020 10:10:36 +0100 > > Neil Armstrong narmstr...@baylibre.com wrote: > > > >> This commit breaks Linux boot on Amlogic libretech-cc, libretech-ac, > >> sei510, > >> sei610 board by automatically disabling the power domain after device > >> removal. > >> This because the power domain associated to the video driver must be kept > >> enabled for linux to boot. > >> > >> The only way is to use the introduced flag DM_FLAG_DEFAULT_PD_CTRL_OFF, > >> which > >> removes automatic handling of power domain. > >> This flag is not a complete solution since it will force reverting to > >> manually > >> enable the power domain from the driver probe. > >> > >> Instead a flag to keep the power domain enabled after removal should be > >> introduced. > >> > >> In the meantime revert this commit until a proper solution is found. > >> > >> This reverts commit 52edfed65de967a86983a55c51ba0727090efc43. > >> > >> Signed-off-by: Neil Armstrong <narmstr...@baylibre.com> > >> --- > >> Hi Simon, Tom, > >> > >> This revert is the simplest fix to make the boards boot again on v2020.04, > >> the goal is to make this behavior happen again with a proper flag to keep > >> the > >> power domain enabled on specific drivers. > >> > >> I can push it on my next PR on my branch, is it ok ? > > > > Please test if these two patches fix the problem: > > http://patchwork.ozlabs.org/patch/1239143 > > http://patchwork.ozlabs.org/patch/1239144 > > Should I apply these through my tree or you plan to push these yourself ? These patches are assigned to Simon in patchwork, and are in his tree under build-testing I think. -- Anatolij
