| Message ID | 20171106052741.445-1-bjorn.andersson@linaro.org (mailing list archive) |
|---|---|
| State | Deferred |
| Headers | show |
> --------- Original Message --------- > Sender : Bjorn Andersson <bjorn.andersson@linaro.org> > > Propagate the error of devfreq_add_device() in devm_devfreq_add_device() > rather than statically returning ENOMEM. This makes it slightly faster > to pinpoint the cause of a returned error. > > Fixes: 8cd84092d35e ("PM / devfreq: Add resource-managed function for devfreq device") > Cc: stable@vger.kernel.org > Acked-by: Chanwoo Choi <cw00.choi@samsung.com> > Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org> > --- > > Resending with Chanwoo's ack, Fixes and Cc stable > > drivers/devfreq/devfreq.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Acked-by: MyungJoo Ham <myungjoo.ham@samsung.com>
diff --git a/drivers/devfreq/devfreq.c b/drivers/devfreq/devfreq.c index 78fb496ecb4e..99c4021fc33b 100644 --- a/drivers/devfreq/devfreq.c +++ b/drivers/devfreq/devfreq.c @@ -737,7 +737,7 @@ struct devfreq *devm_devfreq_add_device(struct device *dev, devfreq = devfreq_add_device(dev, profile, governor_name, data); if (IS_ERR(devfreq)) { devres_free(ptr); - return ERR_PTR(-ENOMEM); + return devfreq; } *ptr = devfreq;
Propagate the error of devfreq_add_device() in devm_devfreq_add_device() rather than statically returning ENOMEM. This makes it slightly faster to pinpoint the cause of a returned error. Fixes: 8cd84092d35e ("PM / devfreq: Add resource-managed function for devfreq device") Cc: stable@vger.kernel.org Acked-by: Chanwoo Choi <cw00.choi@samsung.com> Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org> --- Resending with Chanwoo's ack, Fixes and Cc stable drivers/devfreq/devfreq.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)