All of lore.kernel.org
 help / color / mirror / Atom feed
From: Alistair Popple <alistair@popple.id.au>
To: Alan Stern <stern@rowland.harvard.edu>
Cc: Mark Rutland <mark.rutland@arm.com>,
	devicetree@vger.kernel.org, gregkh@linuxfoundation.org,
	linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org,
	Tony Prisk <linux@prisktech.co.nz>,
	linuxppc-dev@lists.ozlabs.org
Subject: Re: [PATCH 4/7] ECHI Platform: Merge ppc-of EHCI driver into the ehci-platform driver
Date: Mon, 24 Feb 2014 11:28:01 +1100	[thread overview]
Message-ID: <2268977.fdRl41kSok@mexican> (raw)
In-Reply-To: <Pine.LNX.4.44L0.1402211037300.1273-100000@iolanthe.rowland.org>

On Fri, 21 Feb 2014 10:41:50 Alan Stern wrote:
> On Fri, 21 Feb 2014, Alistair Popple wrote:
> > Currently the ppc-of driver uses the compatibility string
> > "usb-ehci". This means platforms that use device-tree and implement an
> > EHCI compatible interface have to either use the ppc-of driver or add
> > a compatible line to the ehci-platform driver. It would be more
> > appropriate for the platform driver to be compatible with "usb-ehci"
> > as non-powerpc platforms are also beginning to utilise device-tree.
> > 
> > This patch merges the device tree property parsing from ehci-ppc-of
> > into the platform driver and adds a "usb-ehci" compatibility
> > string. The existing ehci-ppc-of driver is removed and the 440EPX
> > specific quirks are added to the ehci-platform driver.
> > 
> > Signed-off-by: Alistair Popple <alistair@popple.id.au>
> 
> This patch is also out of date.  The compatibility string used by the
> ehci-platform driver is "generic-ehci".

Ok. I will switch to using "generic-ehci" in the Akebono device tree.

> There remains the question of whether to merge ehci-ppc-of into
> ehci-platform.  This would be a rather invasive change, but I suppose
> we could do it.  With adjustments along the lines suggested by Mark
> Rutland.

As I can use "generic-ehci" to support Akebono I will drop this patch. I can 
look at merging ehci-ppc-of at some later date if desired (thanks Mark & Tony 
for reviewing though).

- Alistair

> Alan Stern

WARNING: multiple messages have this Message-ID (diff)
From: Alistair Popple <alistair@popple.id.au>
To: Alan Stern <stern@rowland.harvard.edu>
Cc: gregkh@linuxfoundation.org, linux-usb@vger.kernel.org,
	devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
	linuxppc-dev@lists.ozlabs.org, Tony Prisk <linux@prisktech.co.nz>,
	Mark Rutland <mark.rutland@arm.com>
Subject: Re: [PATCH 4/7] ECHI Platform: Merge ppc-of EHCI driver into the ehci-platform driver
Date: Mon, 24 Feb 2014 11:28:01 +1100	[thread overview]
Message-ID: <2268977.fdRl41kSok@mexican> (raw)
In-Reply-To: <Pine.LNX.4.44L0.1402211037300.1273-100000@iolanthe.rowland.org>

On Fri, 21 Feb 2014 10:41:50 Alan Stern wrote:
> On Fri, 21 Feb 2014, Alistair Popple wrote:
> > Currently the ppc-of driver uses the compatibility string
> > "usb-ehci". This means platforms that use device-tree and implement an
> > EHCI compatible interface have to either use the ppc-of driver or add
> > a compatible line to the ehci-platform driver. It would be more
> > appropriate for the platform driver to be compatible with "usb-ehci"
> > as non-powerpc platforms are also beginning to utilise device-tree.
> > 
> > This patch merges the device tree property parsing from ehci-ppc-of
> > into the platform driver and adds a "usb-ehci" compatibility
> > string. The existing ehci-ppc-of driver is removed and the 440EPX
> > specific quirks are added to the ehci-platform driver.
> > 
> > Signed-off-by: Alistair Popple <alistair@popple.id.au>
> 
> This patch is also out of date.  The compatibility string used by the
> ehci-platform driver is "generic-ehci".

Ok. I will switch to using "generic-ehci" in the Akebono device tree.

> There remains the question of whether to merge ehci-ppc-of into
> ehci-platform.  This would be a rather invasive change, but I suppose
> we could do it.  With adjustments along the lines suggested by Mark
> Rutland.

As I can use "generic-ehci" to support Akebono I will drop this patch. I can 
look at merging ehci-ppc-of at some later date if desired (thanks Mark & Tony 
for reviewing though).

- Alistair

> Alan Stern

  reply	other threads:[~2014-02-24  0:28 UTC|newest]

