Hi David,

Thank you for the patch.

On Thu, Sep 03, 2026 at 16:35, David Lechner <[email protected]> wrote:

> Check the return value of dev_remap_addr_index() for NULL instead of
> IS_ERR(). This function only ever returns NULL on failure.
>
> Fixes: 92114d7c0a19 ("usb: dwc3: Add dwc3 glue driver for am62")
> Signed-off-by: David Lechner <[email protected]>

Reviewed-by: Mattijs Korpershoek <[email protected]>

> ---
>  drivers/usb/dwc3/dwc3-am62.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/usb/dwc3/dwc3-am62.c b/drivers/usb/dwc3/dwc3-am62.c
> index 8cb5796b6ad..57a34d69fbe 100644
> --- a/drivers/usb/dwc3/dwc3-am62.c
> +++ b/drivers/usb/dwc3/dwc3-am62.c
> @@ -46,7 +46,7 @@ static void dwc3_ti_am62_glue_configure(struct udevice 
> *dev, int index,
>       struct ofnode_phandle_args args;
>  
>       usbss = dev_remap_addr_index(dev, 0);
> -     if (IS_ERR(usbss)) {
> +     if (!usbss) {
>               dev_err(dev, "can't map IOMEM resource\n");
>               return;
>       }
>
> -- 
> 2.43.0

Reply via email to