linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: s.hauer@pengutronix.de (Sascha Hauer)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 0/7] ARM: mx28: add usb host function
Date: Wed, 20 Jul 2011 22:07:58 +0200	[thread overview]
Message-ID: <20110720200757.GP26347@pengutronix.de> (raw)
In-Reply-To: <1311160106-4898-1-git-send-email-tony.lin@freescale.com>

On Wed, Jul 20, 2011 at 07:08:19PM +0800, Tony Lin wrote:
> this patch series is to add mx28 usb host function.
> the driver only support usb host mode on usb port1.
> the driver reuses ehci-mxc.c under usb/host, but did
> some modifications based on it to satisfy mx28 usb host
> requirement. The modifications will not affect other
> platforms.
> 
> Code is based on the branch for-next in sascha's tree http://git.pengutronix.de/git/imx/linux-2.6.git


I don't really like the approach of merging a completely
unrelated architecture into the i.MX platform glue code.

Currently every architecture has it's own very similar
glue code for the EHCI driver which is bad. What is even
worse is that we have multiple (I think 4) drivers for the
gadget part of this chip. Some architectures support OTG,
others not, but all of them have a different way to attach
a USB phy to the EHCI core.

We are on the road to device tree support which should give
us a good opportunity to clean this up (We don't want to have
device tree binding code in each platform driver we have now,
right?)
Given that Freescale is currently working hard on device tree
support I think the effort is better spent in bringing the
existing oftree driver in shape for use on ARM.

Sascha


-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

      parent reply	other threads:[~2011-07-20 20:07 UTC|newest]

Thread overview: 41+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-07-20 11:08 [PATCH 0/7] ARM: mx28: add usb host function Tony Lin
2011-07-20 10:57 ` Felipe Balbi
2011-07-20 11:08 ` [PATCH 1/7] consolidate definitions and structures to share among platforms Tony Lin
2011-07-20 10:59   ` Felipe Balbi
2011-07-20 11:03     ` Lin Tony-B19295
2011-07-20 11:16       ` Felipe Balbi
2011-07-20 11:21         ` Lin Tony-B19295
2011-07-20 11:08 ` [PATCH 2/7] enable usb1 phy power supply Tony Lin
2011-07-20 11:01   ` Felipe Balbi
2011-07-20 11:41   ` Wolfram Sang
2011-07-22  2:16     ` Lin Tony-B19295
2011-07-20 19:08   ` Sascha Hauer
2011-07-22  2:20     ` Lin Tony-B19295
2011-07-20 11:08 ` [PATCH 3/7] add usb phy clocks to clock tree Tony Lin
2011-07-20 11:02   ` Felipe Balbi
2011-07-20 11:05     ` Lin Tony-B19295
2011-07-20 19:10   ` Sascha Hauer
2011-07-22  6:02     ` Lin Tony-B19295
2011-07-22  9:18       ` Sascha Hauer
2011-07-20 11:08 ` [PATCH 4/7] mx28: add usb host phy functions Tony Lin
2011-07-20 11:12   ` Felipe Balbi
2011-07-20 19:26   ` Sascha Hauer
2011-07-22  2:23     ` Lin Tony-B19295
2011-07-21  6:04   ` Lothar Waßmann
2011-07-20 11:08 ` [PATCH 5/7] ehci mxc: make it more flexible to be used for mx28 Tony Lin
2011-07-20 11:05   ` Arnaud Patard (Rtp)
2011-07-20 11:08     ` Lin Tony-B19295
2011-07-20 11:45       ` Wolfram Sang
2011-07-20 11:14   ` Felipe Balbi
2011-07-20 11:21     ` Lin Tony-B19295
2011-07-20 11:23       ` Felipe Balbi
2011-07-20 11:25         ` Lin Tony-B19295
2011-07-20 11:27           ` Felipe Balbi
2011-07-20 13:59   ` Alan Stern
2011-07-22  5:18     ` Lin Tony-B19295
2011-07-20 19:41   ` Sascha Hauer
2011-07-20 11:08 ` [PATCH 6/7] add macro definitions according to ehci-mxc driver change Tony Lin
2011-07-20 19:43   ` Sascha Hauer
2011-07-21  6:06   ` Lothar Waßmann
2011-07-20 11:08 ` [PATCH 7/7] add usb host function to default config Tony Lin
2011-07-20 20:07 ` Sascha Hauer [this message]

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=20110720200757.GP26347@pengutronix.de \
    --to=s.hauer@pengutronix.de \
    --cc=linux-arm-kernel@lists.infradead.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).