Re: [[RFC] 3/5] Expunge old Atmel PWMC driver, replacing it with one that conforms to the PWM API

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Mike Frysinger wrote:
+       ap->clk = clk_get(&pdev->dev, "pwm_clk");
+       if (IS_ERR(ap->clk)) {
+               pr_info("%s: clk_get error %ld\n",
+                       ap->pwm.bus_id, PTR_ERR(ap->clk));
+               ret = -ENODEV;
+               goto err_clk_get;

shouldnt it be:
  ret = PTR_ERR(ap->clk);

Probably, because it's preferable to return the actual error code when it's known, rather than translating all error codes to -ENODEV. Good catch.


b.g.

--
Bill Gatliff
bgat@xxxxxxxxxxxxxxx

--
To unsubscribe from this list: send the line "unsubscribe linux-embedded" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Gstreamer Embedded]     [Linux MMC Devel]     [U-Boot V2]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux ARM Kernel]     [Linux OMAP]     [Linux SCSI]

  Powered by Linux