Thread overview: 69+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-02-21  6:31 [PATCH 0/7] IBM Akebono/PPC476GTR Support Alistair Popple
2014-02-21  6:31 ` Alistair Popple
2014-02-21  6:31 ` Alistair Popple
2014-02-21  6:31 ` [PATCH 1/7] IBM Akebono: Add a SDHCI platform driver Alistair Popple
2014-02-21  6:31   ` Alistair Popple
2014-02-21  6:31   ` Alistair Popple
     [not found]   ` <1392964293-13687-2-git-send-email-alistair-Y4h6yKqj69EXC2x5gXVKYQ@public.gmane.org>
2014-02-21 14:14     ` Arnd Bergmann
2014-02-21 14:14       ` Arnd Bergmann
2014-02-21 14:14       ` Arnd Bergmann
2014-02-24  0:38       ` Alistair Popple
2014-02-24  0:38         ` Alistair Popple
2014-02-21  6:31 ` [PATCH 2/7] IBM Akebono: Add support for a new PHY interface to the IBM emac driver Alistair Popple
2014-02-21  6:31   ` Alistair Popple
2014-02-21  6:31   ` Alistair Popple
2014-02-21 11:18   ` Mark Rutland
2014-02-21 11:18     ` Mark Rutland
2014-02-21 11:18     ` Mark Rutland
2014-02-24  2:09     ` Alistair Popple
2014-02-24  2:09       ` Alistair Popple
2014-02-24  2:09       ` Alistair Popple
2014-02-21  6:31 ` [PATCH 3/7] IBM Akebono: Add support to the OHCI platform driver for PPC476GTR Alistair Popple
2014-02-21  6:31   ` Alistair Popple
2014-02-21  6:31   ` Alistair Popple
2014-02-21 14:16   ` Arnd Bergmann
2014-02-21 14:16     ` Arnd Bergmann
2014-02-21 14:16     ` Arnd Bergmann
2014-02-24  0:20     ` Alistair Popple
2014-02-24  0:20       ` Alistair Popple
2014-02-24  0:20       ` Alistair Popple
2014-02-21 15:34   ` Alan Stern
2014-02-21 15:34     ` Alan Stern
2014-02-21 15:34     ` Alan Stern
2014-02-21  6:31 ` [PATCH 4/7] ECHI Platform: Merge ppc-of EHCI driver into the ehci-platform driver Alistair Popple
2014-02-21  6:31   ` Alistair Popple
2014-02-21  6:31   ` Alistair Popple
2014-02-21 11:48   ` Mark Rutland
2014-02-21 11:48     ` Mark Rutland
2014-02-21 11:48     ` Mark Rutland
2014-02-21 14:21     ` Arnd Bergmann
2014-02-21 14:21       ` Arnd Bergmann
2014-02-21 14:21       ` Arnd Bergmann
2014-02-22  2:32     ` Tony Prisk
2014-02-22  2:32       ` Tony Prisk
2014-02-22  2:32       ` Tony Prisk
2014-02-21 15:41   ` Alan Stern
2014-02-21 15:41     ` Alan Stern
2014-02-21 15:41     ` Alan Stern
2014-02-24  0:28     ` Alistair Popple [this message]
2014-02-24  0:28       ` Alistair Popple
2014-02-21  6:31 ` [PATCH 5/7] IBM Currituck: Clean up board specific code before adding Akebono code Alistair Popple
2014-02-21  6:31   ` Alistair Popple
2014-02-21  6:31   ` Alistair Popple
2014-02-21  6:31 ` [PATCH 6/7] IBM Akebono: Add the Akebono platform Alistair Popple
2014-02-21  6:31   ` Alistair Popple
2014-02-21  6:31 ` [PATCH 7/7] powerpc: Added PCI MSI support using the HSTA module Alistair Popple
2014-02-21  6:31   ` Alistair Popple
2014-02-21  6:31   ` Alistair Popple
2014-02-21 14:33   ` Arnd Bergmann
2014-02-21 14:33     ` Arnd Bergmann
2014-02-21 14:33     ` Arnd Bergmann
2014-02-21 20:41     ` Benjamin Herrenschmidt
2014-02-21 20:41       ` Benjamin Herrenschmidt
2014-02-21 20:41       ` Benjamin Herrenschmidt
2014-02-21 21:53       ` Arnd Bergmann
2014-02-21 21:53         ` Arnd Bergmann
2014-02-21 21:53         ` Arnd Bergmann
2014-02-25  5:54       ` Alistair Popple
2014-02-25  5:54         ` Alistair Popple
2014-02-25  5:54         ` Alistair Popple

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=2268977.fdRl41kSok@mexican \
    --to=alistair@popple.id.au \
    --cc=devicetree@vger.kernel.org \
    --cc=gregkh@linuxfoundation.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-usb@vger.kernel.org \
    --cc=linux@prisktech.co.nz \
    --cc=linuxppc-dev@lists.ozlabs.org \
    --cc=mark.rutland@arm.com \
    --cc=stern@rowland.harvard.edu \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.