Re: [PATCH RESEND 1/2] usb: dwc2: Make PHY optional

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Felipe,

Am 20.11.2015 um 16:27 schrieb Felipe Balbi:

Hi,

Stefan Wahren <info@xxxxxxxxxxxxxx> writes:
Hi Felipe,

Am 19.11.2015 um 20:54 schrieb Felipe Balbi:

Hi John,

John Youn <John.Youn@xxxxxxxxxxxx> writes:
Fixes commit 09a75e85
"usb: dwc2: refactor common low-level hw code to platform.c"

these two lines should be placed ...


The above commit consolidated the low-level phy access into a common
location. This change introduced a check from the gadget requiring
that a PHY is specified. This requirement never existed on the host
side and broke some platforms when it was moved into platform.c.

The gadget doesn't require the PHY either so remove the check.


... here with the following format:

Fixes: 09a75e857790 ("usb: dwc2: refactor common low-level
		hw code to platform.c")

Just is just FYI, as I have already applied another version ;-)


thanks but

https://git.kernel.org/cgit/linux/kernel/git/balbi/usb.git/log/?h=testing/fixes

seems to miss ("usb: dwc2: fix kernel oops during driver probe")

This is what you gent for sending fixes and non-fixes in the same series
:-)

Also, this has been broken for a long time, you didn't include a "Fixes:
foo-bar-baz", you didn't Cc stable (seems like the regression is from
v3.19). It gets pretty difficult for me to apply patches properly if you
can't send them in a way I can understand what to do with them.

If you want patches to get into -rc, send the bare minimum patches
needed to fix the problem and do NOT make them depend on anything else.

i'm sorry about that. I should have added the fixes tag to all the patches.

I tought it wasn't a candidate for Cc stable, since the issue wasn't reproducable before.

The intention of the series was to get USB working for 4.4 for bcm2835.


and

https://git.kernel.org/cgit/linux/kernel/git/balbi/usb.git/log/?h=testing/next

seems to miss ("usb: dwc2: Return errors from PHY")

in my queue

Maybe i forgot to mention that the patch series needs to be applied
complete to fix the regression.

if you have dependencies within the series, let me know now.


There a no dependencies within the series.

Thanks

Stefan

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



[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux