Hello Dinh Nguyen,

The patch 8d736d8a9c44: "usb: dwc2: gadget: Do not fail probe if
there isn't a clock node" from Nov 11, 2014, leads to the following
static checker warning:

        drivers/usb/dwc2/gadget.c:3436 dwc2_gadget_init()
        warn: passing zero to 'PTR_ERR'

drivers/usb/dwc2/gadget.c
  3432          hsotg->clk = devm_clk_get(dev, "otg");
  3433          if (IS_ERR(hsotg->clk)) {
  3434                  hsotg->clk = NULL;

You need to preserve the error code.  NULL means zero means success.

  3435                  dev_err(dev, "cannot get otg clock\n");
  3436                  return PTR_ERR(hsotg->clk);
  3437          }

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

Reply via email to