linux-arm-msm.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Ivan T. Ivanov" <iivanov-NEYub+7Iv8PQT0dZR+AlfA@public.gmane.org>
To: balbi-l0cyMroinI0@public.gmane.org
Cc: gregkh-hQyY1W1yCW8ekmWlsbkhG0B+6BGkLq7r@public.gmane.org,
	linux-usb-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-arm-msm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: Re: [PATCH v2 0/7] usb: phy: msm: Fixes and cleanups
Date: Tue, 16 Jul 2013 14:25:56 +0300	[thread overview]
Message-ID: <1373973956.2657.4.camel@iivanov-dev> (raw)
In-Reply-To: <1373384833-18077-1-git-send-email-iivanov-NEYub+7Iv8PQT0dZR+AlfA@public.gmane.org>


Hi Felipe, 

On Tue, 2013-07-09 at 18:47 +0300, Ivan T. Ivanov wrote:
> From: "Ivan T. Ivanov" <iivanov-NEYub+7Iv8PQT0dZR+AlfA@public.gmane.org>
> 
> Changes since first version.
> * Extend commit messages a little bit.
> 
> Following patches make initial cleanup of usb phy found in the Qualcomm
> chipsets. Changes include:
> * Build time error fix.
> * Move driver to Managed Device Resource allocation.
> * Checkpatch warnings and error fixes
> * Removed usage of global regulators variables.
> 
> Ivan T. Ivanov (7):
>   usb: phy: msm: Move mach depndend code to platform data
>   usb: phy: msm: Migrate to Managed Device Resource allocation
>   usb: phy: msm: Move regulator usage to managed resource allocation
>   usb: phy: msm: Remove unnecessarily check for valid regulators.
>   usb: phy: msm: Fix WARNING: quoted string split across lines
>   usb: phy: msm: Fix WARNING: Prefer seq_puts to seq_printf
>   usb: phy: msm: Lindent the code


Could you take a look at these patches, please.
I would like to clean up this driver and add support for
controllers which can be found in new MSM8974 chipsets.

If there is something, which you would like to see improved
in this driver, please let me know.

Regards,
Ivan

> 
>  arch/arm/mach-msm/board-msm7x30.c |   35 ++++
>  arch/arm/mach-msm/board-qsd8x50.c |   34 ++++
>  drivers/usb/phy/phy-msm-usb.c     |  384 ++++++++++++++-----------------------
>  include/linux/usb/msm_hsusb.h     |    5 +
>  4 files changed, 220 insertions(+), 238 deletions(-)
> 


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

  parent reply	other threads:[~2013-07-16 11:25 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-07-09 15:47 [PATCH v2 0/7] usb: phy: msm: Fixes and cleanups Ivan T. Ivanov
2013-07-09 15:47 ` [PATCH v2 1/7] usb: phy: msm: Move mach depndend code to platform data Ivan T. Ivanov
2013-07-24 12:34   ` Felipe Balbi
2013-07-24 12:36     ` Felipe Balbi
     [not found] ` <1373384833-18077-1-git-send-email-iivanov-NEYub+7Iv8PQT0dZR+AlfA@public.gmane.org>
2013-07-09 15:47   ` [PATCH v2 2/7] usb: phy: msm: Migrate to Managed Device Resource allocation Ivan T. Ivanov
     [not found]     ` <1373384833-18077-3-git-send-email-iivanov-NEYub+7Iv8PQT0dZR+AlfA@public.gmane.org>
2013-07-24 12:38       ` Felipe Balbi
2013-07-25 13:42         ` Ivan T. Ivanov
2013-07-09 15:47   ` [PATCH v2 6/7] usb: phy: msm: Fix WARNING: Prefer seq_puts to seq_printf Ivan T. Ivanov
2013-07-16 11:25   ` Ivan T. Ivanov [this message]
2013-07-09 15:47 ` [PATCH v2 3/7] usb: phy: msm: Move regulator usage to managed resource allocation Ivan T. Ivanov
2013-07-24 12:39   ` Felipe Balbi
2013-07-25 13:43     ` Ivan T. Ivanov
2013-07-26 12:31       ` Ivan T. Ivanov
2013-07-26 20:28         ` Felipe Balbi
2013-07-27  6:54           ` Ivan T. Ivanov
2013-07-09 15:47 ` [PATCH v2 4/7] usb: phy: msm: Remove unnecessarily check for valid regulators Ivan T. Ivanov
2013-07-09 15:47 ` [PATCH v2 5/7] usb: phy: msm: Fix WARNING: quoted string split across lines Ivan T. Ivanov
2013-07-09 15:47 ` [PATCH v2 7/7] usb: phy: msm: Lindent the code Ivan T. Ivanov

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=1373973956.2657.4.camel@iivanov-dev \
    --to=iivanov-neyub+7iv8pqt0dzr+alfa@public.gmane.org \
    --cc=balbi-l0cyMroinI0@public.gmane.org \
    --cc=gregkh-hQyY1W1yCW8ekmWlsbkhG0B+6BGkLq7r@public.gmane.org \
    --cc=linux-arm-msm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=linux-usb-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    /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).