[RESEND PATCH] PM / devfreq: Propagate error from devfreq_add_device()

[RESEND PATCH] PM / devfreq: Propagate error from devfreq_add_device()

[Bjorn Andersson]

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(-)

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;
-- 
2.12.0

RE: [RESEND PATCH] PM / devfreq: Propagate error from devfreq_add_device()

[MyungJoo Ham]

 
 
> --------- 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>

 
 

RE: [RESEND PATCH] PM / devfreq: Propagate error from devfreq_add_device()

[MyungJoo Ham]

 
 
> --------- 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>