On 13 December 2013 21:55, Marek Vasut <ma...@denx.de> wrote: > In case the controller is not initialized, we shall not de-initialize it. > As the control structure will not be filled, we will produce a null ptr > dereference if the controller is not inited. > > Signed-off-by: Marek Vasut <ma...@denx.de> > Cc: Simon Glass <s...@chromium.org> >
Acked-by: Simon Glass <s...@chromium.org> > --- > drivers/usb/host/ehci-hcd.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/drivers/usb/host/ehci-hcd.c b/drivers/usb/host/ehci-hcd.c > index 8bd1eb8..3ef204d 100644 > --- a/drivers/usb/host/ehci-hcd.c > +++ b/drivers/usb/host/ehci-hcd.c > @@ -201,6 +201,9 @@ static int ehci_shutdown(struct ehci_ctrl *ctrl) > int i, ret = 0; > uint32_t cmd, reg; > > + if (!ctrl || !ctrl->hcor) > + return -EINVAL; > + > cmd = ehci_readl(&ctrl->hcor->or_usbcmd); > cmd &= ~(CMD_PSE | CMD_ASE); > ehci_writel(&ctrl->hcor->or_usbcmd, cmd); > -- > 1.8.4.3 > >
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
