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

[Michal Simek]

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.

M