[PATCH v5 resend 1/2] ehci-platform: Add support for controllers with multiple reset lines

Greg Kroah-Hartman
2016-06-08T17:34:19Z

On Wed, Jun 08, 2016 at 06:52:30PM +0200, Hans de Goede wrote:
> Hi,
>
> On 08-06-16 07:22, Greg Kroah-Hartman wrote:
> > On Thu, Jun 02, 2016 at 05:14:05PM +0200, Hans de Goede wrote:
> > > From: Reinder de Haan <patchesrdh@mveas.com>
> > >
> > > At least the EHCI/OHCI found on the Allwinnner H3 SoC needs multiple
> > > reset lines, the controller will not initialize while the reset for
> > > its companion is still asserted, which means we need to de-assert
> > > 2 resets for the controller to work.
> > >
> > > Signed-off-by: Reinder de Haan <patchesrdh@mveas.com>
> > > Signed-off-by: Hans de Goede <hdegoede@redhat.com>
> > > ---
> > > Changes in v2:
> > > -Use the new reset_control_[de]assert_shared reset-controller functions
> > > ---
> > > Documentation/devicetree/bindings/usb/usb-ehci.txt | 2 +-
> > > drivers/usb/host/ehci-platform.c | 41 ++++++++++++----------
> > > 2 files changed, 24 insertions(+), 19 deletions(-)
> >
> > This patch doesn't apply to my usb-testing branch at all, what went
> > wrong?
>
> You merged this patch:
>
> https://git.kernel.org/cgit/linux/kernel/git/gregkh/usb.git/commit/?id=d0e08b0077f49e209bc90305ddf1ca434ac6cc0e
>
> Which does almost the same, except that it only supports dedicated reset
> controllers and not shared reset controllers.
>
> I'll send a new patch which changes things to add support for shared
> reset controllers.
>
> This will also bring the ehci-platform reset handling code inline
> with ohci-platform again.

Thanks, I've merged your new patch now.

greg k-h