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

[Felipe Balbi]

Hi,

Stefan Wahren <info at lategoodbye.de> writes:
> Hi Felipe,
>
> Am 20.11.2015 um 16:27 schrieb Felipe Balbi:
>>
>> Hi,
>>
>> Stefan Wahren <info at lategoodbye.de> writes:
>>> Hi Felipe,
>>>
>>> Am 19.11.2015 um 20:54 schrieb Felipe Balbi:
>>>>
>>>> Hi John,
>>>>
>>>> John Youn <John.Youn at synopsys.com> 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.

okay, in that case, can you make sure testing/fixes and testing/next
contain all your pending patches?

Thanks

-- 
balbi
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 818 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-rpi-kernel/attachments/20151120/d5c739cb/attachment.sig>