Re: [PATCH V3 05/11] USB: OHCI: Properly handle ohci-ep93xx suspend

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello.

On 06/19/2013 06:42 PM, Manjunath Goudar wrote:

        Suspend scenario in case of ohci-ep93xx glue was not
        properly handled as it was not suspending generic part
        of ohci controller. Calling explicitly the ohci_suspend()
        routine in ohci_hcd_ep93xx_drv_suspend() will ensure proper
        handling of suspend scenario.


        Signed-off-by: Manjunath Goudar <manjunath.goudar@xxxxxxxxxx
        <mailto:manjunath.goudar@xxxxxxxxxx>>
        Cc: Arnd Bergmann <arnd@xxxxxxxx <mailto:arnd@xxxxxxxx>>
        Cc: Alan Stern <stern@xxxxxxxxxxxxxxxxxxx
        <mailto:stern@xxxxxxxxxxxxxxxxxxx>>
        Cc: Greg KH <greg@xxxxxxxxx <mailto:greg@xxxxxxxxx>>
        Cc: linux-usb@xxxxxxxxxxxxxxx <mailto:linux-usb@xxxxxxxxxxxxxxx>


        V2:
           -Incase ohci_suspend() fails, return right away without
            executing further.
        V3:
           -Aligned variable "do_wakeup" and "ret".
        ---
           drivers/usb/host/ohci-ep93xx.c |   10 +++++++++-
           1 file changed, 9 insertions(+), 1 deletion(-)


        diff --git a/drivers/usb/host/ohci-__ep93xx.c
        b/drivers/usb/host/ohci-__ep93xx.c
        index 8704e9f..f5f11d1 100644
        --- a/drivers/usb/host/ohci-__ep93xx.c
        +++ b/drivers/usb/host/ohci-__ep93xx.c
        @@ -174,13 +174,21 @@ static int
        ohci_hcd_ep93xx_drv_suspend(__struct platform_device *pdev,
        pm_message_
           {
                 struct usb_hcd *hcd = platform_get_drvdata(pdev);
                 struct ohci_hcd *ohci = hcd_to_ohci(hcd);
        +       bool    do_wakeup = device_may_wakeup(&pdev->dev);
        +       int             ret;


        There was no need to use tabs here, didn't you see that here the
    variables aren't aligned?

    WBR, Sergei

Instead of gmail you can use thunderbird mail there it is showing
  proper alignment.

   Imagine, I'm using Thinderbird. :-D
And you're using HTML reply format, unwelcome on the Linux mailing lists.

Manjunath Goudar

WBR, Sergei

--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux