linux-arm-msm.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Pavankumar Kondeti <pkondeti@codeaurora.org>
To: greg@kroah.com, linux-usb@vger.kernel.org
Cc: linux-arm-msm@vger.kernel.org, davidb@codeaurora.org,
	mina86@mina86.com, Pavankumar Kondeti <pkondeti@codeaurora.org>
Subject: [PATCH V2 0/7] Typo & BUG fixes for MSM USB drivers
Date: Mon,  2 May 2011 11:56:26 +0530	[thread overview]
Message-ID: <1304317593-21808-1-git-send-email-pkondeti@codeaurora.org> (raw)
In-Reply-To: <1303977501-17115-1-git-send-email-pkondeti@codeaurora.org>

This patch series addresses the following issues.

1. EP0 is enabled after enabling pull-up in ci13xxx_udc.  But some simulators
are caching the ep0 max packet size at the time of pull-up enable due to
which USB is not working. Fix this by initializing ep0 at gadget driver
registration.
2. Retrieve the IN endpoint correctly from ep array while processing
endpoint feature requests.
3. Data phase on control OUT is not working.
4. USB does not work some times after resume due to a race condition involved
in clearing in_lpm flag.

Other than above issues, a couple typo mistakes are corrected in this patch
series.

Changelog:

- v2
  - Resending patches with V2 tag :-)
  - logical AND operator is used instead of bitwiese AND for testing flags.
  - Fixes a typo error (s/EBUSY/-EBUSY) pointed out by Michal.


Anji jonnala (1):
  USB: gadget: Initialize ep0 once while registering gadget in
    ci13xxx_udc

Pavankumar Kondeti (6):
  USB: gadget: Fix typo (s/EBUSY/-EBUSY) in ci13xxx_udc
  USB: gadget: Use bitwise AND operator to test flags in ci13xxx_udc
  USB: gadget: Fix unused variable warning in ci13xxx_udc
  USB: gadget: Fix bug in endpoint feature request processing in
    ci13xxx_udc
  USB: gadget: Use ep0out for control OUT data phase in ci13xxx_udc
  USB: OTG: msm: Clear in_lpm flag before enabling the IRQ in resume

 drivers/usb/gadget/ci13xxx_udc.c |   65 ++++++++++++++++++++++++--------------
 drivers/usb/otg/msm_otg.c        |    4 +-
 2 files changed, 43 insertions(+), 26 deletions(-)

-- 
Sent by a consultant of the Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum.

  parent reply	other threads:[~2011-05-02  6:26 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-04-28  7:58 [PATCH 1/5] USB: gadget: Fix unused variable warning in ci13xxx_udc Pavankumar Kondeti
2011-04-28  7:58 ` [PATCH 2/5] USB: gadget: Fix bug in endpoint feature request processing " Pavankumar Kondeti
     [not found] ` <1303977501-17115-1-git-send-email-pkondeti-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>
2011-04-28  7:58   ` [PATCH 3/5] USB: gadget: Use ep0out for control OUT data phase " Pavankumar Kondeti
2011-04-28  7:58 ` [PATCH 4/5] USB: gadget: Initialize ep0 once while registering gadget " Pavankumar Kondeti
2011-04-28  7:58 ` [PATCH 5/5] USB: OTG: msm: Clear in_lpm flag before enabling the IRQ in resume Pavankumar Kondeti
2011-04-28 10:01 ` [PATCH 1/5] USB: gadget: Fix unused variable warning in ci13xxx_udc Michal Nazarewicz
2011-04-28 10:19   ` Pavan Kondeti
     [not found]     ` <4DB93F48.4050803-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>
2011-04-28 16:30       ` David Brown
2011-04-29  5:12 ` [PATCH 1/7] USB: gadget: Fix typo (s/EBUSY/-EBUSY) " Pavankumar Kondeti
2011-04-29  5:12   ` [PATCH 2/7] USB: gadget: Use bitwise AND operator to test flags " Pavankumar Kondeti
2011-04-29  5:12   ` [PATCH 3/7] USB: gadget: Fix unused variable warning " Pavankumar Kondeti
2011-04-29  5:12   ` [PATCH 4/7] USB: gadget: Fix bug in endpoint feature request processing " Pavankumar Kondeti
     [not found]   ` <1304053975-1960-1-git-send-email-pkondeti-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>
2011-04-29  5:12     ` [PATCH 5/7] USB: gadget: Use ep0out for control OUT data phase " Pavankumar Kondeti
2011-04-29 16:04     ` [PATCH 1/7] USB: gadget: Fix typo (s/EBUSY/-EBUSY) " David Brown
     [not found]       ` <8yay62txcwc.fsf-AOX6H5vLt3Uj8izMo0bVsAC/G2K4zDHf@public.gmane.org>
2011-04-30  0:29         ` Greg KH
2011-05-02  5:44           ` Pavan Kondeti
2011-05-02  6:09       ` Pavan Kondeti
2011-04-29  5:12   ` [PATCH 6/7] USB: gadget: Initialize ep0 once while registering gadget " Pavankumar Kondeti
2011-04-29  5:12   ` [PATCH 7/7] USB: OTG: msm: Clear in_lpm flag before enabling the IRQ in resume Pavankumar Kondeti
2011-05-02  6:26 ` Pavankumar Kondeti [this message]
     [not found]   ` <1304317593-21808-1-git-send-email-pkondeti-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>
2011-05-02  6:26     ` [PATCH V2 1/7] USB: gadget: Fix typo (s/EBUSY/-EBUSY) in ci13xxx_udc Pavankumar Kondeti
2011-05-02  6:26     ` [PATCH V2 3/7] USB: gadget: Fix unused variable warning " Pavankumar Kondeti
2011-05-02  6:26   ` [PATCH V2 2/7] USB: gadget: Use bitwise AND operator to test flags " Pavankumar Kondeti
2011-05-02  6:26   ` [PATCH V2 4/7] USB: gadget: Fix bug in endpoint feature request processing " Pavankumar Kondeti
2011-05-02  6:26   ` [PATCH V2 5/7] USB: gadget: Use ep0out for control OUT data phase " Pavankumar Kondeti
2011-05-02  6:26   ` [PATCH V2 6/7] USB: gadget: Initialize ep0 once while registering gadget " Pavankumar Kondeti
2011-05-02  6:26   ` [PATCH V2 7/7] USB: OTG: msm: Clear in_lpm flag before enabling the IRQ in resume Pavankumar Kondeti

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1304317593-21808-1-git-send-email-pkondeti@codeaurora.org \
    --to=pkondeti@codeaurora.org \
    --cc=davidb@codeaurora.org \
    --cc=greg@kroah.com \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-usb@vger.kernel.org \
    --cc=mina86@mina86.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).