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

Hans de Goede
2016-06-08T16:52:30Z

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.

Regards,

Hans