[PATCH v2] usb: dwc3: Use the devm_gpiod_get_optional() API for reset gpio

[Marek Vasut]

On 2/23/23 10:24, Michal Simek wrote:
> 
> 
> On 2/22/23 16:33, Marek Vasut wrote:
>> On 2/22/23 08:01, Michal Simek wrote:
>>>
>>>
>>> On 2/22/23 02:06, Marek Vasut wrote:
>>>> On 1/13/23 06:12, Venkatesh Yadav Abbarapu wrote:
>>>>> As the "reset-gpios" property is optional, don't return the
>>>>> error and just skip the gpio reset sequence.
>>>>>
>>>>> Signed-off-by: Venkatesh Yadav Abbarapu <venkatesh.abbarapu at amd.com>
>>>>> ---
>>>>> Changes in v2:
>>>>> - Replaced the gpio_request_by_name() the API with
>>>>> devm_gpiod_get_optional().
>>>>
>>>> The PX30 seems to fail to build:
>>>>
>>>> https://source.denx.de/u-boot/custodians/u-boot-usb/-/jobs/580968
>>>
>>> Doesn't look usb related.
>>>
>>> +Please report this bug.
>>> +make[4]: *** [scripts/Makefile.build:257: spl/drivers/mmc/mmc.o] 
>>> Error 1
>>> +make[3]: *** [scripts/Makefile.build:397: spl/drivers/mmc] Error 2
>>> +make[2]: *** [scripts/Makefile.spl:533: spl/drivers] Error 2
>>>
>>> Can you please double check it?
>>
>> u-boot/master without this patch built fine, usb/next just built fine 
>> too. I did retrigger the failing build to see if this is transient 
>> failure, but it would be good if someone at xilinx could build-test 
>> the px30 too .
> 
> TBH. I actually was building it myself before I sent this email.

The build passed, so it was a transient error, this patch is now part of 
u-boot/master too.