On Fri, 14 Feb 2020 09:50:54 +0530
Lokesh Vutla lokeshvu...@ti.com wrote:
> On 13/02/20 10:42 PM, Neil Armstrong wrote:
> > Hi,
> >
> > On 03/02/2020 14:59, Oliver Graute wrote:
> >> The conga-imx8 board isn't booting without this revert. Can someone tell me
> >> what I need to Do so that this revert is not necessary?
>
> Can you give more details on where the failure is happening?(logs would really
> help).
>
> >>
> >> This reverts commit 52edfed65de967a86983a55c51ba0727090efc43.
> >> ---
> >> drivers/core/device-remove.c | 5 -----
> >> 1 file changed, 5 deletions(-)
> >>
> >> diff --git a/drivers/core/device-remove.c b/drivers/core/device-remove.c
> >> index 5c8dc4ad70..586fadee0a 100644
> >> --- a/drivers/core/device-remove.c
> >> +++ b/drivers/core/device-remove.c
> >> @@ -16,7 +16,6 @@
> >> #include <dm/uclass.h>
> >> #include <dm/uclass-internal.h>
> >> #include <dm/util.h>
> >> -#include <power-domain.h>
> >>
> >> int device_chld_unbind(struct udevice *dev, struct driver *drv)
> >> {
> >> @@ -193,10 +192,6 @@ int device_remove(struct udevice *dev, uint flags)
> >> }
> >> }
> >>
> >> - if (!(drv->flags & DM_FLAG_DEFAULT_PD_CTRL_OFF) &&
> >> - (dev != gd->cur_serial_dev))
> >> - dev_power_domain_off(dev);
> >> -
> >> if (flags_remove(flags, drv->flags)) {
> >> device_free(dev);
> >>
> >>
> >
> > It also breaks amlogic boards with video, since we can't tell we want
> > to keep the power domains up after boot, it cuts the power domains and
> > thus defeats all the goal of u-boot video..
> > The problem is DM_FLAG_DEFAULT_PD_CTRL_OFF disables auto power domain on
> > aswell, reverting to manual power domain enable.
>
> This doesn't mean U-boot must leave the all power-domains un handled before
> jumping to kernel. Please use DM_FLAG_DEFAULT_PD_CTRL_OFF in you driver and
> enable power_domain in your probe.
No, DM_FLAG_DEFAULT_PD_CTRL_OFF is only for very special cases where
manual power domain handling is required.
> Your case is exactly the reason why
> DM_FLAG_DEFAULT_PD_CTRL_OFF is introduced.
No. For this case the flag DM_FLAG_REMOVE_WITH_PD_ON has been introduced
in my v3 patch, but it was dropped in the merged patch. I've submitted a
new patch for this: http://patchwork.ozlabs.org/patch/1239143
--
Anatolij
